Pasquale Congiusti

Results 534 comments of Pasquale Congiusti

To be squashed and merged once checks pass after the rebase. Thanks.

> What about these tests: > > ❌ TestMetrics (3m0.27s) ❌ TestMetrics/reconciliation_duration_metric (10ms) > > they seems flaky to me, first run they were passing, they are passing locally, but...

@valdar I don't have enough context to review this one, sorry. The immediate impression I have is that from community perspective it would be good to have some design diagram...

I've enabled Quarkus Native tests. However it's very likely they fail due to some infrastructure problem that we have not yet tackled. In such circumstances, please, run the Quarkus Native...

> @christophd @squakez I guess this is now in a good shape to be merged I haven't received any feedback about my last 2 comments. We need to run manually...

Also, several tests have failed and they don't seem to be the usual flaky ones. I've restarted them.

> > Also, several tests have failed and they don't seem to be the usual flaky ones. I've restarted them. > > The native tests seems to be failing constantly...

@lburgazzoli the problem is not the failure of the checks per se. As we've done in other PR, everything that may affect the Quarkus native has to be validated manually...

> I agree that we need some docs but since this PR is part of a larger issue and the rationale was briefly explained in [#3397 (comment)](https://github.com/apache/camel-k/issues/3397#issuecomment-1735525398), then I wonder...

> I'm not complaining about having to run them locally, I'm just noticing the OSX build is almost always failing so I wonder what is the value of such test....