Alan Pevec has posted comments on this change.
Change subject: BZ#727509 Vdsm-reg should get RHEVM certificates ......................................................................
Patch Set 1: Looks good to me, but someone else must approve
(1 inline comment)
.................................................... File vdsm_reg/vdsm-reg-setup.py Line 197: In rhevm.py cert was supposed to be downloaded when cert verification was selected, so it should've worked at least in that flow. But since Pavel filed 727509 while verifying 602338, it seems it wasn't working even with cert verification selected. I'd ask Pavel to quickly verify this patch on his setup.
-- To view, visit http://gerrit.usersys.redhat.com/779 To unsubscribe, visit http://gerrit.usersys.redhat.com/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I09673ab7bc9fa5dc99829eb6b8d6a65bc13d0e73 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: Alan Pevec apevec@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: Yotam Oron yoron@redhat.com