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: move to self-hosted action runner #618

Merged
merged 2 commits into from
Jan 2, 2024

Conversation

bk201
Copy link
Member

@bk201 bk201 commented Dec 28, 2023

While Jenkins instance worked in the past, I propose we move the vagrant installation CI to GitHub self-hosted action runner for

  • Eliminate the need for credentials to log in to view CI logs (good for community users and us)
  • Re-running a failed job updates the status back to PR. Not sure why Jenkins never works.
  • Easier IaC in the source. No separate repo is needed.

Note: The drone CI part doesn't change. We still rely on it to build ISO and publish to the release server.

@bk201 bk201 force-pushed the action-runner branch 4 times, most recently from 4eceffe to db482f2 Compare December 28, 2023 09:36
@bk201 bk201 changed the title ci: move to self-hosted action runner ci: move to self-hosted action runner [CI SKIP] Dec 28, 2023
@bk201 bk201 force-pushed the action-runner branch 7 times, most recently from fd78fa8 to 68b4474 Compare December 28, 2023 13:12
@bk201 bk201 changed the title ci: move to self-hosted action runner [CI SKIP] ci: move to self-hosted action runner Jan 2, 2024
@bk201 bk201 marked this pull request as ready for review January 2, 2024 01:03
@bk201 bk201 force-pushed the action-runner branch 2 times, most recently from 68b4474 to 0b24e29 Compare January 2, 2024 01:22
Signed-off-by: Kiefer Chang <kiefer.chang@suse.com>
Signed-off-by: Kiefer Chang <kiefer.chang@suse.com>
Copy link
Contributor

@tserong tserong left a comment

Choose a reason for hiding this comment

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

LGTM. Also, it seems to work, given it says "all checks passed" :-)

Copy link
Contributor

@Vicente-Cheng Vicente-Cheng left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for this improvement!

@Vicente-Cheng Vicente-Cheng merged commit c7f6b5d into harvester:master Jan 2, 2024
5 checks passed
@bk201
Copy link
Member Author

bk201 commented Jan 2, 2024

@Mergifyio backport v1.2, v1.1

Copy link

mergify bot commented Jan 2, 2024

backport v1.2, v1.1

❌ No backport have been created

  • Backport to branch v1.2, failed

GitHub error: Branch not found

@bk201
Copy link
Member Author

bk201 commented Jan 2, 2024

@Mergifyio backport v1.2

Copy link

mergify bot commented Jan 2, 2024

backport v1.2

🟠 Pending

  • Backport to branch v1.2 in progress

@bk201
Copy link
Member Author

bk201 commented Jan 2, 2024

@Mergifyio backport v1.2

Copy link

mergify bot commented Jan 2, 2024

backport v1.2

✅ Backports have been created

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