Skip to content

chore: promote snap to candidate on release#94

Merged
jnsgruk merged 2 commits intocanonical:mainfrom
rebornplusplus:ROCKS-747/promote-snap
Aug 22, 2023
Merged

chore: promote snap to candidate on release#94
jnsgruk merged 2 commits intocanonical:mainfrom
rebornplusplus:ROCKS-747/promote-snap

Conversation

@rebornplusplus
Copy link

@rebornplusplus rebornplusplus commented Aug 21, 2023

Promote snap to latest/candidate channels upon every GitHub release.

Successfully executing this workflow job would require adding a SNAPCRAFT_STORE_CREDENTIALS secret in this repo. Please let me or @cjdcordeiro know if you don't have access to it (it's inside the credentials for the bot account).

Similar to canonical/pebble#255.

Demo run: https://github.com/rebornplusplus/chisel/actions/runs/5925313450
(not exactly the same behaviour as the demo promoted to latest/beta and ran the workflow job on push)


  • Have you signed the CLA?

Promote snap to latest/candidate channels upon every GitHub release.
@rebornplusplus rebornplusplus force-pushed the ROCKS-747/promote-snap branch from bdab19f to e873286 Compare August 21, 2023 11:21
Copy link
Collaborator

@cjdcordeiro cjdcordeiro left a comment

Choose a reason for hiding this comment

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

Cool, thanks.

Seems to follow the same approach as https://github.com/canonical/pebble/pull/255/files, so it LGTM

As for the new secret, either @jnsgruk or @niemeyer need to add it.

@cjdcordeiro cjdcordeiro requested a review from jnsgruk August 21, 2023 11:23
Double-quotes are not okay in a Github workflow expressions. Change back
to single-quotes.
Copy link
Member

@jnsgruk jnsgruk left a comment

Choose a reason for hiding this comment

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

LGTM. @niemeyer will need to add the creds as I don't have access.

@jnsgruk
Copy link
Member

jnsgruk commented Aug 22, 2023

Okay, that's done. I'll merge this now. Thanks all!

@jnsgruk jnsgruk merged commit 40807a1 into canonical:main Aug 22, 2023
rebornplusplus pushed a commit to rebornplusplus/chisel that referenced this pull request Aug 25, 2023
In canonical#94, the workflow for promoting snaps from latest/edge to
latest/candidate upon GitHub release, was introduced. We didn't have
riscv64 builds for the snap back then. We have it now. [1]

This commit extends the promotion workflow for the riscv64 architecture.

References:
- [1] https://answers.launchpad.net/launchpad/+question/707699
rebornplusplus pushed a commit to rebornplusplus/chisel that referenced this pull request Aug 25, 2023
In canonical#94, the workflow for promoting snaps from latest/edge to
latest/candidate upon GitHub release was introduced. We didn't have
riscv64 builds for the snap back then. We have it now. [1]

This commit extends the promotion workflow for the riscv64 architecture.

References:
- [1] https://answers.launchpad.net/launchpad/+question/707699
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.

3 participants