You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2022/10/11 17:01:37 UTC
[GitHub] [camel-k] daniel-geiger-ibmde opened a new issue, #3739: Unable to set env var OPERATOR_ID in Helm chart
daniel-geiger-ibmde opened a new issue, #3739:
URL: https://github.com/apache/camel-k/issues/3739
As per https://camel.apache.org/camel-k/1.10.x/installation/advanced/multi.html the env var OPERATOR_ID must be set on the operator deployment to set the operator id. However the current helm chart doesn't expose the option to set any custom env vars in templates/operator.yaml.
In addition, default behaviour as described at https://camel.apache.org/camel-k/1.10.x/installation/advanced/multi.html does not apply - at least when deploying operator and integrations to a namespace other than camel-k:
Any Integration CRs annotated with the default "camel.apache.org/operator.id: camel-k" won't get picked up by the operator without OPERATOR_ID env variable set. The operator will only pick up the Integration CR after (manually) deleting the annotation. (So it seems the default operator-id is NULL rather than "camel-k".)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] christophd commented on issue #3739: Unable to set env var OPERATOR_ID in Helm installation
Posted by GitBox <gi...@apache.org>.
christophd commented on issue #3739:
URL: https://github.com/apache/camel-k/issues/3739#issuecomment-1275085434
Hi,
there is a bug with `OPERATOR_ID` env var that will be fixed in `v1.10.1` (see #3657)
There is a workaround to set `KAMEL_OPERATOR_ID` env var. With that the operator should start to pick up the integrations annotated with `camel.apache.org/operator.id: camel-k`.
With the upcoming fix in `v1.10.1` both `OPERATOR_ID` and `KAMEL_OPERATOR_ID` will work as expected.
However this only solves on part of your problem. The missing option on the Helm chart still is a problem. I guess you would expect the Helm chart to set the default `OPERATOR_ID=camel-k` in addition to let the user specify another id, right?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] MehrCurry commented on issue #3739: Unable to set env var OPERATOR_ID in Helm installation
Posted by GitBox <gi...@apache.org>.
MehrCurry commented on issue #3739:
URL: https://github.com/apache/camel-k/issues/3739#issuecomment-1280668077
I have seen the fix in #3657. But it seems again that the helm chart has no possibility to set the environment variables. I missed i something?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] daniel-geiger-ibmde commented on issue #3739: Unable to set env var OPERATOR_ID in Helm installation
Posted by GitBox <gi...@apache.org>.
daniel-geiger-ibmde commented on issue #3739:
URL: https://github.com/apache/camel-k/issues/3739#issuecomment-1275620709
Hi Christoph, thanks for responding so quickly :-)
I totally agree regarding the helm chart. A new optional helm variable `operator.id` with a default value of `kamel-k` would be most clear... at least to me ;-)
Regarding explanation of the default "kamel-k" at https://camel.apache.org/camel-k/1.10.x/installation/advanced/multi.html: I think a brief hint could be helpful, that there will be no default operator id if neither of the env variables `OPERATOR_ID` or `KAMEL_OPERATOR_ID` is set. (From the current wording, one could assume that the default is implemented somewhere in the operator itself.)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] daniel-geiger-ibmde closed issue #3739: Unable to set env var OPERATOR_ID in Helm installation
Posted by GitBox <gi...@apache.org>.
daniel-geiger-ibmde closed issue #3739: Unable to set env var OPERATOR_ID in Helm installation
URL: https://github.com/apache/camel-k/issues/3739
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] daniel-geiger-ibmde commented on issue #3739: Unable to set env var OPERATOR_ID in Helm installation
Posted by GitBox <gi...@apache.org>.
daniel-geiger-ibmde commented on issue #3739:
URL: https://github.com/apache/camel-k/issues/3739#issuecomment-1334073591
@MehrCurry - [this ](https://github.com/apache/camel-k/commit/946be23a77bf24f13e11cb35d4c3c20a9aae0e7b )is the fix.
Works like a charm - thanks guys! :-)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org