-
Notifications
You must be signed in to change notification settings - Fork 80
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
tests: Add tmt test for bootc install with LBI #792
Conversation
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.
Looks nice to me! Thanks for working on this, I'm excited to set a better testing foundation for the project!
xtask/src/xtask.rs
Outdated
// pull some small images that are used for LBI installation tests | ||
cmd!( | ||
sh, | ||
"podman pull quay.io/curl/curl-base:latest quay.io/curl/curl:latest" |
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.
Minor: probably worth factoring out a const TEST_IMAGES: &[&str] = ...
at the top of the file for increased visibility?
This one actually also relates to https://gitlab.com/fedora/bootc/examples/-/merge_requests/50#note_2041291332
which hmmm, I need to finish that PR
@@ -0,0 +1,3 @@ | |||
[Image] | |||
Image=registry.redhat.io/jboss-webserver-5/jws5-rhel8-operator:latest | |||
AuthFile=/root/auth.json |
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.
Is this actually used?
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.
nope, added a comment to explain that it's meant to represent a quadlet image that is not logically bound.
c74f64d
to
6ccf4c8
Compare
This just needs a |
Along with the test, this will install the LBIs on the machine running the tests prior to running the tests. Also refactors the existing LBI test into a separate test plan to make room for the install tests. Signed-off-by: Chris Kyrouac <[email protected]>
6ccf4c8
to
0f09ae2
Compare
Along with the test, this will install the LBIs on the machine running the tests prior to running the tests. Also refactors the existing LBI test into a separate test plan to make room for the install tests.