-
Notifications
You must be signed in to change notification settings - Fork 28k
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
[SPARK-44802][INFRA][FOLLOWUP] Fix to consider JIRA_ACCESS_TOKEN in precheck conditions #42502
Conversation
…recheck conditions
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.
LGTM, I found this issue and prepared to make a followup too
Let's also update L611 with something like |
Thank you, @yaooqinn . |
Co-authored-by: Kent Yao <yao@apache.org>
I addressed the comments. Thank you again, @yaooqinn . |
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.
LGTM
Successfully merged PR #42511 and resolved the correlated JIRA issue w/these updates and w/o USER/PASSWORD. Thanks @dongjoon-hyun for the quick fix. Merged to master |
Thank you so much, @yaooqinn . :) |
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.
LGTM!
…recheck conditions ### What changes were proposed in this pull request? This is a follow-up of apache#42484 to fix the recheck conditions. ### Why are the changes needed? 1. Currently, the merge script fails at the last update stage because the rest of code requires JIRA_USERNAME **and** JIRA_PASSWORD still. ``` Would you like to pick afe1886 into another branch? (y/n): n JIRA_USERNAME and JIRA_PASSWORD not set Exiting without trying to close the associated JIRA. ``` 2. In addition, it shows improper warning at the initial stage. ``` $ dev/merge_spark_pr.py git rev-parse --abbrev-ref HEAD The env-vars JIRA_USERNAME and/or JIRA_PASSWORD are not set. Continue? (y/n): y Which pull request would you like to merge? (e.g. 34): ``` ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Manual tests Closes apache#42502 from dongjoon-hyun/SPARK-44802. Authored-by: Dongjoon Hyun <dongjoon@apache.org> Signed-off-by: Kent Yao <yao@apache.org>
…recheck conditions ### What changes were proposed in this pull request? This is a follow-up of apache#42484 to fix the recheck conditions. ### Why are the changes needed? 1. Currently, the merge script fails at the last update stage because the rest of code requires JIRA_USERNAME **and** JIRA_PASSWORD still. ``` Would you like to pick afe1886 into another branch? (y/n): n JIRA_USERNAME and JIRA_PASSWORD not set Exiting without trying to close the associated JIRA. ``` 2. In addition, it shows improper warning at the initial stage. ``` $ dev/merge_spark_pr.py git rev-parse --abbrev-ref HEAD The env-vars JIRA_USERNAME and/or JIRA_PASSWORD are not set. Continue? (y/n): y Which pull request would you like to merge? (e.g. 34): ``` ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Manual tests Closes apache#42502 from dongjoon-hyun/SPARK-44802. Authored-by: Dongjoon Hyun <dongjoon@apache.org> Signed-off-by: Kent Yao <yao@apache.org>
What changes were proposed in this pull request?
This is a follow-up of #42484 to fix the recheck conditions.
Why are the changes needed?
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Manual tests