Skip to content

[MINOR] Fix Azure CI status check to consider multiple runs on the same commit#11980

Merged
codope merged 1 commit intoapache:masterfrom
yihua:MINOR-fix-azure-ci-comment-check
Sep 21, 2024
Merged

[MINOR] Fix Azure CI status check to consider multiple runs on the same commit#11980
codope merged 1 commit intoapache:masterfrom
yihua:MINOR-fix-azure-ci-comment-check

Conversation

@yihua
Copy link
Contributor

@yihua yihua commented Sep 21, 2024

Change Logs

The hudi-bot comment of the Azure CI status can contain the status of multiple runs on the same commit (see the screenshot below). Right now Github checks only reflect the status of the first run which is inconsistent in the case of multiple runs. As long as there is one successful CI run on a particular commit, we treat the commit passing CI (there can be flakiness outside the scope of the PR or previously cancelled runs). This PR fixes Azure CI status check to consider such multiple runs on the same commit so the Github checks reflect the correct status of the Azure CI.

Screenshot 2024-09-20 at 18 26 17

Impact

Fixes Github checks to be consistent with the status of the Azure CI.

Risk level

none

Documentation Update

N/A

Contributor's checklist

  • Read through contributor's guide
  • Change Logs and Impact were stated clearly
  • Adequate tests were added if applicable
  • CI passed

@github-actions github-actions bot added the size:S PR with lines of changes in (10, 100] label Sep 21, 2024
@hudi-bot
Copy link
Collaborator

CI report:

Bot commands @hudi-bot supports the following commands:
  • @hudi-bot run azure re-run the last Azure build

@codope codope merged commit 0d5bcba into apache:master Sep 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S PR with lines of changes in (10, 100]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants