Arik Hadas has submitted this change and it was merged.
Change subject: core, db: Adding Cpu Qos DB config
......................................................................
core, db: Adding Cpu Qos DB config
Adding the Cpu Qos db config,
the Cpu Qos consists of one parameter: cpu_limit.
The parameter will be added to the qos table.
support for this feature is for all 3.x versions.
Also changed Storage Qos insert and update stored proc to fit common
behavior ("insert"+"storage"/"cpu"+"qos").
Change-Id: I533bcc601ac44289c51f5d56a6da0dd2455116d6
Bug-Url:
https://bugzilla.redhat.com/1084930
Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com>
---
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/FeatureSupported.java
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/ConfigurationValues.java
M
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/DbFacade.java
M
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/DefaultGenericDaoDbFacade.java
M
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/qos/AllQosBaseDaoFacadeImpl.java
A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/qos/CpuQosDao.java
A
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/qos/CpuQosDaoDbFacadeImpl.java
M
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/qos/QosBaseDaoFacadeImpl.java
M backend/manager/modules/dal/src/main/jdbc-resources/engine-daos.properties
M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/FixturesTool.java
A
backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/qos/CpuQosDaoTest.java
M backend/manager/modules/dal/src/test/resources/fixtures.xml
M packaging/dbscripts/qos_sp.sql
A packaging/dbscripts/upgrade/03_06_0280_add_cpu_limit_to_qos_table.sql
M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql
16 files changed, 250 insertions(+), 6 deletions(-)
Approvals:
Arik Hadas: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/27686
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I533bcc601ac44289c51f5d56a6da0dd2455116d6
Gerrit-PatchSet: 10
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Kobi Ianko <kobi(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server