remote-build: configurable timeout/deadline for starting and monitoring build
Metadata
Current evaluation
Merged configurable timeout and deadline handling for remote build start and monitoring. Review feedback was applied before approval, resolving ambiguous timeout errors and reinstating the --launchpad-timeout flag.
Suggested action: —
No scores available.
Issue body
The error was ambiguous when the timeout was hit, and there was no recourse for the user.
Re-instate a configurable timeout as `--launchpad-timeout` but apply it to both the start_build and monitor_build as a deadline. When Launchpad is slow, it may take some time to process the request, but as long as its Pending, we keep waiting for a response (unless deadline is hit).
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged configurable timeout and deadline handling for remote build start and monitoring. Review feedback was applied before approval, resolving ambiguous timeout errors and reinstating the --launchpad-timeout flag. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged cjp256's PR adding a configurable --launchpad-timeout deadline for remote builds. Applies to starting and monitoring, resolving ambiguous errors and handling slow Launchpad responses by waiting while pending. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Adds a configurable --launchpad-timeout deadline for starting and monitoring remote builds, resolving ambiguous timeout errors and handling slow Launchpad responses. Finalized after author addressed review comments and minor issues. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #4426 fix(remote-build): parse `--launchpad-timeout` argument | snapcraft | merged | Merged fix for parsing the --launchpad-timeout argument. Resolves a regression in the fallback remote builder and adds support for the new remote builder. Approved, passed CI, and merged to main. Fixes #4397. | |
| #4427 fix(remote-build): parse `--launchpad-timeout` argument | snapcraft | merged | Merged fix for a regression where the --launchpad-timeout argument was not parsed by the fallback remote builder. Resolves #4397 and was merged into the hotfix/7.5 branch. |