bundle upload does not hard fail for missing credentials

RESOLVED FIXED

Status

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: hwine, Assigned: gps)

Tracking

Details

Attachments

(1 attachment)

when credentials are not installed, or are invalid, for upload of bundles to S3, no explicit error is given. This made diagnosis of bug 1260652 harder than it should have been :)
(Assignee)

Updated

3 years ago
Duplicate of this bug: 1260682
(Assignee)

Comment 2

3 years ago
I'll fix this and the problem of advertising bundles that didn't upload properly.
Assignee: nobody → gps
Status: NEW → ASSIGNED
(Assignee)

Comment 3

3 years ago
Created attachment 8736338 [details]
MozReview Request: sripts/generate-hg-s3-bundles: check for errors (bug 1260681); r?fubar

TIL that Python futures in concurrent.futures swallow exceptions. I
thought they were behaving like Python threads where an exception in any
thread would terminate program execution. It turns out you need to
explicitly check Future.result() or Future.exception() to see if
the thing raised an exception.

Tests demonstrating the script fails under various conditions have been
added.

Review commit: https://reviewboard.mozilla.org/r/43223/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/43223/
Attachment #8736338 - Flags: review?(klibby)
Comment on attachment 8736338 [details]
MozReview Request: sripts/generate-hg-s3-bundles: check for errors (bug 1260681); r?fubar

https://reviewboard.mozilla.org/r/43223/#review39831

lgtm
Attachment #8736338 - Flags: review?(klibby) → review+
(Assignee)

Comment 6

3 years ago
This was fixed 22 days ago.
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.