-
Notifications
You must be signed in to change notification settings - Fork 104
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
Run a KUDO deployment for E2E testing #1243
Conversation
a7c8fde
to
265d9dc
Compare
A Docker image of the KUDO manager is built and deployed in the E2E test cluster. This is done by adding the build KUDO image to the KinD cluster and changing the 'kubctl kudo init' output to ensure that this image will be used when deploying KUDO.
265d9dc
to
ea6e801
Compare
|
||
./bin/kubectl-kudo init --dry-run --output yaml \ | ||
| sed -E -e "/image:/ s/:[[:alnum:]]+/:$VERSION/" -e '/imagePullPolicy/ s/Always/Never/' \ | ||
> test/manifests/kudo.yaml |
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.
Maybe we should have a file in test/manifests/kudo.yaml
with a note that the file will be overwritten all the time? Just to prevent that somebody tries to put a static config there?
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.
Good point! The tests currently creates artifacts all over the place and doesn't bother with cleaning up temporaries once they're done. This is something that need to be fixed at some point, but independently from this PR. How about having a separate manifests folder for these generated files?
Co-Authored-By: Andreas Neumann <aneumann@mesosphere.com>
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.
🚢
A Docker image of the KUDO manager is built and deployed in the E2E test cluster. This is done by adding the build KUDO image to the KinD cluster and changing the 'kubectl kudo init' output to ensure that this image will be used when deploying KUDO. Co-Authored-By: Andreas Neumann <aneumann@mesosphere.com> Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
What this PR does / why we need it:
A Docker image of the KUDO manager is built and deployed in the E2E test cluster.
This is done by adding the build KUDO image to the KinD cluster and changing the 'kubctl kudo init' output to ensure that this image will be used when deploying KUDO.
Fixes #1177