1261101 - Add full update snippet contents to the update results dictionary

Status: RESOLVED FIXED

(Testing :: Firefox UI Tests, defect)

Description

[Henrik Skupin [:whimboo][⌚️UTC+2]]

In case of failures in update tests related to busted update snippets it would be great to have all the details listed in the log or even better in the update results dictionary as printed at the end of the test.

The URL of the update snippet is getting added here:

https://dxr.mozilla.org/mozilla-central/source/testing/puppeteer/firefox/firefox_puppeteer/api/software_update.py#229

> 'url_aus': self.get_update_url(True),

So we could use urllib or similar to actually retrieve the content of that remote file.

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Re-thinking that I feel it would be too much content inside the update results dictionary and lines won't be that readable. I would suggest we output the content via the info logger.

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Bug 1261101 - Add full update snippet contents to the update results dictionary.

Review commit: https://reviewboard.mozilla.org/r/62122/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/62122/

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Ok, I reverted my opinion from above and it should be better to add it to the update result dictionary. With that we can store the whole dict as json file on disk and upload it to taskcluster.

Comment 4

[Maja Frydrychowicz :impossibus (was :maja_zf) (needinfo me)]

Comment on attachment 8767705 [details]
Bug 1261101 - Add full update snippet contents to the update results dictionary.

https://reviewboard.mozilla.org/r/62122/#review59212

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a4abd7b926c7
Add full update snippet contents to the update results dictionary. r=maja_zf

Comment 6

[Gregory Szorc [:gps]]

https://hg.mozilla.org/mozilla-central/rev/a4abd7b926c7

Comment 7

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Would be great to get this test only change onto mozilla-aurora. It will help us a lot with investigation of test failures of update tests.

Please make sure that the patch on bug 1282080 lands first on mozilla-aurora. Thanks.

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/e2a02d7d4e71