feat: use platform name in build instance name
Metadata
Current evaluation
Merged in canonical/craft-application#601 as 5ae7f878: build instance names now use the platform, preventing environment reuse across platforms and fixing #600.
Resolution: closed resolved
Reason: PR #601 was merged as commit 5ae7f878. It replaced build-on/build-for architectures with the platform in instance names, preventing environments from being reused across platforms.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [x] Have you followed the guidelines for contributing?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `tox`?
-----
Changes the build instance name to use the platform rather than the `build-on` and `build-for` architectures.
This prevents the same build environment being used for multiple platforms. See https://github.com/canonical/charmcraft/pull/2069 for examples.
Fixes #600
Unblocks https://github.com/canonical/charmcraft/pull/2069
(CRAFT-3859)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged in canonical/craft-application#601 as 5ae7f878: build instance names now use the platform, preventing environment reuse across platforms and fixing #600. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged commit 5ae7f878 changes build instance names to use the platform, preventing environment reuse across platforms and fixing canonical/craft-application#600. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged. Changes build instance naming to use platform names instead of architectures, preventing shared environments. Approved by two reviewers, the +40/-10 line update across four files fixes #600 and unblocks dependent work. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
canonical/craft-application#600
(confidence 100%)
PR #601 explicitly fixes this platform-based build-environment naming issue.
-
Related To:
canonical/charmcraft#2069
(confidence 95%)
PR #601 states that this naming change unblocks the Charmcraft PR.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #600 Use platform for build environment names | craft-application | closed | Fixed in commit 5ae7f878: build instance names now use the platform name instead of build-on/build-for, preventing platform environment collisions. | |
| #84 fix: do not strip base from platform names | craft-platforms | merged | Merged PR #84 via commit 7680af2: shorthand platform names now retain their base, preventing duplicate BuildInfo names required by craft-application. | |
| #483 feat: support "--platform" for "clean" | craft-application | merged | Merged PR #483 via commit 07e89deb, adding --platform to clean so one managed instance can be removed, and fixing craft-application#425. | |
| #947 feat!(platforms): prevent use of '/' in platform names | craft-application | merged | Merged PR #947 as commit f7f3b1ff, adding validation that rejects platform names containing "/" and fixing #941 (CRAFT-4876). |