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
[EWS] Support PRs when sending build failure emails (Follow-up fix) #144
[EWS] Support PRs when sending build failure emails (Follow-up fix) #144
Conversation
5be02e0
to
f2e33dc
Compare
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.
r+ with comments.
Tools/CISupport/ews-build/steps.py
Outdated
@@ -2168,7 +2167,7 @@ def send_email_for_new_build_failure(self): | |||
if pr_number and not self.should_send_email_for_pr(pr_number): | |||
return | |||
if not patch_id and not (pr_number and sha): | |||
print('Unrecognized change type') | |||
self._addToLog('Unrecognized change type') |
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.
I don't think we will ever hit it. Some other steps will check similar check fail much earlier. But that's ok print to UI here as well.
Tools/CISupport/ews-build/steps.py
Outdated
|
||
if not change_author: | ||
print('Failed to determine change author for hash {} belonging to PR #{}'.format(sha, patch_id)) | ||
self._addToLog('Failed to determine change author for hash {} belonging to PR #{}'.format(sha, patch_id)) |
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.
Might want to mention something about contributors.json as well. So that people know what they can do to fix it. Might want to include github username in this message as well. Something like:
Unable to determine email address for [github_username] from contributors.json. Skipping sending email.
f2e33dc
to
15e21e6
Compare
Left a comment: 15e21e6#r66766126 |
15e21e6