Change in vdsm[master]: debian: add supervdsm_api
gerrit-hooks has posted comments on this change. Change subject: debian: add supervdsm_api .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found. -- To view, visit https://gerrit.ovirt.org/54466 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2779caab6a4ec877faf9d0ba5141ca91cfadffb3 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Martin PolednikGerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Francesco Romani Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik Gerrit-Reviewer: Milan Zamazal Gerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: debian: add supervdsm_api
Dan Kenigsberg has submitted this change and it was merged. Change subject: debian: add supervdsm_api .. debian: add supervdsm_api Whole supervdsm_api package was missing in debian installation according to the debian vdsm.install. This patch fixes the creation of the package. Change-Id: I2779caab6a4ec877faf9d0ba5141ca91cfadffb3 Signed-off-by: Martin PolednikReviewed-on: https://gerrit.ovirt.org/54466 Continuous-Integration: Jenkins CI Reviewed-by: Francesco Romani Reviewed-by: Milan Zamazal Reviewed-by: Dan Kenigsberg --- M debian/vdsm.install 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Dan Kenigsberg: Looks good to me, approved Francesco Romani: Looks good to me, but someone else must approve Martin Polednik: Verified Milan Zamazal: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/54466 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2779caab6a4ec877faf9d0ba5141ca91cfadffb3 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Martin Polednik Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Francesco Romani Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik Gerrit-Reviewer: Milan Zamazal Gerrit-Reviewer: gerrit-hooks ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: debian: add supervdsm_api
Dan Kenigsberg has posted comments on this change. Change subject: debian: add supervdsm_api .. Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/54466 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2779caab6a4ec877faf9d0ba5141ca91cfadffb3 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Martin PolednikGerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Francesco Romani Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik Gerrit-Reviewer: Milan Zamazal Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: debian: add supervdsm_api
Martin Polednik has posted comments on this change. Change subject: debian: add supervdsm_api .. Patch Set 1: Verified+1 according to current codebase this should be fine and contains all files that rpm specfile does -- To view, visit https://gerrit.ovirt.org/54466 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2779caab6a4ec877faf9d0ba5141ca91cfadffb3 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Martin PolednikGerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Francesco Romani Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik Gerrit-Reviewer: Milan Zamazal Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: debian: add supervdsm_api
Milan Zamazal has posted comments on this change. Change subject: debian: add supervdsm_api .. Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/54466 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2779caab6a4ec877faf9d0ba5141ca91cfadffb3 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Martin PolednikGerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Francesco Romani Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik Gerrit-Reviewer: Milan Zamazal Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: debian: add supervdsm_api
Francesco Romani has posted comments on this change. Change subject: debian: add supervdsm_api .. Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/54466 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2779caab6a4ec877faf9d0ba5141ca91cfadffb3 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Martin PolednikGerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Francesco Romani Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik Gerrit-Reviewer: Milan Zamazal Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: debian: add supervdsm_api
Martin Polednik has posted comments on this change. Change subject: debian: add supervdsm_api .. Patch Set 1: also calling for verification from someone who can either 1) trusts the spec enough 2) owns debian installation. :) @mzamazal? -- To view, visit https://gerrit.ovirt.org/54466 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2779caab6a4ec877faf9d0ba5141ca91cfadffb3 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Martin PolednikGerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Francesco Romani Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik Gerrit-Reviewer: Milan Zamazal Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: debian: add supervdsm_api
gerrit-hooks has posted comments on this change. Change subject: debian: add supervdsm_api .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url. * Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.6', 'ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3']) -- To view, visit https://gerrit.ovirt.org/54466 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2779caab6a4ec877faf9d0ba5141ca91cfadffb3 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Martin PolednikGerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: debian: add supervdsm_api
Martin Polednik has uploaded a new change for review. Change subject: debian: add supervdsm_api .. debian: add supervdsm_api Whole supervdsm_api package was missing in debian installation according to the debian vdsm.install. This patch fixes the creation of the package. Change-Id: I2779caab6a4ec877faf9d0ba5141ca91cfadffb3 Signed-off-by: Martin Polednik--- M debian/vdsm.install 1 file changed, 6 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/66/54466/1 diff --git a/debian/vdsm.install b/debian/vdsm.install index a9c197b..73fe45b 100644 --- a/debian/vdsm.install +++ b/debian/vdsm.install @@ -125,6 +125,12 @@ ./usr/share/vdsm/storage/threadPool.py ./usr/share/vdsm/storage/volume.py ./usr/share/vdsm/sitecustomize.py +./usr/share/vdsm/supervdsm_api/__init__.py +./usr/share/vdsm/supervdsm_api/hwinfo.py +./usr/share/vdsm/supervdsm_api/mkimage.py +./usr/share/vdsm/supervdsm_api/ksm.py +./usr/share/vdsm/supervdsm_api/network.py +./usr/share/vdsm/supervdsm_api/virt.py ./usr/share/vdsm/supervdsmServer ./usr/share/vdsm/v2v.py ./usr/share/vdsm/vdsm -- To view, visit https://gerrit.ovirt.org/54466 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I2779caab6a4ec877faf9d0ba5141ca91cfadffb3 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Martin Polednik ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches