aplyusnin opened a new pull request, #891:
URL: https://github.com/apache/flink-kubernetes-operator/pull/891

   Tests that perform requests are usualy fail now on CI due to race 
conditions. 
   
   To fix it, I addeв busywait for metrics with timeout as it is done in 
RestApiMetricsCollectorTest#testJmMetricCollection().
   Also I added an extra delay in the 
RestApiMetricsCollectorTest#testJmMetricCollection(), it helps to avoid CI 
failing on my workload runs. 
   
   Alternative fix:
   Add blocks inside tests until requests are performed, but this requreis a 
lot of modifications 
   
   ## Does this pull request potentially affect one of the following parts:
   
     - Dependencies (does it add or upgrade a dependency): no
     - The public API, i.e., is any changes to the `CustomResourceDescriptors`: 
no
     - Core observer or reconciler logic that is regularly executed: no
   
   ## Documentation
   
     - Does this pull request introduce a new feature? no
     - If yes, how is the feature documented? not documented
   


-- 
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: issues-unsubscr...@flink.apache.org

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

Reply via email to