New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HDDS-4995. Skip CI for draft pull requests #2060
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall, LGTM.
Minor comment inline.
79a9d39
to
520ba00
Compare
@adoroszlai , how can github actions be enabled on forks ? |
@mukul1987 https://github.com/mukul1987/hadoop-ozone/actions/workflows/post-commit.yml - I think you as repo owner should see a button: "Enable workflow". |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1, thanks for the improvement @adoroszlai
Apache organization has a shared github actions pool (180 executors AFAIK) for all the Apache projects, we need to minimize the action executions when it's reasonable....
Thanks @dineshchitlangia and @elek for the review. |
What changes were proposed in this pull request?
Github pull requests can be created as (or converted to) draft. This is useful for gathering early feedback from other contributors about proof-of-concept code. Such code may not even compile, and style issues are usually ignored.
Since further commits are generally expected in this case, I think it makes sense to skip CI build/test until the PR is marked as "ready for review". The goal is to save CI cycles for other issues.
If CI feedback is desired, the author should enable workflows in their fork so we can check push builds there.
https://issues.apache.org/jira/browse/HDDS-4995
How was this patch tested?
Tested with a draft PR in my fork: adoroszlai#12
Builds:
Also with this PR: