ua: do not re-enable enabled services for snapcraft_legacy
Metadata
Current evaluation
Merged a fix for snapcraft_legacy preventing re-enablement of already-active UA services to avoid errors. Approved, retargeted to the hotfix branch, and merged after an unrelated test failure was resolved separately.
Suggested action: —
No scores available.
Issue body
- [X] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [X] Have you successfully run `make lint`?
- [X] Have you successfully run `pytest tests/unit`?
-----
Same as https://github.com/snapcore/snapcraft/pull/3960, but for `snapcraft_legacy`.
If a UA service is already enabled, do not attempt to re-enable the service because UA will raise and error.
source: https://bugs.launchpad.net/bugs/1989500
(CRAFT-1451)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged a fix for snapcraft_legacy preventing re-enablement of already-active UA services to avoid errors. Approved, retargeted to the hotfix branch, and merged after an unrelated test failure was resolved separately. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix for snapcraft_legacy preventing re-enabling of active Ubuntu Advantage services, resolving error 1989500. Applied to main and the hotfix branch, with an unrelated spread failure handled separately. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main and hotfix branches. Prevents snapcraft_legacy from re-enabling already active UA services, resolving a Launchpad error. Merged despite one unrelated spread failure. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3890 ua: enable ua services in legacy | snapcraft | merged | Merged baseline implementation backporting UA services support from core22 to legacy snapcraft. Introduces ua-services project key and --enable-experimental-ua-services flag. Approved by maintainer, passed CI, and integrated into main. | |
| #1989500 Can't run twice when ua-services specified | snapcraft (launchpad) | closed | Fixed. Reported that snapcraft fails on repeated runs with ua-services due to ua enable returning non-zero for already-enabled services. Resolved by updating execution logic to handle existing service states. | |
| #3960 ua: do not re-enable enabled services | snapcraft | merged | Merged into hotfix/7.2.2. Prevents Ubuntu Advantage from raising errors when attempting to re-enable already enabled services. Rebased and merged alongside fixes for unrelated failing tests. |