remote-build: remove need to specify user
Metadata
Current evaluation
Merged #2825, removing the remote-build user requirement and adding automatic Launchpad login; it also fixed public-upload handling and cleaned up LaunchpadClient.
Resolution: closed resolved
Reason: PR canonical/snapcraft#2825 was approved and merged, resolving the no-user workflow. Commits 994c8e1dd, 2c2aaca3b, and 621eea18e added automatic login, corrected AcceptPublicUploadError handling, and removed _waiting.
Evaluated by: mr-cal-bot
No scores available.
Issue body
A few other tweaks/cleanup while we're here.
remote-build: remove requirement to specify user …
Thanks to @cjwatson for the great idea! :+1:
https://bugs.launchpad.net/snapcraft/+bug/1852482
remote-build: login automatically when initialized …
remote-build: fix AcceptPublicUploadError option …
remote-build: remove _waiting from LaunchpadClient …
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged #2825, removing the remote-build user requirement and adding automatic Launchpad login; it also fixed public-upload handling and cleaned up LaunchpadClient. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2825 removed remote-build's user requirement, added automatic Launchpad login, fixed AcceptPublicUploadError handling, and cleaned up LaunchpadClient. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged remote-build changes removed the need to specify a user, enabled automatic login, fixed AcceptPublicUploadError handling, and cleaned up LaunchpadClient. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in canonical/snapcraft#2825: remote-build no longer requires --user, logs in automatically, and includes related Launchpad client fixes. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged after two approvals and CI. Removes remote build user specification, adds automatic login on init, fixes AcceptPublicUploadError, and deletes unused _waiting from LaunchpadClient across five files. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged remote-build changes removing the user specification requirement, enabling automatic login on init, fixing the AcceptPublicUploadError option, and removing the unused _waiting attribute from LaunchpadClient, resolving bug #1852482. |
Update history
No update history recorded yet.
Related work
-
Related To:
launchpad#1852482
(confidence 90%)
The PR credits this bug with motivating removal of the explicit remote-build user requirement.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #224 fix(remotebuild): create the launchpad instance correctly | craft-application | merged | Merged PR #224 as commit 87bc56a7, fixing remotebuild to create the Launchpad instance correctly. | |
| #2780 remote-build: option renames for arch, user, and accept-public-upload | snapcraft | merged | Merged #2780 renamed remote-build --arch to --build-on and Launchpad-prefixed the user and public-upload options (commits 6ef1767ba, b8e7121e2). | |
| #2690 remote-build: error when --user is required | snapcraft | merged | Merged PR #2690 (04a0d9331), restoring fail-fast remote-build validation when neither --user nor a configured username is available. | |
| #4481 Clean Launchpad repositories when remote-build errors are raised | snapcraft | closed | Launchpad cleanup for remote-build errors was implemented in commit 663dcdfab; the author confirmed the request was completed. | |
| #3021 remote-build: remove artifact sanity check | snapcraft | merged | Merged PR #3021 (commit e4a878561) removed remote-build's artifact arch sanity check, so Launchpad results with arch 'multi' from build-on are downloaded instead of rejected. Fixes LP bug #1870730. | |
| #1827679 Update remote build login when Launchpad infrastructure is ready | snapcraft (launchpad) | closed | Remote builds now authenticate with Launchpad personal access tokens instead of SSH keys via snapcraft#4270 (commit 7c80a055b). | |
| #1185 docs: rework remote build guide | craft-application | closed | Merged in commit 4c49d82b (#1185), recommending Launchpad’s web UI by default and simplifying the remote-build guide. | |
| #4270 remote-build: Use personal access tokens | snapcraft | merged | Merged PR #4270 as 7c80a055b, replacing deprecated Launchpad bearer macaroons with scoped, revocable, short-lived personal access tokens for remote builds. |