build providers: fix base change warning message
Metadata
Current evaluation
Merged after approval and passing CI. The pull request fixes the base change warning message in build providers, updating one file with four additions and two deletions.
Suggested action: —
No scores available.
Issue body
Signed-off-by: Chris Patterson <chris.patterson@canonical.com>
- [ ] 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 `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged after approval and passing CI. The pull request fixes the base change warning message in build providers, updating one file with four additions and two deletions. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix correcting an inaccurate base change warning message in build providers. The maintainer-authored pull request was accepted and integrated without additional comments or revisions. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix correcting an inaccurate base change warning message within build providers. The update ensures warnings properly trigger during base configuration modifications, improving build environment feedback. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3182 build providers: improve warning for unknown base | snapcraft | merged | Merged after approval and passing CI checks. Improves the warning message for unknown bases in build providers, modifying one file with seven additions and four deletions. | |
| #4505 chore: remove deprecated parameter `build-base` | snapcraft | merged | Merged to remove the deprecated build-base parameter from craft-providers, eliminating a deprecation warning. Approved, passed CI and tests, and modified three files. The fix was already on the main branch. | |
| #3180 build providers: nice message on bad base | snapcraft | merged | Merged implementation adding a new exception to handle invalid bases in LXD build providers, providing clearer error messages. Resolves SNAPCRAFT-1H2. Approved by one reviewer, passed CI, and modified 4 files (+77/-14). | |
| #5537 docs: update build provider procedures | snapcraft | merged | Merged documentation update to build provider procedures, resolving issue #5519. Approved by two reviewers and passed most CI checks despite minor integration test failures. Changes span four files with 100 lines added and 98 removed. | |
| #2474 build providers: remove dead code | snapcraft | merged | Merged removal of dead code for legacy cleanbuild support in build providers. Approved by one reviewer, passed CI, and modified three files (+54/-165 lines). | |
| #2508 Build provider errors | snapcraft | merged | Merged after approval and passing CI checks. Resolved build provider errors across two files (+353/-85 lines). Codecov reported 90.62% diff coverage. | |
| #2460 build providers: recreate instance if base changed | snapcraft | merged | Merged fix ensuring build providers recreate instances when the project base changes. Previously, base updates were ignored after initial provisioning. This resolves Launchpad bug #1794506 and was approved and merged into master. | |
| #2374 build providers: destroy on create failures | snapcraft | merged | Merged. Resolves a bug where build providers and VMs remained running after launch failures. Adds cleanup logic and tests across seven files, approved by a reviewer and passing CI checks. | |
| #3105 build provider: clean incompatible build-environments | snapcraft | merged | Merged updates to clean incompatible build environments caused by base mismatches, missing base records, or version differences. Implemented Debian-compliant version comparison and updated test scenarios. Approved and merged. | |
| #2935 build providers: remove tzdata workaround | snapcraft | merged | Merged changes remove an outdated tzdata workaround from build providers. The underlying issue was resolved by adding --preserve-env in a prior commit, rendering the workaround obsolete. Approved and passed CI. |