Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Watch environment variables are generated incorrectly in OLM bundle #855

Closed
jcechace opened this issue Mar 20, 2024 · 1 comment · Fixed by #856
Closed

Watch environment variables are generated incorrectly in OLM bundle #855

jcechace opened this issue Mar 20, 2024 · 1 comment · Fixed by #856
Assignees
@metacosm

Comments

@jcechace
Copy link
Contributor

The following duplicity gets generated in OLM CSV

              containers:
              - env:
                - name: KUBERNETES_NAMESPACE
                  valueFrom:
                    fieldRef:
                      fieldPath: metadata.namespace
                - name: QUARKUS_OPERATOR_SDK_CONTROLLERS_DEBEZIUMSERVER_NAMESPACES
                  value: "{watchNamespaces}"
                - name: QUARKUS_OPERATOR_SDK_CONTROLLERS_DEBEZIUMSERVER_NAMESPACES
                  valueFrom:
                    fieldRef:
                      fieldPath: "metadata.annotations['olm.targetNamespaces']"

The problem exists at minimum since 6.6.3
metacosm added a commit that referenced this issue Mar 20, 2024
@metacosm
fix: do not modify deployment as it is live and reused elsewhere
e1fd070 
Fixes #855

Signed-off-by: Chris Laprun <claprun@redhat.com>
@metacosm metacosm linked a pull request Mar 20, 2024 that will close this issue
fix: remove duplicated and wrong env variable #856
Merged
@metacosm metacosm self-assigned this Mar 20, 2024
@metacosm
Copy link
Member

Please check the associated PR and let me if this fixes your issue.

metacosm added a commit that referenced this issue Mar 21, 2024
@metacosm
fix: do not modify deployment as it is live and reused elsewhere
341f162 
Fixes #855

Signed-off-by: Chris Laprun <claprun@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@metacosm metacosm

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: remove duplicated and wrong env variable quarkiverse/quarkus-operator-sdk
2 participants
@metacosm @jcechace