Change in vdsm[master]: multipath: Bump multipath configuration version
automat...@ovirt.org has posted comments on this change. Change subject: multipath: Bump multipath configuration version .. Patch Set 2: * Update tracker::#1124450::OK -- To view, visit http://gerrit.ovirt.org/34840 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I95aea50fb251bd6d22426fa63edbe4bbd770a06f Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer Gerrit-Reviewer: Allon Mureinik Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Federico Simoncelli Gerrit-Reviewer: Nir Soffer Gerrit-Reviewer: Yeela Kaplan Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: multipath: Bump multipath configuration version
Nir Soffer has abandoned this change. Change subject: multipath: Bump multipath configuration version .. Abandoned Seems that this is not very popular, we will wait for the next change to bump the version. -- To view, visit http://gerrit.ovirt.org/34840 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: abandon Gerrit-Change-Id: I95aea50fb251bd6d22426fa63edbe4bbd770a06f Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer Gerrit-Reviewer: Allon Mureinik Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Federico Simoncelli Gerrit-Reviewer: Nir Soffer Gerrit-Reviewer: Yeela Kaplan Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: multipath: Bump multipath configuration version
Nir Soffer has posted comments on this change. Change subject: multipath: Bump multipath configuration version .. Patch Set 2: Waiting for Federico review with this. -- To view, visit http://gerrit.ovirt.org/34840 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I95aea50fb251bd6d22426fa63edbe4bbd770a06f Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer Gerrit-Reviewer: Allon Mureinik Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Federico Simoncelli Gerrit-Reviewer: Nir Soffer Gerrit-Reviewer: Yeela Kaplan Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: multipath: Bump multipath configuration version
Dan Kenigsberg has posted comments on this change. Change subject: multipath: Bump multipath configuration version .. Patch Set 2: Code-Review-1 Per Nir's private communication, there's no real need in bumping the version here: if the host is already installed, it won't suddenly need to boot from a iscsi root fs. -- To view, visit http://gerrit.ovirt.org/34840 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I95aea50fb251bd6d22426fa63edbe4bbd770a06f Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer Gerrit-Reviewer: Allon Mureinik Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Federico Simoncelli Gerrit-Reviewer: Nir Soffer Gerrit-Reviewer: Yeela Kaplan Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: multipath: Bump multipath configuration version
Yeela Kaplan has posted comments on this change. Change subject: multipath: Bump multipath configuration version .. Patch Set 2: Code-Review-1 This should be on the same patch as the previous one. There is no reason to separate them as this patch is only a side effect and a requirement of the previous patch. -- To view, visit http://gerrit.ovirt.org/34840 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I95aea50fb251bd6d22426fa63edbe4bbd770a06f Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer Gerrit-Reviewer: Allon Mureinik Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Federico Simoncelli Gerrit-Reviewer: Nir Soffer Gerrit-Reviewer: Yeela Kaplan Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: multipath: Bump multipath configuration version
Nir Soffer has posted comments on this change. Change subject: multipath: Bump multipath configuration version .. Patch Set 2: Verified+1 Verified that new multipath configuration is updated when installing on host with older vdsm version. -- To view, visit http://gerrit.ovirt.org/34840 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I95aea50fb251bd6d22426fa63edbe4bbd770a06f Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer Gerrit-Reviewer: Allon Mureinik Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Federico Simoncelli Gerrit-Reviewer: Nir Soffer Gerrit-Reviewer: Yeela Kaplan Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: multipath: Bump multipath configuration version
oVirt Jenkins CI Server has posted comments on this change. Change subject: multipath: Bump multipath configuration version .. Patch Set 2: Build Failed http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/13413/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12623/ : FAILURE http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created_staging/189/ : FAILURE http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/13575/ : FAILURE -- To view, visit http://gerrit.ovirt.org/34840 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I95aea50fb251bd6d22426fa63edbe4bbd770a06f Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer Gerrit-Reviewer: Allon Mureinik Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Federico Simoncelli Gerrit-Reviewer: Yeela Kaplan Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: multipath: Bump multipath configuration version
Allon Mureinik has posted comments on this change. Change subject: multipath: Bump multipath configuration version .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/34840 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I95aea50fb251bd6d22426fa63edbe4bbd770a06f Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer Gerrit-Reviewer: Allon Mureinik Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Federico Simoncelli Gerrit-Reviewer: Yeela Kaplan Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: multipath: Bump multipath configuration version
oVirt Jenkins CI Server has posted comments on this change. Change subject: multipath: Bump multipath configuration version .. Patch Set 1: Build Failed http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/13403/ : FAILURE http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/13242/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12452/ : FAILURE -- To view, visit http://gerrit.ovirt.org/34840 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I95aea50fb251bd6d22426fa63edbe4bbd770a06f Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer Gerrit-Reviewer: Allon Mureinik Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Federico Simoncelli Gerrit-Reviewer: Yeela Kaplan Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: multipath: Bump multipath configuration version
Nir Soffer has uploaded a new change for review. Change subject: multipath: Bump multipath configuration version .. multipath: Bump multipath configuration version Systems configured with older vdsm version includes bad path to scsi_id, which cause failure to boot from multipath device. Bumping version ensure that badly configured systems are updated when multipath configuration is checked. Change-Id: I95aea50fb251bd6d22426fa63edbe4bbd770a06f Bug-Url: https://bugzilla.redhat.com/1124450 Signed-off-by: Nir Soffer --- M vdsm/storage/multipath.py 1 file changed, 3 insertions(+), 2 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/40/34840/1 diff --git a/vdsm/storage/multipath.py b/vdsm/storage/multipath.py index b9c9ecb..89e768b 100644 --- a/vdsm/storage/multipath.py +++ b/vdsm/storage/multipath.py @@ -53,8 +53,9 @@ OLD_TAGS = ["# RHAT REVISION 0.2", "# RHEV REVISION 0.3", "# RHEV REVISION 0.4", "# RHEV REVISION 0.5", "# RHEV REVISION 0.6", "# RHEV REVISION 0.7", -"# RHEV REVISION 0.8", "# RHEV REVISION 0.9"] -MPATH_CONF_TAG = "# RHEV REVISION 1.0" +"# RHEV REVISION 0.8", "# RHEV REVISION 0.9", +"# RHEV REVISION 1.0"] +MPATH_CONF_TAG = "# RHEV REVISION 1.1" MPATH_CONF_PRIVATE_TAG = "# RHEV PRIVATE" STRG_MPATH_CONF = ( "\n\n" -- To view, visit http://gerrit.ovirt.org/34840 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I95aea50fb251bd6d22426fa63edbe4bbd770a06f Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches