← Back to issue list

docker: add the architecture

View original Github issue

Metadata

Project
snapcraft
Number
#1994
Type
pull request
State
merged
Author
sergiusens
Labels
Created
Updated
Closed

Current evaluation

Configured SNAP_ARCH for Docker images and added explicit failure if missing. Merged following successful CI checks and two reviewer approvals.

Suggested action:

No scores available.

Issue body

Setup the correct SNAP_ARCH for our docker images, also blow up in a more obvious way if it is missing. Signed-off-by: Sergio Schvezov <sergio.schvezov@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] 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 unit`? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Configured SNAP_ARCH for Docker images and added explicit failure if missing. Merged following successful CI checks and two reviewer approvals.
qwen3.6-35b-a3b-mtp-q6 Configures SNAP_ARCH for Docker images and adds explicit error handling when the architecture is missing. Merged after maintainer approval.
qwen3.6-35b-a3b-mtp-q6 Merged. Configured SNAP_ARCH for Docker images and added explicit failure handling if the architecture is missing, ensuring reliable container builds.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#3210 docker: install snapd dependency snapcraft merged Merged. Adds snapd installation to Docker images to satisfy snapcraft's new snap pack requirement. Approved by two reviewers, passed CI, and modified five files.
72%
#2014 integration tests: snap tests shouldn't be arch-specific snapcraft merged Merged a change removing architecture-specific constraints from snap integration tests. Approved by one reviewer with a two-line modification across a single file.
71%
#3987 ci: remove docker before building snap snapcraft merged Merged into main after reviewer approval and passing CI checks. The change modifies four files to remove Docker before building the snap, with no impact on overall code coverage.
71%