July 2015 - Engine-commits - oVirt List Archives

Change in ovirt-node-plugin-vdsm[ovirt-3.5]: page: Be more specififc about the Engine password
by fabiand＠redhat.com 31 Jul '15

31 Jul '15

Fabian Deutsch has submitted this change and it was merged. Change subject: page: Be more specififc about the Engine password ...................................................................... page: Be more specififc about the Engine password Setting an engine password will enable SSH and set#s the root password. This was not said explicitly before, but should be said, to give the user a sense for any potential risk. Change-Id: I7d1a309bafe545c4a71e8ef315a9330c964cc732 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1245143 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> Reviewed-on: https://gerrit.ovirt.org/44236 Continuous-Integration: Jenkins CI Reviewed-by: Fabian Deutsch <fabiand(a)redhat.com> Tested-by: Fabian Deutsch <fabiand(a)redhat.com> Reviewed-on: https://gerrit.ovirt.org/44252 Continuous-Integration: Fabian Deutsch <fabiand(a)redhat.com> --- M src/engine_page.py 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/44252 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7d1a309bafe545c4a71e8ef315a9330c964cc732 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: ovirt-3.5 Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-node-plugin-vdsm[master]: page: Be more specififc about the Engine password
by fabiand＠redhat.com 31 Jul '15

31 Jul '15

Fabian Deutsch has submitted this change and it was merged. Change subject: page: Be more specififc about the Engine password ...................................................................... page: Be more specififc about the Engine password Setting an engine password will enable SSH and set#s the root password. This was not said explicitly before, but should be said, to give the user a sense for any potential risk. Change-Id: I7d1a309bafe545c4a71e8ef315a9330c964cc732 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1245143 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> Reviewed-on: https://gerrit.ovirt.org/44236 Continuous-Integration: Jenkins CI Reviewed-by: Fabian Deutsch <fabiand(a)redhat.com> Tested-by: Fabian Deutsch <fabiand(a)redhat.com> --- M src/engine_page.py 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/44236 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7d1a309bafe545c4a71e8ef315a9330c964cc732 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: Remove outdated patternfly css fix in CPPV
by gshereme＠redhat.com 31 Jul '15

31 Jul '15

Greg Sheremeta has submitted this change and it was merged. Change subject: webadmin: Remove outdated patternfly css fix in CPPV ...................................................................... webadmin: Remove outdated patternfly css fix in CPPV One of the Patternfly-Webkit css workarounds is now obsolete and broke the ClusterPolicyPopupView dialog by moving input fields out of their einclosing horizontal panel. Change-Id: Icb795d3f9d5f0713373d6092a1700c02dd53f2ea Bug-Url: https://bugzilla.redhat.com/1238266 Signed-off-by: Roman Mohr <rmohr(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/scheduling/ClusterPolicyPopupView.ui.xml M packaging/branding/ovirt.brand/ovirt-patternfly-compat.css 2 files changed, 1 insertion(+), 7 deletions(-) Approvals: Dudi Maroshi: Looks good to me, but someone else must approve Roman Mohr: Verified Greg Sheremeta: Looks good to me, approved; Passed CI tests Vojtech Szocs: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/44077 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icb795d3f9d5f0713373d6092a1700c02dd53f2ea Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Dudi Maroshi <dudi(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: template main tab -> vm subtab -> vm name may disa...
by tjelinek＠redhat.com 31 Jul '15

31 Jul '15

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: template main tab -> vm subtab -> vm name may disappear ...................................................................... webadmin: template main tab -> vm subtab -> vm name may disappear The problem was that the vm name in this subtab did not have the size set. If the webadmin was opened in a screen where the name did not fit in, it was collapsed completely. Fixed by setting it the same width as the host name has. Change-Id: Icfd85eb615f6c7ecec378015594d85fdf0864ccb Bug-Url: https://bugzilla.redhat.com/1209185 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/template/SubTabTemplateVmView.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Martin Betak: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/44230 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icfd85eb615f6c7ecec378015594d85fdf0864ccb Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: dropped engine related fc21 jobs
by dcaroest＠redhat.com 31 Jul '15

31 Jul '15

David Caro has submitted this change and it was merged. Change subject: dropped engine related fc21 jobs ...................................................................... dropped engine related fc21 jobs Dropped engine related fc21 jobs to save jenkins resources as announced on oVirt mailing lists. Change-Id: I1e5a07a088d71d214deb24dd5968f91af6d7b327 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/yaml/builders/copy-create-job-artifact.yaml M jobs/confs/yaml/jobs/ebay-cors-filter/ebay-cors-filter.yaml M jobs/confs/yaml/jobs/jasper-reports/jasper-reports_create-rpms.yaml M jobs/confs/yaml/jobs/ovirt-dwh/ovirt-dwh_create-rpms.yaml M jobs/confs/yaml/jobs/ovirt-engine-sdk-java/ovirt-engine-sdk-java_create-rpms.yaml M jobs/confs/yaml/jobs/ovirt-engine-wildfly-overlay/ovirt-engine-wildfly-overlay.yaml M jobs/confs/yaml/jobs/ovirt-engine-wildfly/ovirt-engine-wildfly.yaml M jobs/confs/yaml/jobs/ovirt-image-uploader/ovirt-image-uploader_create-rpms.yaml M jobs/confs/yaml/jobs/ovirt-iso-uploader/ovirt-iso-uploader_create-rpms.yaml M jobs/confs/yaml/jobs/ovirt-log-collector/ovirt-log-collector_create-rpms.yaml M jobs/confs/yaml/jobs/ovirt-reports/ovirt-reports_create-rpms.yaml M jobs/confs/yaml/jobs/ovirt/publish-rpms-nightly-master.yaml M jobs/confs/yaml/jobs/unboundid-ldapsdk/unboundid-ldapsdk_create-rpms.yaml M jobs/confs/yaml/jobs/vdsm-jsonrpc-java/vdsm-jsonrpc-java_create-rpms.yaml 14 files changed, 16 insertions(+), 32 deletions(-) Approvals: Sandro Bonazzola: Verified; Ready for review Jenkins CI: Passed CI tests David Caro: Looks good to me; Ready for merge -- To view, visit https://gerrit.ovirt.org/44223 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1e5a07a088d71d214deb24dd5968f91af6d7b327 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-image-uploader[master]: Added standard ci scripts
by sbonazzo＠redhat.com 31 Jul '15

31 Jul '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: Added standard ci scripts ...................................................................... Added standard ci scripts This way it can run completely inside mock Change-Id: Ia812dcd02d49453f05315f3727aa006183346153 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M .gitignore A automation/build-artifacts.packages A automation/build-artifacts.repos A automation/build-artifacts.sh A automation/check-patch.packages A automation/check-patch.sh 6 files changed, 34 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/44123 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia812dcd02d49453f05315f3727aa006183346153 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-image-uploader Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Keith Robertson <kroberts(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <kiril.nesenko(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: gluster: Corrected an issue with time conversion logic
by sabose＠redhat.com 31 Jul '15

31 Jul '15

Sahina Bose has submitted this change and it was merged. Change subject: gluster: Corrected an issue with time conversion logic ...................................................................... gluster: Corrected an issue with time conversion logic In the API convertTime() while getting input time value as no of millis, Calendar.HOUR was used which is problematic and there used to be 12 Hrs difference after converting the time to other time zone. Corrected the same to use Calendar.HOUR_OF_DAY. Change-Id: I795d53e2a7a333864087bfe3a0b85fb6eb9375d0 Bug-Url: https://bugzilla.redhat.com/1244714 Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/GlusterUtil.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Shubhendu Tripathi: Verified Sahina Bose: Looks good to me, approved Ramesh N: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/43827 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I795d53e2a7a333864087bfe3a0b85fb6eb9375d0 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: gluster: Enabled to disable CLI volume snapshot schedule again
by sabose＠redhat.com 31 Jul '15

31 Jul '15

Sahina Bose has submitted this change and it was merged. Change subject: gluster: Enabled to disable CLI volume snapshot schedule again ...................................................................... gluster: Enabled to disable CLI volume snapshot schedule again If gluster meta volume gets deleted and re-created from gluster CLI the disabling of CLI schedule needs to happen again while syncing the meta volume back to engine again. Change-Id: I6e00a14ebcd87141edd941d7752350cc2af64fa8 Bug-Url: https://bugzilla.redhat.com/1240231 Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterSyncJob.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/GlusterSyncJobTest.java 2 files changed, 29 insertions(+), 17 deletions(-) Approvals: Jenkins CI: Passed CI tests Shubhendu Tripathi: Verified Sahina Bose: Looks good to me, approved Ramesh N: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/43302 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6e00a14ebcd87141edd941d7752350cc2af64fa8 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: ovirt-engine-extension-aaa-ldap: branched 3.5
by sbonazzo＠redhat.com 31 Jul '15

31 Jul '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-engine-extension-aaa-ldap: branched 3.5 ...................................................................... ovirt-engine-extension-aaa-ldap: branched 3.5 Change-Id: Ie0f406f3f4d170bad8af0eaf1d6f2e1e5f3fe2d1 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/yaml/jobs/ovirt-engine-extension-aaa-ldap/ovirt-engine-extension-aaa-ldap_create-rpms.yaml 1 file changed, 11 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Looks good to me Sandro Bonazzola: Verified; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/44226 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie0f406f3f4d170bad8af0eaf1d6f2e1e5f3fe2d1 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.5-gluster]: engine : Fix brick state change messages on start volume
by sabose＠redhat.com 31 Jul '15

31 Jul '15

Sahina Bose has submitted this change and it was merged. Change subject: engine : Fix brick state change messages on start volume ...................................................................... engine : Fix brick state change messages on start volume An attempt to sync the volume capacity details as part of start volume was made. As part of this a new issue of appearance of unwanted brick state change messages was observed. The cause for this is attempting to sync volume advanced details for the volume with status not updated. Hence, this patch fetches the volume newly from db and then uses it to get advanced details. Change-Id: I0c259601de0e8edfdb0ced91e582db3f31731f3e Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1235560 Signed-off-by: Anmol Babu <anbabu(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/StartGlusterVolumeCommand.java 1 file changed, 12 insertions(+), 1 deletion(-) Approvals: anmolbabu: Verified Sahina Bose: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/44225 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0c259601de0e8edfdb0ced91e582db3f31731f3e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0