- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: dashboards: update dwh views version
by Code Review 15 Feb '17

15 Feb '17

>From Alexander Wels <awels(a)redhat.com>: Alexander Wels has submitted this change and it was merged. Change subject: dashboards: update dwh views version ...................................................................... dashboards: update dwh views version Change-Id: I2d845ac01c6788aff5e1aaf6d60d83e5504b95a1 Bug-Url: https://bugzilla.redhat.com/1421942 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/server/dashboard/dao/ClusterDwhDAO.properties M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/server/dashboard/dao/HostDwhDAO.properties M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/server/dashboard/dao/StorageDomainDwhDAO.properties M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/server/dashboard/dao/VmDwhDAO.properties 4 files changed, 52 insertions(+), 52 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, but someone else must approve Alexander Wels: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/72238 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2d845ac01c6788aff5e1aaf6d60d83e5504b95a1 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: LabelFilterPolicyUnit type inference
by Code Review 15 Feb '17

15 Feb '17

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: core: LabelFilterPolicyUnit type inference ...................................................................... core: LabelFilterPolicyUnit type inference Change-Id: I32e7d07df5c8c35bca66ec3e6eb3ae11a523ac9b Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/LabelFilterPolicyUnit.java 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified -- To view, visit https://gerrit.ovirt.org/72059 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I32e7d07df5c8c35bca66ec3e6eb3ae11a523ac9b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-ha[v2.1.z]: jsonrpc: avoid using vdscli
by Code Review 15 Feb '17

15 Feb '17

>From Martin Sivák <msivak(a)redhat.com>: Martin Sivák has submitted this change and it was merged. Change subject: jsonrpc: avoid using vdscli ...................................................................... jsonrpc: avoid using vdscli Change-Id: I8c565347f4e7e2dfbd9cdd44198b6846b8f50de8 Bug-Url: https://bugzilla.redhat.com/1101554 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M ovirt_hosted_engine_ha/lib/storage_backends.py 1 file changed, 23 insertions(+), 22 deletions(-) Approvals: Denis Chaplygin: Looks good to me, but someone else must approve Martin Sivák: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/72300 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8c565347f4e7e2dfbd9cdd44198b6846b8f50de8 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: v2.1.z Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Denis Chaplygin <dchaplyg(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-ha[master]: jsonrpc: avoid using vdscli
by Code Review 15 Feb '17

15 Feb '17

>From Martin Sivák <msivak(a)redhat.com>: Martin Sivák has submitted this change and it was merged. Change subject: jsonrpc: avoid using vdscli ...................................................................... jsonrpc: avoid using vdscli Change-Id: I8c565347f4e7e2dfbd9cdd44198b6846b8f50de8 Bug-Url: https://bugzilla.redhat.com/1101554 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M ovirt_hosted_engine_ha/lib/storage_backends.py 1 file changed, 23 insertions(+), 22 deletions(-) Approvals: Denis Chaplygin: Verified; Looks good to me, but someone else must approve Sandro Bonazzola: Looks good to me, but someone else must approve Martin Sivák: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/72068 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8c565347f4e7e2dfbd9cdd44198b6846b8f50de8 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Denis Chaplygin <dchaplyg(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-guest-agent[ovirt-4.1]: win32: Do not attempt to login when the desktop is unlocked
by Code Review 15 Feb '17

15 Feb '17

>From Vinzenz Feenstra <vfeenstr(a)redhat.com>: Vinzenz Feenstra has submitted this change and it was merged. Change subject: win32: Do not attempt to login when the desktop is unlocked ...................................................................... win32: Do not attempt to login when the desktop is unlocked When the desktop is unlocked we should not try to login because we're calling the 'Secure Attention Sequence' which will open the fullscreen window with the options to lock the screen, change password etc. This is a disturbing side effect that should be avoided. After searching for the best possible solution I came to the conclusion that the solution introduced in this patch is good enough, even though it won't work right after starting the agent. It will start working after the first login or unlock of the screen. That means also that this functionality won't be working after a restart of the service, since the agent won't be aware of the fact that it has been only restarted. I also can't say by default that the screen is unlocked, because that would break the SSO on startup and other cases. Therefore I assumed this as the currently 'good enough solution' as this is also a niche problem because for this feature to be useful the option to take no action on console disconnect has to be active in the ovirt-engine. The default is the lock screen behavior which will intiate the locking of the screen in case of a disconnect of the console. Bug-Url: https://bugzilla.redhat.com/1365925 Backport-To: 4.1 Change-Id: I52e92dc00eb34f9f6afc98dc4743a7d793de858b Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> (cherry picked from commit c3cfd269d66def2a3b1bf5cfcfd878493aa99473) --- M ovirt-guest-agent/GuestAgentWin32.py M ovirt-guest-agent/OVirtAgentLogic.py 2 files changed, 11 insertions(+), 1 deletion(-) Approvals: Vinzenz Feenstra: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/72356 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I52e92dc00eb34f9f6afc98dc4743a7d793de858b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: ovirt-4.1 Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-guest-agent[master]: win32: Do not attempt to login when the desktop is unlocked
by Code Review 15 Feb '17

15 Feb '17

>From Vinzenz Feenstra <vfeenstr(a)redhat.com>: Vinzenz Feenstra has submitted this change and it was merged. Change subject: win32: Do not attempt to login when the desktop is unlocked ...................................................................... win32: Do not attempt to login when the desktop is unlocked When the desktop is unlocked we should not try to login because we're calling the 'Secure Attention Sequence' which will open the fullscreen window with the options to lock the screen, change password etc. This is a disturbing side effect that should be avoided. After searching for the best possible solution I came to the conclusion that the solution introduced in this patch is good enough, even though it won't work right after starting the agent. It will start working after the first login or unlock of the screen. That means also that this functionality won't be working after a restart of the service, since the agent won't be aware of the fact that it has been only restarted. I also can't say by default that the screen is unlocked, because that would break the SSO on startup and other cases. Therefore I assumed this as the currently 'good enough solution' as this is also a niche problem because for this feature to be useful the option to take no action on console disconnect has to be active in the ovirt-engine. The default is the lock screen behavior which will intiate the locking of the screen in case of a disconnect of the console. Bug-Url: https://bugzilla.redhat.com/1365925 Backport-To: 4.1 Change-Id: I52e92dc00eb34f9f6afc98dc4743a7d793de858b Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M ovirt-guest-agent/GuestAgentWin32.py M ovirt-guest-agent/OVirtAgentLogic.py 2 files changed, 11 insertions(+), 1 deletion(-) Approvals: Vinzenz Feenstra: Verified; Looks good to me, approved; Passed CI tests Milan Zamazal: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/71223 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I52e92dc00eb34f9f6afc98dc4743a7d793de858b Gerrit-PatchSet: 7 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: Jenkins CI Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Milan Zamazal <mzamazal(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-ha[v2.1.z]: Add a method to get all editable keys on shared storage
by Code Review 15 Feb '17

15 Feb '17

>From Martin Sivák <msivak(a)redhat.com>: Martin Sivák has submitted this change and it was merged. Change subject: Add a method to get all editable keys on shared storage ...................................................................... Add a method to get all editable keys on shared storage Add a method to get all the keys that can be edited. That method gives the user more info about what can be edited for each editable file. Change-Id: I28a4dec3b854592c13a7f5b755531bd36e4642bb Bug-Url:https://bugzilla.redhat.com/1417583 Signed-off-by: Jenny Tokar <jtokar(a)redhat.com> --- M ovirt_hosted_engine_ha/client/client.py M ovirt_hosted_engine_ha/env/config.py 2 files changed, 24 insertions(+), 0 deletions(-) Approvals: Martin Sivák: Looks good to me, approved Jenkins CI: Passed CI tests Jenny Tokar: Verified -- To view, visit https://gerrit.ovirt.org/72346 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I28a4dec3b854592c13a7f5b755531bd36e4642bb Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: v2.1.z Gerrit-Owner: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>

1 0

Change in ovirt-hosted-engine-ha[v2.1.z]: Provide more info in error message
by Code Review 15 Feb '17

15 Feb '17

>From Martin Sivák <msivak(a)redhat.com>: Martin Sivák has submitted this change and it was merged. Change subject: Provide more info in error message ...................................................................... Provide more info in error message Add a list of supported types to the error message when trying to set or get shared storage configuration. Change-Id: I68e0e1b32fc6bd15f8b38cd00795342d6b14e683 Bug-Url:https://bugzilla.redhat.com/1417583 Signed-off-by: Jenny Tokar <jtokar(a)redhat.com> --- M ovirt_hosted_engine_ha/env/config.py 1 file changed, 6 insertions(+), 3 deletions(-) Approvals: Martin Sivák: Looks good to me, approved Jenkins CI: Passed CI tests Jenny Tokar: Verified -- To view, visit https://gerrit.ovirt.org/72347 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I68e0e1b32fc6bd15f8b38cd00795342d6b14e683 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: v2.1.z Gerrit-Owner: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>

1 0

Change in jenkins[master]: lago: restrict jobs to run on same distro vm
by Code Review 15 Feb '17

15 Feb '17

>From Eyal Edri <eedri(a)redhat.com>: Eyal Edri has submitted this change and it was merged. Change subject: lago: restrict jobs to run on same distro vm ...................................................................... lago: restrict jobs to run on same distro vm Lately I've encountered several errors on virt-sysprep stage, which I think are the result of a *really* weird infrastructure combination lago jobs use: el7-host -> fc25/24 vm -> el7 mock -> starting a fc24/el7.3 vm with lago At the moment, I see no point in investigating this weird scenario deeply, as it happens only here. This patch tries to reduce the noise by restricting lago jobs to run on the same distro VM, so it will look like: el7-host -> X vm -> X mock -> starting a fc24 vm If this doesn't resolve the issue, the next escalation point could be moving to bare-metals(instead of 'testing' L3 nested virtualisation on each lago job) Change-Id: Id0175e55ba1e5bec0fb7e2eaa30518e947ab2a94 Signed-off-by: Nadav Goldin <ngoldin(a)redhat.com> --- M jobs/confs/projects/lago/lago_standard.yaml 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Looks good to me; Ready for merge Nadav Goldin: Verified; Ready for review Eyal Edri: Looks good to me, approved Daniel Belenky: Looks good to me Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/71800 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id0175e55ba1e5bec0fb7e2eaa30518e947ab2a94 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Nadav Goldin <ngoldin(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <barak.korren(a)gmail.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Daniel Belenky <dbelenky(a)redhat.com> Gerrit-Reviewer: David Caro <david(a)dcaro.es> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nadav Goldin <ngoldin(a)redhat.com> Gerrit-Reviewer: Pavel Zhukov <pavel(a)zhukoff.net> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: scheduling: Honor the Start Running On option
by Code Review 15 Feb '17

15 Feb '17

>From Martin Sivák <msivak(a)redhat.com>: Martin Sivák has submitted this change and it was merged. Change subject: scheduling: Honor the Start Running On option ...................................................................... scheduling: Honor the Start Running On option We should treat the VM's option Start Running On more seriously. This patch makes it 99 times more important than other policy units. That means we will honor it unless there are 99 better hosts according to other policy units. Change-Id: Iadadbf7b2a6b27e5f9db7aa9fdaf15b8b43c33d4 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1421713 Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/InternalClusterPolicies.java 1 file changed, 4 insertions(+), 4 deletions(-) Approvals: Denis Chaplygin: Looks good to me, but someone else must approve Martin Sivák: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Yanir Quinn: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/72277 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iadadbf7b2a6b27e5f9db7aa9fdaf15b8b43c33d4 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Andrej Krejcir <akrejcir(a)redhat.com> Gerrit-Reviewer: Denis Chaplygin <dchaplyg(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Phillip Bailey <phbailey(a)redhat.com> Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Yanir Quinn <yquinn(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0