Skip to content

Conversation

@adoroszlai
Copy link
Contributor

What changes were proposed in this pull request?

dependabot pushes dependency version bumps to the main repo instead of its own fork. It then creates a pull request without waiting for the push workflow to succeed. Thus both workflows test the same state of code, which is unnecessary.

I propose completely skipping push build for dependabot's changes to save CI resources.

https://issues.apache.org/jira/browse/HDDS-9698

How was this patch tested?

Build skipped in branch dependabot-HDDS-9698 in my fork:
https://github.com/adoroszlai/ozone/actions/runs/6876171772

but ran normally in branch HDDS-9698:
https://github.com/adoroszlai/ozone/actions/runs/6876180867

as well as for PR created from branch dependabot-HDDS-9698:
https://github.com/adoroszlai/ozone/actions/runs/6876212003

Copy link
Member

@kaijchen kaijchen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @adoroszlai

@kerneltime kerneltime merged commit a0e19bd into apache:master Nov 15, 2023
@adoroszlai adoroszlai deleted the HDDS-9698 branch November 16, 2023 20:52
@adoroszlai
Copy link
Contributor Author

Thanks @kaijchen, @kerneltime for the review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants