DonalEvans commented on pull request #6757:
URL: https://github.com/apache/geode/pull/6757#issuecomment-902018065


   > Alright, so even after re-running the task 
"concourse-ci/distributed-test-openjdk8" via an empty commit, it's still 
failing, so not sure if we are ok with this to still get the PR approved.
   
   It's (sadly) pretty normal for there to be a flaky failure in the 
pre-checkin, so as long as it's been confirmed that the cause of the job 
failing is a known issue (the most recent one seems to be 
[GEODE-8411](https://issues.apache.org/jira/browse/GEODE-8411)), it's fine to 
merge the PR. It's often helpful for reviewers if you can identify the cause of 
the flaky failure and leave a comment saying what it is, so they know it's 
nothing to worry about.
   
   That being said, I would appreciate it if you could address [this 
comment](https://github.com/apache/geode/pull/6757#issuecomment-900595283) to 
update the test names to be more descriptive before this gets merged.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to