From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has posted comments on this change.
Change subject: automation: check-merged artifacts bugfix
......................................................................
Patch Set 26: Code-Review-1
(1 comment)
https://gerrit.ovirt.org/#/c/69809/26/automation/prepare-yum-conf
File automation/prepare-yum-conf:
Line 58: _, env_info, vm_name, distro, prefix = sys.argv
Line 59:
Line 60: with tf.NamedTemporaryFile() as tempfile:
Line 61: try:
Line 62: open(YUM_CONF, 'r')
marking with -1 since this should not exist here.
Line 63: except OSError as e:
Line 64: if e.errno == errno.ENOENT:
Line 65: open(YUM_CONF, 'w')
Line 66: else:
--
To view, visit
https://gerrit.ovirt.org/69809
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I53603d61b036914baafdab860b13fda32ffeca19
Gerrit-PatchSet: 26
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Leon Goldberg <lgoldber(a)redhat.com>
Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Gal Ben Haim <galbh2(a)gmail.com>
Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Leon Goldberg <lgoldber(a)redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
Gerrit-HasComments: Yes