- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: restapi: Add IPv6 fields to HostNic
by juan.hernandez＠redhat.com 20 Apr '16

20 Apr '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Add IPv6 fields to HostNic ...................................................................... restapi: Add IPv6 fields to HostNic Change-Id: I420315342fbf08b2034fa4ed3a7177282a669171 Bug-Url: https://bugzilla.redhat.com/1080725 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendHostNicsResourceTest.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostNicMapper.java 2 files changed, 112 insertions(+), 38 deletions(-) Approvals: Juan Hernandez: Looks good to me, approved Jenkins CI: Passed CI tests Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/55476 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I420315342fbf08b2034fa4ed3a7177282a669171 Gerrit-PatchSet: 29 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: remove support for fc22
by sbonazzo＠redhat.com 20 Apr '16

20 Apr '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: core: remove support for fc22 ...................................................................... core: remove support for fc22 Remove support for fc22 from check-patch.sh script. Change-Id: I2c72491cf7c33460aa9467e50071883f5c76d6e5 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M automation/check-patch.sh 1 file changed, 0 insertions(+), 7 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Dan Kenigsberg: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/56363 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2c72491cf7c33460aa9467e50071883f5c76d6e5 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-java[master]: Fix connect timeout of HTTP connection
by juan.hernandez＠redhat.com 20 Apr '16

20 Apr '16

Juan Hernandez has submitted this change and it was merged. Change subject: Fix connect timeout of HTTP connection ...................................................................... Fix connect timeout of HTTP connection setConnectionRequestTimeout set the timeout in milliseconds used when requesting a connection from the connection manager. setConnectTimeout set the timeout in milliseconds until a connection is established. We have to use setConnectTimeout. Change-Id: Ie4ff807613a8ae48312544a590f2eaca864cdae4 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M sdk/src/main/java/org/ovirt/engine/sdk4/internal/ConnectionBuilder45.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Ondra Machacek: Verified Juan Hernandez: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56346 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie4ff807613a8ae48312544a590f2eaca864cdae4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-java[master]: Add SSO support
by juan.hernandez＠redhat.com 20 Apr '16

20 Apr '16

Juan Hernandez has submitted this change and it was merged. Change subject: Add SSO support ...................................................................... Add SSO support Use SSO service for authentication. When authenticating the sdk should obtain access_token from SSO and use Bearer authentication to access API using the access token. The SSO access token is revoked at the end of the session to terminate the session on SSO and clean up the engine session. Change-Id: I93eefb798b7987ec08ba287f08375c739e19a4ea Bug-Url: https://bugzilla.redhat.com/1285885 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M automation/build-artifacts.packages M automation/check-patch.packages M packaging/spec.el7.in M packaging/spec.fc23.in M sdk/pom.xml M sdk/src/main/java/org/ovirt/engine/sdk4/ConnectionBuilder.java M sdk/src/main/java/org/ovirt/engine/sdk4/internal/ConnectionBuilder42.java M sdk/src/main/java/org/ovirt/engine/sdk4/internal/ConnectionBuilder45.java M sdk/src/main/java/org/ovirt/engine/sdk4/internal/HttpConnection.java A sdk/src/main/java/org/ovirt/engine/sdk4/internal/SsoUtils.java 10 files changed, 287 insertions(+), 50 deletions(-) Approvals: Ondra Machacek: Verified Juan Hernandez: Looks good to me, approved Ravi Nori: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56109 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I93eefb798b7987ec08ba287f08375c739e19a4ea Gerrit-PatchSet: 12 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(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: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-dwh[master]: history: fix bug in 3.6 views of data center statistics
by Shirly Radco 20 Apr '16

20 Apr '16

Shirly Radco has submitted this change and it was merged. Change subject: history: fix bug in 3.6 views of data center statistics ...................................................................... history: fix bug in 3.6 views of data center statistics Change-Id: I26b2533431e7dbb7d0a64fcc67f3179bf19ac468 Bug-Url: https://bugzilla.redhat.com/1328769 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M packaging/dbscripts/create_views_3_6.sql 1 file changed, 3 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Shirly Radco: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56383 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I26b2533431e7dbb7d0a64fcc67f3179bf19ac468 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: aaa: Add service to invoke SSO Logout
by mperina＠redhat.com 20 Apr '16

20 Apr '16

Martin Peřina has submitted this change and it was merged. Change subject: aaa: Add service to invoke SSO Logout ...................................................................... aaa: Add service to invoke SSO Logout Add a service sso-logout to invoke logout given an access token and scope Change-Id: Idd11e1af4d705b8b6ee6e7a9e621670c3f79dad5 Bug-Url: https://bugzilla.redhat.com/1092744 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- A backend/manager/modules/services/src/main/java/org/ovirt/engine/core/services/SSOLogoutServlet.java M backend/manager/modules/services/src/main/webapp/WEB-INF/web.xml 2 files changed, 58 insertions(+), 0 deletions(-) Approvals: Ondra Machacek: Looks good to me, but someone else must approve Martin Peřina: Looks good to me, approved Juan Hernandez: Looks good to me, but someone else must approve Ravi Nori: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/55031 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idd11e1af4d705b8b6ee6e7a9e621670c3f79dad5 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: stop polling the database for vm-jobs
by ahadas＠redhat.com 20 Apr '16

20 Apr '16

Arik Hadas has submitted this change and it was merged. Change subject: core: stop polling the database for vm-jobs ...................................................................... core: stop polling the database for vm-jobs Now that we have the up-to-date status of the VM-jobs in memory, the callback of the merge operation can query that in-memory repository instead of querying the database. Change-Id: Ia597bc449b17e5da768998eb5e577dfedb745535 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MergeCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MergeCommandCallback.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmJobDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmJobDaoImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmJobDaoTest.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmJobsMonitoring.java M packaging/dbscripts/vm_jobs_sp.sql 7 files changed, 33 insertions(+), 84 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Greg Padgett: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Shmuel Leib Melamud: Looks good to me, but someone else must approve Ala Hino: Looks good to me, but someone else must approve Moti Asayag: Looks good to me, but someone else must approve Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/54442 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia597bc449b17e5da768998eb5e577dfedb745535 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: engine: use IpAddressPredicate in IP address comparisons
by alkaplan＠redhat.com 20 Apr '16

20 Apr '16

Alona Kaplan has submitted this change and it was merged. Change subject: engine: use IpAddressPredicate in IP address comparisons ...................................................................... engine: use IpAddressPredicate in IP address comparisons Use IpAddressPredicate in different IP address comparisons instead of the simple string comparisons. Change-Id: Ib77da9b7cde92649b9accbb9dabffcc781bf9c72 Bug-Url: https://bugzilla.redhat.com/1080725 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/NetworkAttachmentValidator.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/validator/NetworkAttachmentValidatorTest.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java 3 files changed, 124 insertions(+), 57 deletions(-) Approvals: Alona Kaplan: Looks good to me, approved; Passed CI tests Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/56092 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib77da9b7cde92649b9accbb9dabffcc781bf9c72 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Remove unused, empty VmQuery class
by amureini＠redhat.com 20 Apr '16

20 Apr '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: Remove unused, empty VmQuery class ...................................................................... core: Remove unused, empty VmQuery class Change-Id: I1e3b065780a5bbd9e1f91ce3675caac4ab1732ed Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmQuery.java 1 file changed, 0 insertions(+), 4 deletions(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/56372 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1e3b065780a5bbd9e1f91ce3675caac4ab1732ed 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: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(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]: engine: add IPv6 to isInSync logic
by alkaplan＠redhat.com 20 Apr '16

20 Apr '16

Alona Kaplan has submitted this change and it was merged. Change subject: engine: add IPv6 to isInSync logic ...................................................................... engine: add IPv6 to isInSync logic Change-Id: Id8ca6ac4780353f57faf4c5eb161b7b32d0399ed Bug-Url: https://bugzilla.redhat.com/1080725 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/network/ReportedConfigurationType.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/network/ReportedConfigurations.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/NetworkInSyncWithVdsNetworkInterface.java M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/NetworkInSyncWithVdsNetworkInterfaceTest.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/NetworkOutOfSyncPanel.java 6 files changed, 384 insertions(+), 132 deletions(-) Approvals: Alona Kaplan: Looks good to me, approved; Passed CI tests Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/56056 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id8ca6ac4780353f57faf4c5eb161b7b32d0399ed Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0