s/ISOURL/ISO_URL/ (changed upstream)
ClosedPublic

Authored by adamwill on Feb 29 2016, 11:25 PM.

Details

Summary

I just pushed the latest openqa package update stable. That
package backports the changes to the asset download code I got
merged upstream, and one consequence of that is that we have
to use ISO_URL for downloading ISOs now, not ISOURL. So, we'd
better change that. Please review this ASAP so we don't wind
up with the packages updated on the prod boxes and the tests
stop working :)

You'll want at least openqa-4.3-17 to test this, before -17,
ISOURL was correct.

Test Plan

Update openqa to 4.3-17 or later, run the scheduler
for a compose you don't have downloaded yet, and make sure
the downloading works.

Diff Detail

Repository
rOPENQA fedora_openqa
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
adamwill retitled this revision from to s/ISOURL/ISO_URL/ (changed upstream).Feb 29 2016, 11:25 PM
adamwill updated this object.
adamwill edited the test plan for this revision. (Show Details)
adamwill added reviewers: jskladan, garretraziel.
garretraziel accepted this revision.Mar 1 2016, 12:28 PM

Yup, changed it on my machine a while ago (I used openQA for updates-testing) and it worked.

This revision is now accepted and ready to land.Mar 1 2016, 12:28 PM
This revision was automatically updated to reflect the committed changes.