Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: Publish ISO images on branch push (v1.4) #809

Merged
merged 6 commits into from
Aug 14, 2024

Conversation

tserong
Copy link
Contributor

@tserong tserong commented Aug 14, 2024

Problem:
ISOs are only published on pushes to the main harvester repo. Sometimes we need an ISO built when there's only been a change in harvester-installer, for example when doing testing of the head of one of the stable branches, or of master.

Solution:
Duplicate the relevant ISO publishing bits from https://github.com/harvester/harvester/blob/master/.github/workflows/build.yml

Related Issue:
harvester/harvester#6296

Test plan:
Merge this PR and make sure an ISO is published to releases.rancher.com.

Related issue: harvester/harvester#6296

Signed-off-by: Tim Serong <tserong@suse.com>
(cherry picked from commit 1b3ea7c)
Fixes: 1b3ea7c
Signed-off-by: Tim Serong <tserong@suse.com>
(cherry picked from commit 43b0a1e)
Fixes: 1b3ea7c
Signed-off-by: Tim Serong <tserong@suse.com>
(cherry picked from commit e7de316)
Fixes: 1b3ea7c
Signed-off-by: Tim Serong <tserong@suse.com>
(cherry picked from commit c7b29b6)
Yet another subtle little inconsistency between this repo and the
main harvester repo :-/

Fixes: 1b3ea7c
Signed-off-by: Tim Serong <tserong@suse.com>
(cherry picked from commit d3cd1ad)
Fixes: e7de316
Signed-off-by: Tim Serong <tserong@suse.com>
(cherry picked from commit b34983e)
@tserong tserong requested a review from bk201 August 14, 2024 02:41
@bk201 bk201 merged commit fcd48af into harvester:v1.4 Aug 14, 2024
6 checks passed
@tserong tserong deleted the wip-backport-6296-v1.4 branch August 14, 2024 03:19
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