Differential D868

fix atomic scheduling
ClosedPublic
Actions

Authored by garretraziel on May 26 2016, 11:22 AM.

Details

Reviewers
adamwill
jskladan
Commits
rOPENQAad77f380a832: fix atomic scheduling
Summary

This fixed atomic scheduling. I have changed images structure without
changing it for atomic scheduling. It should fix those pesky fedmsg errors.

Test Plan

Schedule atomic on newest compose.

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.
garretraziel retitled this revision from to fix atomic scheduling.May 26 2016, 11:22 AM
garretraziel updated this object.
garretraziel edited the test plan for this revision. (Show Details)
garretraziel added reviewers: adamwill, jskladan.
garretraziel added a comment.May 26 2016, 11:23 AM

Pushing directly as it is simple fix.

Closed by commit rOPENQAad77f380a832: fix atomic scheduling (authored by garretraziel). · Explain WhyMay 26 2016, 11:23 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents

Diff 2229

View Options

scheduler/fedora_openqa_schedule/schedule.py

Show First 20 Lines • Show All 65 Lines • ▼ Show 20 Line(s)63def _get_atomic_installer(location, cid):
66 are not done with Pungi 4 at present so do not have the metadata 66​ are not done with Pungi 4 at present so do not have the metadata
67 we want to use. This should die as soon as all composes we want to 67​ we want to use. This should die as soon as all composes we want to
68 test are done in Pungi 4 style. 68​ test are done in Pungi 4 style.
69 """ 69​ """
70 url = '{0}/Cloud_Atomic/x86_64/iso/Fedora-Cloud_Atomic-x86_64-{1}.iso'.format( 70​ url = '{0}/Cloud_Atomic/x86_64/iso/Fedora-Cloud_Atomic-x86_64-{1}.iso'.format(
71 location, cid) 71​ location, cid)
72 try: 72​ try:
73 resp = urlopen(url) 73​ resp = urlopen(url)
74 return [(url, 'Atomic-boot-iso', 'x86_64', 0)] 74​ return [('Atomic-boot-iso', 'x86_64', 0, {'ISO_URL': url})]
75 except (ValueError, URLError, HTTPError): 75​ except (ValueError, URLError, HTTPError):
76 raise TriggerException("Compose not found, or failed!") 76​ raise TriggerException("Compose not found, or failed!")
77 77​
78 78​
79def _get_compose_id(location): 79​def _get_compose_id(location):
80 """Given a compose location, find the compose ID. Really we'd like 80​ """Given a compose location, find the compose ID. Really we'd like
81 taskotron to give us this, as fedmsg provides it, but taskotron is 81​ taskotron to give us this, as fedmsg provides it, but taskotron is
82 kinda tied to giving us just *one* variable property of the fedmsg 82​ kinda tied to giving us just *one* variable property of the fedmsg
▲ Show 20 Lines • Show All 230 Lines • Show Last 20 Lines