oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 22:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master-libfapi_create-rpms-el6
Dan Kenigsberg has submitted this change and it was merged.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Move multipath configuration to vdsm-tool configurator
Previously multipath is reconfigured o
Dan Kenigsberg has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 21: Code-Review+2
--
To view, visit http://gerrit.ovirt.org/30909
To unsubscribe, visit htt
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 21:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc21
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 20:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc21
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 19:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc21
Yeela Kaplan has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 21: Verified+1
--
To view, visit http://gerrit.ovirt.org/30909
To unsubscribe, visit http://g
Yeela Kaplan has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 20: Verified+1
--
To view, visit http://gerrit.ovirt.org/30909
To unsubscribe, visit http://g
Saggi Mizrahi has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 20: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/30909
To unsubscribe, visit http
Saggi Mizrahi has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 18: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/30909
To unsubscribe, visit http
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 18:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc21
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 17:
(1 comment)
http://gerrit.ovirt.org/#/c/30909/17/lib/vdsm/tool/configurators/multipath.py
F
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 17:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12
Yeela Kaplan has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 16:
(4 comments)
http://gerrit.ovirt.org/#/c/30909/16/lib/vdsm/tool/configurators/multipath.p
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 16:
(4 comments)
http://gerrit.ovirt.org/#/c/30909/16/lib/vdsm/tool/configurators/multipath.py
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 16:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12
Yaniv Bronhaim has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 15: Code-Review+1
(1 comment)
this part looks good imo
http://gerrit.ovirt.org/#/c/30909/1
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 15:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1309
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 14:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1291
Yeela Kaplan has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 13:
(3 comments)
Separating into two steps / patches:
1. move to backup file from rotate file
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 13:
We may aim for different backup solution, but it is not related to this patch.
--
To view,
Dan Kenigsberg has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 13:
Back in http://lists.ovirt.org/pipermail/devel/2014-June/007897.html we agreed
to take
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 13: Code-Review-1
This patch does now two unrelated changes:
1. Keep one backup of multipath.con
Dan Kenigsberg has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 13:
(1 comment)
http://gerrit.ovirt.org/#/c/30909/13/lib/vdsm/tool/configurators/multipath.
Dan Kenigsberg has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 13:
Just to be clear, I refused to see rotateFiles in utils in its current fragile
form (ig
Yeela Kaplan has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 13:
moving to use backup instead of rotateFiles is not because 'Dan doesn't like
it'.
The ch
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 13:
This is not the only way to go. If Dan has a problem with rotateFiles, we can
fix it.
We s
Yeela Kaplan has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 13:
It doesn't make sense to change the use of rotateFiles to copy in a different
following p
Yaniv Bronhaim has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 13:
I intend to agree with nir about the code changes. most of the comments I had
were abou
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 13:
(3 comments)
I think we are going in the wrong direction. This patch should be simple move
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 13:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc20
Yaniv Bronhaim has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 12: Code-Review+1
(1 comment)
my comment there although its as previous implementation in m
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 12:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc20
Yaniv Bronhaim has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 11: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/30909
To unsubscribe, visit htt
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 11: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/30909
To unsubscribe, visit http://
Yeela Kaplan has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 11: Verified+1
--
To view, visit http://gerrit.ovirt.org/30909
To unsubscribe, visit http://g
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 11:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc20
Yeela Kaplan has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 10: Verified+1
--
To view, visit http://gerrit.ovirt.org/30909
To unsubscribe, visit http://g
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 10: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/30909
To unsubscribe, visit http://
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 10:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_e
Yeela Kaplan has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 9:
(4 comments)
http://gerrit.ovirt.org/#/c/30909/9//COMMIT_MSG
Commit Message:
Line 11:
Li
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 9:
(4 comments)
http://gerrit.ovirt.org/#/c/30909/9/lib/vdsm/tool/configurators/multipath.py
Fi
Yaniv Bronhaim has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 9:
(5 comments)
http://gerrit.ovirt.org/#/c/30909/9/lib/vdsm/tool/configurator.py
File lib/
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 9:
(1 comment)
The patch is in great shape now.
Unfortunately a rebase is required after http:
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 9:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 8:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 7: Code-Review-1
(6 comments)
Need to update for new Configurator interface
http://gerrit.ovir
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 7:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/113
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 6:
I think the rebase should be easy - just move the new class to its own module.
--
To view,
Yaniv Bronhaim has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 6:
now it requires big rebase... i warned you
--
To view, visit http://gerrit.ovirt.org/30
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 6:
(2 comments)
http://gerrit.ovirt.org/#/c/30909/6/lib/vdsm/tool/configurator.py
File lib/vdsm
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 6:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 5:
(1 comment)
http://gerrit.ovirt.org/#/c/30909/5/lib/vdsm/tool/configurator.py
File lib/vdsm/
Yeela Kaplan has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 5:
(5 comments)
http://gerrit.ovirt.org/#/c/30909/5/lib/vdsm/tool/configurator.py
File lib/vd
Yaniv Bronhaim has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 5: -Code-Review
(1 comment)
so still need to improve the comments. not only me complaining
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 5: Code-Review-1
(5 comments)
Usage of service_status looks wrong, and some minor documentation
Yaniv Bronhaim has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 5: Code-Review+1
oh well .. at least I know now what the tags and the 2 empty lines mean
--
Yeela Kaplan has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 4:
(3 comments)
force should not override existing configuration if it contains PRIVATE_TAG
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 5:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el
Yaniv Bronhaim has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 4: Code-Review-1
(3 comments)
I'm sorry, i don't understand how the tags work. I know this
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 4:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/106
Yeela Kaplan has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 3:
(3 comments)
http://gerrit.ovirt.org/#/c/30909/3/lib/vdsm/tool/configurator.py
File lib/vd
Yaniv Bronhaim has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 3:
(3 comments)
http://gerrit.ovirt.org/#/c/30909/3/lib/vdsm/tool/configurator.py
File lib/
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 3:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/106
Yeela Kaplan has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 2:
(7 comments)
http://gerrit.ovirt.org/#/c/30909/2/lib/vdsm/tool/configurator.py
File lib/vd
Yaniv Bronhaim has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 2:
(7 comments)
http://gerrit.ovirt.org/#/c/30909/2/lib/vdsm/tool/configurator.py
File lib/
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 2:
(7 comments)
http://gerrit.ovirt.org/#/c/30909/2/lib/vdsm/tool/configurator.py
File lib/vdsm
Yaniv Bronhaim has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 2:
(4 comments)
http://gerrit.ovirt.org/#/c/30909/2/lib/vdsm/tool/configurator.py
File lib/
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 2:
(3 comments)
http://gerrit.ovirt.org/#/c/30909/2/lib/vdsm/tool/configurator.py
File lib/vdsm
Yaniv Bronhaim has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 2:
(4 comments)
http://gerrit.ovirt.org/#/c/30909/2/lib/vdsm/tool/configurator.py
File lib/
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 2: Code-Review-1
(7 comments)
There are some issues in the old code from multipath that must be
Yaniv Bronhaim has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 2: Code-Review-1
(8 comments)
http://gerrit.ovirt.org/#/c/30909/2/lib/vdsm/tool/configurato
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 2:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/106
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 1:
> 3. maybe we should close the bug as duplicate of the current bug-url attached
> to this pa
Yeela Kaplan has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 2: Verified+1
--
To view, visit http://gerrit.ovirt.org/30909
To unsubscribe, visit http://ge
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 1:
> 1. as far as I could tell, there is no special check when vdsm starts, it
> just uses vdsm
Yeela Kaplan has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 1:
(3 comments)
Nir:
Thank you for the review... some answers to your comments:
1. as far as
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 1:
(1 comment)
http://gerrit.ovirt.org/#/c/30909/1//COMMIT_MSG
Commit Message:
Line 7: Move mu
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 1:
(2 comments)
So much nicer when we have several simple patches!
Whats missing:
1. checking
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 1:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/105
Yeela Kaplan has uploaded a new change for review.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Move multipath configuration to vdsm-tool configurator
Previously multipath is reconfigured on each vds
Yeela Kaplan has abandoned this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Abandoned
replaced by a new patch
--
To view, visit http://gerrit.ovirt.org/26123
To unsubscribe, visit http://
Yeela Kaplan has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 3:
(10 comments)
http://gerrit.ovirt.org/#/c/26123/3/lib/vdsm/tool/configurator.py
File lib/v
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 3:
(10 comments)
To make it easier to review, please separate patches where you remove code
wi
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 3:
This patch changes very old code - please start by rebasing on master.
Please break this to
Yaniv Bronhaim has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 3: Code-Review-1
and don't forget to rebase and add toolTests for that
--
To view, visit h
Dan Kenigsberg has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 3: Code-Review-1
(2 comments)
very partial review
http://gerrit.ovirt.org/#/c/26123/3/lib/
Dan Kenigsberg has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 2:
(2 comments)
http://gerrit.ovirt.org/#/c/26123/2/lib/vdsm/utils.py
File lib/vdsm/utils.p
Nir Soffer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 3: Code-Review-1
Please fix pep8 violations:
lib/vdsm/tool/configurator.py:226:17: E128 continua
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 3:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/884
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 3:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit/9779/
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 3: Code-Review-1 Verified-1
Build Failed
http://jenkins.ovirt.org/job/vdsm_master
Itamar Heim has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 2:
ping
--
To view, visit http://gerrit.ovirt.org/26123
To unsubscribe, visit http://gerrit.o
mooli tayer has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 2:
Regarding yaniv previous comment the tool tests were merged:
http://gerrit.ovirt.org/#/c/25
Yaniv Bronhaim has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 2: Code-Review-1
(5 comments)
hope that during next week http://gerrit.ovirt.org/#/c/25263/
Douglas Schilling Landgraf has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 2:
(1 comment)
http://gerrit.ovirt.org/#/c/26123/2//COMMIT_MSG
Commit Message:
Douglas Schilling Landgraf has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 2:
(8 comments)
Hello Yeela, few comments.
http://gerrit.ovirt.org/#/c/26123/2
Douglas Schilling Landgraf has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 2: Code-Review-1
I got from cherry-pick:
Automatic cherry-pick failed. After re
Dan Kenigsberg has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 2: Code-Review-1
(3 comments)
Very very partial review.
Note the persistent pep8 errors
l
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Move multipath configuration to vdsm-tool configurator
..
Patch Set 2:
Build Unstable
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6874/ :
1 - 100 of 103 matches
Mail list logo