You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "jovanpet (via GitHub)" <gi...@apache.org> on 2025/10/21 03:06:17 UTC
[PR] Issue #6334 Flaky TestKedaAuthentication unit test [camel-k]
jovanpet opened a new pull request, #6337:
URL: https://github.com/apache/camel-k/pull/6337
<!-- 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.
-->
https://github.com/issues/assigned?issue=apache%7Ccamel-k%7C6334
**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
Re: [PR] Issue #6334 Flaky TestKedaAuthentication unit test [camel-k]
Posted by "jovanpet (via GitHub)" <gi...@apache.org>.
jovanpet commented on PR #6337:
URL: https://github.com/apache/camel-k/pull/6337#issuecomment-3426848441
Thanks for quick turnaround!
--
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
Re: [PR] Issue #6334 Flaky TestKedaAuthentication unit test [camel-k]
Posted by "jovanpet (via GitHub)" <gi...@apache.org>.
jovanpet commented on PR #6337:
URL: https://github.com/apache/camel-k/pull/6337#issuecomment-3427330321
Hey guys is it normal for advanced to fail multiple times?
--
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
Re: [PR] Issue #6334 Flaky TestKedaAuthentication unit test [camel-k]
Posted by "gansheer (via GitHub)" <gi...@apache.org>.
gansheer merged PR #6337:
URL: https://github.com/apache/camel-k/pull/6337
--
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
Re: [PR] Issue #6334 Flaky TestKedaAuthentication unit test [camel-k]
Posted by "gansheer (via GitHub)" <gi...@apache.org>.
gansheer commented on PR #6337:
URL: https://github.com/apache/camel-k/pull/6337#issuecomment-3428023093
> Hey guys is it normal for advanced to fail multiple times? Also I usually have Merge PR button on my local machine. Does this work differently with open source or auto-merges?
Yes, unfortunately. We have some instability in the test run. It has gotten better but not the best it could yet.
--
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
Re: [PR] Issue #6334 Flaky TestKedaAuthentication unit test [camel-k]
Posted by "squakez (via GitHub)" <gi...@apache.org>.
squakez commented on PR #6337:
URL: https://github.com/apache/camel-k/pull/6337#issuecomment-3426601825
Didn't know that. Thanks for checking @gansheer - approved then!
--
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
Re: [PR] Issue #6334 Flaky TestKedaAuthentication unit test [camel-k]
Posted by "gansheer (via GitHub)" <gi...@apache.org>.
gansheer commented on PR #6337:
URL: https://github.com/apache/camel-k/pull/6337#issuecomment-3426175284
> Thanks for the PR. I think this is the same as we had. The problem with the flakiness is that sometimes you get elements either in position 0 or 1, so it's not reliable. Ideally you need two `assert.Contains` in order to validate the output regardless of the array position.
Actually I think it's good according to the doc: https://pkg.go.dev/github.com/stretchr/testify/assert#ElementsMatch
--
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