← Back to issue list

fix: get non-store CI working on Charmcraft 2.7

View original Github issue

Metadata

Project
charmcraft
Number
#2518
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged to fix non-store CI configuration for Charmcraft 2.7. Adjusted test setup and permissions without altering application functionality. Approved by two reviewers and integrated alongside a companion permissions update.

Suggested action: —

No scores available.

Issue body

A grab-bag of changes to charmcraft 2.7 to get CI working. No functional changes to the app. This may still include failures within the spread tests themselves, but the test *setup* should be correct. Docs failure is because charmcraft 2.7 doesn't have sphinx docs. Accompanying permissions change: https://github.com/canonical/canonical-repo-automation/pull/537 CHARMCRAFT-669 --- - [x] I've followed the [contribution guidelines](https://github.com/canonical/charmcraft/blob/main/CONTRIBUTING.md). - [x] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/). - [x] I've successfully run `make lint && make test`. - [ ] I've added or updated any relevant documentation. - [ ] I've updated the relevant release notes.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged to fix non-store CI configuration for Charmcraft 2.7. Adjusted test setup and permissions without altering application functionality. Approved by two reviewers and integrated alongside a companion permissions update.
qwen3.6-35b-a3b-mtp-q6 — — Merged CI configuration updates for Charmcraft 2.7 to enable non-store testing. Contains no functional app changes, resolves CHARMCRAFT-669, and pairs with an accompanying permissions update.
qwen3.6-35b-a3b-mtp-q6 — — Merged updates to Charmcraft 2.7 CI configuration, resolving non-store test setup and documentation generation issues without functional app changes. Includes an accompanying permissions update for CHARMCRAFT-669.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#49 ci: test the charmcraft actions in more places craft-actions merged Merged after two approvals and passing all CI checks. The update expands the continuous integration pipeline to test charmcraft actions across additional build environments, modifying a single file with minor configuration changes.
74%
#580 requirements: update to latest released versions charmcraft merged Merged update to latest released dependency versions, introducing craft-store for improved Charmhub error handling. Approved by one reviewer with passing CI checks across two files.
73%
#405 fix: update packages typing craft-store merged Merged fix updating packages typing to a mapping, resolving a charmcraft login validation error for string charm names. Approved by two reviewers and passed CI. Prepares the craft-store release.
73%
#2122 build(deps): update patch dependencies for Charmcraft 2.7 charmcraft merged Merged after two approvals and passing CI. Updates patch dependencies for Charmcraft 2.7 to resolve issues including jinja2. Changes modify three files with minor version bumps.
72%
#1075 ci: only require CHARMCRAFT_AUTH for store spread charmcraft merged Merged CI workflow updates that restrict CHARMCRAFT_AUTH requirements to store tests only. General tests run independently, while store tests are limited to Ubuntu 22.04, marked manual, and reduced in repetition. Three files modified.
72%
#1950 fix: make charmcraft work with snap parallel installs charmcraft merged Merged to resolve issue #1770, enabling charmcraft to operate with snap parallel installs. Approved by two reviewers, the update modifies five files with 55 additions and 15 deletions.
71%
#2275 fix(charm): workaround to allow charmcraft-wide constraints charmcraft merged Merged a workaround enabling charmcraft-wide constraints, resolving issue #2259 and CHARMCRAFT-614. Approved by two reviewers, the change modifies 15 files with 425 additions and passed CI checks prior to integration.
71%
#2031 ci: use charmcraft from the store for weekly spread runs charmcraft merged Merged a CI update that switches weekly spread runs to pull charmcraft from the store instead of building a snap. This saves build time as edge sync delays are rare. Approved by two reviewers and passed all checks.
70%
#476 Fixed permissions bits after extracting the charm's content to analyze (CRAFT-429) charmcraft merged Merged fix for permission bits after extracting charm content for analysis (CRAFT-429). Approved by two reviewers, passed CI checks, and modified two files with 47 additions and 13 deletions.
70%