← Back to issue list

docs: replace google for ci and fix tests

View original Github issue

Metadata

Project
charmcraft
Number
#2338
Type
pull request
State
merged
Author
javierdelapuente
Labels
Created
Updated
Closed

Current evaluation

Merged after approval. Replaced Google CI with self-hosted GitHub runners for spread-docs, updated test configurations for prepare/restore and retries, and adjusted spread doc tests. CI passed with minor policy exceptions.

Suggested action:

No scores available.

Issue body

This PR tries to improve the spread-docs, as they were not working in the CI. For that: - Stop using google and instead run the tests in the self hosted github runners. The spread tests now use the new backend ci using the [.extension](https://github.com/canonical/snapcraft/blob/hotfix/8.9/snapcraft/templates/test/spread/.extension) from snapcraft. Thanks @lengau for this information. - Several changes in all spread doc tests, mostly related to prepare/restore and retries.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged after approval. Replaced Google CI with self-hosted GitHub runners for spread-docs, updated test configurations for prepare/restore and retries, and adjusted spread doc tests. CI passed with minor policy exceptions.
qwen3.6-35b-a3b-mtp-q6 Merged. Replaced Google CI with self-hosted GitHub runners for spread-doc tests. Updated test configuration to use snapcraft's backend extension, adding prepare/restore steps and retry logic to fix CI execution.
qwen3.6-35b-a3b-mtp-q6 Merged external PR replacing Google CI with self-hosted GitHub runners for spread-doc tests. Updated test configurations for prepare/restore and retry logic, resolving CI failures.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2126 ci(docs): spread test documentation charmcraft merged Merged changes adding a CI workflow for spread testing documentation and adjusting tasks for Google runners. Approved by two reviewers, the update modifies five files (+100/-22 lines) and resolves CRAFT-3798.
76%
#143 docs: make docs spread-testable craft-store merged Merged documentation spread-testing infrastructure setup. Approved by two reviewers and passing CI, the change modifies six files (+136/-41 lines) to enable local spread test execution for documentation.
73%
#2029 Run documentation spread tests in CI charmcraft open Maintainer task to create a CI job for docs spread tests. The workflow (spread-docs.yaml) now exists and runs on PRs/pushes/weekly. The google backend failure reported by @erinecon was addressed by switching backends. Issue remains open but work appears complete.
73%
#4715 ci: adjust tags for spread runs snapcraft merged Merged. Adjusted CI tags for spread runs to support Canonical's self-hosted runners. Approved by two reviewers, passed CI, and maintained coverage. Modified two files with six additions and six deletions.
71%
#1279 test(spread): migrate from google to openstack rockcraft merged Merged migration of spread tests from Google to OpenStack. Approved by two reviewers. CI checks for source-spread-tests and docs-spread-tests failed due to offline runners, with remaining proxy-related failures deferred for piecemeal fixes.
70%