-
Notifications
You must be signed in to change notification settings - Fork 124
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
test: small tweaks to cri-o test process #151
Conversation
Signed-off-by: Peter Hunt <pehunt@redhat.com>
@@ -232,6 +230,7 @@ install_testing_deps() { | |||
echo "Installing latest upstream version of BATS" | |||
ooe.sh git clone https://github.com/bats-core/bats-core.git /tmp/bats | |||
cd /tmp/bats | |||
echo "Testing against bats commit: $(git rev-parse HEAD)" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if you're keeping this long term, you might want to tweak the space/tab issue here.
Signed-off-by: Peter Hunt <pehunt@redhat.com>
@@ -76,6 +76,7 @@ ooe.sh sudo dnf -y install \ | |||
openssl \ | |||
openssl-devel \ | |||
ostree-devel \ | |||
parallel \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
spacing/tab?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😃
Signed-off-by: Peter Hunt <pehunt@redhat.com>
@@ -22,7 +22,7 @@ install_ooe | |||
echo "Updating packages" | |||
ooe.sh sudo dnf update -y | |||
|
|||
ooe.sh sudo dnf -y module install cri-o:1.13 | |||
ooe.sh sudo dnf -y module install cri-o:1.17 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we already want to use 1.18 ❓
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
baby steps 😆 this PR doesn't even work properly right now lol
(really I want to debug why we're seeing
Error: Duplicate test name(s) in file "/var/tmp/go/src/github.com/cri-o/cri-o/test/./ctr_seccomp.bats": test_ctr_seccomp_profiles_runtime-2fdefault
and also not failing because we don't have parallel installed)Signed-off-by: Peter Hunt pehunt@redhat.com