← 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 PR #4353 (commit 2eddbfe6c) added handling of SNAPCRAFT_REMOTE_BUILD_STRATEGY (disable-fallback/force-fallback) for core22-and-older bases, closing issue #4319 (CRAFT-1974).

Resolution: closed resolved

Reason: The PR itself was merged into main as commit 2eddbfe6c and implements the envvar control logic requested in issue #4319; the legacy fallback remained available via force-fallback until the legacy remote builder was later removed in #5994.

Evaluated by: mr-cal-bot

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
stealth/space-bunny-alpha — closed resolved Merged PR #4353 (commit 2eddbfe6c) added handling of SNAPCRAFT_REMOTE_BUILD_STRATEGY (disable-fallback/force-fallback) for core22-and-older bases, closing issue #4319 (CRAFT-1974).
stealth/space-bunny-alpha — closed resolved Merged #4353 added SNAPCRAFT_REMOTE_BUILD_STRATEGY with disable-fallback and force-fallback modes for core22 and older, fixing #4319 (CRAFT-1974).
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.

Update history

No update history recorded yet.

Related work

  • Related To: snapcraft#4319 (confidence 98%)

    Issue explicitly fixed by this PR (CRAFT-1974), closed alongside it

  • Related To: snapcraft#5994 (confidence 70%)

    Later commit b08f1a66e removed the legacy remote builder, making the force-fallback path obsolete in Snapcraft 9.0

Related issues

Issue Project State Summary Similarity
#4319 remote-build: add control logic for envvar `SNAPCRAFT_REMOTE_BUILD_STRATEGY` snapcraft closed Implemented via merged PR #4353 (commit 2eddbfe6c, CRAFT-1974), adding SNAPCRAFT_REMOTE_BUILD_STRATEGY handling for disable-fallback/force-fallback; legacy builder later removed in #4724 (Snapcraft 9.0).
86%
#4724 feat(remote)!: use craft-application remote build for core22 snapcraft merged Merged PR #4724 (9905f2831) switched core22 remote builds to craft-application, added a core20 disable-fallback opt-in, removed --build-on, and fixed #4710.
74%
#4318 remote-build: add control logic for snap base snapcraft closed Base-aware remote-build routing was merged in PR #4343 (commit 128e0acb2), using legacy code for core20/core22 and the new path for devel/core24 and newer.
72%
#4343 feat(remote-build): check base before executing snapcraft merged Merged into main as commit 128e0acb2. Added base validation before remote-build, allowing fallback for core20/core22 while rejecting unsupported bases and fixing #4318.
72%
#5676 remote build: remove legacy remote builder snapcraft closed Fixed in merged PR #5994 (commit b08f1a66e) for Snapcraft 9.0 by removing the legacy remote builder, selection logic, strategy env var, and legacy credential support.
71%
#4498 fix(remote): fallback to legacy snapcraft if under shallow git clone snapcraft merged Merged as 1e285148e: remote builds under shallow Git clones fall back to legacy Snapcraft on core20/core22; unsupported bases and devel report an error.
71%
#4653 feat(craft-application): support remote build snapcraft closed Closed unmerged; superseded by canonical/snapcraft#4664 (commit 57d184af7), which landed the same craft-application remote build support for core24.
70%
#5239 refactor: use craft-application's remote-build snapcraft merged Merged PR #5239 as f31d99498, replacing Snapcraft’s legacy remote-build path with craft-application’s command for CRAFT-4044.
70%