You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
While working on this PR #11032, which has changes on the operator but also need a new bridge (not released yet), I needed to run the system tests by using a custom bridge image.
For mostly all of them I needed to specify the bridge image within the .azure/templates/steps/system_test_general.yaml file by setting something like:
Usually, for the operator there is no need to make any changes, because the STs are getting the image from the PR build while for Helm Charts STs you need to tweak the values.yaml as above.
For the bridge image it's worse because you need to hard-code stuff into the HelmResource class.
It would be useful to simplify and improve the configurability of Helm Charts STs.
The text was updated successfully, but these errors were encountered:
It turned out these changes for Helm Charts are also not enough.
The build would fail in the pipeline with following error related to a naming check:
FAIL strimzi operator deployment strimzi-kafka-operator/tests/operator_deployment_test.yaml
- should have latest image with default image registry when no image is provided
- asserts[1] `matchRegex` fail
Template: strimzi-kafka-operator/templates/060-Deployment-strimzi-cluster-operator.yaml
DocumentIndex: 0
Path: spec.template.spec.containers[0].image
Expected to match:
quay.io/strimzi/operator:(latest|[0-9]+\.[0-9]+\.[0-9]+)
Actual:
quay.io/ppatierno/operator:bridge-config
scholzj
changed the title
Improve the way to configure custom HTTP bridge image for Helm Charts system tests
[ST] Improve the way to configure custom HTTP bridge image for Helm Charts system tests
Jan 22, 2025
While working on this PR #11032, which has changes on the operator but also need a new bridge (not released yet), I needed to run the system tests by using a custom bridge image.
For mostly all of them I needed to specify the bridge image within the .azure/templates/steps/system_test_general.yaml file by setting something like:
For the Helm Charts tests (from the
HelmChartST
class) I actually needed two changes.One for running the right operator by modifying its info here https://github.com/strimzi/strimzi-kafka-operator/blob/main/packaging/helm-charts/helm3/strimzi-kafka-operator/values.yaml#L15 like:
and then hard-coding the bridge image data here https://github.com/strimzi/strimzi-kafka-operator/blob/main/systemtest/src/main/java/io/strimzi/systemtest/resources/operator/specific/HelmResource.java#L59 like:
Usually, for the operator there is no need to make any changes, because the STs are getting the image from the PR build while for Helm Charts STs you need to tweak the values.yaml as above.
For the bridge image it's worse because you need to hard-code stuff into the
HelmResource
class.It would be useful to simplify and improve the configurability of Helm Charts STs.
The text was updated successfully, but these errors were encountered: