-
Notifications
You must be signed in to change notification settings - Fork 41
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
OSSM-3778 Run against each supported SMCP version #490
Conversation
ea8c139
to
f663b46
Compare
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.
It would be also nice to update Readme. e.g. how to run only given control plane version and what's the default behavior.
Dockerfile
Outdated
&& curl -Lo ./oc.tar.gz https://mirror.openshift.com/pub/openshift-v4/clients/ocp/stable/openshift-client-linux.tar.gz \ | ||
&& tar -xf oc.tar.gz \ | ||
&& rm -f oc.tar.gz \ | ||
&& curl -Lo ./golang.tar.gz https://go.dev/dl/go1.16.15.linux-amd64.tar.gz \ | ||
&& curl -Lo ./golang.tar.gz https://go.dev/dl/go1.19.8.linux-amd64.tar.gz \ |
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.
Any reason not to use the latest stable version 1.20.3?
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.
fixed.
&& tar -xf golang.tar.gz -C / \ | ||
&& rm -f golang.tar.gz \ | ||
&& microdnf update \ | ||
&& microdnf clean all | ||
|
||
ENV GOROOT=/go | ||
ENV TEST_GROUP ${TEST_GROUP} | ||
ENV GOPATH=/root/go | ||
ENV GODEBUG "x509ignoreCN=0" |
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.
This might not be needed anymore, but I'm going to try this in my other PR
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.
yup, let's remove it in a follow-up PR if you see it's no longer needed.
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, but also we need to update Readme file with all the options that we have now to run against the differents smcp version and could be good also to add some documentation about running from the container
I'll revise the README and ask you for a review of just the readme again. |
No description provided.