January 2017 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: Sync db script numbering with ovirt-engine-4.1 branch
by Code Review 12 Jan '17

12 Jan '17

>From Martin Peřina <mperina(a)redhat.com>: Martin Peřina has submitted this change and it was merged. Change subject: core: Sync db script numbering with ovirt-engine-4.1 branch ...................................................................... core: Sync db script numbering with ovirt-engine-4.1 branch In order not to break upgrade from ovirt-engine-4.1 branch and from existing master installation, we need to rename all existing 4.1 upgrade script which numbering differs from ovirt-engine-4.1 branch. Change-Id: I41b53b08b8b53892c6eb7ef9f839d041a78b237f Signed-off-by: Martin Perina <mperina(a)redhat.com> --- R packaging/dbscripts/upgrade/04_02_0010_add_vm_leases.sql R packaging/dbscripts/upgrade/04_02_0020_add_glustervolume_id_to_storage_server_connection.sql R packaging/dbscripts/upgrade/04_02_0030_remove_PackageNamesForCheckUpdate_general_config_value.sql R packaging/dbscripts/upgrade/04_02_0040_add_vm_configuration_broken_to_snapshots.sql R packaging/dbscripts/upgrade/04_02_0050_set_blank_template_version_name.sql R packaging/dbscripts/upgrade/04_02_0060_add_network_usage_to_vm_statistics.sql R packaging/dbscripts/upgrade/04_02_0070_create_storage_dr.sql 7 files changed, 0 insertions(+), 0 deletions(-) Approvals: Martin Peřina: Verified Ravi Nori: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/69758 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I41b53b08b8b53892c6eb7ef9f839d041a78b237f Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(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: Add 4.2 support
by Code Review 12 Jan '17

12 Jan '17

>From Martin Peřina <mperina(a)redhat.com>: Martin Peřina has submitted this change and it was merged. Change subject: core: Add 4.2 support ...................................................................... core: Add 4.2 support - Introduces 4.2 version - Upon new installation created default data center and cluster with version 4.2 - Updates configuration values to support version 4.2 - Allows to add db upgrade scripts for version 4.2 NOTE: Before merging this patch we need to merge a patch to VDSM to add version 4.2 to supported engines and supported cluster levels, otherwise we won't be able to add a host into 4.2 cluster. Change-Id: I387167814e61db9ef8f6f2468b17b2475d038caf Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/VersionStorageFormatUtil.java M backend/manager/modules/compat/src/main/java/org/ovirt/engine/core/compat/Version.java M packaging/dbscripts/common_sp.sql M packaging/dbscripts/data/00300_insert_storage_pool.sql M packaging/dbscripts/data/00400_insert_vds_groups.sql M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql M packaging/setup/plugins/ovirt-engine-common/base/core/misc.py 7 files changed, 25 insertions(+), 16 deletions(-) Approvals: Martin Sivák: Looks good to me, but someone else must approve Martin Peřina: Verified Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, but someone else must approve Sahina Bose: Looks good to me, but someone else must approve Moti Asayag: Looks good to me, approved Arik Hadas: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/69757 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I387167814e61db9ef8f6f2468b17b2475d038caf Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(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: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Fix HotUnplugMemorySupported definition
by Code Review 12 Jan '17

12 Jan '17

>From Martin Peřina <mperina(a)redhat.com>: Martin Peřina has submitted this change and it was merged. Change subject: core: Fix HotUnplugMemorySupported definition ...................................................................... core: Fix HotUnplugMemorySupported definition HotUnplugMemorySupported is disabled in the code and so far it was not enabled in any version, so there's no need to have the value in db and we can rely on the default defined in ConfigValues. When the feature becomes enabled in future, we will update values in db accordingly. Change-Id: I3b3d042dc4d331f11272cbbbf23eacf5e09004dd Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Martin Peřina: Verified Jenkins CI: Passed CI tests Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/69829 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3b3d042dc4d331f11272cbbbf23eacf5e09004dd Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(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: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Fix QemuimgCommitSupported definition
by Code Review 12 Jan '17

12 Jan '17

>From Martin Peřina <mperina(a)redhat.com>: Martin Peřina has submitted this change and it was merged. Change subject: core: Fix QemuimgCommitSupported definition ...................................................................... core: Fix QemuimgCommitSupported definition QemuimgCommitSupportedM option was enabled in 4.1: 1. New installations - For versions up to 4.0 (included) feature is disabled by adding false value into db - For versions greater than 4.0 we will rely on default value in ConfigValues class 2. Upgrades - For versions up to 4.0 (included) the feature is already disabled in db - For version 4.1 we enable the feature by updating db value (as the feature was enabled during 4.1 development) - For versions greater than 4.0 we will rely on default value in ConfigValues class Change-Id: Ic6d452f7b11e4f0935aff78f18fe1da0e634b340 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Martin Peřina: Verified Jenkins CI: Passed CI tests Liron Aravot: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/69828 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic6d452f7b11e4f0935aff78f18fe1da0e634b340 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Fix DataOperationsByHSM definition
by Code Review 12 Jan '17

12 Jan '17

>From Martin Peřina <mperina(a)redhat.com>: Martin Peřina has submitted this change and it was merged. Change subject: core: Fix DataOperationsByHSM definition ...................................................................... core: Fix DataOperationsByHSM definition DataOperationsByHSM option was enabled in 4.1: 1. New installations - For versions up to 4.0 (included) feature is disabled by adding false value into db - For versions greater than 4.0 we will rely on default value in ConfigValues class 2. Upgrades - For versions up to 4.0 (included) the feature is already disabled in db - For version 4.1 we enable the feature by updating db value (as the feature was enabled during 4.1 development) - For versions greater than 4.0 we will rely on default value in ConfigValues class Change-Id: I01482044adf729150740c90f8b3cbc09851e5b5c Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Martin Peřina: Verified Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, but someone else must approve Moti Asayag: Looks good to me, but someone else must approve Liron Aravot: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/69754 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I01482044adf729150740c90f8b3cbc09851e5b5c Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(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]: core: Fix SshSoftFencingCommand definition
by Code Review 12 Jan '17

12 Jan '17

>From Martin Peřina <mperina(a)redhat.com>: Martin Peřina has submitted this change and it was merged. Change subject: core: Fix SshSoftFencingCommand definition ...................................................................... core: Fix SshSoftFencingCommand definition SshSoftFencingCommand contains the same value for all version, so there's no need to have the same values in db for each version and we can rely on defaults defined in ConfigValues class. Change-Id: Idae417013b5f277ed07c0d668754efaae986510e Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 1 file changed, 0 insertions(+), 2 deletions(-) Approvals: Martin Peřina: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/69827 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idae417013b5f277ed07c0d668754efaae986510e Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(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: Fix VdsFenceOptionMapping and VdsFenceType definition
by Code Review 12 Jan '17

12 Jan '17

>From Martin Peřina <mperina(a)redhat.com>: Martin Peřina has submitted this change and it was merged. Change subject: core: Fix VdsFenceOptionMapping and VdsFenceType definition ...................................................................... core: Fix VdsFenceOptionMapping and VdsFenceType definition Support for ilo_ssh has been added in 4.1 and backported to 3.6.z/4.0.z, so for new installation we can relay on defaults defined in ConfigValues class and in db only handle upgrades for 3.6, 4.0 and 4.1. Change-Id: I500e43554c67d9a23a56bfc53b04034b2345f618 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 2 files changed, 19 insertions(+), 4 deletions(-) Approvals: Martin Peřina: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/69755 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I500e43554c67d9a23a56bfc53b04034b2345f618 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(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[ovirt-engine-4.1]: gluster: show replica count in new volume popup
by Code Review 12 Jan '17

12 Jan '17

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: gluster: show replica count in new volume popup ...................................................................... gluster: show replica count in new volume popup Gluster Volume Type is initialized to 'Replicate' before listeners are attached. So changing the order so that volume type is initialized after attaching listeners. Change-Id: Ic0d52f9e1e49e8c5c08bfb321221f4c2107a9b54 Bug-Url: https://bugzilla.redhat.com/1366167 Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/VolumePopupView.ui.xml 2 files changed, 3 insertions(+), 4 deletions(-) Approvals: Jenkins CI: Passed CI tests Sahina Bose: Looks good to me, approved Ramesh N: Verified -- To view, visit https://gerrit.ovirt.org/70097 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic0d52f9e1e49e8c5c08bfb321221f4c2107a9b54 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Ramesh N <rnachimu(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: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Normalize repo names in centos-6 mock configs
by Code Review 12 Jan '17

12 Jan '17

>From Barak Korren <bkorren(a)redhat.com>: Barak Korren has submitted this change and it was merged. Change subject: Normalize repo names in centos-6 mock configs ...................................................................... Normalize repo names in centos-6 mock configs We need this so mirror injection can work Change-Id: I3c11b964fec7b836dbcc54f49463388ba9d889cf Jira-Ticket: https://ovirt-jira.atlassian.net/browse/OVIRT-575 Signed-off-by: Barak Korren <bkorren(a)redhat.com> --- D mock_configs/epel-6-i386.cfg M mock_configs/epel-6-x86_64.cfg M mock_configs/epel-6-x86_64_proxied.cfg 3 files changed, 6 insertions(+), 103 deletions(-) Approvals: Barak Korren: Verified; Looks good to me, approved; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/69432 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3c11b964fec7b836dbcc54f49463388ba9d889cf Gerrit-PatchSet: 3 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Adapt mock_runner.sh to use mirrors
by Code Review 12 Jan '17

12 Jan '17

>From Barak Korren <bkorren(a)redhat.com>: Barak Korren has submitted this change and it was merged. Change subject: Adapt mock_runner.sh to use mirrors ...................................................................... Adapt mock_runner.sh to use mirrors - Added a parameter to mock_runner.sh to enable it to use mirrors. - Changed moch config files for CentOS 7 to standardize repo names so mirrors can be used. - Excluded *-release packages from epel and centos-extras repos so we won't have extra *.repo files slipping into the environment - Made sure distro default repo files are removed from the mock environment - Added a CI test to verify we don't have stray repo files - Added a name to the ci-tool repo configured for check-patch on el7 to allow it to be mirrored in the future Change-Id: I282d68060e6073e0b25b992f1e17b6b06b71830f Signed-off-by: Barak Korren <bkorren(a)redhat.com> Jira-Ticket: https://ovirt-jira.atlassian.net/browse/OVIRT-575 --- M automation/check-patch.repos.el7 M automation/check-patch.sh M mock_configs/epel-7-x86_64.cfg M mock_configs/epel-7-x86_64_proxied.cfg M mock_configs/mock_runner.sh 5 files changed, 79 insertions(+), 54 deletions(-) Approvals: Eyal Edri: Looks good to me Barak Korren: Verified; Looks good to me, approved; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/69250 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I282d68060e6073e0b25b992f1e17b6b06b71830f Gerrit-PatchSet: 10 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <david(a)dcaro.es> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nadav Goldin <ngoldin(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0