Change in vdsm[master]: XXX: Add _getDeletedImagePath until patch merged

2015-12-11 Thread alitke
Adam Litke has posted comments on this change. Change subject: XXX: Add _getDeletedImagePath until patch merged .. Patch Set 2: Code-Review-2 -- To view, visit https://gerrit.ovirt.org/50218 To unsubscribe, visit https://gerri

Change in vdsm[master]: XXX: All outstanding VolumeMetadata changes squashed

2015-12-11 Thread alitke
Adam Litke has posted comments on this change. Change subject: XXX: All outstanding VolumeMetadata changes squashed .. Patch Set 1: Code-Review-2 -- To view, visit https://gerrit.ovirt.org/50362 To unsubscribe, visit https://g

Change in vdsm[master]: sdm: Create the storage/sdm directory

2015-12-11 Thread alitke
Adam Litke has abandoned this change. Change subject: sdm: Create the storage/sdm directory .. Abandoned Already merged. -- To view, visit https://gerrit.ovirt.org/50219 To unsubscribe, visit https://gerrit.ovirt.org/settings

Change in vdsm[master]: sdm: Create the storage/sdm directory

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: sdm: Create the storage/sdm directory .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found -- To view, visit https://gerrit.ovirt.org/50219 To unsubscribe

Change in vdsm[master]: XXX: All outstanding VolumeMetadata changes squashed

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: XXX: All outstanding VolumeMetadata changes squashed .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure

Change in vdsm[master]: storagefakelib: Add FakeResourceManager

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: storagefakelib: Add FakeResourceManager .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header match

Change in vdsm[master]: storage: Split newMetadata into makeMetadata and createMetadata

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: storage: Split newMetadata into makeMetadata and createMetadata .. Patch Set 8: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found

Change in vdsm[master]: XXX: Add _getDeletedImagePath until patch merged

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: XXX: Add _getDeletedImagePath until patch merged .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure hea

Change in vdsm[master]: XXX: All outstanding VolumeMetadata changes squashed

2015-12-11 Thread alitke
Adam Litke has uploaded a new change for review. Change subject: XXX: All outstanding VolumeMetadata changes squashed .. XXX: All outstanding VolumeMetadata changes squashed Change-Id: I5db8bb8bdf405b8760ecbe103dcbb1a1907ca1db S

Change in vdsm[master]: sdm: Add create_volume job

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: sdm: Add create_volume job .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url:

Change in vdsm[master]: storage: add get_volume_artifacts_class to the domain manifest

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: storage: add get_volume_artifacts_class to the domain manifest .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found,

Change in vdsm[master]: fileVolume: refactor _putMetadata to support volume artifacts

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: fileVolume: refactor _putMetadata to support volume artifacts .. Patch Set 4: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found,

Change in vdsm[master]: fileVolume: define file extension for volume artifacts metadata

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: fileVolume: define file extension for volume artifacts metadata .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found

Change in vdsm[master]: jobs: Add error property

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: jobs: Add error property .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '

Change in vdsm[master]: Introduce VolumeArtifacts

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: Introduce VolumeArtifacts .. Patch Set 8: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '

Change in vdsm[master]: jobs: Only report progress if set

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: jobs: Only report progress if set .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bu

Change in vdsm[master]: sdm: add create_volume_container API stub

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: sdm: add create_volume_container API stub .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header mat

Change in vdsm[master]: sdm: Add create_volume job

2015-12-11 Thread alitke
Adam Litke has posted comments on this change. Change subject: sdm: Add create_volume job .. Patch Set 1: (17 comments) https://gerrit.ovirt.org/#/c/50221/1/tests/sdm_verbs_test.py File tests/sdm_verbs_test.py: Line 38:

Change in vdsm[master]: sdm: Add create_volume job

2015-12-11 Thread alitke
Adam Litke has posted comments on this change. Change subject: sdm: Add create_volume job .. Patch Set 2: (7 comments) https://gerrit.ovirt.org/#/c/50221/2/vdsm/storage/sdm/api/create_volume.py File vdsm/storage/sdm/api/create

Change in vdsm[master]: jobs: Only report progress if set

2015-12-11 Thread alitke
Adam Litke has uploaded a new change for review. Change subject: jobs: Only report progress if set .. jobs: Only report progress if set Not all Jobs have deterministic progress and some jobs will complete so quickly that progres

Change in vdsm[master]: jobs: Only report progress if set

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: jobs: Only report progress if set .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bu

Change in vdsm[master]: jobs: Add error property

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: jobs: Add error property .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '

Change in vdsm[master]: jobs: Add error property

2015-12-11 Thread alitke
Adam Litke has uploaded a new change for review. Change subject: jobs: Add error property .. jobs: Add error property When a Job encounters an error, there should be a standardized way for that error to be stored so that code wo

Change in vdsm[master]: sdm: add create_volume_container API stub

2015-12-11 Thread alitke
Adam Litke has posted comments on this change. Change subject: sdm: add create_volume_container API stub .. Patch Set 2: (7 comments) https://gerrit.ovirt.org/#/c/50220/2/client/vdsClient.py File client/vdsClient.py: Line 106

Change in vdsm[master]: service: change vdsm KillMode to mixed

2015-12-11 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: service: change vdsm KillMode to mixed .. Patch Set 6: if this patch is not needed anymore, please abandon. -- To view, visit https://gerrit.ovirt

Change in vdsm[master]: vdsm standard CI arrangement

2015-12-11 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: vdsm standard CI arrangement .. Patch Set 26: Code-Review-1 -- To view, visit https://gerrit.ovirt.org/48268 To unsubscribe, visit https://gerrit.ovirt.org/setting

Change in vdsm[master]: vdsm standard CI arrangement

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: vdsm standard CI arrangement .. Patch Set 26: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Ur

Change in vdsm[master]: deleting references due to memory leaks in the following mod...

2015-12-11 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: deleting references due to memory leaks in the following modules. adding GC thread every 30sec to vdsm .. Patch Set 4: (3 comments) https://gerrit.ovirt.org/#/c/4

Change in vdsm[master]: deleting references due to memory leaks in the following mod...

2015-12-11 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: deleting references due to memory leaks in the following modules. adding GC thread every 30sec to vdsm .. Patch Set 4: (4 comments) There is no chance that someth

Change in vdsm[master]: clean refrences inorder to save memory.

2015-12-11 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: clean refrences inorder to save memory. .. Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/49346/1/vdsm/storage/misc.py File vdsm/storage/misc.py: Line 220:

Change in vdsm[master]: deleting references due to memory leaks in the following mod...

2015-12-11 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: deleting references due to memory leaks in the following modules. adding GC thread every 30sec to vdsm .. Patch Set 4: Code-Review-2 Please provide information sug

Change in vdsm[master]: clean refrences inorder to save memory.

2015-12-11 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: clean refrences inorder to save memory. .. Patch Set 1: Code-Review-2 - This path is full of unrelated changes - We don't have any tests - We don't have any reason

Change in vdsm[master]: schema: Fix asserts that had side effect

2015-12-11 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: schema: Fix asserts that had side effect .. Patch Set 3: Verified+1 -- To view, visit https://gerrit.ovirt.org/34362 To unsubscribe, visit https://gerrit.ovirt.org

Change in vdsm[master]: schema: Fix asserts that had side effect

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: schema: Fix asserts that had side effect .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matc

Change in vdsm[master]: sdm: add create_volume_container API stub

2015-12-11 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: sdm: add create_volume_container API stub .. Patch Set 2: Code-Review-1 (7 comments) Mostly great, need little fix in api. https://gerrit.ovirt.org/#/c/50220/2/cl

Change in vdsm[ovirt-3.5]: Add an empty metadata qos element to the created domain

2015-12-11 Thread fromani
Francesco Romani has posted comments on this change. Change subject: Add an empty metadata qos element to the created domain .. Patch Set 3: seems fine now. Waiting until monday because 50341, otherwise fine. -- To view, visi

Change in vdsm[ovirt-3.5]: Add an empty metadata qos element to the created domain

2015-12-11 Thread fromani
Francesco Romani has posted comments on this change. Change subject: Add an empty metadata qos element to the created domain .. Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/49655 To unsubscribe, visit

Change in vdsm[ovirt-3.5]: Handle empty QoS section with no cpu limit information

2015-12-11 Thread fromani
Francesco Romani has posted comments on this change. Change subject: Handle empty QoS section with no cpu limit information .. Patch Set 2: seems fine, and solves the issue seen on 49655. Waiting until monday if Martin wants t

Change in vdsm[ovirt-3.5]: Handle empty QoS section with no cpu limit information

2015-12-11 Thread fromani
Francesco Romani has posted comments on this change. Change subject: Handle empty QoS section with no cpu limit information .. Patch Set 2: Verified+1 -- To view, visit https://gerrit.ovirt.org/50341 To unsubscribe, visit http

Change in vdsm[master]: storage: Split newMetadata into makeMetadata and createMetadata

2015-12-11 Thread alitke
Adam Litke has posted comments on this change. Change subject: storage: Split newMetadata into makeMetadata and createMetadata .. Patch Set 7: Code-Review-1 (1 comment) https://gerrit.ovirt.org/#/c/44045/7/vdsm/storage/volume.

Change in vdsm[master]: migration: add 'transferring' property

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: migration: add 'transferring' property .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matche

Change in vdsm[master]: migration: don't mess up with _status fields

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: migration: don't mess up with _status fields .. Patch Set 7: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: virt: do not use status after getStat()

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: virt: do not use status after getStat() .. Patch Set 17: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matc

Change in vdsm[master]: vm: report "MigrationSource" only if transferring

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: vm: report "MigrationSource" only if transferring .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure he

Change in vdsm[master]: migration: make status field private

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: migration: make status field private .. Patch Set 7: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches

Change in vdsm[master]: migration: rename method

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: migration: rename method .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '

Change in vdsm[master]: virt: enhance migration.SourceThread.stop()

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: virt: enhance migration.SourceThread.stop() .. Patch Set 19: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: migration: return copy of internal status

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: migration: return copy of internal status .. Patch Set 7: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header mat

Change in vdsm[master]: migration: track and report abort reason

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: migration: track and report abort reason .. Patch Set 8: * #1154397::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1154397::OK, public bug * Check P

Change in vdsm[master]: sdm: Add create_volume job

2015-12-11 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: sdm: Add create_volume job .. Patch Set 2: (2 comments) https://gerrit.ovirt.org/#/c/50221/2/vdsm/storage/sdm/api/create_volume.py File vdsm/storage/sdm/api/create

Change in vdsm[ovirt-3.5]: Handle empty QoS section with no cpu limit information

2015-12-11 Thread fromani
Francesco Romani has posted comments on this change. Change subject: Handle empty QoS section with no cpu limit information .. Patch Set 2: Continuous-Integration+1 -- To view, visit https://gerrit.ovirt.org/50341 To unsubscri

Change in vdsm[ovirt-3.5]: Handle empty QoS section with no cpu limit information

2015-12-11 Thread fromani
Francesco Romani has posted comments on this change. Change subject: Handle empty QoS section with no cpu limit information .. Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/50341 To unsubscribe, visit h

Change in vdsm[master]: sdm: Add create_volume job

2015-12-11 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: sdm: Add create_volume job .. Patch Set 2: Code-Review-1 (5 comments) https://gerrit.ovirt.org/#/c/50221/2/vdsm/storage/sdm/api/sdm_job.py File vdsm/storage/sdm/ap

Change in vdsm[ovirt-3.5]: Handle empty QoS section with no cpu limit information

2015-12-11 Thread Jenkins CI
Jenkins CI has posted comments on this change. Change subject: Handle empty QoS section with no cpu limit information .. Patch Set 2: Continuous-Integration-1 Propagate review hook: Continuous Integration value inherited from p

Change in vdsm[master]: virt: add run_async helper

2015-12-11 Thread mbetak
Martin Betak has posted comments on this change. Change subject: virt: add run_async helper .. Patch Set 5: Code-Review+1 (1 comment) Nice job! I really like the simplification over the previous implementations. https://gerri

Change in vdsm[ovirt-3.5]: Handle empty QoS section with no cpu limit information

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: Handle empty QoS section with no cpu limit information .. Patch Set 2: Verified-1 * #1289007::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1289007:

Change in vdsm[ovirt-3.5]: Handle empty QoS section with no cpu limit information

2015-12-11 Thread msivak
Martin Sivák has uploaded a new change for review. Change subject: Handle empty QoS section with no cpu limit information .. Handle empty QoS section with no cpu limit information The 3.5 stats collecting code expected the cpu l

Change in vdsm[ovirt-3.5]: Handle empty QoS section with no cpu limit information

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: Handle empty QoS section with no cpu limit information .. Patch Set 1: Verified-1 * #1219903::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1219903:

Change in vdsm[master]: migration: rename method

2015-12-11 Thread Jenkins CI
Jenkins CI has posted comments on this change. Change subject: migration: rename method .. Patch Set 2: Continuous-Integration+1 Propagate review hook: Continuous Integration value inherited from patch 1 -- To view, visit htt

Change in vdsm[master]: migration: add 'transferring' property

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: migration: add 'transferring' property .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matche

Change in vdsm[master]: migration: rename method

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: migration: rename method .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '

Change in vdsm[master]: vm: report "MigrationSource" only if transferring

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: vm: report "MigrationSource" only if transferring .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure he

Change in vdsm[master]: migration: track and report abort reason

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: migration: track and report abort reason .. Patch Set 7: * #1154397::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1154397::OK, public bug * Check P

Change in vdsm[master]: virt: vm: Update time on VM after resume

2015-12-11 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: virt: vm: Update time on VM after resume .. Patch Set 10: Code-Review+1 Waiting for Francesco approval. -- To view, visit https://gerrit.ovirt.org/48860 To unsubs

Change in vdsm[master]: storagefakelib: Add FakeResourceManager

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: storagefakelib: Add FakeResourceManager .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header match

Change in vdsm[master]: sdm: Add create_volume job

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: sdm: Add create_volume job .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url:

Change in vdsm[master]: storage: add get_volume_artifacts_class to the domain manifest

2015-12-11 Thread alitke
Adam Litke has uploaded a new change for review. Change subject: storage: add get_volume_artifacts_class to the domain manifest .. storage: add get_volume_artifacts_class to the domain manifest When working with a domain manifes

Change in vdsm[master]: sdm: add create_volume_container API stub

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: sdm: add create_volume_container API stub .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header mat

Change in vdsm[master]: storage: add get_volume_artifacts_class to the domain manifest

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: storage: add get_volume_artifacts_class to the domain manifest .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found,

Change in vdsm[master]: storagefakelib: Add FakeResourceManager

2015-12-11 Thread alitke
Adam Litke has uploaded a new change for review. Change subject: storagefakelib: Add FakeResourceManager .. storagefakelib: Add FakeResourceManager Change-Id: I0302b48d984ce7eb8ce2326ab4bb033430c032f9 Signed-off-by: Adam Litke

Change in vdsm[master]: Introduce VolumeArtifacts

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: Introduce VolumeArtifacts .. Patch Set 7: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '

Change in vdsm[master]: storage: add a context manager for the domainLock

2015-12-11 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: storage: add a context manager for the domainLock .. Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/50272 To unsubscribe, visit https://gerr

Change in vdsm[master]: storage: add a context manager for the domainLock

2015-12-11 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: storage: add a context manager for the domainLock .. Patch Set 3: Continuous-Integration+1 -- To view, visit https://gerrit.ovirt.org/50272 To unsubscribe, visit h

Change in vdsm[master]: storage: add a context manager for the domainLock

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: storage: add a context manager for the domainLock .. Patch Set 4: * Update tracker: IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found. -- To vie

Change in vdsm[master]: storage: add a context manager for the domainLock

2015-12-11 Thread nsoffer
Nir Soffer has submitted this change and it was merged. Change subject: storage: add a context manager for the domainLock .. storage: add a context manager for the domainLock Change-Id: Id7b831d4fe5a67f6998f31978f2399fdebdb3ceb

Change in vdsm[master]: vm: report "MigrationSource" only if transferring

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: vm: report "MigrationSource" only if transferring .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure he

Change in vdsm[master]: migration: rename method

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: migration: rename 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: '

Change in vdsm[master]: migration: rename method

2015-12-11 Thread fromani
Francesco Romani has uploaded a new change for review. Change subject: migration: rename method .. migration: rename method TODO Change-Id: I9f8558f85c3a9824e0fd7884fe7beed2b239f591 Signed-off-by: Francesco Romani --- M vdsm/v

Change in vdsm[master]: vm: report "MigrationSource" only if transferring

2015-12-11 Thread fromani
Francesco Romani has uploaded a new change for review. Change subject: vm: report "MigrationSource" only if transferring .. vm: report "MigrationSource" only if transferring When a Vm failed to migrate for whatever reason, the r

Change in vdsm[master]: migration: add 'transferring' property

2015-12-11 Thread fromani
Francesco Romani has uploaded a new change for review. Change subject: migration: add 'transferring' property .. migration: add 'transferring' property Add 'transferring' property to migration.SourceThread, to let the client cod

Change in vdsm[master]: migration: add 'transferring' property

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: migration: add 'transferring' property .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matche

Change in vdsm[master]: storage: add a context manager for the domainLock

2015-12-11 Thread alitke
Adam Litke has posted comments on this change. Change subject: storage: add a context manager for the domainLock .. Patch Set 3: Verified+1 -- To view, visit https://gerrit.ovirt.org/50272 To unsubscribe, visit https://gerrit.

Change in vdsm[master]: storage: add a context manager for the domainLock

2015-12-11 Thread alitke
Adam Litke has posted comments on this change. Change subject: storage: add a context manager for the domainLock .. Patch Set 2: (2 comments) https://gerrit.ovirt.org/#/c/50272/2/tests/manifest_tests.py File tests/manifest_tes

Change in vdsm[master]: storage: add a context manager for the domainLock

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: storage: add a context manager for the domainLock .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure he

Change in vdsm[master]: virt: add run_async helper

2015-12-11 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: add run_async helper .. Patch Set 5: Verified+1 tested the unthrottled path using patched VDSM running (and migrating) vms. Tested the other paths usin

Change in vdsm[master]: virt: add run_async helper

2015-12-11 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: add run_async helper .. Patch Set 5: Martin, please share your thoughts! -- To view, visit https://gerrit.ovirt.org/49570 To unsubscribe, visit https:

Change in vdsm[master]: storage: add a context manager for the domainLock

2015-12-11 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: storage: add a context manager for the domainLock .. Patch Set 2: Code-Review+1 (2 comments) Looks good, but we can simplify the failing test even more. https://g

Change in vdsm[ovirt-3.5]: Add an empty metadata qos element to the created domain

2015-12-11 Thread fromani
Francesco Romani has posted comments on this change. Change subject: Add an empty metadata qos element to the created domain .. Patch Set 3: caused by appendMetadata - we need a try/except IndexError in sampling.py in _sampleC

Change in vdsm[ovirt-3.5]: Add an empty metadata qos element to the created domain

2015-12-11 Thread fromani
Francesco Romani has posted comments on this change. Change subject: Add an empty metadata qos element to the created domain .. Patch Set 3: Code-Review-1 sorry for late noticing: smoke test revelead a bug Thread-77::ERROR::20

Change in vdsm[ovirt-3.5]: Add an empty metadata qos element to the created domain

2015-12-11 Thread fromani
Francesco Romani has posted comments on this change. Change subject: Add an empty metadata qos element to the created domain .. Patch Set 3: Continuous-Integration+1 jenkins failure unrelated -- To view, visit https://gerrit.

Change in vdsm[ovirt-3.5]: Add an empty metadata qos element to the created domain

2015-12-11 Thread fromani
Francesco Romani has posted comments on this change. Change subject: Add an empty metadata qos element to the created domain .. Patch Set 3: ...sorry, I misread 3.5 for 3.6. Will recheck the tests and merge -- To view, visit

Change in vdsm[ovirt-3.5]: Add an empty metadata qos element to the created domain

2015-12-11 Thread fromani
Francesco Romani has posted comments on this change. Change subject: Add an empty metadata qos element to the created domain .. Patch Set 3: will check test failure, it should be bogus. Will be merged after the 3.6.1 branch cr

Change in vdsm[master]: storage: add a context manager for the domainLock

2015-12-11 Thread alitke
Adam Litke has posted comments on this change. Change subject: storage: add a context manager for the domainLock .. Patch Set 2: Verified+1 Verified with unit tests as before. -- To view, visit https://gerrit.ovirt.org/50272

Change in vdsm[master]: virt: vm: Update time on VM after resume

2015-12-11 Thread mzamazal
Milan Zamazal has posted comments on this change. Change subject: virt: vm: Update time on VM after resume .. Patch Set 10: Verified+1 Verified by suspending and resuming a VM with and without qemu-guest-agent running and chec

Change in vdsm[master]: virt: vm: Update time on VM after resume

2015-12-11 Thread mzamazal
Milan Zamazal has posted comments on this change. Change subject: virt: vm: Update time on VM after resume .. Patch Set 9: (5 comments) https://gerrit.ovirt.org/#/c/48860/9/tests/vmTests.py File tests/vmTests.py: Line 1564:

Change in vdsm[master]: virt: vm: Update time on VM after resume

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: virt: vm: Update time on VM after resume .. Patch Set 10: * #1156194::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1156194::OK, public bug * Check

Change in vdsm[master]: packaging: updating for VDSM 4.17.4 on Debian jessie

2015-12-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: packaging: updating for VDSM 4.17.4 on Debian jessie .. Patch Set 35: * Update tracker: IGNORE, no Bug-Url found -- To view, visit https://gerrit.ovirt.org/3773

Change in vdsm[master]: packaging: updating for VDSM 4.17.4 on Debian jessie

2015-12-11 Thread stirabos
Simone Tiraboschi has abandoned this change. Change subject: packaging: updating for VDSM 4.17.4 on Debian jessie .. Abandoned Moved to https://gerrit.ovirt.org/49257 -- To view, visit https://gerrit.ovirt.org/37737 To unsubs

Change in vdsm[master]: schema: Fix asserts that had side effect

2015-12-11 Thread alitke
Adam Litke has posted comments on this change. Change subject: schema: Fix asserts that had side effect .. Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/34362 To unsubscribe, visit https://gerrit.ovirt.

Change in vdsm[master]: tests: Add a live merge functional test

2015-12-11 Thread alitke
Adam Litke has restored this change. Change subject: tests: Add a live merge functional test .. Restored -- To view, visit https://gerrit.ovirt.org/29824 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageT

  1   2   >