build providers: clean up LXD startup message
Metadata
Current evaluation
Merged PR snapcraft#2936 (commit bc605d706) trimmed the misleading LXD cloud-init startup log message, removing an implementation detail that users misread as data being sent to the cloud.
Resolution: closed resolved
Reason: The PR was approved and merged as commit bc605d706 into snapcraft, changing a single line in the LXD build provider to drop the confusing cloud-init waiting message; follow-up #2940 (9daee0bba) later removed cloud-init usage entirely.
Evaluated by: mr-cal-bot
No scores available.
Issue body
This message about waiting for cloud-init is an implementation detail.
This has confused users, some thought that their data was being
_sent to the cloud_ which is not what is happening considering
what cloud-init is meant to be doing.
The code base is also moving away from cloud-init to profile containers
for Snapcraft, it might as well be ready for that moment.
Signed-off-by: Sergio Schvezov <sergio.schvezov@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 |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR snapcraft#2936 (commit bc605d706) trimmed the misleading LXD cloud-init startup log message, removing an implementation detail that users misread as data being sent to the cloud. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2936 (commit bc605d706) removed the misleading LXD cloud-init startup message, clarifying that users' data was not being sent to the cloud. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged removal of a confusing LXD startup message about waiting for cloud-init, which misled users into thinking data was being sent to the cloud. The change aligns with the transition to profile containers and was approved. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged removal of a confusing LXD startup message about waiting for cloud-init. The message misled users into thinking data was being sent to the cloud. The change aligns with the shift from cloud-init to profile containers. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/snapcraft#2940
(confidence 85%)
Follow-up PR merged as 9daee0bba, removing the cloud-init provisioning this message referred to
-
Related To:
canonical/snapcraft#2227
(confidence 70%)
Earlier commit a0dac5a92 'lxd: wait for cloud-init' introduced the waiting behaviour/message being cleaned up
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3281 build providers: hide systemd setup for LXD | snapcraft | merged | Merged in PR #3281 (commit 8029036c6), hiding verbose LXD systemd setup output by default while preserving it in internal debug logs. | |
| #3033 build providers: do not print network test output for LXD | snapcraft | merged | Merged PR #3033 via commit e335de69e, suppressing getent hosts snapcraft.io output during LXD build-provider environment setup. | |
| #2509 build providers: initial support for LXD | snapcraft | merged | Merged PR #2509 (commit 067024732) adding initial LXD build-provider support: `snapcraft --use-lxd` runs all lifecycle steps and clean inside an LXD container. Released via beta/lxd channel; LP bug #1805221 closed. | |
| #1647 lxd: better surfacing of errors | snapcraft | merged | Merged in snapcraft#1647 as commit 71f48bdc9, improving LXD error handling so callers return silent, correctly coded errors without printing messages twice. | |
| #3115 build providers: ignore missing LXD instance when cleaning project | snapcraft | merged | Merged PR #3115 (commit 8ce59011a) makes project cleanup ignore missing LXD instances deleted manually. | |
| #3112 spread: remove dead code for lxd setup and add debug prints | snapcraft | merged | Merged PR #3112 in commit cf2fec383, removing unreachable LXD setup code and adding debug output for intermittent LXD snap-install failures. |