Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2014-01-12 Thread iheim
Itamar Heim has abandoned this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Abandoned ok, abandoning per last comment. -- To view, visit http://gerrit.ovirt.org/4526 To unsubscribe, visit http:

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2014-01-12 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 8: I created http://gerrit.ovirt.org/#/c/21772/ to replace this work. It is too old to rebase an

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2014-01-06 Thread iheim
Itamar Heim has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 8: ping -- To view, visit http://gerrit.ovirt.org/4526 To unsubscribe, visit http://gerrit.ovirt.o

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2013-11-27 Thread lilei
Lei Li has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 8: I am glad that this still could be useful, it'd be great if you could help on the rebase the pushing!

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2013-11-27 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 8: Code-Review-1 I created http://gerrit.ovirt.org/#/c/21772/ to replace this work. It is too ol

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2013-11-26 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 8: This patch contains very usable parts. I prefer to split it a bit and get it merged. Lei Li,

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2013-11-26 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 8: Didn't notice that patch. reviewing it -- To view, visit http://gerrit.ovirt.org/4526 To uns

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2013-10-25 Thread iheim
Itamar Heim has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 8: still relevant or should be abandoned? -- To view, visit http://gerrit.ovirt.org/4526 To unsubs

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2012-11-13 Thread lilei
Lei Li has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 8: I am petty sure that I have tested the code and pass the build... Take a look at the build error from

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2012-11-13 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 8: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_manual_gerrit/11/ :

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2012-11-13 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 8: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_manual_gerrit/11/ -- To v

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2012-11-13 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 7: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_manual_gerrit/10/ :

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2012-11-13 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 7: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_manual_gerrit/10/ -- To v

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2012-09-14 Thread Ryan Harper
Ryan Harper has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 6: I would prefer that you didn't submit this (9 inline comments) .

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2012-08-08 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 6: Build Successful http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/182/ : SUCC

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2012-08-08 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 6: No score Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/182/ -

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2012-07-01 Thread wenyi
Wenyi Gao has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 6: (1 inline comment) File vdsm_tool/passwd.py Li

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2012-07-01 Thread shaohef
ShaoHe Feng has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 6: No score -- To view, visit http://gerrit.ovirt.org/4526 To unsubscribe, visit http://gerrit.ovirt

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2012-07-01 Thread shaohef
ShaoHe Feng has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 6: No score -- To view, visit http://gerrit.ovirt.org/4526 To unsubscribe, visit http://gerrit.ovirt

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2012-07-01 Thread lilei
Lei Li has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 6: (1 inline comment) Shaohe: Please give more details about which vdsm-tool command failed..and the whic

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2012-07-01 Thread shaohef
ShaoHe Feng has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 6: Fails $ sudo ls /sbin/initctl ls: cannot access /sbin/initctl: No such file or directory -- To v

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2012-07-01 Thread shaohef
ShaoHe Feng has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 6: I would prefer that you didn't submit this (2 inline comments) .

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2012-07-01 Thread wenyi
Wenyi Gao has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 6: (1 inline comment) File vdsm_tool/passwd.py Li

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2012-06-29 Thread lilei
Lei Li has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 6: And thanks to Xu He Jie and Zhou ZhengSheng for their help with the hacking of importing vdsm_tool.

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2012-06-20 Thread lilei
Lei Li has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 5: Hi Dan, 1) For the unit tests, I will add it later. 2) For the constants, I will git rid of them if y

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2012-06-19 Thread lilei
Lei Li has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 4: (1 inline comment) File vdsm-tool/unregister.py L

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2012-06-19 Thread lilei
Lei Li has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 4: (6 inline comments) File vdsm/constants.py.in Lin

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2012-06-19 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 4: I would prefer that you didn't submit this (6 inline comments) much better, but there's still

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2012-06-12 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 2: I would prefer that you didn't submit this (2 inline comments) ..

Change in vdsm[master]: Move and encapsulate preun section into vdsm-tool

2012-06-11 Thread lilei
Lei Li has posted comments on this change. Change subject: Move and encapsulate preun section into vdsm-tool .. Patch Set 1: (3 inline comments) File vdsm.spec.in Line 372: %