August 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: Fix creation of template from VM containing direct luns
by ofrenkel＠redhat.com 25 Aug '13

25 Aug '13

Omer Frenkel has submitted this change and it was merged. Change subject: engine: Fix creation of template from VM containing direct luns ...................................................................... engine: Fix creation of template from VM containing direct luns Change-Id: I49a41624529301e26a14f2a69e2b364d6d380ffa Bug-Url: https://bugzilla.redhat.com/999951 Signed-off-by: Sergey Gotliv <sgotliv(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/VmDeviceUtils.java 1 file changed, 5 insertions(+), 1 deletion(-) Approvals: Daniel Erez: Looks good to me, but someone else must approve Sergey Gotliv: Verified Omer Frenkel: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18453 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I49a41624529301e26a14f2a69e2b364d6d380ffa Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sergey Gotliv <sgotliv(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Sergey Gotliv <sgotliv(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in otopi[master]: core: Add more debug info for buildSequence
by Alon Bar-Lev 23 Aug '13

23 Aug '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: core: Add more debug info for buildSequence ...................................................................... core: Add more debug info for buildSequence Change-Id: Ieedbf3ed2dfdb743768448819c0fcfe336b8f4e0 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M src/otopi/context.py 1 file changed, 6 insertions(+), 1 deletion(-) Approvals: Alon Bar-Lev: Looks good to me, approved Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/18423 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieedbf3ed2dfdb743768448819c0fcfe336b8f4e0 Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: check access rights on storage
by sbonazzo＠redhat.com 23 Aug '13

23 Aug '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: check access rights on storage ...................................................................... packaging: setup: check access rights on storage during customization also check for access permissions on storage location avoiding to fail storage domain creation in a later stage. Change-Id: I5c97ee7a8b98d0f41a906dd9507ace88d44da38d Bug-Url:https://bugzilla.redhat.com/999865 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/storage/storage.py 1 file changed, 27 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18421 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5c97ee7a8b98d0f41a906dd9507ace88d44da38d Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Leonid Natapov <leonidnatapov(a)gmail.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)gmail.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: handle bond and bridge interfaces
by sbonazzo＠redhat.com 23 Aug '13

23 Aug '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: handle bond and bridge interfaces ...................................................................... packaging: setup: handle bond and bridge interfaces Previously all existing bond and bridge interfaces where listed as possible nics for creating the management bridge. If a bond interface was specified the bridge creation failed. Now the bridge are not listed anymore and bonds are listed only if they have slave interfaces. Enslaved interfaces and interfaces used by other bridges are not listed anymore as available for management bridge creation. If a bond interface with slaves is specified now should work correctly. Change-Id: If19dfa2c2b9b02ba8d997ad1758614fad3470eb5 Bug-Url: https://bugzilla.redhat.com/999577 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-hosted-engine-setup.spec.in M src/plugins/ovirt-hosted-engine-setup/network/bridge.py 2 files changed, 57 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/18406 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If19dfa2c2b9b02ba8d997ad1758614fad3470eb5 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Leonid Natapov <leonidnatapov(a)gmail.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)gmail.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: start vdsmd only if not running
by sbonazzo＠redhat.com 23 Aug '13

23 Aug '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: start vdsmd only if not running ...................................................................... packaging: setup: start vdsmd only if not running in LATE_SETUP stage start vdsmd only if it is not already running, avoiding errors due to 'cannot start VDS daemon when another instance runs' Bug-Url: https://bugzilla.redhat.com/999542 Change-Id: Ic12fec8ef572ba9ad0bc5b3773fa8e1e1d16c4fc Reported-By: Leonid Natapov <lnatapov(a)redhat.com> Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/system/vdsmenv.py 1 file changed, 9 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Alex Lourie: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/18372 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic12fec8ef572ba9ad0bc5b3773fa8e1e1d16c4fc Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Leonid Natapov <leonidnatapov(a)gmail.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)gmail.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-release[master]: bump release to 8-1
by mburns＠redhat.com 22 Aug '13

22 Aug '13

Michael Burns has submitted this change and it was merged. Change subject: bump release to 8-1 ...................................................................... bump release to 8-1 also add appropriate changelog Change-Id: I5acac4dbefc2121fd3247a91a70c8014b09fd22f Signed-off-by: Mike Burns <mburns(a)redhat.com> --- M configure.ac M ovirt-release.spec.in 2 files changed, 10 insertions(+), 1 deletion(-) Approvals: Michael Burns: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18428 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5acac4dbefc2121fd3247a91a70c8014b09fd22f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Michael Burns <mburns(a)redhat.com> Gerrit-Reviewer: Michael Burns <mburns(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-release[master]: add glusterfs epel repo to EL6 config
by mburns＠redhat.com 22 Aug '13

22 Aug '13

Michael Burns has submitted this change and it was merged. Change subject: add glusterfs epel repo to EL6 config ...................................................................... add glusterfs epel repo to EL6 config To ensure that we include glusterfs since vdsm now depends on it. Include the repos because we don't want to have to maintain all the gluster updates on ovirt.org. Note: Fedora gets the updates automatically Change-Id: I89bb7e231d45579f7e72f8e085798d2680b287e5 Signed-off-by: Mike Burns <mburns(a)redhat.com> --- M Makefile.am A glusterfs-epel.repo M ovirt-release.spec.in 3 files changed, 25 insertions(+), 0 deletions(-) Approvals: Michael Burns: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18427 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I89bb7e231d45579f7e72f8e085798d2680b287e5 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Michael Burns <mburns(a)redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Michael Burns <mburns(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Use container id for memory lock in import process.
by mlipchuk＠redhat.com 22 Aug '13

22 Aug '13

Maor Lipchuk has submitted this change and it was merged. Change subject: core: Use container id for memory lock in import process. ...................................................................... core: Use container id for memory lock in import process. When importing a VM we use memory lock on the VM in the export domain so it will not get deleted in the process. When the import process is being done with clone, a new vm id is generated and the lock at the end command is being regenerated with the new VM guid instead of the original one from the export domain. The proposed fix is to use the container id from the parameters when generating the shared lock. Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/998907 Change-Id: Ie849300bdc98020aeca90deeddafd12295e8211a --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Maor Lipchuk: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18416 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie849300bdc98020aeca90deeddafd12295e8211a Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Liron Ar <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Sergey Gotliv <sgotliv(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in mom[master]: Fix ever increasing CPU usage
by agl＠us.ibm.com 22 Aug '13

22 Aug '13

Adam Litke has submitted this change and it was merged. Change subject: Fix ever increasing CPU usage ...................................................................... Fix ever increasing CPU usage In the HostKSM collector we are calculating the number of jiffies that have been used since the last collection period. Unfortunately we never reset self.last_jiff so our count accumulates indefinitely. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=999615 Signed-off-by: Adam Litke <agl(a)us.ibm.com> Change-Id: I434dfc454776c8eeb1e2a074132d7b94ca68883f Reviewed-on: http://gerrit.ovirt.org/18420 Reviewed-by: Martin Sivák <msivak(a)redhat.com> --- M mom/Collectors/HostKSM.py 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Adam Litke: Verified; Looks good to me, approved Martin Sivák: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/18420 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I434dfc454776c8eeb1e2a074132d7b94ca68883f Gerrit-PatchSet: 2 Gerrit-Project: mom Gerrit-Branch: master Gerrit-Owner: Adam Litke <agl(a)us.ibm.com> Gerrit-Reviewer: Adam Litke <agl(a)us.ibm.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Mei Liu <liumbj(a)linux.vnet.ibm.com> Gerrit-Reviewer: Royce Lv <lvroyce(a)linux.vnet.ibm.com>

1 0

Change in ovirt-log-collector[ovirt-log-collector-3.3]: Fix --ticket-number option.
by knesenko＠redhat.com 22 Aug '13

22 Aug '13

Kiril Nesenko has submitted this change and it was merged. Change subject: Fix --ticket-number option. ...................................................................... Fix --ticket-number option. Bug-Url: https://bugzilla.redhat.com/977836 Change-Id: Ia5425f88392f309df2b6278b927d78b957220a61 Signed-off-by: Kiril Nesenko <knesenko(a)redhat.com> --- M src/__main__.py 1 file changed, 28 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Kiril Nesenko: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18419 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia5425f88392f309df2b6278b927d78b957220a61 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-log-collector Gerrit-Branch: ovirt-log-collector-3.3 Gerrit-Owner: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0