Skip to content

feat: add weekly snap promotions - #492

Merged
lucabello merged 1 commit into
mainfrom
feat/promote-snaps
Aug 26, 2026
Merged

feat: add weekly snap promotions#492
lucabello merged 1 commit into
mainfrom
feat/promote-snaps

Conversation

@lucabello

Copy link
Copy Markdown
Contributor

This PR does two things:

  • add a promotion mechanism for snaps from /edge to /stable on all their available tracks, to be run automatically on a weekly basis;
  • improve the promote-charm-train workflow and justfile recipe to read from manifest.yaml instead of using a hardcoded list of charms.

@lucabello
lucabello requested a review from a team as a code owner August 25, 2026 10:11
- name: Promote snap
env:
SNAPCRAFT_STORE_CREDENTIALS: ${{ secrets.SNAPCRAFT_STORE_CREDENTIALS }}
run: just promote-snap-train ${{ matrix.snap }}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems like a blind promotion, not a "low pass filter" of one week delay. Was this intentional?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like the plan is to wait a week after upstream releases to avoid promoting "quickly-fixed bugs" to /stable. Ideally the workflow would track all new releases to /edge, and whenever one has been released for over a week, we would promote that to /stable.

There are some corner cases here:

  • If we get a new version every week (though unlikely) we wouldn't be promoting to /stable anymore (maybe it'd be a sign the project itself is not stable?)

@lucabello
lucabello merged commit acd32be into main Aug 26, 2026
1 check passed
@lucabello
lucabello deleted the feat/promote-snaps branch August 26, 2026 09:32
@lucabello

Copy link
Copy Markdown
Contributor Author

Merging this so we at least have a baseline, as currently nothing is promoted to /stable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants