← Back to issue list

integrations: add the snapcraft Dockerfile

View original Github issue

Metadata

Project
snapcraft
Number
#1370
Type
pull request
State
merged
Author
come-maiz
Labels
Created
Updated
Closed

Current evaluation

Merged the snapcraft Dockerfile to centralize its source and add a required environment variable for snapcraft 2.31. Approved by one reviewer and passed CI checks.

Suggested action:

No scores available.

Issue body

LP: #[1699218](https://bugs.launchpad.net/snapcraft/+bug/1699218) The source for our snapcore/snapcraft docker image is in a location that probably only @sergiusens knows. It should be in this repo, so we can all find it and touch it. In particular, the `ENV` entry added in this file is required by snapcraft 2.31, which has just landed into the archive. So we need to hurry and update the image using this as the source. I've asked @niemeyer to give me access to update the image: https://forum.snapcraft.io/t/in-progress-snapcraft-2-31/682/9

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged the snapcraft Dockerfile to centralize its source and add a required environment variable for snapcraft 2.31. Approved by one reviewer and passed CI checks.
qwen3.6-35b-a3b-mtp-q6 Merged a Dockerfile for the snapcraft image to centralize its source and add a required ENV variable for snapcraft 2.31. The change was integrated into the repository to streamline image maintenance and updates.
qwen3.6-35b-a3b-mtp-q6 Merged the snapcraft Dockerfile to centralize the image source and add an environment variable required for snapcraft 2.31. The file was adapted from the ubuntu/docker-snapcraft repository.

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.
74%
#2245 docker: support for testing snapcraft in proposed snapcraft merged Merged additions for a xenial-proposed snapcraft test Dockerfile and cleanup of the standard Dockerfile. Changes received approval, passed CI checks, and were validated through successful container execution.
74%
#2353 tests: remove dependency on snapcraft for integration tests snapcraft merged Merged after reviewer approval and passing CI. The change removes the snapcraft dependency from integration tests to improve test independence, modifying 27 files with 956 additions and 733 deletions.
73%
#4607 fix(application): support snap/craft environment variables snapcraft merged Merged after three approvals and passing CI. The PR adds snap/craft environment variable support across two files. Initial spread test failures and merge conflicts from a concurrent PR were resolved before final integration.
72%
#1625 tests: use the snapcraft snap for integration tests snapcraft merged Merged after two approvals. The PR updates integration tests to run against the installed snapcraft snap, aligning with user environments. Despite a reported Go runtime address space conflict in CI, the changes were accepted and merged.
71%
#2883 docker: add core18 snap that snapcraft now uses as a base snapcraft merged Merged the addition of the core18 snap as snapcraft's Docker base, updating related comments for accuracy. Approved by one reviewer, the change was integrated after a minor authorship clarification.
71%
#1576 docker: add the environment variable to setup core snapcraft merged Merged a one-line Dockerfile update adding the SNAPCRAFT_SETUP_CORE environment variable to enable straightforward snapcraft builds with classic confinement. The change was approved, passed CI checks, and successfully integrated.
70%