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 2021/01/21 11:47:23 UTC
[GitHub] [camel-k] nicolaferraro opened a new issue #1930: Warnings on getting operator versions
nicolaferraro opened a new issue #1930:
URL: https://github.com/apache/camel-k/issues/1930
I see these messages in CI:
```
Could not parse <nil> (error: Invalid Semantic Version)
Warning: you're using Camel K 1.4.0-SNAPSHOT client against a cluster operator
```
Something related to the operator version check? @astefanutti , @squakez
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] squakez commented on issue #1930: Warnings on getting operator versions
Posted by GitBox <gi...@apache.org>.
squakez commented on issue #1930:
URL: https://github.com/apache/camel-k/issues/1930#issuecomment-764645473
Most likely yes. The way it works, every "online" command try to get the cluster operator version. It matches against the client, and if major.minor differs return the warning.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] astefanutti commented on issue #1930: Warnings on getting operator versions
Posted by GitBox <gi...@apache.org>.
astefanutti commented on issue #1930:
URL: https://github.com/apache/camel-k/issues/1930#issuecomment-765307459
A default IntegrationPlatform is created by the `kamel install` command. So the check is too early for that command, as the version status has not be reconciled yet :)
I think we can exclude the `install` command from the list of commands for which the check applies, as, while it is "online", it does not interact with the operator, since it has yet to be installed. Actually, as far as the check is concern, the right semantic should be "interacts with the operator", rather than "online".
Otherwise, we can skip the check when the `.status.version` field is `nil`.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] astefanutti closed issue #1930: Warnings on getting operator versions
Posted by GitBox <gi...@apache.org>.
astefanutti closed issue #1930:
URL: https://github.com/apache/camel-k/issues/1930
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] squakez commented on issue #1930: Warnings on getting operator versions
Posted by GitBox <gi...@apache.org>.
squakez commented on issue #1930:
URL: https://github.com/apache/camel-k/issues/1930#issuecomment-764645473
Most likely yes. The way it works, every "online" command try to get the cluster operator version. It matches against the client, and if major.minor differs return the warning.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] astefanutti closed issue #1930: Warnings on getting operator versions
Posted by GitBox <gi...@apache.org>.
astefanutti closed issue #1930:
URL: https://github.com/apache/camel-k/issues/1930
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] squakez commented on issue #1930: Warnings on getting operator versions
Posted by GitBox <gi...@apache.org>.
squakez commented on issue #1930:
URL: https://github.com/apache/camel-k/issues/1930#issuecomment-765298784
I managed to replicate the issue locally. I am not sure how the `e2e` tests are working but the operator that is installed is failing:
```
Error from server (BadRequest): container "camel-k-operator" in pod "camel-k-operator-7c5db4c5c9-799c8" is waiting to start: trying and failing to pull image
```
Neither there is an integration platform installed in the temporary namespace. Is that the expected behavior of the test procedure? if that is the case we may include a skip warning flag in order to avoid cluttering those tests.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org