-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Regression: [ews] check-status-on-other-ewses step is not working #9182
Regression: [ews] check-status-on-other-ewses step is not working #9182
Conversation
EWS run on previous version of this PR (hash a59a70a)
|
Tools/CISupport/ews-build/steps.py
Outdated
self._addToLog('stdio', 'Failed to access {}\n'.format(url)) | ||
except Exception as e: | ||
yield self._addToLog('stdio', 'Failed to access {}\n'.format(url)) | ||
yield self._addToLog('stdio', ' {}\n'.format(str(e))) |
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.
these two log lines can be combined in one. Also can use python 3 f-strings.
@@ -5004,16 +5004,17 @@ def get_change_status(self, change_id, queue): | |||
try: | |||
response = yield TwistedAdditions.request(url, logger=lambda content: self._addToLog('stdio', content)) | |||
if response.status_code != 200: |
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.
but why was response.status_code non-200 in the first place?
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.
That's not the case we're hitting, we're hitting the Exception
a59a70a
to
dbb0e34
Compare
EWS run on current version of this PR (hash dbb0e34)
|
To be clear, this PR is intended to shed light on the situation, not fix the problem. We can't reproduce this problem on our staging instance and are hoping logging can help narrow the issue now. |
https://bugs.webkit.org/show_bug.cgi?id=251214 rdar://104701249 Reviewed by Aakash Jain. Add logging to the code from 259315@main to log the raised exception. * Tools/CISupport/ews-build/steps.py: (CheckStatusOnEWSQueues.get_change_status): Canonical link: https://commits.webkit.org/259438@main
dbb0e34
to
18d5a67
Compare
Committed 259438@main (18d5a67): https://commits.webkit.org/259438@main Reviewed commits have been landed. Closing PR #9182 and removing active labels. |
18d5a67
dbb0e34
🛠 mac🛠 wpe🛠 🧪 win🛠 gtk🛠 wincairo🧪 ios-wk2🧪 api-mac🧪 gtk-wk2🧪 api-ios🧪 mac-wk1🧪 api-gtk🛠 tv🧪 mac-wk2🛠 tv-sim🧪 mac-AS-debug-wk2🛠 watch🧪 mac-wk2-stress🛠 watch-sim