You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "christophd (via GitHub)" <gi...@apache.org> on 2023/05/08 13:13:13 UTC
[GitHub] [camel-k] christophd opened a new pull request, #4341: [Backport 1.10.x] fix: Always uninstall IntegrationPlatform resources
christophd opened a new pull request, #4341:
URL: https://github.com/apache/camel-k/pull/4341
Manual backport of #4334
--
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] squakez merged pull request #4341: [Backport 1.10.x] fix: Always uninstall IntegrationPlatform resources
Posted by "squakez (via GitHub)" <gi...@apache.org>.
squakez merged PR #4341:
URL: https://github.com/apache/camel-k/pull/4341
--
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] christophd commented on pull request #4341: [Backport 1.10.x] fix: Always uninstall IntegrationPlatform resources
Posted by "christophd (via GitHub)" <gi...@apache.org>.
christophd commented on PR #4341:
URL: https://github.com/apache/camel-k/pull/4341#issuecomment-1539661468
I think we can merge as the failing jobs are not related to the changes of this PR
@squakez WDYT?
--
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] squakez commented on pull request #4341: [Backport 1.10.x] fix: Always uninstall IntegrationPlatform resources
Posted by "squakez (via GitHub)" <gi...@apache.org>.
squakez commented on PR #4341:
URL: https://github.com/apache/camel-k/pull/4341#issuecomment-1539837024
> I think we can merge as the failing jobs are not related to the changes of this PR
>
> @squakez WDYT?
I prefer we don't. I am pretty sure the failures are not related, but if we go on that path, we'll always do the same, ending up not trusting our tests at all. Unfortunately on release branches, the suite is long to run, so, any re-run takes time. I'm retriggering the checks for those suite of test which we know is not broken.
--
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] squakez commented on pull request #4341: [Backport 1.10.x] fix: Always uninstall IntegrationPlatform resources
Posted by "squakez (via GitHub)" <gi...@apache.org>.
squakez commented on PR #4341:
URL: https://github.com/apache/camel-k/pull/4341#issuecomment-1539945213
Okey. So, this seems to be failing after the merge of https://github.com/apache/camel-k/pull/4243. Hence I'm merging this. However, @claudio4j, would you mind to have a look at this problem? Otherwise it's going to be difficult to rely on our suite of 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.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org