-
Notifications
You must be signed in to change notification settings - Fork 1.4k
[ST] Specify correct registry, org, and tag for Helm and remove unused code for OCP #10955
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
[ST] Specify correct registry, org, and tag for Helm and remove unused code for OCP #10955
Conversation
…d code for OCP Signed-off-by: Lukas Kral <[email protected]>
/azp run helm-acceptance |
Azure Pipelines successfully started running 1 pipeline(s). |
@strimzi-ci run tests --profile=acceptance --install-type=bundle --cluster-version=4.17 --cluster-type=ocp |
|
❌ Test Summary ❌TEST_PROFILE: acceptance ❗ Test Failures ❗
Re-run command: |
@strimzi-ci run tests --profile=acceptance --install-type=olm --cluster-version=4.17 --cluster-type=ocp |
|
❌ Test Summary ❌TEST_PROFILE: acceptance ❗ Test Failures ❗
Re-run command: |
@strimzi-ci run tests --profile=acceptance --install-type=olm --cluster-version=4.17 --cluster-type=ocp |
|
❌ Test Summary ❌TEST_PROFILE: acceptance ❗ Test Failures ❗
Re-run command: |
|
✔️ Test Summary ✔️TEST_PROFILE: acceptance |
Type of change
Description
In #10947 I changed the default values for the image registry, org, and tag values -> from our specific values to the empty one. Together with that, the check for changing the image parts was changed as well.
Now, this works for the K8s clusters and regular installation type, but it doesn't work for Helm.
Also, I'm removing unused code for adding the image-puller role, as it was not used for a long time in our tests on OCP -> and now it caused issues with running the tests, when the
STRIMZI_ORG
was empty.Checklist