Skip to content

Commit

Permalink
Fix env var naming issue (#581)
Browse files Browse the repository at this point in the history
* Fix env var naming issue

Fix an environment variable naming issue in the CSV for STO when
attempting to install alertmanager disconnected. The env var being
looked up should have a postfix of _IMAGE to match the other env vars.

Found in testing by vkmc.

* Run operator-sdk generate bundle

Run the generate bundle command to make sure everything is in sync.
Fixes CI found issue in previous commit.
  • Loading branch information
leifmadsen authored Feb 26, 2024
1 parent c9c4065 commit 6568039
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -297,7 +297,7 @@ spec:
value: <<RELATED_IMAGE_OAUTH_PROXY>>:<<RELATED_IMAGE_OAUTH_PROXY_TAG>>
- name: RELATED_IMAGE_PROMETHEUS_IMAGE
value: <<RELATED_IMAGE_PROMETHEUS>>:<<RELATED_IMAGE_PROMETHEUS_TAG>>
- name: RELATED_IMAGE_ALERTMANAGER
- name: RELATED_IMAGE_ALERTMANAGER_IMAGE
value: <<RELATED_IMAGE_ALERTMANAGER>>:<<RELATED_IMAGE_ALERTMANAGER_TAG>>
image: <<OPERATOR_IMAGE>>:<<OPERATOR_TAG>>
imagePullPolicy: Always
Expand Down
2 changes: 1 addition & 1 deletion deploy/operator.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ spec:
value: <<RELATED_IMAGE_OAUTH_PROXY>>:<<RELATED_IMAGE_OAUTH_PROXY_TAG>>
- name: RELATED_IMAGE_PROMETHEUS_IMAGE
value: <<RELATED_IMAGE_PROMETHEUS>>:<<RELATED_IMAGE_PROMETHEUS_TAG>>
- name: RELATED_IMAGE_ALERTMANAGER
- name: RELATED_IMAGE_ALERTMANAGER_IMAGE
value: <<RELATED_IMAGE_ALERTMANAGER>>:<<RELATED_IMAGE_ALERTMANAGER_TAG>>
volumes:
- name: runner
Expand Down

0 comments on commit 6568039

Please sign in to comment.