← Back to issue list

ci: adjust tags for spread runs

View original Github issue

Metadata

Project
snapcraft
Number
#4715
Type
pull request
State
merged
Author
lengau
Labels
Created
Updated
Closed

Current evaluation

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.

Suggested action:

No scores available.

Issue body

This ensures we can use any machine with the spread-installed tag. Necessary for upcoming changes to Canonical's self-hosted runners. Spread will fail until a repo admin adjusts the runner tags. - [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `tox run -m lint`? - [ ] Have you successfully run `tox run -e test-py310`? (supported versions: `py39`, `py310`, `py311`, `py312`) -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b 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.
qwen3.6-35b-a3b-mtp-q6 Merged adjustments to CI tags for spread runs, ensuring compatibility with Canonical's self-hosted runners. The change passed all automated tests and achieved full coverage on modified lines before integration.
qwen3.6-35b-a3b-mtp-q6 Merged to adjust CI tags for spread runs, enabling compatibility with Canonical's self-hosted runners. Passed all tests and coverage checks, allowing any machine with the spread-installed tag to be utilized.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#524 ci: adjust tags for spread runs rockcraft closed Merged a configuration update adjusting spread runner tags to support Canonical's self-hosted runners. Approved by two reviewers with passing CI. The change modifies tags across two files to prevent execution failures.
94%
#1628 ci: use the correct tags for large spread runs charmcraft merged Merged a one-line CI fix to apply correct tags for large spread runs. Approved by three reviewers with no unresolved comments.
85%
#573 ci: run spread tests on 'spread-installed' runners rockcraft merged Merged a two-file, four-line CI configuration change to run spread tests on spread-installed runners. Approved by three reviewers and integrated.
81%
#293 ci: run on self-hosted runners fetch-service merged CI configuration updated to run on self-hosted runners. Merged after approval from two reviewers, modifying one file with 15 additions and 5 deletions.
77%
#309 ci: switch to public runners craft-store merged Merged the CI configuration update to switch to public runners. Approved by two reviewers with all continuous integration checks passing. The change modifies a single file with two additions and one deletion.
76%
#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.
75%
#3662 tests: update spread url snapcraft merged Merged update to spread tarball URLs in test files. Approved by a reviewer, passed CI checks, modified two files with minimal changes, and maintained existing code coverage.
75%
#280 ci, spread: add "large spreads" suite rockcraft merged Merged addition of a new large spreads test suite to CI. Configured to run weekly via cron or manually via label. Approved by two reviewers and successfully integrated.
74%
#127 Fix spread tests imagecraft merged Merged after receiving approval from two reviewers and passing all CI checks. The changes fix spread tests across 11 files (+49/-38 lines).
74%
#1083 ci: fix spread test missing PATH charmcraft merged Merged a fix for a CI spread test missing the PATH variable. The change updated four files and received reviewer approval. Post-merge validation confirmed the spread tests for configure and remove passed successfully.
74%