Re: [PR] test(health): #3997 structured conditions for scaled integrations [camel-k]

2023-11-20 Thread via GitHub
squakez merged PR #4917: URL: https://github.com/apache/camel-k/pull/4917 -- 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:

Re: [PR] test(health): #3997 structured conditions for scaled integrations [camel-k]

2023-11-17 Thread via GitHub
rinaldodev commented on PR #4917: URL: https://github.com/apache/camel-k/pull/4917#issuecomment-1817116312 > Thanks, I will. Some of the tests are flaky and I'm trying to figure out why. Some of the original tests are flaky because sometimes routes are considered UP for a while

Re: [PR] test(health): #3997 structured conditions for scaled integrations [camel-k]

2023-11-14 Thread via GitHub
rinaldodev commented on PR #4917: URL: https://github.com/apache/camel-k/pull/4917#issuecomment-1811428155 > Feel free to move the PR to ready to review or add a comment when you are okey to merge it. Thanks, I will. Some of the tests are flaky and I'm trying to figure out why. --

Re: [PR] test(health): #3997 structured conditions for scaled integrations [camel-k]

2023-11-14 Thread via GitHub
squakez commented on PR #4917: URL: https://github.com/apache/camel-k/pull/4917#issuecomment-1810229208 Feel free to move the PR to ready to review or add a comment when you are okey to merge it. -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] test(health): #3997 structured conditions for scaled integrations [camel-k]

2023-11-13 Thread via GitHub
squakez commented on PR #4917: URL: https://github.com/apache/camel-k/pull/4917#issuecomment-1808045568 > @squakez I was considering adding tests for more scenarios. Do you think its worth it? The more scenario we cover the better. However, for the specific issue I think it should

Re: [PR] test(health): #3997 structured conditions for scaled integrations [camel-k]

2023-11-13 Thread via GitHub
rinaldodev commented on PR #4917: URL: https://github.com/apache/camel-k/pull/4917#issuecomment-1808037160 @squakez I was considering adding tests for more scenarios. Do you think its worth it? -- This is an automated message from the Apache Git Service. To respond to the message, please