build providers: shell in provider if debug is used
Metadata
Current evaluation
Merged changes enabling shell access in build providers during debug sessions. Approved by a reviewer, passed CI, and updated five files with 112 additions and 21 deletions.
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] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)?
- [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [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 changes enabling shell access in build providers during debug sessions. Approved by a reviewer, passed CI, and updated five files with 112 additions and 21 deletions. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Implements shell access in build providers when debug mode is enabled. The maintainer merged the changes despite Codecov reporting 0% diff coverage, enabling better debugging workflows. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a maintainer pull request enabling an interactive shell in the build provider when debug mode is active. The change was integrated into the main branch despite a Codecov report noting zero diff coverage. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2249 build providers: provide support to shell in | snapcraft | merged | Merged. Adds shell access support to build providers, resolving LP #1782775. Approved by reviewer, passed CI checks, and implemented across 8 files. | |
| #2254 build providers: add support for --shell | snapcraft | merged | Merged after approval by two reviewers. Adds --shell support to build providers across three files. Passed all unit and static tests with 100% diff coverage. | |
| #2253 build-providers: add support for --shell-after | snapcraft | merged | Merged pull request adding --shell-after support to build-providers. Approved by one reviewer with 100% diff coverage. Changes span 3 files, adding 87 lines and removing 6. | |
| #1404 cleanbuild: ensure we enter a shell on failures on --debug | snapcraft | merged | Merged to ensure cleanbuild enters a shell on failures when --debug is enabled, regardless of flag positioning. Approved by two reviewers, updating four files with 65 additions and 37 deletions. | |
| #2275 build providers: use the provider if exported | snapcraft | merged | Merged an update to build provider logic that enables using a provider if it is exported. Approved by a reviewer, passed CI checks, and integrated into the codebase. | |
| #2474 build providers: remove dead code | snapcraft | merged | Merged removal of dead code for legacy cleanbuild support in build providers. Approved by one reviewer, passed CI, and modified three files (+54/-165 lines). | |
| #3718 parts: run debug shell on build environment | snapcraft | merged | Merged. Adds debug, shell, and shell-after flags to launch a build environment shell during parts errors or to replace lifecycle steps. Approved by reviewer, rebased, and passed CI before integration. | |
| #5537 docs: update build provider procedures | snapcraft | merged | Merged documentation update to build provider procedures, resolving issue #5519. Approved by two reviewers and passed most CI checks despite minor integration test failures. Changes span four files with 100 lines added and 98 removed. | |
| #2374 build providers: destroy on create failures | snapcraft | merged | Merged. Resolves a bug where build providers and VMs remained running after launch failures. Adds cleanup logic and tests across seven files, approved by a reviewer and passing CI checks. | |
| #2515 build providers: support for provider setup | snapcraft | merged | Merged after approval and passing CI. Implemented provider setup support by relocating checks and initialization logic into provider implementations. Modified ten files (+212/-121 lines) and resolves LP #1821586. |