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.

Sign up for GitHub

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

Jump to bottom

Fix runner issue #949

Merged
merged 2 commits into from
Mar 5, 2024
Merged

Fix runner issue #949

merged 2 commits into from
Mar 5, 2024

Conversation

JooyoungPark73
Copy link
Contributor

Fix #947 and other runner related issues

  • Remove deprecated node setup script written in bash and migrate to go script
  • Fix integration tests not working due to some missing packages and log name collision
  • Make gVisor cri test to use proper yaml file: stub does not work with gVisor
  • Let runner use specific branch to test runner upgrades

@JooyoungPark73 JooyoungPark73 marked this pull request as draft March 5, 2024 07:22
@JooyoungPark73 JooyoungPark73 marked this pull request as ready for review March 5, 2024 07:28
@JooyoungPark73
fix gh runner gvisor cri test
f1e6bb2 
Signed-off-by: JooYoung Park <[email protected]>
@JooyoungPark73 JooyoungPark73 requested a review from lrq619 March 5, 2024 09:15
@JooyoungPark73 JooyoungPark73 merged commit c68a514 into main Mar 5, 2024
23 checks passed
@JooyoungPark73 JooyoungPark73 deleted the fix-runner-issue branch March 5, 2024 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lrq619 lrq619 lrq619 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

gVisor not working on branch bump-k8s
2 participants
@JooyoungPark73 @lrq619