← Back to issue list

tests: check that architecture "all" works

View original Github issue

Metadata

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

Current evaluation

Merged as f136def1, adding tests for architecture-independent charms using build-for: all and fixing #1698; prerequisite #1894 was merged.

Resolution: closed resolved

Reason: PR #1895 was merged as commit f136def1 and added coverage for architecture-independent charms using build-for: all, resolving #1698. The required build-plan refactor in #1894 was also merged.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Fixes #1698, requires https://github.com/canonical/charmcraft/pull/1894

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged as f136def1, adding tests for architecture-independent charms using build-for: all and fixing #1698; prerequisite #1894 was merged.
stealth/space-bunny-alpha — closed resolved Merged as f136def1, adding tests for architecture-independent charms using build-for: all and fixing #1698; prerequisite #1894 was merged.
stealth/space-bunny-alpha — closed resolved PR #1895 merged as f136def1, adding tests covering `build-for: all` for arch-independent charms. Closed #1698, building on craft-platforms refactor #1894 and `build-for: all` support from #1876.
stealth/space-bunny-alpha — closed resolved Merged as f136def1 via #1895, adding architecture "all" tests and resolving #1698 after prerequisite #1894 moved build-plan handling to craft-platforms.
stealth/space-bunny-alpha — closed resolved Merged in charmcraft PR #1895 after two approvals, adding tests for architecture "all" across 5 files (+70/−38). It fixes #1698 and depends on merged PR #1894.
qwen/qwen3.6-35b-a3b — — Merged following two approvals. Introduces tests for architecture all, resolving issue 1698. Depends on PR 1894. Updated 5 files (+70/-38).
qwen3.6-35b-a3b-mtp-q6 — — Merged to resolve issue #1698 by adding tests that verify architecture all functionality. Integration depended on prerequisite PR #1894.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/charmcraft#1698 (confidence 100%)

    PR #1895 explicitly fixes the issue about build-for: all for architecture-independent charms.

  • Related To: canonical/charmcraft#1894 (confidence 100%)

    Merged prerequisite PR that moved build-plan handling to craft-platforms.

Related issues

Issue Project State Summary Similarity
#1698 build-for architecture 'all' for arch-independent charms not supported charmcraft closed Support for build-for: all landed in #1894 (f1755e8f), with architecture-independent charm tests merged in #1895 (f136def1).
82%
#1400 fix: always include architectures in charm manifest charmcraft merged Merged canonical/charmcraft#1400 to always serialize architecture fields in generated charm manifests, including inferred defaults; rendering and tests were updated across nine files.
73%
#1818 fix(lifecycle): allow building multi-arch charms charmcraft merged Merged to main as commit c787ea23 (PR #1818), letting Charmcraft's lifecycle builds handle charms with multiple run-on architectures; landed alongside Ruff 0.6.0 fixes from #1825. Fixes #1697.
71%
#1198 tests: add charm library multiple versions update test charmcraft merged Merged PR #1198 (commit 154bf5d6) added a 120-line regression test covering updates across multiple charm-library versions, linked to #1020/CRAFT-1916.
70%