← Back to issue list

feat(remote-build): check SNAPCRAFT_REMOTE_BUILD_STRATEGY envvar

View original Github issue

Metadata

Project
snapcraft
Number
#4353
Type
pull request
State
merged
Author
mr-cal
Labels
Created
Updated
Closed

Current evaluation

Merged implementation of SNAPCRAFT_REMOTE_BUILD_STRATEGY environment variable to control remote build fallback for core22 and older bases. Approved by two reviewers, passed CI, and achieved 100% diff coverage. Resolves #4319.

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] Have you successfully run `make lint`? - [x] Have you successfully run `pytest tests/unit`? ----- Valid values for the environment variable are: - `disable-fallback`: uses new remote-build code - `force-fallback`: uses legacy remote-build code This envar is used for bases core22 and older. Bases newer than core22 use the new remote-build code. Fixes #4319 (CRAFT-1974)

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged implementation of SNAPCRAFT_REMOTE_BUILD_STRATEGY environment variable to control remote build fallback for core22 and older bases. Approved by two reviewers, passed CI, and achieved 100% diff coverage. Resolves #4319.
qwen3.6-35b-a3b-mtp-q6 Merged into main. Adds SNAPCRAFT_REMOTE_BUILD_STRATEGY environment variable to toggle remote build strategy for core22 and older bases. Resolves #4319 with 100% test coverage.
qwen3.6-35b-a3b-mtp-q6 Merged into main. Introduces the SNAPCRAFT_REMOTE_BUILD_STRATEGY environment variable to toggle between legacy and new remote-build code for core22 and older bases. Achieved 100% diff coverage and resolves #4319.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#4319 remote-build: add control logic for envvar `SNAPCRAFT_REMOTE_BUILD_STRATEGY` snapcraft closed Implemented control logic for SNAPCRAFT_REMOTE_BUILD_STRATEGY to toggle between new and legacy remote-build execution. The feature was merged and the issue closed as completed.
81%
#4498 fix(remote): fallback to legacy snapcraft if under shallow git clone snapcraft merged Merged a fix falling back to legacy remote builds for shallow git clones, which cannot be pushed by design. Supports only core20 and core22 bases. Approved by two reviewers, passed CI, and modified 8 files.
73%
#4343 feat(remote-build): check base before executing snapcraft merged Merged after reviewer approval and passing CI. Adds base validation before remote-build execution, raising errors for unsupported bases and enabling fallback for core20/core22. Resolves #4318.
73%
#1128 feat(remotebuild): add build-path to snap recipes craft-application merged Merged. Adds build-path to snap recipes and RemoteBuildService, resolving #1106 and unblocking snapcraft #6287. Exposed as --project-dir in the remote-build command. Approved by two reviewers and passed CI.
72%
#4895 fix(remotebuild): require core20 snaps to use the legacy remote builder snapcraft merged Merged. Requires core20 snaps to use the legacy remote builder since the new builder cannot parse core20 snapcraft.yaml files. Resolves #4886. Approved and passed CI.
72%
#196 feat: remote build service craft-application merged Merged the remote build service feature, migrating code from snapcraft. Approved by two reviewers, passed CI checks, and introduced 2316 lines across 26 files.
71%
#4395 feat(remote-build): new command logic snapcraft merged Merged into main. Introduces a RemoteBuild class to orchestrate remote builds, adds local project directory caching, and implements post-build cache cleanup. Resolves #4324. Approved by three reviewers with passing CI and 97.88% diff coverage.
71%
#2720 remote build: switch from core to core18 snapcraft merged Merged change switches remote build base from core to core18, aligning with snapcraft edge. Approved, passed CI, and modified one file.
71%
#3821 meta: architectures snapcraft merged Merged adding build-on and build-for architecture support for core22 snaps. Introduced the SNAPCRAFT_BUILD_FOR environment variable to limit builds. Temporarily drafted to fix a duplicate build regression before final merge.
71%
#4724 feat(remote)!: use craft-application remote build for core22 snapcraft merged Replaced deprecated remote build code with craft-application for core22. Core20 support enabled via environment variable. The --build-on flag was removed. Fixes #4710. Merged after approval and passing CI checks.
70%