← Back to issue list

remote build: add option to skip public upload question

View original Github issue

Metadata

Project
snapcraft
Number
#2590
Type
pull request
State
merged
Author
cmatsuoka
Labels
Created
Updated
Closed

Current evaluation

Merged. Introduces a flag to skip the public upload confirmation prompt during remote builds, supporting non-interactive environments. Approved by two reviewers, passed CI, and includes tty-aware prompts with error handling.

Suggested action:

No scores available.

Issue body

Add an option to prevent user interaction when running on a noninteractive environment while still acknowledging that uploaded code will be public. Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com> - [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 `./runtests.sh static`? - [x] Have you successfully run `./runtests.sh tests/unit`? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged. Introduces a flag to skip the public upload confirmation prompt during remote builds, supporting non-interactive environments. Approved by two reviewers, passed CI, and includes tty-aware prompts with error handling.
qwen3.6-35b-a3b-mtp-q6 Merged an update adding a flag to skip the public upload confirmation prompt during remote builds, enabling non-interactive environments. Includes tty-aware prompt handling and improved error messaging for failed checks.
qwen3.6-35b-a3b-mtp-q6 Merged into remote-build. Adds a flag to skip the public upload confirmation prompt, enabling non-interactive environments. Includes tty-aware prompt adjustments and explicit error messaging for validation failures.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2567 remote build: add warning before sending data snapcraft merged Merged a change adding a warning prompt before sending data to remote builders, alerting users to public availability and requesting confirmation. Approved by one reviewer, passed CI, and integrated with a six-line addition.
73%
#2825 remote-build: remove need to specify user snapcraft merged 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.
73%
#3553 cli: make remote-build more user-friendly snapcraft merged Merged improvements to remote-build CLI. Aligns --recover with --status, suppresses unnecessary public data prompts when checking status, and adds tests. Approved and merged.
72%
#2798 cli: improve the remote-build upload messaging snapcraft merged Merged changes improving CLI remote-build upload messaging. The update replaces public with publicly available to clarify artifact visibility. Approved by reviewer, passed CI, and integrated with minimal code changes.
72%
#2837 remote-build: gpg-signing and usability fixes snapcraft merged Merged changes disabling GPG signing for git commits to fix remote-build failures. Improves error messaging and commit formatting. Approved by reviewer, passed CI, and integrated into the codebase.
70%
#2642 remote-build: detect early build errors snapcraft merged Merged. Improved remote build error detection to distinguish between pending builders and actual failures like missing config or timeouts. Approved by reviewer, passed CI, and modified 3 files (+131/-47 lines). Minor import fix applied.
70%