-
Notifications
You must be signed in to change notification settings - Fork 71
tests/e2e: allow to superseed images in operator.sh #341
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
base: main
Are you sure you want to change the base?
tests/e2e: allow to superseed images in operator.sh #341
Conversation
`./tests/e2e/operator.sh install` is still tightly coupled with the build phase, which happen to push images to a local registry, so the install tries to always start the registry service. Now suppose that you want to run `./tests/e2e/operator.sh install` stand-alone to install images from quay.io, then the registry service locally is not needed. Signed-off-by: Wainer dos Santos Moschetta <[email protected]>
By exporting $KATA_PAYLOAD_IMG variable the `./tests/e2e/operator.sh install` will replace the default kata-payload image. Signed-off-by: Wainer dos Santos Moschetta <[email protected]>
stevenhorsman
left a comment
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.
LGTM, Thanks Wainer
|
lgtm |
| "install": install only, | ||
| "wait_for_stabilization": wait for CoCo pods to be stable | ||
| "uninstall": uninstall the operator. | ||
| environment variables : |
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.
I really like this section, could you please (in a separate commit) add the RUNTIMECLASS, which is already supported to be set?
ldoktor
left a comment
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.
Overall it looks good, only the already-supported-to-be-set variable is missing in the usage message. Could you please add it (Ideally as a first commit to introduce this section, alternatively just as a part of the current first commit which defines the section)
|
lgtm |
beraldoleal
left a comment
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.
Except from the @ldoktor comments, LGTM.
I started working on adapting the Kata CI scripts to install from this operator. I will be trying to re-use the scripts in
tests/e2e/as much as possible, for example,tests/e2e/operator.sh installto install and check Kata Containers in the cluster's nodes. So far I faced two problems withoperator.shthough:operator.shuses the built-and-pushed-to-local-registry images so it always tries to start a local registry service. If it is installing from quay.io (or ghcr.io), for example, it makes no sense the local registry.So this PR address those two problems. Also by making
operator.shmore flexible, this will likely to solve issues when we start improving the operator CI pipeline as suggested in #309If that helps on review, here goes a sample of the code I'm writing for the Kata CI side:
I tested locally by running: