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

e2e: fixes for openSUSE #1111

Merged
merged 1 commit into from
Sep 6, 2024
Merged

e2e: fixes for openSUSE #1111

merged 1 commit into from
Sep 6, 2024

Conversation

marquiz
Copy link
Contributor

@marquiz marquiz commented Sep 6, 2024

  • distro's crictl is too old to support cri v1 -> install it from tarball
  • vm-install-containernetworking failed so install cni-plugins directly from the distro.
  • install container runtime(s) from the distro repo (as they're new enough there)
  • remove failing snapd hack.

@marquiz
e2e: fixes for openSUSE
86a75cc 
- distro's crictl is too old to support cri v1 -> install it from
  tarball
- vm-install-containernetworking failed so install cni-plugins directly
  from the distro.
- install container runtime(s) from the distro repo (as they're new
  enough there)
- remove failing snapd hack.
klihub

This comment was marked as resolved.

Sign in to view

klihub
klihub approved these changes Sep 6, 2024
View reviewed changes
Copy link
Contributor

@klihub klihub left a comment

Choose a reason for hiding this comment

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

LGTM.

@klihub
Copy link
Contributor

klihub commented Sep 6, 2024

BTW: it is this commit that breaks govm for us.

@marquiz
Copy link
Contributor Author

marquiz commented Sep 6, 2024

BTW: it is this commit that breaks govm for us.

K, I didn't investigate that far. Is there an easy fix for this (on our side)?

@klihub klihub merged commit ba771ad into intel:master Sep 6, 2024
6 checks passed
@marquiz marquiz deleted the devel/opensuse-fixes branch September 6, 2024 11:43
@marquiz marquiz mentioned this pull request Sep 6, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@klihub klihub klihub approved these changes

@kad kad Awaiting requested review from kad kad is a code owner

@mythi mythi Awaiting requested review from mythi mythi is a code owner

@askervin askervin Awaiting requested review from askervin askervin is a code owner

@fmuyassarov fmuyassarov Awaiting requested review from fmuyassarov fmuyassarov is a code owner

Assignees

@askervin askervin

@klihub klihub

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@marquiz @klihub @askervin