← Back to issue list

integrations: use the snapcore/snapcraft docker image in travis

View original Github issue

Metadata

Project
snapcraft
Number
#1331
Type
pull request
State
merged
Author
filibtester
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #1331 as commit 6d777f770, switching generated Travis CI integrations from the Xenial container to the snapcore/snapcraft Docker image; existing projects were not auto-migrated.

Resolution: closed resolved

Reason: Approved by two reviewers and merged by come-maiz as commit 6d777f770, updating snapcraft/integrations/travis.py to use the snapcore/snapcraft image. Per cprov's comment, `enable-ci travis --refresh` still only updates store credentials, so old integrations kept the legacy image.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Thanks for helping us make a better Snapcraft! Checklist: - Have you signed the contributor licence agreement? https://www.ubuntu.com/legal/contributors - Is there a reported a bug for the problem you are fixing? https://bugs.launchpad.net/snapcraft/ If there is none, please report it. Once you have the BUG_NUMBER, replace it here: LP: [#BUG_NUMBER](https://bugs.launchpad.net/snapcraft/+bug/BUG_NUMBER) - Have you read our contribution guide? [CONTRIBUTING.md](CONTRIBUTING.md)

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #1331 as commit 6d777f770, switching generated Travis CI integrations from the Xenial container to the snapcore/snapcraft Docker image; existing projects were not auto-migrated.
stealth/space-bunny-alpha — closed resolved PR #1331 by filibtester merged as commit 6d777f770, switching the Travis CI integration template to the snapcore/snapcraft Docker image (+7/-9 across 3 files). Reviewers agreed existing integrations keep the old image, no migration.
stealth/space-bunny-alpha — closed resolved PR #1331 by filibtester was merged (commit 6d777f770), making snapcraft's `enable-ci travis` integration generate .travis.yml configs that use the snapcore/snapcraft Docker image instead of a bare xenial container.
stealth/space-bunny-alpha — closed resolved Merged PR #1331 as commit 6d777f770, switching Travis CI integrations to the snapcore/snapcraft Docker image. Existing integrations kept the old image without automated migration.
qwen/qwen3.6-35b-a3b — — Merged the Travis CI configuration update to use the snapcore/snapcraft Docker image. Reviewers approved, concluding that manual migration for existing projects is acceptable despite lacking an automated refresh command.
qwen3.6-35b-a3b-mtp-q6 — — Merged. Updates Travis CI configuration to use the snapcore/snapcraft Docker image. Reviewers approved the change despite it not updating existing integrations, deeming it safe and straightforward.

Update history

No update history recorded yet.

Related work

  • Superseded By: snapcraft#3164 (confidence 80%)

    Commit ccb612e5a 'cli: remove enable-ci command (#3164)' later removed the enable-ci integration code entirely.

  • Related To: snapcraft#5073 (confidence 60%)

    Commit 5e91afb6a removed Travis CI from the project entirely, making the integration obsolete.

  • Related To: snapcraft#597 (confidence 45%)

    Earlier PR #597 introduced the Xenial-based Travis execution this PR replaced with the snapcore/snapcraft image.

Related issues

Issue Project State Summary Similarity
#597 Use a xenial docker container for travis executions snapcraft merged Merged snapcraft#597 as ee28121ba, switching Travis executions to a Xenial Docker container; unit, integration, and snap tests passed.
77%
#1653685 clean up travis.yaml snapcraft (launchpad) closed Resolved by merged snapcore/snapcraft PR #1012 (commit b66bdebe9), which cleaned up Travis and changed container build phases to use docker exec.
74%
#2217 Travis test snapcraft merged Merged in snapcraft#2217 via commit 281defaef, improving Travis test speed and simplifying CI across 26 files.
72%
#1679441 the enable-ci travis command is not using the snapcraft docker image snapcraft (launchpad) closed Fixed by snapcore/snapcraft#1331, merged as commit 6d777f770; Travis enable-ci now uses the snapcore/snapcraft Docker image.
72%
#1644381 Add support for enabling ci systems (start with travis) snapcraft (launchpad) closed CI setup support was implemented in snapcore/snapcraft PR #901 via commit d7d550741, adding the initial enable-ci command for Travis CI.
70%
#1370 integrations: add the snapcraft Dockerfile snapcraft merged Merged #1370 in commit 778e4e234, adding the Snapcraft Dockerfile and ENV setting required by Snapcraft 2.31.
70%