squakez commented on PR #5119:
URL: https://github.com/apache/camel-k/pull/5119#issuecomment-1926467434

   > 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 now failing in CI :/
   > 
   > They are also marked as "problematic" but CI runs without the skip 
problematic config? :/
   
   No, I don't think they are flaky. It fails at line:
   ```
   Expect(platformReconciled).NotTo(BeNil())
   ```
   I suspect that the operator perform and provide metrics which we may have 
lost if running the reconciliation loop outside the main process.


-- 
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-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to