Change in vdsm[master]: network: restore sriov devices number of vfs

2015-04-30 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: network: restore sriov devices number of vfs
..


Patch Set 9:

* 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.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/40088
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I76b898019840ffe65939ffad4a1e98829ad3c887
Gerrit-PatchSet: 9
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Francesco Romani from...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Martin Polednik mpoled...@redhat.com
Gerrit-Reviewer: Ondřej Svoboda osvob...@redhat.com
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]: network: wait for all udev events after configuring SRIOV

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: network: wait for all udev events after configuring SRIOV
..


Patch Set 2: -Verified

Build Started (1/3) - 
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18405/

-- 
To view, visit https://gerrit.ovirt.org/40400
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I04d63ca35aab585051c3d0a5f2652dbf2f91b080
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Martin Polednik mpoled...@redhat.com
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]: vm: disappeared domain are not ready for commands

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: vm: disappeared domain are not ready for commands
..


Patch Set 29: -Verified

Build Started (1/2) - 
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18402/

-- 
To view, visit https://gerrit.ovirt.org/39370
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I63ba190b2f2830292ec282c1ec5c8eb8e42f3530
Gerrit-PatchSet: 29
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: sampling: more detailed stale sampling reporting

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: sampling: more detailed stale sampling reporting
..


Patch Set 3: -Verified

Build Started (1/2) - 
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18397/

-- 
To view, visit https://gerrit.ovirt.org/40390
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I00cb8f602fc3f26d62dd34bafe05b8169e184b91
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: periodic: simplify per_vm_operation

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: periodic: simplify per_vm_operation
..


Patch Set 5: -Verified

Build Started (1/2) - 
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18403/

-- 
To view, visit https://gerrit.ovirt.org/40327
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I27bff3c36b0507eaa77383083e224478d7e3861c
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: gluster: List activate-on-create volume snapshot option

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: gluster: List activate-on-create volume snapshot option
..


Patch Set 1:

Build Started (2/2) - 
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1637/

-- 
To view, visit https://gerrit.ovirt.org/40416
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I50ad0f482fc3d65e489c0a5c27212b84058c4c73
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
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]: vm: disappeared domain are not ready for commands

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: vm: disappeared domain are not ready for commands
..


Patch Set 29: Verified-1

Build Failed 

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1633/ : FAILURE

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18402/ : SUCCESS

-- 
To view, visit https://gerrit.ovirt.org/39370
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I63ba190b2f2830292ec282c1ec5c8eb8e42f3530
Gerrit-PatchSet: 29
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: periodic: simplify per_vm_operation

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: periodic: simplify per_vm_operation
..


Patch Set 5: Verified-1

Build Failed 

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1634/ : FAILURE

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18403/ : SUCCESS

-- 
To view, visit https://gerrit.ovirt.org/40327
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I27bff3c36b0507eaa77383083e224478d7e3861c
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: spec: Drop el6 support

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: spec: Drop el6 support
..


Patch Set 2: Verified-1

Build Failed 

http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc21_created/878/ : 
FAILURE

http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc22_created/104/ : 
FAILURE

http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-el7_created/887/ : 
FAILURE

http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc20_created/1432/ 
: FAILURE

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1641/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18410/ : SUCCESS

-- 
To view, visit https://gerrit.ovirt.org/40419
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I52bf73f539d9036fbc3468b4faf8a8d6556c9560
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov dkuzn...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: Yeela Kaplan ykap...@redhat.com
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]: tool: Remove upstart handling in libvirt configurator

2015-04-30 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: tool: Remove upstart handling in libvirt configurator
..


Patch Set 2:

* 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.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/40420
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic71430103cb04a5636b4b41371852cbf68495db7
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov dkuzn...@redhat.com
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]: spec: Drop el6 support

2015-04-30 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: spec: Drop el6 support
..


Patch Set 2:

* 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.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/40419
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I52bf73f539d9036fbc3468b4faf8a8d6556c9560
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov dkuzn...@redhat.com
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]: network: restore sriov devices number of vfs

2015-04-30 Thread mpolednik
Martin Polednik has posted comments on this change.

Change subject: network: restore sriov devices number of vfs
..


Patch Set 7:

(1 comment)

https://gerrit.ovirt.org/#/c/40088/7/vdsm/network/api.py
File vdsm/network/api.py:

Line 550: with open(os.path.join(dir_path, device_name), 'w') as f:
Line 551: f.write(str(numvfs))
Line 552: 
Line 553: 
Line 554: def changeNumvfs(device_name, num_vfs):
 that's intended, since this is new code, and a parameter name. I will actua
'numvfs' is the name used everywhere in kernel and documentation, I don't see a 
reason to break the systemwide consistency. The method, according to pep8 and 
this consistency, should be called 'change_numvfs' and the formal parameter 
'numvfs'.
Line 555: Change number of virtual functions of a device in a 'safe' 
way. If the
Line 556: connectivity to engine is lost as a consequence, the value is 
restored.
Line 557: Since calling this verb is currently blocked by engine if this 
device is
Line 558: already in use, the logic does not attempt to do anything 
further, such as


-- 
To view, visit https://gerrit.ovirt.org/40088
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I76b898019840ffe65939ffad4a1e98829ad3c887
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Francesco Romani from...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Martin Polednik mpoled...@redhat.com
Gerrit-Reviewer: Ondřej Svoboda osvob...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: network: restore sriov devices number of vfs

2015-04-30 Thread mpolednik
Martin Polednik has posted comments on this change.

Change subject: network: restore sriov devices number of vfs
..


Patch Set 9: Code-Review-1

Please do not create stack naming inconsistence with num_vfs.

-- 
To view, visit https://gerrit.ovirt.org/40088
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I76b898019840ffe65939ffad4a1e98829ad3c887
Gerrit-PatchSet: 9
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Francesco Romani from...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Martin Polednik mpoled...@redhat.com
Gerrit-Reviewer: Ondřej Svoboda osvob...@redhat.com
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]: sampling: hoststats: rationalize getInterfaceStats

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: sampling: hoststats: rationalize getInterfaceStats
..


Patch Set 1:

Build Successful 

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1645/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18414/ : SUCCESS

-- 
To view, visit https://gerrit.ovirt.org/40426
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I556149271fb2786a6ff7341642989ea767eb2392
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: network: restore sriov devices number of vfs

2015-04-30 Thread mpolednik
Martin Polednik has posted comments on this change.

Change subject: network: restore sriov devices number of vfs
..


Patch Set 9:

(Small correction: numvfs is used for documentation and generic pci driver, the 
drivers themself use num_vfs. I still vote for numvfs due to consistence and 
pci/iov.c compatibility)

-- 
To view, visit https://gerrit.ovirt.org/40088
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I76b898019840ffe65939ffad4a1e98829ad3c887
Gerrit-PatchSet: 9
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Francesco Romani from...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Martin Polednik mpoled...@redhat.com
Gerrit-Reviewer: Ondřej Svoboda osvob...@redhat.com
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]: packaging: Move libvirt_password out of PKI

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: packaging: Move libvirt_password out of PKI
..


Patch Set 3:

Build Started (4/6) - 
http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc20_created/1434/

-- 
To view, visit https://gerrit.ovirt.org/37049
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia7b5f81e4320834711e52e39f092c6e90796a485
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov dkuzn...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
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]: spec: Drop el6 support

2015-04-30 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: spec: Drop el6 support
..


Patch Set 3:

* 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.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/40419
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I52bf73f539d9036fbc3468b4faf8a8d6556c9560
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov dkuzn...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: Yeela Kaplan ykap...@redhat.com
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]: packaging: Move libvirt_password out of PKI

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: packaging: Move libvirt_password out of PKI
..


Patch Set 3: Verified-1

Build Failed 

http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc22_created/106/ : 
FAILURE

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1651/ : FAILURE

http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc21_created/880/ : 
SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-el7_created/889/ : 
SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc20_created/1434/ 
: SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18420/ : SUCCESS

-- 
To view, visit https://gerrit.ovirt.org/37049
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia7b5f81e4320834711e52e39f092c6e90796a485
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov dkuzn...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
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]: spec: Drop el6 support

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: spec: Drop el6 support
..


Patch Set 3:

Build Started (5/6) - 
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1653/

-- 
To view, visit https://gerrit.ovirt.org/40419
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I52bf73f539d9036fbc3468b4faf8a8d6556c9560
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov dkuzn...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: Yeela Kaplan ykap...@redhat.com
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]: migration: Use jsonrpc during migration of vms

2015-04-30 Thread piotr . kliczewski
Piotr Kliczewski has posted comments on this change.

Change subject: migration: Use jsonrpc during migration of vms
..


Patch Set 9: Code-Review+1

-- 
To view, visit https://gerrit.ovirt.org/36701
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie38334e6cdcc4d7899bd5e836b3196567fc0bfd8
Gerrit-PatchSet: 9
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yeela Kaplan ykap...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Dima Kuznetsov dkuzn...@redhat.com
Gerrit-Reviewer: Francesco Romani from...@redhat.com
Gerrit-Reviewer: Michal Skrivanek michal.skriva...@redhat.com
Gerrit-Reviewer: Piotr Kliczewski piotr.kliczew...@gmail.com
Gerrit-Reviewer: Saggi Mizrahi smizr...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: Yeela Kaplan ykap...@redhat.com
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]: sslutils: move createSSLContext from clientIF

2015-04-30 Thread piotr . kliczewski
Piotr Kliczewski has posted comments on this change.

Change subject: sslutils: move createSSLContext from clientIF
..


Patch Set 5: Code-Review+1

-- 
To view, visit https://gerrit.ovirt.org/36699
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I8c7e324759892232d16a4913314f1dfa76ea85b8
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yeela Kaplan ykap...@redhat.com
Gerrit-Reviewer: Francesco Romani from...@redhat.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Piotr Kliczewski piotr.kliczew...@gmail.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: Yaniv Bronheim ybron...@redhat.com
Gerrit-Reviewer: Yeela Kaplan ykap...@redhat.com
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]: network: restore sriov devices number of vfs

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: network: restore sriov devices number of vfs
..


Patch Set 9:

Build Started (2/3) - 
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1635/

-- 
To view, visit https://gerrit.ovirt.org/40088
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I76b898019840ffe65939ffad4a1e98829ad3c887
Gerrit-PatchSet: 9
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Francesco Romani from...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Martin Polednik mpoled...@redhat.com
Gerrit-Reviewer: Ondřej Svoboda osvob...@redhat.com
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]: virt: rename SampleVMs for clarity

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: virt: rename SampleVMs for clarity
..


Patch Set 4:

Build Successful 

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18401/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1632/ : SUCCESS

-- 
To view, visit https://gerrit.ovirt.org/40370
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iadf91e0e3b7ffd4d76eff36e4906441496adb71c
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: network: wait for all udev events after configuring SRIOV

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: network: wait for all udev events after configuring SRIOV
..


Patch Set 2:

Build Started (2/3) - 
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1636/

-- 
To view, visit https://gerrit.ovirt.org/40400
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I04d63ca35aab585051c3d0a5f2652dbf2f91b080
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Martin Polednik mpoled...@redhat.com
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]: gluster: List activate-on-create volume snapshot option

2015-04-30 Thread shtripat
Shubhendu Tripathi has uploaded a new change for review.

Change subject: gluster: List activate-on-create volume snapshot option
..

gluster: List activate-on-create volume snapshot option

Modified the verb glusterSnapshotConfigList to list
option activate-on-create as well.

Change-Id: I50ad0f482fc3d65e489c0a5c27212b84058c4c73
Signed-off-by: Shubhendu Tripathi shtri...@redhat.com
Bug-URL: https://bugzilla.redhat.com/1216056
---
M vdsm/gluster/cli.py
M vdsm/rpc/vdsmapi-gluster-schema.json
2 files changed, 7 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/16/40416/1

diff --git a/vdsm/gluster/cli.py b/vdsm/gluster/cli.py
index c50b877..8d68271 100644
--- a/vdsm/gluster/cli.py
+++ b/vdsm/gluster/cli.py
@@ -1390,7 +1390,8 @@
 
 returns {'system':{'snap-max-hard-limit': 'hardlimit',
'snap-max-soft-limit': 'softLimit',
-   'auto-delete': 'enable/disable'},
+   'auto-delete': 'enable/disable',
+   'activate-on-create': 'enable/disable'},
  'volume':{'name' :
   {'snap-max-hard-limit: 'hardlimit'}
   }
@@ -1403,6 +1404,8 @@
 'snapConfig/systemConfig/softLimit').text
 systemConfig['auto-delete'] = tree.find(
 'snapConfig/systemConfig/autoDelete').text
+systemConfig['activate-on-create'] = tree.find(
+'snapConfig/systemConfig/activateOnCreate').text
 
 volumeConfig = {}
 for el in tree.findall('snapConfig/volumeConfig/volume'):
diff --git a/vdsm/rpc/vdsmapi-gluster-schema.json 
b/vdsm/rpc/vdsmapi-gluster-schema.json
index 935adff..7f7c936 100644
--- a/vdsm/rpc/vdsmapi-gluster-schema.json
+++ b/vdsm/rpc/vdsmapi-gluster-schema.json
@@ -1865,11 +1865,13 @@
 #
 # @auto-delete: Auto delete enabled/disabled
 #
+# @activate-on-create:  Auto activate enabled/disabled
+#
 # @Since: 4.17.0
 ##
 {'type': 'SystemSnapshotConfig',
  'data': {'snap-max-hard-limit': 'str', 'snap-max-soft-limit': 'str',
-  'auto-delete': 'str'}}
+  'auto-delete': 'str', 'activate-on-create': 'str'}}
 
 ##
 #


-- 
To view, visit https://gerrit.ovirt.org/40416
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I50ad0f482fc3d65e489c0a5c27212b84058c4c73
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: WIP: rename stats cache to vm stats cache

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: WIP: rename stats cache to vm stats cache
..


Patch Set 4: Verified-1

Build Failed 

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1631/ : FAILURE

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18400/ : SUCCESS

-- 
To view, visit https://gerrit.ovirt.org/40371
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If38c49f686dfc2bc0994d444ff24c7736f2e951b
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: gluster: List activate-on-create volume snapshot option

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: gluster: List activate-on-create volume snapshot option
..


Patch Set 1:

Build Started (1/2) - 
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18406/

-- 
To view, visit https://gerrit.ovirt.org/40416
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I50ad0f482fc3d65e489c0a5c27212b84058c4c73
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
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]: virt: move libvirt helpers in a module

2015-04-30 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: virt: move libvirt helpers in a module
..


Patch Set 24:

* 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.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/39332
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I35ceeb12bc96bea12281ce9c9a1481342dd3afa9
Gerrit-PatchSet: 24
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Francesco Romani from...@redhat.com
Gerrit-Reviewer: Martin Polednik mpoled...@redhat.com
Gerrit-Reviewer: Vinzenz Feenstra vfeen...@redhat.com
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]: sampling: hoststats: kill sample() method

2015-04-30 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: sampling: hoststats: kill sample() method
..


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.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/40425
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I8b8903c27c34a4ba7f737aba1d88d67abec08816
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: sampling: hoststats: switch to SampleWindow

2015-04-30 Thread fromani
Francesco Romani has uploaded a new change for review.

Change subject: sampling: hoststats: switch to SampleWindow
..

sampling: hoststats: switch to SampleWindow

HostStatsThread has its own sampling storage code,
remniscent of VmStatsThread before the switch
of SampleWindow.

The requirements and the use cases are already almost
completely covered by SampleWindow, so this patch
ports HostStatsThread to it, to reduce duplication.

Change-Id: I5ec5712ee1fb69ed3d8dbc28ca4e6511998270c9
Signed-off-by: Francesco Romani from...@redhat.com
---
M vdsm/virt/sampling.py
1 file changed, 4 insertions(+), 7 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/28/40428/1

diff --git a/vdsm/virt/sampling.py b/vdsm/virt/sampling.py
index d59ce2c..cbe53ef 100644
--- a/vdsm/virt/sampling.py
+++ b/vdsm/virt/sampling.py
@@ -555,7 +555,6 @@
 
 A thread that periodically samples host statistics.
 
-AVERAGING_WINDOW = 5
 _CONNLOG = logging.getLogger('connectivity')
 
 def __init__(self, log):
@@ -565,7 +564,7 @@
 self.daemon = True
 self._log = log
 self._stopEvent = threading.Event()
-self._samples = []
+self._samples = SampleWindow(size=5)
 
 self._pid = os.getpid()
 self._ncpus = max(os.sysconf('SC_NPROCESSORS_ONLN'), 1)
@@ -584,15 +583,13 @@
 try:
 sample = HostSample(self._pid)
 self._samples.append(sample)
-if len(self._samples) == 1:
+prev, last = self._samples.last_pair()
+if prev is None:
 self._CONNLOG.debug('%s', sample.to_connlog())
 else:
-diff = sample.connlog_diff(self._samples[-2])
+diff = sample.connlog_diff(prev)
 if diff:
 self._CONNLOG.debug('%s', diff)
-
-if len(self._samples)  self.AVERAGING_WINDOW:
-self._samples.pop(0)
 except TimeoutError:
 self._log.exception(Timeout while sampling stats)
 self._stopEvent.wait(self._sampleInterval)


-- 
To view, visit https://gerrit.ovirt.org/40428
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5ec5712ee1fb69ed3d8dbc28ca4e6511998270c9
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: sampling: hoststats: rationalize getInterfaceStats

2015-04-30 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: sampling: hoststats: rationalize getInterfaceStats
..


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.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/40426
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I556149271fb2786a6ff7341642989ea767eb2392
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: sampling: hoststats: rationalize getInterfaceStats

2015-04-30 Thread fromani
Francesco Romani has uploaded a new change for review.

Change subject: sampling: hoststats: rationalize getInterfaceStats
..

sampling: hoststats: rationalize getInterfaceStats

HostStatsThread._getInterfaceStats() is supposed
to gather network stats.

But this method also setups the empty stats with
default value.
This initialization doesn't belong here, so this
patch moves this code outside this method, in the
calling one.

Change-Id: I556149271fb2786a6ff7341642989ea767eb2392
Signed-off-by: Francesco Romani from...@redhat.com
---
M vdsm/virt/sampling.py
1 file changed, 16 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/26/40426/1

diff --git a/vdsm/virt/sampling.py b/vdsm/virt/sampling.py
index 7349b9f..710bba9 100644
--- a/vdsm/virt/sampling.py
+++ b/vdsm/virt/sampling.py
@@ -604,13 +604,25 @@
 return None
 
 def get(self):
-stats = self._getInterfacesStats()
-stats['cpuSysVdsmd'] = stats['cpuUserVdsmd'] = 0.0
-stats['elapsedTime'] = int(time.time() - self.startTime)
+stats = {
+'cpuUser': 0.0,
+'cpuSys': 0.0,
+'cpuIdle': 100.0,
+'rxRate': 0.0,
+'txRate': 0.0,
+'cpuSysVdsmd': 0.0,
+'cpuUserVdsmd': 0.0,
+'elapsedTime': int(time.time() - self.startTime)
+}
+
 if len(self._samples)  2:
 return stats
+
+stats.update(self._getInterfacesStats())
+
 hs0, hs1 = self._samples[0], self._samples[-1]
 interval = hs1.timestamp - hs0.timestamp
+
 jiffies = (hs1.pidcpu.user - hs0.pidcpu.user) % (2 ** 32)
 stats['cpuUserVdsmd'] = jiffies / interval
 jiffies = (hs1.pidcpu.sys - hs0.pidcpu.sys) % (2 ** 32)
@@ -680,8 +692,7 @@
 * rxRate
 * txRate
 
-stats = {'cpuUser': 0.0, 'cpuSys': 0.0, 'cpuIdle': 100.0,
- 'rxRate': 0.0, 'txRate': 0.0}
+stats = {}
 if len(self._samples)  2:
 return stats
 hs0, hs1 = self._samples[0], self._samples[-1]


-- 
To view, visit https://gerrit.ovirt.org/40426
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I556149271fb2786a6ff7341642989ea767eb2392
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: network: wait for all udev events after configuring SRIOV

2015-04-30 Thread mpolednik
Martin Polednik has posted comments on this change.

Change subject: network: wait for all udev events after configuring SRIOV
..


Patch Set 2: Code-Review+1

-- 
To view, visit https://gerrit.ovirt.org/40400
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I04d63ca35aab585051c3d0a5f2652dbf2f91b080
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Martin Polednik mpoled...@redhat.com
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]: WIP: sampling: make _getCpuCoresStats a function

2015-04-30 Thread fromani
Francesco Romani has uploaded a new change for review.

Change subject: WIP: sampling: make _getCpuCoresStats a function
..

WIP: sampling: make _getCpuCoresStats a function

Change-Id: If602a84096db7aa7d1a6672b84d62287e08b0ac2
Signed-off-by: Francesco Romani from...@redhat.com
---
M vdsm/virt/sampling.py
1 file changed, 31 insertions(+), 31 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/29/40429/1

diff --git a/vdsm/virt/sampling.py b/vdsm/virt/sampling.py
index cbe53ef..f61514b 100644
--- a/vdsm/virt/sampling.py
+++ b/vdsm/virt/sampling.py
@@ -649,40 +649,10 @@
 stats['bootTime'] = self._boot_time()
 
 stats['numaNodeMemFree'] = hs1.numaNodeMem.nodesMemSample
-stats['cpuStatistics'] = self._getCpuCoresStats()
+stats['cpuStatistics'] = _get_cpu_core_stats(hs0, hs1)
 
 stats['v2vJobs'] = v2v.get_jobs_status()
 return stats
-
-def _getCpuCoresStats(self):
-
-:returns: a dict that with the following formats:
-
-{'cpuId': {'numaNodeIndex': int, 'cpuSys': 'str',
- 'cpuIdle': 'str', 'cpuUser': 'str'}, ...}
-
-cpuCoreStats = {}
-for nodeIndex, numaNode in caps.getNumaTopology().iteritems():
-cpuCores = numaNode['cpus']
-for cpuCore in cpuCores:
-coreStat = {}
-coreStat['nodeIndex'] = int(nodeIndex)
-hs0, hs1 = self._samples[0], self._samples[-1]
-interval = hs1.timestamp - hs0.timestamp
-jiffies = (hs1.cpuCores.getCoreSample(cpuCore)['user'] -
-   hs0.cpuCores.getCoreSample(cpuCore)['user']) % \
-(2 ** 32)
-coreStat['cpuUser'] = (%.2f % (jiffies / interval))
-jiffies = (hs1.cpuCores.getCoreSample(cpuCore)['sys'] -
-   hs0.cpuCores.getCoreSample(cpuCore)['sys']) % \
-(2 ** 32)
-coreStat['cpuSys'] = (%.2f % (jiffies / interval))
-coreStat['cpuIdle'] = (%.2f %
-   max(0.0, 100.0 -
-   float(coreStat['cpuUser']) -
-   float(coreStat['cpuSys'])))
-cpuCoreStats[str(cpuCore)] = coreStat
-return cpuCoreStats
 
 def _getInterfacesStats(self):
 
@@ -754,6 +724,36 @@
 return stats
 
 
+def _get_cpu_core_stats(hs0, hs1):
+
+:returns: a dict that with the following formats:
+
+{'cpuId': {'numaNodeIndex': int, 'cpuSys': 'str',
+ 'cpuIdle': 'str', 'cpuUser': 'str'}, ...}
+
+cpuCoreStats = {}
+for nodeIndex, numaNode in caps.getNumaTopology().iteritems():
+cpuCores = numaNode['cpus']
+for cpuCore in cpuCores:
+coreStat = {}
+coreStat['nodeIndex'] = int(nodeIndex)
+interval = hs1.timestamp - hs0.timestamp
+jiffies = (hs1.cpuCores.getCoreSample(cpuCore)['user'] -
+   hs0.cpuCores.getCoreSample(cpuCore)['user']) % \
+(2 ** 32)
+coreStat['cpuUser'] = (%.2f % (jiffies / interval))
+jiffies = (hs1.cpuCores.getCoreSample(cpuCore)['sys'] -
+   hs0.cpuCores.getCoreSample(cpuCore)['sys']) % \
+(2 ** 32)
+coreStat['cpuSys'] = (%.2f % (jiffies / interval))
+coreStat['cpuIdle'] = (%.2f %
+   max(0.0, 100.0 -
+   float(coreStat['cpuUser']) -
+   float(coreStat['cpuSys'])))
+cpuCoreStats[str(cpuCore)] = coreStat
+return cpuCoreStats
+
+
 def _getLinkSpeed(dev):
 if dev.isNIC():
 speed = netinfo.nicSpeed(dev.name)


-- 
To view, visit https://gerrit.ovirt.org/40429
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If602a84096db7aa7d1a6672b84d62287e08b0ac2
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: sampling: hoststats: kill sample() method

2015-04-30 Thread fromani
Francesco Romani has uploaded a new change for review.

Change subject: sampling: hoststats: kill sample() method
..

sampling: hoststats: kill sample() method

HostStatsThread.sample() is a two-line method used only once.
Remove it for the sake of code brevity, with no reduction
of code clarity, possibly an increase of.

Change-Id: I8b8903c27c34a4ba7f737aba1d88d67abec08816
Signed-off-by: Francesco Romani from...@redhat.com
---
M vdsm/virt/sampling.py
1 file changed, 1 insertion(+), 5 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/25/40425/1

diff --git a/vdsm/virt/sampling.py b/vdsm/virt/sampling.py
index 8878120..7349b9f 100644
--- a/vdsm/virt/sampling.py
+++ b/vdsm/virt/sampling.py
@@ -569,17 +569,13 @@
 def stop(self):
 self._stopEvent.set()
 
-def sample(self):
-hs = HostSample(self._pid)
-return hs
-
 def run(self):
 try:
 # wait a bit before starting to sample
 time.sleep(self._sampleInterval)
 while not self._stopEvent.isSet():
 try:
-sample = self.sample()
+sample = HostSample(self._pid)
 self._samples.append(sample)
 if len(self._samples) == 1:
 self._CONNLOG.debug('%s', sample.to_connlog())


-- 
To view, visit https://gerrit.ovirt.org/40425
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8b8903c27c34a4ba7f737aba1d88d67abec08816
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: sampling: less verbose debug in _getInterfaceStats

2015-04-30 Thread fromani
Francesco Romani has uploaded a new change for review.

Change subject: sampling: less verbose debug in _getInterfaceStats
..

sampling: less verbose debug in _getInterfaceStats

HostStatsThread._getInterfaceStats needs, among other
things, to compute network tx/rx rate.
To help troubleshoot rate miscalculation, there is
a debug log which shows all the samples used for
computation.

The problem here is that the log cycles over the
internal samples. We want to use SampleWindow
in HostStatsThread to reduce code duplication,
but SampleWindow doesn't give access to samples yet.

To move further, we drop this detailed log for the time
being. Future patches may want to reintroduce it
on top of (enhanced) SampleWindow.

Change-Id: I0a6063c9ac6f9f477a03881e32d0c3821038092f
Signed-off-by: Francesco Romani from...@redhat.com
---
M vdsm/virt/sampling.py
1 file changed, 1 insertion(+), 7 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/24/40424/1

diff --git a/vdsm/virt/sampling.py b/vdsm/virt/sampling.py
index f5640d0..8878120 100644
--- a/vdsm/virt/sampling.py
+++ b/vdsm/virt/sampling.py
@@ -710,13 +710,7 @@
 if txRate  100 or rxRate  100:
 txRate = min(txRate, 100.0)
 rxRate = min(rxRate, 100.0)
-self._log.debug('Rate above 100%%. DEBUG: ifid %s interval: '
-'%s thisRx %s thisTx %s samples %s', ifid,
-interval, thisRx, thisTx,
-[(hs.timestamp, hs.interfaces[ifid].rx,
- hs.interfaces[ifid].tx)
- for hs in self._samples if
- ifid in hs.interfaces])
+self._log.debug('Rate above 100%%.')
 iface = hs1.interfaces[ifid]
 stats['network'][ifid] = {'name': ifid, 'speed': str(ifrate),
   'rxDropped': str(iface.rxDropped),


-- 
To view, visit https://gerrit.ovirt.org/40424
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I0a6063c9ac6f9f477a03881e32d0c3821038092f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: WIP: sampling: add last_pair() method to SampleWindow

2015-04-30 Thread fromani
Francesco Romani has uploaded a new change for review.

Change subject: WIP: sampling: add last_pair() method to SampleWindow
..

WIP: sampling: add last_pair() method to SampleWindow

HostStats thread need to access the last two stored samples.
To cope with this requirement, add last_pair() to SampleWindow class.

Change-Id: If73a6c0dc3503f6bcb02300517bbd2ff1424bcc3
Signed-off-by: Francesco Romani from...@redhat.com
---
M vdsm/virt/sampling.py
1 file changed, 7 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/27/40427/1

diff --git a/vdsm/virt/sampling.py b/vdsm/virt/sampling.py
index 710bba9..d59ce2c 100644
--- a/vdsm/virt/sampling.py
+++ b/vdsm/virt/sampling.py
@@ -366,6 +366,13 @@
 _, last_sample = self._samples[-1]
 return last_sample
 
+def last_pair(self):
+if len(self._samples)  2:
+return None, None
+_, prev_last_sample = self._samples[-2]
+_, last_sample = self._samples[-1]
+return prev_last_sample, last_sample
+
 
 StatsSample = namedtuple('StatsSample',
  ['first_value', 'last_value',


-- 
To view, visit https://gerrit.ovirt.org/40427
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If73a6c0dc3503f6bcb02300517bbd2ff1424bcc3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: packaging: Move libvirt_password out of PKI

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: packaging: Move libvirt_password out of PKI
..


Patch Set 3:

Build Started (2/6) - 
http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-el7_created/889/

-- 
To view, visit https://gerrit.ovirt.org/37049
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia7b5f81e4320834711e52e39f092c6e90796a485
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov dkuzn...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
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]: WIP

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: WIP
..


Patch Set 1:

Build Started (2/2) - 
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1650/

-- 
To view, visit https://gerrit.ovirt.org/40431
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I39c2e6e4bca286a513992b7231f1356e8dd871a1
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: network: wait for all udev events after configuring SRIOV

2015-04-30 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: network: wait for all udev events after configuring SRIOV
..


Patch Set 2:

* 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.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/40400
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I04d63ca35aab585051c3d0a5f2652dbf2f91b080
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Martin Polednik mpoled...@redhat.com
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]: gluster: List activate-on-create volume snapshot option

2015-04-30 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: gluster: List activate-on-create volume snapshot option
..


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.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/40416
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I50ad0f482fc3d65e489c0a5c27212b84058c4c73
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: List activate-on-create volume snapshot option

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: gluster: List activate-on-create volume snapshot option
..


Patch Set 1: Verified-1

Build Failed 

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1637/ : FAILURE

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18406/ : SUCCESS

-- 
To view, visit https://gerrit.ovirt.org/40416
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I50ad0f482fc3d65e489c0a5c27212b84058c4c73
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
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]: network: wait for all udev events after configuring SRIOV

2015-04-30 Thread danken
Dan Kenigsberg has posted comments on this change.

Change subject: network: wait for all udev events after configuring SRIOV
..


Patch Set 2: Code-Review+2

-- 
To view, visit https://gerrit.ovirt.org/40400
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I04d63ca35aab585051c3d0a5f2652dbf2f91b080
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Martin Polednik mpoled...@redhat.com
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]: tool: Remove upstart handling in libvirt configurator

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: tool: Remove upstart handling in libvirt configurator
..


Patch Set 2: Verified-1

Build Failed 

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1640/ : FAILURE

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18409/ : SUCCESS

-- 
To view, visit https://gerrit.ovirt.org/40420
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic71430103cb04a5636b4b41371852cbf68495db7
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov dkuzn...@redhat.com
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]: Move udevadm to lib/vdsm

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: Move udevadm to lib/vdsm
..


Patch Set 5:

Build Started (8/9) - 
http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-el7-x86_64_merged/577/

-- 
To view, visit https://gerrit.ovirt.org/40119
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I113bd85820805e60cbf6a7266ed788ec3ba48637
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
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[ovirt-3.5]: vdsm: Fix memory leak in netlink/link.py

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: vdsm: Fix memory leak in netlink/link.py
..


Patch Set 2:

Build Started (4/4) - 
http://jenkins.ovirt.org/job/vdsm_3.5_create-rpms-fc20-x86_64_merged/219/

-- 
To view, visit https://gerrit.ovirt.org/40134
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I883ce5f5c1ad0c0813d5d659de71761126a8fc80
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Petr Horáček phora...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: John Taylor jtt77...@yahoo.com
Gerrit-Reviewer: Petr Horáček phora...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
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]: virt: move libvirt helpers in a module

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: virt: move libvirt helpers in a module
..


Patch Set 24:

Build Started (5/6) - 
http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc22_created/105/

-- 
To view, visit https://gerrit.ovirt.org/39332
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I35ceeb12bc96bea12281ce9c9a1481342dd3afa9
Gerrit-PatchSet: 24
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Francesco Romani from...@redhat.com
Gerrit-Reviewer: Martin Polednik mpoled...@redhat.com
Gerrit-Reviewer: Vinzenz Feenstra vfeen...@redhat.com
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]: stomp: add an option to create a stomp client

2015-04-30 Thread piotr . kliczewski
Piotr Kliczewski has posted comments on this change.

Change subject: stomp: add an option to create a stomp client
..


Patch Set 6: Code-Review+1

-- 
To view, visit https://gerrit.ovirt.org/38491
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5ad4c79130c0ca1c4a5bd01343eafd3d8bf36231
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yeela Kaplan ykap...@redhat.com
Gerrit-Reviewer: Dima Kuznetsov dkuzn...@redhat.com
Gerrit-Reviewer: Francesco Romani from...@redhat.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Piotr Kliczewski piotr.kliczew...@gmail.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: Yaniv Bronheim ybron...@redhat.com
Gerrit-Reviewer: Yeela Kaplan ykap...@redhat.com
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]: WIP: virt: better timeout for periodic operations

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: WIP: virt: better timeout for periodic operations
..


Patch Set 32:

Build Started (1/6) - 
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18396/

-- 
To view, visit https://gerrit.ovirt.org/39333
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I735d314532c6852889e3ac188651acff044888ed
Gerrit-PatchSet: 32
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: WIP: fix again stats_cache cleaning

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: WIP: fix again stats_cache cleaning
..


Patch Set 2: -Verified

Build Started (1/2) - 
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1625/

-- 
To view, visit https://gerrit.ovirt.org/40398
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I98d388b4684be61936316ef749707e4be418af94
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: WIP: virt: better timeout for periodic operations

2015-04-30 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: WIP: virt: better timeout for periodic operations
..


Patch Set 32:

* 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.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/39333
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I735d314532c6852889e3ac188651acff044888ed
Gerrit-PatchSet: 32
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: WIP: fix again stats_cache cleaning

2015-04-30 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: WIP: fix again stats_cache cleaning
..


Patch Set 2:

* 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.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/40398
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I98d388b4684be61936316ef749707e4be418af94
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: sampling: introduce expensive checks

2015-04-30 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: sampling: introduce expensive checks
..


Patch Set 3:

* 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.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/40391
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7b3bb707dd60de194eedfc2e3de1efbf05574ff7
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: WIP: virt: better timeout for periodic operations

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: WIP: virt: better timeout for periodic operations
..


Patch Set 32:

Build Started (3/6) - 
http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc22_created/102/

-- 
To view, visit https://gerrit.ovirt.org/39333
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I735d314532c6852889e3ac188651acff044888ed
Gerrit-PatchSet: 32
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: virt: rename SampleVMs for clarity

2015-04-30 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: virt: rename SampleVMs for clarity
..


Patch Set 4:

* 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.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/40370
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iadf91e0e3b7ffd4d76eff36e4906441496adb71c
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: network: restore sriov devices number of vfs

2015-04-30 Thread ibarkan
Ido Barkan has posted comments on this change.

Change subject: network: restore sriov devices number of vfs
..


Patch Set 7:

(3 comments)

https://gerrit.ovirt.org/#/c/40088/7/vdsm/network/api.py
File vdsm/network/api.py:

Line 529: raise ConfigNetworkError(ne.ERR_LOST_CONNECTION,
Line 530:  'connectivity check failed')
Line 531: 
Line 532: 
Line 533: def _update_num_vfs(device_name, num_vfs):
 reintroduced the num_vfs to numvfs consistency
see below comment
Line 534: with open(_SYSFS_SRIOV_NUMVFS.format(device_name), 'w', 0) as f:
Line 535: # Zero needs to be written first in order to remove previous 
VFs.
Line 536: # Trying to just write the number (if n  0 VF's existed 
before)
Line 537: # results in 'write error: Device or resource busy'


Line 550: with open(os.path.join(dir_path, device_name), 'w') as f:
Line 551: f.write(str(numvfs))
Line 552: 
Line 553: 
Line 554: def changeNumvfs(device_name, num_vfs):
 num_vfs - numvfs again
that's intended, since this is new code, and a parameter name. I will actually 
want to rename changeNumvfs to change_num_vfs later.
Line 555: Change number of virtual functions of a device in a 'safe' 
way. If the
Line 556: connectivity to engine is lost as a consequence, the value is 
restored.
Line 557: Since calling this verb is currently blocked by engine if this 
device is
Line 558: already in use, the logic does not attempt to do anything 
further, such as


https://gerrit.ovirt.org/#/c/40088/7/vdsm/vdsm-restore-net-config
File vdsm/vdsm-restore-net-config:

Line 55: missing_devices = []  # devices that no longer exist
Line 56: 
Line 57: if not os.path.exists(_VIRTUAL_FUNCTIONS_PATH):
Line 58: os.makedirs(_VIRTUAL_FUNCTIONS_PATH)
Line 59: return 
 Trailing whitespace
Done
Line 60: 
Line 61: for file_name in os.listdir(_VIRTUAL_FUNCTIONS_PATH):
Line 62: if file_name not in existing_sriov_devices:
Line 63: missing_devices.append(file_name)


-- 
To view, visit https://gerrit.ovirt.org/40088
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I76b898019840ffe65939ffad4a1e98829ad3c887
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Francesco Romani from...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Martin Polednik mpoled...@redhat.com
Gerrit-Reviewer: Ondřej Svoboda osvob...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: vm: disappeared domain are not ready for commands

2015-04-30 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: vm: disappeared domain are not ready for commands
..


Patch Set 29:

* 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.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/39370
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I63ba190b2f2830292ec282c1ec5c8eb8e42f3530
Gerrit-PatchSet: 29
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: WIP: tests: add tests for sampling.SampleVMs

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: WIP: tests: add tests for sampling.SampleVMs
..


Patch Set 18: -Verified

Build Started (1/2) - 
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18395/

-- 
To view, visit https://gerrit.ovirt.org/40053
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id66dbd420ca29d08ae4063dc83b858be34b8940f
Gerrit-PatchSet: 18
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
Gerrit-Reviewer: Francesco Romani from...@redhat.com
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]: periodic: improve isolation in VM dispatching

2015-04-30 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: periodic: improve isolation in VM dispatching
..


Patch Set 4:

* 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.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/40339
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id6ed5f1b7c332cdfe23d78571ac135a2c1714ba7
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: sampling: more detailed stale sampling reporting

2015-04-30 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: sampling: more detailed stale sampling reporting
..


Patch Set 3:

* 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.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/40390
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I00cb8f602fc3f26d62dd34bafe05b8169e184b91
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: WIP: fix again stats_cache cleaning

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: WIP: fix again stats_cache cleaning
..


Patch Set 2:

Build Started (2/2) - 
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18394/

-- 
To view, visit https://gerrit.ovirt.org/40398
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I98d388b4684be61936316ef749707e4be418af94
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: WIP: virt: better timeout for periodic operations

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: WIP: virt: better timeout for periodic operations
..


Patch Set 32:

Build Started (4/6) - 
http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-el7_created/885/

-- 
To view, visit https://gerrit.ovirt.org/39333
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I735d314532c6852889e3ac188651acff044888ed
Gerrit-PatchSet: 32
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: WIP: rename stats cache to vm stats cache

2015-04-30 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: WIP: rename stats cache to vm stats cache
..


Patch Set 4:

* 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.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/40371
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If38c49f686dfc2bc0994d444ff24c7736f2e951b
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: WIP: tests: add tests for sampling.SampleVMs

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: WIP: tests: add tests for sampling.SampleVMs
..


Patch Set 18:

Build Started (2/2) - 
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1626/

-- 
To view, visit https://gerrit.ovirt.org/40053
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id66dbd420ca29d08ae4063dc83b858be34b8940f
Gerrit-PatchSet: 18
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
Gerrit-Reviewer: Francesco Romani from...@redhat.com
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]: WIP: virt: better timeout for periodic operations

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: WIP: virt: better timeout for periodic operations
..


Patch Set 32:

Build Started (2/6) - 
http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc21_created/876/

-- 
To view, visit https://gerrit.ovirt.org/39333
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I735d314532c6852889e3ac188651acff044888ed
Gerrit-PatchSet: 32
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: WIP: virt: better timeout for periodic operations

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: WIP: virt: better timeout for periodic operations
..


Patch Set 32:

Build Started (6/6) - 
http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc20_created/1430/

-- 
To view, visit https://gerrit.ovirt.org/39333
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I735d314532c6852889e3ac188651acff044888ed
Gerrit-PatchSet: 32
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: WIP: virt: better timeout for periodic operations

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: WIP: virt: better timeout for periodic operations
..


Patch Set 32:

Build Started (5/6) - 
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1627/

-- 
To view, visit https://gerrit.ovirt.org/39333
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I735d314532c6852889e3ac188651acff044888ed
Gerrit-PatchSet: 32
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: periodic: simplify per_vm_operation

2015-04-30 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: periodic: simplify per_vm_operation
..


Patch Set 5:

* 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.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/40327
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I27bff3c36b0507eaa77383083e224478d7e3861c
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: Move udevadm to lib/vdsm

2015-04-30 Thread ibarkan
Ido Barkan has posted comments on this change.

Change subject: Move udevadm to lib/vdsm
..


Patch Set 4:

as described above:
verified by importing udevadm from vdsm
 from vdsm import udevadm

-- 
To view, visit https://gerrit.ovirt.org/40119
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I113bd85820805e60cbf6a7266ed788ec3ba48637
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: periodic: improve isolation in VM dispatching

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: periodic: improve isolation in VM dispatching
..


Patch Set 4:

Build Started (2/2) - 
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1629/

-- 
To view, visit https://gerrit.ovirt.org/40339
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id6ed5f1b7c332cdfe23d78571ac135a2c1714ba7
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: Move udevadm to lib/vdsm

2015-04-30 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: Move udevadm to lib/vdsm
..


Patch Set 5:

* Update tracker::IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url found.

-- 
To view, visit https://gerrit.ovirt.org/40119
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I113bd85820805e60cbf6a7266ed788ec3ba48637
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: Move udevadm to lib/vdsm

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: Move udevadm to lib/vdsm
..


Patch Set 5:

Build Started (4/9) - 
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc20-x86_64_merged/979/

-- 
To view, visit https://gerrit.ovirt.org/40119
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I113bd85820805e60cbf6a7266ed788ec3ba48637
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
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]: Move udevadm to lib/vdsm

2015-04-30 Thread danken
Dan Kenigsberg has submitted this change and it was merged.

Change subject: Move udevadm to lib/vdsm
..


Move udevadm to lib/vdsm

udevadm is a handy tool that other parts of vdsm could use beside
storage.

Change-Id: I113bd85820805e60cbf6a7266ed788ec3ba48637
Signed-off-by: Ido Barkan ibar...@redhat.com
Reviewed-on: https://gerrit.ovirt.org/40119
Reviewed-by: Dan Kenigsberg dan...@redhat.com
Reviewed-by: Nir Soffer nsof...@redhat.com
---
M debian/vdsm-python.install
M debian/vdsm.install
M lib/vdsm/Makefile.am
R lib/vdsm/udevadm.py
M vdsm.spec.in
M vdsm/storage/Makefile.am
M vdsm/storage/multipath.py
M vdsm/storage/storageServer.py
8 files changed, 6 insertions(+), 6 deletions(-)

Approvals:
  Nir Soffer: Looks good to me, but someone else must approve
  Ido Barkan: Verified
  Dan Kenigsberg: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/40119
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I113bd85820805e60cbf6a7266ed788ec3ba48637
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: Move udevadm to lib/vdsm

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: Move udevadm to lib/vdsm
..


Patch Set 5:

Build Started (3/9) - 
http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-fc22-x86_64_merged/120/

-- 
To view, visit https://gerrit.ovirt.org/40119
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I113bd85820805e60cbf6a7266ed788ec3ba48637
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
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]: Move udevadm to lib/vdsm

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: Move udevadm to lib/vdsm
..


Patch Set 5:

Build Started (6/9) - 
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc21-x86_64_merged/944/

-- 
To view, visit https://gerrit.ovirt.org/40119
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I113bd85820805e60cbf6a7266ed788ec3ba48637
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
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]: Move udevadm to lib/vdsm

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: Move udevadm to lib/vdsm
..


Patch Set 5:

Build Started (7/9) - 
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc22-x86_64_merged/120/

-- 
To view, visit https://gerrit.ovirt.org/40119
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I113bd85820805e60cbf6a7266ed788ec3ba48637
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
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]: Move udevadm to lib/vdsm

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: Move udevadm to lib/vdsm
..


Patch Set 5:

Build Started (5/9) - 
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-el7-x86_64_merged/981/

-- 
To view, visit https://gerrit.ovirt.org/40119
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I113bd85820805e60cbf6a7266ed788ec3ba48637
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
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]: Move udevadm to lib/vdsm

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: Move udevadm to lib/vdsm
..


Patch Set 5:

Build Started (1/9) - 
http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-fc21-x86_64_merged/566/

-- 
To view, visit https://gerrit.ovirt.org/40119
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I113bd85820805e60cbf6a7266ed788ec3ba48637
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
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]: Move udevadm to lib/vdsm

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: Move udevadm to lib/vdsm
..


Patch Set 5:

Build Started (2/9) - 
http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-el7-x86_64_merged/576/

-- 
To view, visit https://gerrit.ovirt.org/40119
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I113bd85820805e60cbf6a7266ed788ec3ba48637
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
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]: tool: Remove upstart handling in libvirt configurator

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: tool: Remove upstart handling in libvirt configurator
..


Patch Set 2:

Build Started (2/2) - 
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1640/

-- 
To view, visit https://gerrit.ovirt.org/40420
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic71430103cb04a5636b4b41371852cbf68495db7
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov dkuzn...@redhat.com
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]: spec: Drop el6 support

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: spec: Drop el6 support
..


Patch Set 2:

Build Started (1/6) - 
http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-el7_created/887/

-- 
To view, visit https://gerrit.ovirt.org/40419
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I52bf73f539d9036fbc3468b4faf8a8d6556c9560
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov dkuzn...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: Yeela Kaplan ykap...@redhat.com
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]: unbreak pyflakes

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: unbreak pyflakes
..


Patch Set 3:

Build Failed 

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_merged/51/ : 0

http://jenkins.ovirt.org/job/vdsm_master_create-rpms-el7-x86_64_merged/980/ : 
FAILURE

http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc21-x86_64_merged/943/ : 
SUCCESS

http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-fc22-x86_64_merged/119/
 : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-el7-x86_64_merged/575/
 : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc20-x86_64_merged/978/ : 
SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc22-x86_64_merged/119/ : 
SUCCESS

http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-fc21-x86_64_merged/565/
 : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-fc20-x86_64_merged/561/
 : SUCCESS

-- 
To view, visit https://gerrit.ovirt.org/40408
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4e3fb9943659339124b76089ebcd23a9b07c268a
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
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]: spec: Require libvirt-python supporting blockCopy

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: spec: Require libvirt-python supporting blockCopy
..


Patch Set 6: Verified-1

Build Failed 

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1616/ : ABORTED

http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc21_created/875/ : 
FAILURE

http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc22_created/101/ : 
FAILURE

http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc20_created/1429/ 
: FAILURE

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18386/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-el7_created/884/ : 
SUCCESS

-- 
To view, visit https://gerrit.ovirt.org/40069
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifbafbfeb61f392bf420caa4589df7cdac92a6568
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Adam Litke ali...@redhat.com
Gerrit-Reviewer: Ala Hino ah...@redhat.com
Gerrit-Reviewer: Allon Mureinik amure...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Francesco Romani from...@redhat.com
Gerrit-Reviewer: Fred Rolland froll...@redhat.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
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]: WIP: make _getInterfaceStats a function

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: WIP: make _getInterfaceStats a function
..


Patch Set 1: Verified-1

Build Failed 

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1649/ : FAILURE

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18418/ : SUCCESS

-- 
To view, visit https://gerrit.ovirt.org/40430
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I0b5796f3463585794d2696ce73f8357cb0fd1f78
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: WIP

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: WIP
..


Patch Set 1: Code-Review-1 Verified-1

Build Failed 

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1650/ : FAILURE

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18419/ : UNSTABLE

-- 
To view, visit https://gerrit.ovirt.org/40431
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I39c2e6e4bca286a513992b7231f1356e8dd871a1
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: packaging: Move libvirt_password out of PKI

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: packaging: Move libvirt_password out of PKI
..


Patch Set 3:

Build Started (6/6) - 
http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc21_created/880/

-- 
To view, visit https://gerrit.ovirt.org/37049
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia7b5f81e4320834711e52e39f092c6e90796a485
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov dkuzn...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
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]: spec: Drop el6 support

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: spec: Drop el6 support
..


Patch Set 3:

Build Started (6/6) - 
http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc21_created/881/

-- 
To view, visit https://gerrit.ovirt.org/40419
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I52bf73f539d9036fbc3468b4faf8a8d6556c9560
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov dkuzn...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: Yeela Kaplan ykap...@redhat.com
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]: tool: Remove upstart handling in libvirt configurator

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: tool: Remove upstart handling in libvirt configurator
..


Patch Set 3: Verified-1

Build Failed 

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1652/ : FAILURE

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18421/ : SUCCESS

-- 
To view, visit https://gerrit.ovirt.org/40420
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic71430103cb04a5636b4b41371852cbf68495db7
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov dkuzn...@redhat.com
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]: network: no need to log when network persistence path doesn'...

2015-04-30 Thread ibarkan
Ido Barkan has posted comments on this change.

Change subject: network: no need to log when network persistence path doesn't 
exist.
..


Patch Set 1: Verified+1

-- 
To view, visit https://gerrit.ovirt.org/40415
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6f998b5ec7a5d0d07e35afdcc9a9d76b42b3c6a0
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
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]: network: no need to log when network persistence path doesn'...

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: network: no need to log when network persistence path doesn't 
exist.
..


Patch Set 1:

Build Started (2/3) - 
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1624/

-- 
To view, visit https://gerrit.ovirt.org/40415
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6f998b5ec7a5d0d07e35afdcc9a9d76b42b3c6a0
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
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]: network: better logging on network restoration.

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: network: better logging on network restoration.
..


Patch Set 3:

Build Started (2/3) - 
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1623/

-- 
To view, visit https://gerrit.ovirt.org/39214
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I0241e7379f799e65bbeba3e13a288e52e5dbc734
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
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]: network: better logging on network restoration.

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: network: better logging on network restoration.
..


Patch Set 3:

Build Started (1/3) - 
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18392/

-- 
To view, visit https://gerrit.ovirt.org/39214
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I0241e7379f799e65bbeba3e13a288e52e5dbc734
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
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]: network: no need to log when network persistence path doesn'...

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: network: no need to log when network persistence path doesn't 
exist.
..


Patch Set 1:

Build Started (1/3) - 
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18393/

-- 
To view, visit https://gerrit.ovirt.org/40415
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6f998b5ec7a5d0d07e35afdcc9a9d76b42b3c6a0
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
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]: sampling: introduce expensive checks

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: sampling: introduce expensive checks
..


Patch Set 3:

Build Started (1/2) - 
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18399/

-- 
To view, visit https://gerrit.ovirt.org/40391
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7b3bb707dd60de194eedfc2e3de1efbf05574ff7
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: WIP: rename stats cache to vm stats cache

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: WIP: rename stats cache to vm stats cache
..


Patch Set 4:

Build Started (1/2) - 
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18400/

-- 
To view, visit https://gerrit.ovirt.org/40371
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If38c49f686dfc2bc0994d444ff24c7736f2e951b
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: virt: rename SampleVMs for clarity

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: virt: rename SampleVMs for clarity
..


Patch Set 4:

Build Started (1/2) - 
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18401/

-- 
To view, visit https://gerrit.ovirt.org/40370
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iadf91e0e3b7ffd4d76eff36e4906441496adb71c
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: sampling: more detailed stale sampling reporting

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: sampling: more detailed stale sampling reporting
..


Patch Set 3:

Build Started (2/2) - 
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1628/

-- 
To view, visit https://gerrit.ovirt.org/40390
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I00cb8f602fc3f26d62dd34bafe05b8169e184b91
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: network: restore sriov devices number of vfs

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: network: restore sriov devices number of vfs
..


Patch Set 9:

Build Started (1/3) - 
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18404/

-- 
To view, visit https://gerrit.ovirt.org/40088
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I76b898019840ffe65939ffad4a1e98829ad3c887
Gerrit-PatchSet: 9
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Francesco Romani from...@redhat.com
Gerrit-Reviewer: Ido Barkan ibar...@redhat.com
Gerrit-Reviewer: Martin Polednik mpoled...@redhat.com
Gerrit-Reviewer: Ondřej Svoboda osvob...@redhat.com
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]: periodic: improve isolation in VM dispatching

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: periodic: improve isolation in VM dispatching
..


Patch Set 4:

Build Started (1/2) - 
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18398/

-- 
To view, visit https://gerrit.ovirt.org/40339
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id6ed5f1b7c332cdfe23d78571ac135a2c1714ba7
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani from...@redhat.com
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]: stomp: add an option to create a stomp client

2015-04-30 Thread fromani
Francesco Romani has posted comments on this change.

Change subject: stomp: add an option to create a stomp client
..


Patch Set 6: Code-Review-1

Please see Nir's comment in v5. I share most of Nir's concerns.

-- 
To view, visit https://gerrit.ovirt.org/38491
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5ad4c79130c0ca1c4a5bd01343eafd3d8bf36231
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yeela Kaplan ykap...@redhat.com
Gerrit-Reviewer: Dima Kuznetsov dkuzn...@redhat.com
Gerrit-Reviewer: Francesco Romani from...@redhat.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Piotr Kliczewski piotr.kliczew...@gmail.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: Yaniv Bronheim ybron...@redhat.com
Gerrit-Reviewer: Yeela Kaplan ykap...@redhat.com
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


  1   2   3   4   >