Change in ovirt-engine-sdk-ruby[sdk_4.0]: [maven-release-plugin] prepare release 4.0.3
by juan.hernandez＠redhat.com
Juan Hernandez has submitted this change and it was merged.
Change subject: [maven-release-plugin] prepare release 4.0.3
......................................................................
[maven-release-plugin] prepare release 4.0.3
Change-Id: I6de20b389811508b1fb7f5bbbab085456f2715c5
---
M generator/pom.xml
M pom.xml
M sdk/pom.xml
3 files changed, 4 insertions(+), 4 deletions(-)
Approvals:
Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests
--
To view, visit https://gerrit.ovirt.org/65216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6de20b389811508b1fb7f5bbbab085456f2715c5
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine-sdk-ruby
Gerrit-Branch: sdk_4.0
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>
7 years, 6 months
Change in ovirt-engine-sdk-ruby[sdk_4.0]: Update CHANGES.adoc for release 4.0.3
by juan.hernandez＠redhat.com
Juan Hernandez has submitted this change and it was merged.
Change subject: Update CHANGES.adoc for release 4.0.3
......................................................................
Update CHANGES.adoc for release 4.0.3
This patch updates the CHANGES.adoc file, documenting the relevant
changes in release 4.0.3.
Change-Id: I19d71abdf602b8480d674392f7a098c879e7c673
Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com>
---
M sdk/CHANGES.adoc
1 file changed, 6 insertions(+), 0 deletions(-)
Approvals:
Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests
--
To view, visit https://gerrit.ovirt.org/65215
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I19d71abdf602b8480d674392f7a098c879e7c673
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine-sdk-ruby
Gerrit-Branch: sdk_4.0
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>
7 years, 6 months
Change in ovirt-engine-sdk-ruby[sdk_4.0]: Remove 'curb' dependency
by juan.hernandez＠redhat.com
Juan Hernandez has submitted this change and it was merged.
Change subject: Remove 'curb' dependency
......................................................................
Remove 'curb' dependency
Currently the SDK requires the 'curb' gem, but this gem isn't available
in some of the target distributions, in particular it isn't available in
Red Hat Enterprise Linux 7. In addition 'curb' doesn't support some
features that we want to use in the future, like streaming of results.
To address these issues this patch replaces the code that uses 'curb'
with code that calls directly 'libcurl'.
Change-Id: Ia3a894fe3d622c92d94d853c0588222428e0c652
Bug-Url: https://bugzilla.redhat.com/1377682
Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com>
(cherry picked from commit 4a7b662cbb3f4b7d449c299800b1a70d3c82f5f5)
---
M automation/build.packages
M generator/src/main/java/org/ovirt/sdk/ruby/ServicesGenerator.java
M packaging/spec.el7.in
M packaging/spec.fc23.in
M packaging/spec.fc24.in
M sdk/ext/ovirtsdk4c/extconf.rb
A sdk/ext/ovirtsdk4c/ov_http_client.c
A sdk/ext/ovirtsdk4c/ov_http_client.h
A sdk/ext/ovirtsdk4c/ov_http_request.c
A sdk/ext/ovirtsdk4c/ov_http_request.h
A sdk/ext/ovirtsdk4c/ov_http_response.c
A sdk/ext/ovirtsdk4c/ov_http_response.h
M sdk/ext/ovirtsdk4c/ovirtsdk4c.c
M sdk/lib/ovirtsdk4/http.rb
M sdk/lib/ovirtsdk4/probe.rb
M sdk/ovirt-engine-sdk.gemspec
M sdk/spec/clusters_service_spec.rb
M sdk/spec/connection_create_spec.rb
M sdk/spec/connection_timeout_spec.rb
M sdk/spec/connection_use_spec.rb
M sdk/spec/data_centers_service_spec.rb
M sdk/spec/probe_spec.rb
M sdk/spec/service_spec.rb
M sdk/spec/spec_helper.rb
M sdk/spec/storage_domain_service_spec.rb
M sdk/spec/storage_domains_service_spec.rb
M sdk/spec/vm_service_spec.rb
M sdk/spec/vms_service_spec.rb
28 files changed, 1,684 insertions(+), 587 deletions(-)
Approvals:
Juan Hernandez: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/65214
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia3a894fe3d622c92d94d853c0588222428e0c652
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine-sdk-ruby
Gerrit-Branch: sdk_4.0
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>
7 years, 6 months
Change in ovirt-engine-sdk-ruby[master]: Remove 'curb' dependency
by juan.hernandez＠redhat.com
Juan Hernandez has submitted this change and it was merged.
Change subject: Remove 'curb' dependency
......................................................................
Remove 'curb' dependency
Currently the SDK requires the 'curb' gem, but this gem isn't available
in some of the target distributions, in particular it isn't available in
Red Hat Enterprise Linux 7. In addition 'curb' doesn't support some
features that we want to use in the future, like streaming of results.
To address these issues this patch replaces the code that uses 'curb'
with code that calls directly 'libcurl'.
Change-Id: Ia3a894fe3d622c92d94d853c0588222428e0c652
Bug-Url: https://bugzilla.redhat.com/1377682
Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com>
---
M automation/build.packages
M generator/src/main/java/org/ovirt/sdk/ruby/ServicesGenerator.java
M packaging/spec.el7.in
M packaging/spec.fc23.in
M packaging/spec.fc24.in
M sdk/ext/ovirtsdk4c/extconf.rb
A sdk/ext/ovirtsdk4c/ov_http_client.c
A sdk/ext/ovirtsdk4c/ov_http_client.h
A sdk/ext/ovirtsdk4c/ov_http_request.c
A sdk/ext/ovirtsdk4c/ov_http_request.h
A sdk/ext/ovirtsdk4c/ov_http_response.c
A sdk/ext/ovirtsdk4c/ov_http_response.h
M sdk/ext/ovirtsdk4c/ovirtsdk4c.c
M sdk/lib/ovirtsdk4/http.rb
M sdk/lib/ovirtsdk4/probe.rb
M sdk/ovirt-engine-sdk.gemspec
M sdk/spec/clusters_service_spec.rb
M sdk/spec/connection_create_spec.rb
M sdk/spec/connection_timeout_spec.rb
M sdk/spec/connection_use_spec.rb
M sdk/spec/data_centers_service_spec.rb
M sdk/spec/probe_spec.rb
M sdk/spec/service_spec.rb
M sdk/spec/spec_helper.rb
M sdk/spec/storage_domain_service_spec.rb
M sdk/spec/storage_domains_service_spec.rb
M sdk/spec/vm_service_spec.rb
M sdk/spec/vms_service_spec.rb
28 files changed, 1,684 insertions(+), 587 deletions(-)
Approvals:
Juan Hernandez: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/60694
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia3a894fe3d622c92d94d853c0588222428e0c652
Gerrit-PatchSet: 7
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: Ondra Machacek <omachace(a)redhat.com>
Gerrit-Reviewer: Yaniv Kaul <ykaul(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
7 years, 6 months
Change in ovirt-engine[master]: restapi: Gluster role for network
by juan.hernandez＠redhat.com
Juan Hernandez has submitted this change and it was merged.
Change subject: restapi: Gluster role for network
......................................................................
restapi: Gluster role for network
Support adding gluster role for network
Bug-Url: https://bugzilla.redhat.com/1379225
Change-Id: I891cf5a26f6724a5665a7db9b6b3766b1be664e4
Signed-off-by: Sahina Bose <sabose(a)redhat.com>
---
M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/NetworkMapper.java
1 file changed, 4 insertions(+), 0 deletions(-)
Approvals:
Juan Hernandez: Looks good to me, approved
Jenkins CI: Passed CI tests
Sahina Bose: Verified
--
To view, visit https://gerrit.ovirt.org/64287
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I891cf5a26f6724a5665a7db9b6b3766b1be664e4
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sahina Bose <sabose(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
7 years, 6 months
Change in ovirt-engine[ovirt-engine-4.0]: restapi: Don't populate NetworkAttachment.network.name
by piotr.kliczewski＠gmail.com
Piotr Kliczewski has submitted this change and it was merged.
Change subject: restapi: Don't populate NetworkAttachment.network.name
......................................................................
restapi: Don't populate NetworkAttachment.network.name
Currently when the list of network attachments of a host is retrieved we
return a link to the network that includes also the network name:
GET /ovirt-engine/api/hosts/123/networkattachments
<network_attachments>
<network_attachment id="..." href="...">
...
<network id="..." href="...>
<name>mynetwork</name> <-- This should't appear
</network>
</network_attachment>
...
</network_attachments>
This is not correct, as output links should not have any content other
than the "id" and "href" attributes. This patch fixes that issue.
Change-Id: Ie0eb5ba57a03f0e89c844d22969015ddf67a4134
Bug-Url: https://bugzilla.redhat.com/1382341
Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com>
(cherry picked from commit 27209c687d3f30228ea33c20a80b39bb46d6d7c3)
---
M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/NetworkAttachmentMapper.java
1 file changed, 0 insertions(+), 9 deletions(-)
Approvals:
Juan Hernandez: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/65177
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie0eb5ba57a03f0e89c844d22969015ddf67a4134
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
7 years, 6 months
Change in ovirt-engine[ovirt-engine-4.0.5]: aaa: Consolidate SSO session validation requests
by piotr.kliczewski＠gmail.com
Piotr Kliczewski has submitted this change and it was merged.
Change subject: aaa: Consolidate SSO session validation requests
......................................................................
aaa: Consolidate SSO session validation requests
Engine makes multiple queries to SSO
to validate each exisitng session. This
could cause performance issue when there
are thousands of sessions. Combining all
requests into one request to fetch all
the statuses in one request can resolve
the performance issue.
When SSO clears an engine session associated
with a token by invoking TerminateSessionsForToken
SSO_ACCESS_TOKEN_PARAMETER value in the
map is null. This can lead to problems when
equality is checked with the contents of
the map for the token.
Change-Id: Ie63dd4e85ac4e4b25cd2884d07d84483dc8add17
Bug-Url: https://bugzilla.redhat.com/1379805
Signed-off-by: Ravi Nori <rnori(a)redhat.com>
---
M backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/SsoOAuthServiceUtils.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/SessionDataContainer.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/aaa/SessionDataContainerTest.java
M backend/manager/modules/enginesso/src/main/java/org/ovirt/engine/core/sso/search/DirectorySearch.java
M backend/manager/modules/enginesso/src/main/java/org/ovirt/engine/core/sso/utils/SsoConstants.java
5 files changed, 58 insertions(+), 25 deletions(-)
Approvals:
Martin Peřina: Looks good to me, approved
Ravi Nori: Verified
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/65189
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie63dd4e85ac4e4b25cd2884d07d84483dc8add17
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0.5
Gerrit-Owner: Ravi Nori <rnori(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
7 years, 6 months
Change in ovirt-engine[ovirt-engine-4.0]: engine: Add extra debug information in AsyncTaskManager
by piotr.kliczewski＠gmail.com
Piotr Kliczewski has submitted this change and it was merged.
Change subject: engine: Add extra debug information in AsyncTaskManager
......................................................................
engine: Add extra debug information in AsyncTaskManager
Add extra debug information in AsyncTaskManager
that logs polling and updating task statuses.
Change-Id: Id3a84f280c24e8bd47096a0a7929223a4033d2ff
Bug-Url: https://bugzilla.redhat.com/1374171
Signed-off-by: Ravi Nori <rnori(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/AsyncTaskManager.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/SPMAsyncTask.java
2 files changed, 31 insertions(+), 1 deletion(-)
Approvals:
Martin Peřina: Looks good to me, approved
Ravi Nori: Verified
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/65186
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id3a84f280c24e8bd47096a0a7929223a4033d2ff
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Ravi Nori <rnori(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
7 years, 6 months
Change in ovirt-engine[master]: core: InstallVdsInternalCommandTest @Spy
by amureini＠redhat.com
Allon Mureinik has submitted this change and it was merged.
Change subject: core: InstallVdsInternalCommandTest @Spy
......................................................................
core: InstallVdsInternalCommandTest @Spy
Use the @Spy annotation to spy the command being tested in
order to allow injecting mocks in subsequent patches.
Change-Id: Id0a1cc86eec32c96d6d1c76173d7bbc24f81b18b
Signed-off-by: Allon Mureinik <amureini(a)redhat.com>
---
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/hostdeploy/InstallVdsInternalCommandTest.java
1 file changed, 7 insertions(+), 11 deletions(-)
Approvals:
Tal Nisan: Looks good to me, approved
Ravi Nori: Looks good to me, but someone else must approve
Allon Mureinik: Verified; Passed CI tests
--
To view, visit https://gerrit.ovirt.org/64630
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id0a1cc86eec32c96d6d1c76173d7bbc24f81b18b
Gerrit-PatchSet: 10
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: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
7 years, 6 months
Change in ovirt-engine[master]: webadmin: Improve column control menu
by awels＠redhat.com
Alexander Wels has submitted this change and it was merged.
Change subject: webadmin: Improve column control menu
......................................................................
webadmin: Improve column control menu
1. replaced JavaScript/jQuery JSNI code with proper Java/GWT
code to avoid performance issues & leaks
2. replaced up/down arrow icons on menu items with drag'n'drop
Side note: HTML5 DnD spec is a _total_ mess.
Change-Id: I14b47620db16b25657e4a7197a84db38fe956420
Bug-Url: https://bugzilla.redhat.com/1285499
Bug-Url: https://bugzilla.redhat.com/1368101
Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/ColumnContextMenu.java
A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/ColumnContextMenu.ui.xml
A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/ColumnContextMenuItem.java
A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/ColumnContextMenuItem.ui.xml
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/ColumnController.java
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/ColumnResizeCellTable.java
D frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/css/ColumnContextMenu.css
D frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/images/checkbox_no.png
D frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/images/checkbox_yes.png
D frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/images/move_down.png
D frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/images/move_up.png
11 files changed, 328 insertions(+), 375 deletions(-)
Approvals:
Alexander Wels: Verified; Looks good to me, approved; Passed CI tests
Vojtech Szocs: Verified
--
To view, visit https://gerrit.ovirt.org/54186
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I14b47620db16b25657e4a7197a84db38fe956420
Gerrit-PatchSet: 12
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Vojtech Szocs <vszocs(a)redhat.com>
Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com>
Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Scott Dickerson <sdickers(a)redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
7 years, 6 months