July 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: Report downtime for each live migration
by ofrenkel＠redhat.com 25 Jul '13

25 Jul '13

Omer Frenkel has submitted this change and it was merged. Change subject: engine: Report downtime for each live migration ...................................................................... engine: Report downtime for each live migration Added downtime to the end migration event log Change-Id: I07fe88ad4aa6778c7180cf1041770af1ec21098b Bug-url: https://bugzilla.redhat.com/970711 Signed-off-by: Shahar Havivi <shavivi(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/MigrateVmParameters.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 3 files changed, 19 insertions(+), 1 deletion(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16399 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I07fe88ad4aa6778c7180cf1041770af1ec21098b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-host-deploy[master]: vdsm: packages: enforce network service
by Alon Bar-Lev 25 Jul '13

25 Jul '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: vdsm: packages: enforce network service ...................................................................... vdsm: packages: enforce network service current implementation enforces network service only if bridge plugin is enabled. While in ovirt-engine-3.3 the bridge is optional. new implementation perform this unconditionally in the packages plugin. Change-Id: I96ca90aa975c92439ba33309381462a052e44c5e Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/plugins/ovirt-host-deploy/vdsm/bridge.py M src/plugins/ovirt-host-deploy/vdsm/packages.py 2 files changed, 11 insertions(+), 7 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17304 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I96ca90aa975c92439ba33309381462a052e44c5e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Antoni Segura Puimedon <asegurap(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: ssh install recommend screen
by sbonazzo＠redhat.com 25 Jul '13

25 Jul '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: ssh install recommend screen ...................................................................... packaging: setup: ssh install recommend screen added detection of an execution done through ssh without using screen. Added a confirmation dialog for allowing the user to abort the installation. Change-Id: Ib9d7a3b5d18f418adfc9b0545abe44b1cce6851f Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/core/Makefile.am M src/plugins/ovirt-hosted-engine-setup/core/__init__.py A src/plugins/ovirt-hosted-engine-setup/core/shell.py 3 files changed, 73 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17258 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib9d7a3b5d18f418adfc9b0545abe44b1cce6851f 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: Lev Veyde <lveyde(a)gmail.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(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: own config directory
by sbonazzo＠redhat.com 25 Jul '13

25 Jul '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: own config directory ...................................................................... packaging: own config directory Ensure that the configuration directory are created on the system and owned by the rpm package. Avoid to create manually those directory when copying the config files to another host where this package is already installed. Change-Id: Ib6a6aef07e1dca4fdbc175dcf4ea002016437b7a Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-hosted-engine-setup.spec.in M src/bin/Makefile.am 2 files changed, 6 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17259 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib6a6aef07e1dca4fdbc175dcf4ea002016437b7a 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: Lev Veyde <lveyde(a)gmail.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in otopi[master]: network: firewalld: query version from python client
by Alon Bar-Lev 25 Jul '13

25 Jul '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: network: firewalld: query version from python client ...................................................................... network: firewalld: query version from python client querying firewalld version is required as firewalld interfaces (both api and cli) are unstable. current implementation queries firewalld version using command-line, however, in order to query the version the firewalld service must be started. starting firewalld in order to detect firewalld is bad requirement, it has also some side effects, as once firewalld is started we actually effect the host. new implementation uses an undocumented firewalld python library to query the client version, which should be sufficient to our purposes. Change-Id: I3db43d6dc808261a1f687ea29fe23bd752ffade8 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=988075 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/plugins/otopi/network/firewalld.py 1 file changed, 23 insertions(+), 31 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved Sandro Bonazzola: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/17302 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3db43d6dc808261a1f687ea29fe23bd752ffade8 Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-guest-agent[master]: agent: Report FQDN of the machine
by bazulay＠redhat.com 25 Jul '13

25 Jul '13

Barak Azulay has submitted this change and it was merged. Change subject: agent: Report FQDN of the machine ...................................................................... agent: Report FQDN of the machine Sending now the fqdn message on refresh and startup Change-Id: I5d91de94b9ebd4448e9761ce597d970029ca3837 Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M ovirt-guest-agent/GuestAgentLinux2.py M ovirt-guest-agent/GuestAgentWin32.py M ovirt-guest-agent/OVirtAgentLogic.py 3 files changed, 11 insertions(+), 0 deletions(-) Approvals: Vinzenz Feenstra: Verified; Looks good to me, but someone else must approve Barak Azulay: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16123 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5d91de94b9ebd4448e9761ce597d970029ca3837 Gerrit-PatchSet: 10 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: master Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com>

1 0

Change in ovirt-dwh[master]: packaging: setup: updated 'su' utility location
by ydary＠redhat.com 25 Jul '13

25 Jul '13

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: updated 'su' utility location ...................................................................... packaging: setup: updated 'su' utility location Change-Id: I51615f333267a3e5a324e5e9b32ac357eb0bfc66 Bug-Url: https://bugzilla.redhat.com/988298 Signed-off-by: Alex Lourie <alourie(a)redhat.com> --- M packaging/common_utils.py 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: Alex Lourie: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17301 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I51615f333267a3e5a324e5e9b32ac357eb0bfc66 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-guest-agent[master]: agent: Disable GDM plugin building on Fedora 19+
by bazulay＠redhat.com 25 Jul '13

25 Jul '13

Barak Azulay has submitted this change and it was merged. Change subject: agent: Disable GDM plugin building on Fedora 19+ ...................................................................... agent: Disable GDM plugin building on Fedora 19+ Fedora 19 does no longer have a GDM with plugin capabilities. To allow proper building of the package gdm plugins are no longer built for Fedora 19 and higher. Change-Id: Iff3f5a3c7c12d9c6ef198aefe163d9633d8364b3 Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M ovirt-guest-agent.spec 1 file changed, 9 insertions(+), 3 deletions(-) Approvals: Vinzenz Feenstra: Verified; Looks good to me, but someone else must approve Barak Azulay: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16920 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iff3f5a3c7c12d9c6ef198aefe163d9633d8364b3 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: master Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com>

1 0

Change in ovirt-guest-agent[master]: agent: Missing BuildRequirements in spec files
by bazulay＠redhat.com 25 Jul '13

25 Jul '13

Barak Azulay has submitted this change and it was merged. Change subject: agent: Missing BuildRequirements in spec files ...................................................................... agent: Missing BuildRequirements in spec files The following two packages are missing in the build requires * python-pep8 * gcc-c++ Change-Id: I997a0fd3653688271cfd5ad7ee584d8b6ed1c11d Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M ovirt-guest-agent.rhel6.spec M ovirt-guest-agent.spec 2 files changed, 3 insertions(+), 0 deletions(-) Approvals: Vinzenz Feenstra: Verified; Looks good to me, but someone else must approve Barak Azulay: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16919 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I997a0fd3653688271cfd5ad7ee584d8b6ed1c11d Gerrit-PatchSet: 4 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: master Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com>

1 0

Change in ovirt-guest-agent[master]: agent: Default value for configure should be without-dist
by bazulay＠redhat.com 25 Jul '13

25 Jul '13

Barak Azulay has submitted this change and it was merged. Change subject: agent: Default value for configure should be without-dist ...................................................................... agent: Default value for configure should be without-dist Change-Id: Ie1f84f06f7a188e59c181a1190ab5a133873cee7 Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M configure.ac 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Vinzenz Feenstra: Verified; Looks good to me, but someone else must approve Barak Azulay: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16918 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie1f84f06f7a188e59c181a1190ab5a133873cee7 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: master Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com>

1 0