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/01/20 10:38:45 UTC
[GitHub] [camel-k] bouskaJ opened a new pull request #2895: [TEST] - check IntegrationPlatform namespace with global operator
bouskaJ opened a new pull request #2895:
URL: https://github.com/apache/camel-k/pull/2895
<!-- Description -->
<!--
Enter your extended release note in the below block. If the PR requires
additional action from users switching to the new release, include the string
"action required". If no release note is required, write "NONE".
You can (optionally) mark this PR with labels "kind/bug" or "kind/feature" to make sure
the text is added to the right section of the release notes.
-->
**Release Note**
```release-note
NONE
```
--
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] bouskaJ commented on pull request #2895: [TEST] - check IntegrationPlatform namespace with global operator
Posted by GitBox <gi...@apache.org>.
bouskaJ commented on pull request #2895:
URL: https://github.com/apache/camel-k/pull/2895#issuecomment-1019951451
Hi @astefanutti I have rebased this PR. The CI failed on various errors not related to this PR
kuberenetes and openshift workflow:
```
=== RUN TestHealthTrait/Readiness_condition_with_stopped_route
Integration "java" created
=== CONT TestHealthTrait
health_test.go:77:
Timed out after 60.001s.
Expected
<string>: "...p[consumer:..."
to equal |
<string>: "...p[context:U..."
=== CONT TestHealthTrait/Readiness_condition_with_stopped_route
```
upgrade:
```
1 integrations have been rebuilt
=== CONT TestOLMAutomaticUpgrade
olm_upgrade_test.go:166:
Timed out after 60.027s.
Expected
<string>:
to contain substring
<string>: 1.8
=== CONT TestOLMAutomaticUpgrade/Integration_upgrade
```
Is there still some ongoing activity to fix that?
--
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] bouskaJ commented on pull request #2895: [TEST] - check IntegrationPlatform namespace with global operator
Posted by GitBox <gi...@apache.org>.
bouskaJ commented on pull request #2895:
URL: https://github.com/apache/camel-k/pull/2895#issuecomment-1017595913
CI failures do not seem to be related to this PR. Can you rerun the CI?
--
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] bouskaJ commented on pull request #2895: [TEST] - check IntegrationPlatform namespace with global operator
Posted by GitBox <gi...@apache.org>.
bouskaJ commented on pull request #2895:
URL: https://github.com/apache/camel-k/pull/2895#issuecomment-1018334430
@astefanutti thank you for your review. I reworked it using existing functions.
--
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] bouskaJ commented on pull request #2895: [TEST] - check IntegrationPlatform namespace with global operator
Posted by GitBox <gi...@apache.org>.
bouskaJ commented on pull request #2895:
URL: https://github.com/apache/camel-k/pull/2895#issuecomment-1018427972
@astefanutti do you know what's wrong with the CI?
--
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] bouskaJ commented on pull request #2895: [TEST] - check IntegrationPlatform namespace with global operator
Posted by GitBox <gi...@apache.org>.
bouskaJ commented on pull request #2895:
URL: https://github.com/apache/camel-k/pull/2895#issuecomment-1022030289
@astefanutti all checks have passed. Thank you for your fixes.
--
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] astefanutti commented on pull request #2895: [TEST] - check IntegrationPlatform namespace with global operator
Posted by GitBox <gi...@apache.org>.
astefanutti commented on pull request #2895:
URL: https://github.com/apache/camel-k/pull/2895#issuecomment-1018334833
@bouskaJ perfect, looks great!
--
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] astefanutti commented on pull request #2895: [TEST] - check IntegrationPlatform namespace with global operator
Posted by GitBox <gi...@apache.org>.
astefanutti commented on pull request #2895:
URL: https://github.com/apache/camel-k/pull/2895#issuecomment-1018429859
@bouskaJ some tests are failing because of the upgrade done for the new release. It's not related to your PR. We're going to have them fixed and you'll be able to rebase.
--
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] astefanutti merged pull request #2895: [TEST] - check IntegrationPlatform namespace with global operator
Posted by GitBox <gi...@apache.org>.
astefanutti merged pull request #2895:
URL: https://github.com/apache/camel-k/pull/2895
--
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] astefanutti commented on pull request #2895: [TEST] - check IntegrationPlatform namespace with global operator
Posted by GitBox <gi...@apache.org>.
astefanutti commented on pull request #2895:
URL: https://github.com/apache/camel-k/pull/2895#issuecomment-1019959875
> Hi @astefanutti I have rebased this PR. The CI failed on various errors not related to this PR
@bouskaJ Thanks for the update.
>
> kuberenetes and openshift workflow:
>
> ```
> === RUN TestHealthTrait/Readiness_condition_with_stopped_route
> Integration "java" created
> === CONT TestHealthTrait
> health_test.go:77:
> Timed out after 60.001s.
> Expected
> <string>: "...p[consumer:..."
> to equal |
> <string>: "...p[context:U..."
> === CONT TestHealthTrait/Readiness_condition_with_stopped_route
> ```
This will be fixed with #2911.
> upgrade:
>
> ```
> 1 integrations have been rebuilt
> === CONT TestOLMAutomaticUpgrade
> olm_upgrade_test.go:166:
> Timed out after 60.027s.
> Expected
> <string>:
> to contain substring
> <string>: 1.8
> === CONT TestOLMAutomaticUpgrade/Integration_upgrade
> ```
>
> Is there still some ongoing activity to fix that?
I'm working on it. I've fixed it for branches with #2904, but it's not working for cross-channel upgrade in #2901. It seems the channel change does not trigger the upgrade. I'm still trying to understand what's going, maybe if you could have a look to check if I'm missing something obvious, that would be great.
--
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] astefanutti commented on pull request #2895: [TEST] - check IntegrationPlatform namespace with global operator
Posted by GitBox <gi...@apache.org>.
astefanutti commented on pull request #2895:
URL: https://github.com/apache/camel-k/pull/2895#issuecomment-1022031591
@bouskaJ great, thanks a lot for your patience. Let's merge it!
--
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