- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: webadmin: AbstractSubTabTree methods
by amureini＠redhat.com 18 Jan '16

18 Jan '16

Allon Mureinik has submitted this change and it was merged. Change subject: webadmin: AbstractSubTabTree methods ...................................................................... webadmin: AbstractSubTabTree methods Removed unused methods isRootSelectionEnabled() and isNodeSelectionEnabled(). Change-Id: I487fedf0df8dbe886368598796ef8cd47bec7ae3 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/tree/AbstractSubTabTree.java 1 file changed, 0 insertions(+), 8 deletions(-) Approvals: Daniel Erez: Looks good to me, approved Allon Mureinik: Verified; Passed CI tests Objections: Tal Nisan: I would prefer that you didn't submit this -- To view, visit https://gerrit.ovirt.org/52240 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I487fedf0df8dbe886368598796ef8cd47bec7ae3 Gerrit-PatchSet: 6 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: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: BaseBackendResource commented out code
by amureini＠redhat.com 18 Jan '16

18 Jan '16

Allon Mureinik has submitted this change and it was merged. Change subject: restapi: BaseBackendResource commented out code ...................................................................... restapi: BaseBackendResource commented out code Removed commented out code. History should be tracked by the SCM, not by commenting out old code. Change-Id: Id89af65807e86966cba4a6fd4e839a7e7177e1ae Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BaseBackendResource.java 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Tal Nisan: Looks good to me, approved Juan Hernandez: Looks good to me, approved Allon Mureinik: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/52235 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id89af65807e86966cba4a6fd4e839a7e7177e1ae Gerrit-PatchSet: 4 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: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Remove BaseBackendResource#combine
by amureini＠redhat.com 18 Jan '16

18 Jan '16

Allon Mureinik has submitted this change and it was merged. Change subject: restapi: Remove BaseBackendResource#combine ...................................................................... restapi: Remove BaseBackendResource#combine Removed unused method. Change-Id: Ic1322d17a8314fddc93a211b6b2d687a98d6fb7c Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BaseBackendResource.java 1 file changed, 0 insertions(+), 10 deletions(-) Approvals: Juan Hernandez: Looks good to me, approved Allon Mureinik: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/52234 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic1322d17a8314fddc93a211b6b2d687a98d6fb7c Gerrit-PatchSet: 4 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: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: engine: Fix NPE in Glance import image as template
by tnisan＠redhat.com 18 Jan '16

18 Jan '16

Tal Nisan has submitted this change and it was merged. Change subject: engine: Fix NPE in Glance import image as template ...................................................................... engine: Fix NPE in Glance import image as template In AddVmTemplateCommand, the vm variable is not available in the flow of importing image as template from Glance. Using it will cause a NullPointerException. Instead of using the vm for getting the compatibility version, we use the master VM from the parameters or the cluster. Change-Id: I066e33fc1414675799fe3ffb655bfe4204e1ba3d Bug-Url: https://bugzilla.redhat.com/1299233 Signed-off-by: Fred Rolland <frolland(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmTemplateCommand.java 1 file changed, 5 insertions(+), 2 deletions(-) Approvals: Jenkins CI: Passed CI tests Freddy Rolland: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/51938 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I066e33fc1414675799fe3ffb655bfe4204e1ba3d Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Fred Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core, webadmin: Not validating HostNetworkQos values
by tnisan＠redhat.com 18 Jan '16

18 Jan '16

Tal Nisan has submitted this change and it was merged. Change subject: core,webadmin: Not validating HostNetworkQos values ...................................................................... core,webadmin: Not validating HostNetworkQos values • Missing @Valid • added code so that HostSetupNetworksCommand validates NetworkAttachment and Bonds instances passed in its parameters class. Change-Id: I50cf6787bdc43d2dba6e726d62d13f08d3a5ab47 Bug-Url: https://bugzilla.redhat.com/1271220 Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/HostSetupNetworksCommand.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/common/businessentities/SeparateNewAndModifiedInstances.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/network/NetworkAttachment.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/EngineMessage.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 7 files changed, 71 insertions(+), 2 deletions(-) Approvals: Martin Mucha: Verified Alona Kaplan: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/51484 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I50cf6787bdc43d2dba6e726d62d13f08d3a5ab47 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[master]: Add "connection" attribute to structs and lists
by juan.hernandez＠redhat.com 18 Jan '16

18 Jan '16

Juan Hernandez has submitted this change and it was merged. Change subject: Add "connection" attribute to structs and lists ...................................................................... Add "connection" attribute to structs and lists This patch adds a "connection" attribute to struct and list types. It will contain a reference to the connection that created the object, and will be used by later patches to implement support for following links. Change-Id: Ic6d7c1768c232dac3159d560a1bb5852c550d107 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/ruby/ReadersGenerator.java M generator/src/main/java/org/ovirt/sdk/ruby/ServicesGenerator.java M sdk/lib/ovirt/sdk/v4/types/list.rb M sdk/lib/ovirt/sdk/v4/types/struct.rb M sdk/spec/vm_reader_spec.rb 5 files changed, 89 insertions(+), 10 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/51915 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic6d7c1768c232dac3159d560a1bb5852c550d107 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[master]: Add "href" attribute to structs and lists
by juan.hernandez＠redhat.com 18 Jan '16

18 Jan '16

Juan Hernandez has submitted this change and it was merged. Change subject: Add "href" attribute to structs and lists ...................................................................... Add "href" attribute to structs and lists This patch adds the "href" attribute to struct and list types, and mofies the generator of the XML readers and writers so that they handle this attribute. Change-Id: I926d32c97a0eff2da624622e522118115f0d947b Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/ruby/ReadersGenerator.java M generator/src/main/java/org/ovirt/sdk/ruby/WritersGenerator.java M sdk/lib/ovirt/sdk/v4/types/list.rb M sdk/lib/ovirt/sdk/v4/types/struct.rb M sdk/spec/vm_reader_spec.rb M sdk/spec/vm_writer_spec.rb 6 files changed, 141 insertions(+), 5 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/51914 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I926d32c97a0eff2da624622e522118115f0d947b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[master]: Use "List" instead of "Array"
by juan.hernandez＠redhat.com 18 Jan '16

18 Jan '16

Juan Hernandez has submitted this change and it was merged. Change subject: Use "List" instead of "Array" ...................................................................... Use "List" instead of "Array" Currently when a list of objects is received from the server it is converted to an instance of the Ruby "Array" class. This patch introduces a new "List" class, that extends the Ruby "Array" class, currently with no methods. List of objects returned from the SDK will be of this type. Later patches will take advantage of this "List" class, for example to add methods to support following links. The patch also renames the base "Type" class to "Struct", as that better reflects the difference between struct and list types. Change-Id: I1cbcd1eab8bd8b95d7d4a0e0537ccd8a47e81f64 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/ruby/ReadersGenerator.java M generator/src/main/java/org/ovirt/sdk/ruby/RubyNames.java M generator/src/main/java/org/ovirt/sdk/ruby/TypesGenerator.java C sdk/lib/ovirt/sdk/v4/types/list.rb R sdk/lib/ovirt/sdk/v4/types/struct.rb M sdk/spec/vm_reader_spec.rb A sdk/spec/vm_spec.rb 7 files changed, 79 insertions(+), 33 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/51913 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1cbcd1eab8bd8b95d7d4a0e0537ccd8a47e81f64 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[master]: storage: resetting spUUID to a random value only if really n...
by sbonazzo＠redhat.com 18 Jan '16

18 Jan '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: storage: resetting spUUID to a random value only if really needed ...................................................................... storage: resetting spUUID to a random value only if really needed hosted-engine-setup generates answer file with a blank SP UUID for additional hosts cause it correctly disconnected its SD from the boostrap storage pool. If we want to use the answerfile for not-interactively replaing the scenario we need a valid spUUID value otherwise we are not able to create the bootstrap storage pool. On the other side, passing an answerfile via CLI is a valid alternative instead of download it from another host via scp. And in that case we should honour the BLANK UUID. So resetting spUUID to blank only if really needed to create a new boostrap storage pool. Change-Id: I67569bc6902a176565269f2bcbec569b51549181 Bug-Url: https://bugzilla.redhat.com/1277010 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/storage/storage.py 1 file changed, 11 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/51853 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I67569bc6902a176565269f2bcbec569b51549181 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6.2]: core: hosted-engine: set the os type to linux on import
by tnisan＠redhat.com 18 Jan '16

18 Jan '16

Tal Nisan has submitted this change and it was merged. Change subject: core: hosted-engine: set the os type to linux on import ...................................................................... core: hosted-engine: set the os type to linux on import The imported VM is obviously linux so setting the type the the most basic linux os instead of "other" This fixes a bug when editing the VM because the values of the console are in the limits of linux and not valid for "other" os. Change-Id: I3520d6d0501e82aa135bdc896f8d994e02e30d74 Bug-Url: https://bugzilla.redhat.com/1158096 Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/HostedEngineImporter.java 1 file changed, 8 insertions(+), 0 deletions(-) Approvals: Tal Nisan: Passed CI tests Roy Golan: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/51932 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3520d6d0501e82aa135bdc896f8d994e02e30d74 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6.2 Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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