- Engine-commits - oVirt List Archives

Change in ovirt-log-collector[master]: inventory: Add header/desc to installed packages in engine
by Code Review 22 Aug '17

22 Aug '17

>From Douglas Schilling Landgraf <dougsland(a)redhat.com>: Douglas Schilling Landgraf has submitted this change and it was merged. ( https://gerrit.ovirt.org/80928 ) Change subject: inventory: Add header/desc to installed packages in engine ...................................................................... inventory: Add header/desc to installed packages in engine Change-Id: I5c854cb645a064332395bc987690d0625f41668f Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M src/inventory_report/produceReport/produceReport.sh 1 file changed, 5 insertions(+), 3 deletions(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/80928 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5c854cb645a064332395bc987690d0625f41668f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-log-collector Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-log-collector[master]: inventory: Add copyright header for the SQLs files
by Code Review 22 Aug '17

22 Aug '17

>From Douglas Schilling Landgraf <dougsland(a)redhat.com>: Douglas Schilling Landgraf has submitted this change and it was merged. ( https://gerrit.ovirt.org/80927 ) Change subject: inventory: Add copyright header for the SQLs files ...................................................................... inventory: Add copyright header for the SQLs files Change-Id: I088901bc4a27ed896e562e115c1570fc4351a1c0 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M src/inventory_report/produceReport/sqls/agent_passwords.sql M src/inventory_report/produceReport/sqls/async_tasks_query_running_tasks.sql M src/inventory_report/produceReport/sqls/bookmarks_query_name_value.sql M src/inventory_report/produceReport/sqls/bookmarks_query_number_of_rows.sql M src/inventory_report/produceReport/sqls/business_entity_snapshot_query_running_compensation_tasks.sql M src/inventory_report/produceReport/sqls/cleanup.sql M src/inventory_report/produceReport/sqls/cleanup_procedures_for_reporting_agent_passwords_as_csv.sql M src/inventory_report/produceReport/sqls/cluster_query_check_datacenter.sql M src/inventory_report/produceReport/sqls/cluster_query_migration_policies.sql M src/inventory_report/produceReport/sqls/cluster_query_minimum_3_6_compat_version.sql M src/inventory_report/produceReport/sqls/cluster_query_show_all_clusters.sql M src/inventory_report/produceReport/sqls/cluster_query_vms_per_cluster.sql M src/inventory_report/produceReport/sqls/command_entities_query_get_running_commands.sql M src/inventory_report/produceReport/sqls/database_size.sql M src/inventory_report/produceReport/sqls/datacenter_query_minimum_3_6_compat_version.sql M src/inventory_report/produceReport/sqls/datacenter_show_all.sql M src/inventory_report/produceReport/sqls/dws_query_check_if_its_running.sql M src/inventory_report/produceReport/sqls/engine_versions_through_all_upgrades.sql M src/inventory_report/produceReport/sqls/hosts_create_related_lookup_tables.sql M src/inventory_report/produceReport/sqls/hosts_query_check_health.sql M src/inventory_report/produceReport/sqls/hosts_query_check_pretty_name.sql M src/inventory_report/produceReport/sqls/images_query_invalid_status.sql M src/inventory_report/produceReport/sqls/images_query_vm_snapshot_id_zero.sql M src/inventory_report/produceReport/sqls/lun_storage_server_connection_map_query_number_connection_map.sql M src/inventory_report/produceReport/sqls/luns_query_all_data.sql M src/inventory_report/produceReport/sqls/mac_pools_query_show_data_based_on_datacenter_and_cluster.sql M src/inventory_report/produceReport/sqls/network_query_cluster_with_non_ovirtmgmt_as_management_network.sql M src/inventory_report/produceReport/sqls/networks_table_not_using_network_attachments.sql M src/inventory_report/produceReport/sqls/networks_table_using_network_attachments.sql M src/inventory_report/produceReport/sqls/prepare_procedures_for_reporting_agent_passwords_as_csv.sql M src/inventory_report/produceReport/sqls/providers_query_show_data.sql M src/inventory_report/produceReport/sqls/storage_create_related_lookup_tables.sql M src/inventory_report/produceReport/sqls/storage_domains_lun_data.sql M src/inventory_report/produceReport/sqls/storage_domains_query_data.sql M src/inventory_report/produceReport/sqls/storage_domains_query_get_failing_domains.sql M src/inventory_report/produceReport/sqls/users_query_system_users.sql M src/inventory_report/produceReport/sqls/vdc_options_query_AAA_legacy.sql M src/inventory_report/produceReport/sqls/vdc_options_query_checkmixedrhelversion.sql M src/inventory_report/produceReport/sqls/vms_check_no_timezone_set.sql M src/inventory_report/produceReport/sqls/vms_create_related_lookup_tables.sql M src/inventory_report/produceReport/sqls/vms_query_check_obsolete_cluster.sql M src/inventory_report/produceReport/sqls/vms_query_health.sql M src/inventory_report/produceReport/sqls/vms_query_linux_and_others_vms_with_incorrect_timezone.sql M src/inventory_report/produceReport/sqls/vms_query_pinned_virtual_machines.sql M src/inventory_report/produceReport/sqls/vms_query_total_number_of_virtual_machines_in_engine.sql M src/inventory_report/produceReport/sqls/vms_query_total_number_of_virtual_machines_linux_other_OS.sql M src/inventory_report/produceReport/sqls/vms_query_total_number_of_virtual_machines_windows_OS.sql M src/inventory_report/produceReport/sqls/vms_query_windows_vms_with_incorrect_timezone.sql M src/inventory_report/produceReport/sqls/vms_with_cluster_lower_3_6_with_virtio_serial_console.sql 49 files changed, 590 insertions(+), 0 deletions(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/80927 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I088901bc4a27ed896e562e115c1570fc4351a1c0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-log-collector Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: Add firewall type field to cluster
by Code Review 22 Aug '17

22 Aug '17

>From Martin Peřina <mperina(a)redhat.com>: Martin Peřina has submitted this change and it was merged. ( https://gerrit.ovirt.org/79847 ) Change subject: webadmin: Add firewall type field to cluster ...................................................................... webadmin: Add firewall type field to cluster This patch add support to choose a firewall type supported by cluster, this firewall is later used on all hosts in a cluster. Change-Id: I9e672aae2646742f954e46da12139ecca5c35ab0 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1462811 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/LocalizedEnums.java M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums.properties 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/cluster/ClusterPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.ui.xml M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/ApplicationConstants.properties 11 files changed, 64 insertions(+), 0 deletions(-) Approvals: Tomas Jelinek: Looks good to me, but someone else must approve Ondra Machacek: Verified Martin Peřina: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/79847 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9e672aae2646742f954e46da12139ecca5c35ab0 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Execute ansible-playbook after otopi host-deploy
by Code Review 22 Aug '17

22 Aug '17

>From Martin Peřina <mperina(a)redhat.com>: Martin Peřina has submitted this change and it was merged. ( https://gerrit.ovirt.org/78504 ) Change subject: core: Execute ansible-playbook after otopi host-deploy ...................................................................... core: Execute ansible-playbook after otopi host-deploy This patch add new class called AnsibleExecutor, which executes external command 'ansible-playbook'. The user can build a command which will be executed, by adding attributes like additional variables, hostnames, private key, etc. Currently there is executed playbook called 'ovirt-host-deploy', right after the otopi host-deploy is executed successfully. This playbook is shipped via ovirt/ovirt-ansible github repository[1]. As developer you have to clone this repo and install it to your dev env. For RPM users, the ovirt-engine RPM have dependency on this repository. Currently the ovirt-host-deploy playbook, will only execute the ovirt-host-deploy role, which only implemente firewalld configuration on the host. [1] https://github.com/ovirt/ovirt-ansible Change-Id: I630f10ceb451276053f0c44704dbf691f5498b7d Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1462811 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M Makefile M README.adoc M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/hostdeploy/InstallVdsInternalCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/hostdeploy/VdsDeployBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/hostdeploy/VdsDeployIptablesUnit.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/common/utils/ansible/AnsibleCommandBuilder.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/common/utils/ansible/AnsibleConstants.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/common/utils/ansible/AnsibleExecutor.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/common/utils/ansible/AnsibleReturnCode.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/common/utils/ansible/AnsibleVerbosity.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/common/utils/ansible/AnsibleCommandBuilderTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/FeatureSupported.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties M ovirt-engine.spec.in 16 files changed, 780 insertions(+), 5 deletions(-) Approvals: Ondra Machacek: Verified Martin Peřina: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/78504 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I630f10ceb451276053f0c44704dbf691f5498b7d Gerrit-PatchSet: 22 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Dominik Holler <dholler(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: Ondra Machacek <omachace(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[master]: restapi: Add mapper support for firewall type of cluster
by Code Review 22 Aug '17

22 Aug '17

>From Martin Peřina <mperina(a)redhat.com>: Martin Peřina has submitted this change and it was merged. ( https://gerrit.ovirt.org/79848 ) Change subject: restapi: Add mapper support for firewall type of cluster ...................................................................... restapi: Add mapper support for firewall type of cluster This patch add a mapping for cluster to support also firewall type. Change-Id: I473151159fb36966e9932d8eed842c3a32045447 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1462811 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/ClusterMapper.java A backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/FirewallTypeMapper.java 2 files changed, 52 insertions(+), 0 deletions(-) Approvals: Ondra Machacek: Verified Ori Liel: Looks good to me, approved Martin Peřina: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/79848 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I473151159fb36966e9932d8eed842c3a32045447 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine 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: 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: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Add firewall type field to cluster
by Code Review 22 Aug '17

22 Aug '17

>From Martin Peřina <mperina(a)redhat.com>: Martin Peřina has submitted this change and it was merged. ( https://gerrit.ovirt.org/79846 ) Change subject: core: Add firewall type field to cluster ...................................................................... core: Add firewall type field to cluster This patch add a firewall type to cluster, so we can manage the type of the firewall to be used for all hosts in cluster. Change-Id: I65527ce4a383312fb6085a8045f07d4906161438 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1462811 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddClusterCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ClusterOperationCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateClusterCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/Cluster.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/network/DefaultFirewallType.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/network/FirewallType.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/ClusterDaoImpl.java M backend/manager/modules/dal/src/test/resources/fixtures.xml M packaging/dbscripts/clusters_sp.sql A packaging/dbscripts/upgrade/04_02_0500_add_firewall_type_to_cluster.sql 10 files changed, 133 insertions(+), 17 deletions(-) Approvals: Martin Peřina: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/79846 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I65527ce4a383312fb6085a8045f07d4906161438 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(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: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: packaging: setup: Allow ovirt user to access engine_id_rsa
by Code Review 22 Aug '17

22 Aug '17

>From Martin Peřina <mperina(a)redhat.com>: Martin Peřina has submitted this change and it was merged. ( https://gerrit.ovirt.org/80664 ) Change subject: packaging: setup: Allow ovirt user to access engine_id_rsa ...................................................................... packaging: setup: Allow ovirt user to access engine_id_rsa ... so that the engine can use it for ansible. Change-Id: I4c520e63d9cfbae44a9546db1f3d8cb534022f58 Bug-Url: https://bugzilla.redhat.com/1462811 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/pki/ssh.py 1 file changed, 20 insertions(+), 2 deletions(-) Approvals: Ondra Machacek: Verified Martin Peřina: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/80664 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4c520e63d9cfbae44a9546db1f3d8cb534022f58 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Ido Rosenzwig <irosenzw(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Rafael Martins <rafael(a)rafaelmartins.eng.br> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yaniv Kaul <ykaul(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-log-collector[master]: inventory: improve nfs storage domain
by Code Review 22 Aug '17

22 Aug '17

>From Douglas Schilling Landgraf <dougsland(a)redhat.com>: Douglas Schilling Landgraf has submitted this change and it was merged. ( https://gerrit.ovirt.org/80926 ) Change subject: inventory: improve nfs storage domain ...................................................................... inventory: improve nfs storage domain - Only display the header if we have data available - Add copyright to sql file - put the inner join in one single line Change-Id: Id186d79fe1c366d5f89284e5ed10dbea0d9a2eee Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M src/inventory_report/produceReport/produceReport.sh M src/inventory_report/produceReport/sqls/storage_domains_nfs_path.sql 2 files changed, 16 insertions(+), 9 deletions(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/80926 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id186d79fe1c366d5f89284e5ed10dbea0d9a2eee Gerrit-PatchSet: 1 Gerrit-Project: ovirt-log-collector Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-log-collector[master]: inventory: fix query for mac pools
by Code Review 22 Aug '17

22 Aug '17

>From Douglas Schilling Landgraf <dougsland(a)redhat.com>: Douglas Schilling Landgraf has submitted this change and it was merged. ( https://gerrit.ovirt.org/80925 ) Change subject: inventory: fix query for mac pools ...................................................................... inventory: fix query for mac pools There are SQL schema that clusters tablet might exist and not have mac_pool_id. Adding double check. Change-Id: I5f444b6e22a1fb229c36e43cfd4e714f172daa98 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M src/inventory_report/produceReport/sqls/mac_pools_query_show_data_based_on_datacenter_and_cluster.sql 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/80925 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5f444b6e22a1fb229c36e43cfd4e714f172daa98 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-log-collector Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-log-collector[master]: inventory: HE - Check if HA is in Global Maint
by Code Review 22 Aug '17

22 Aug '17

>From Douglas Schilling Landgraf <dougsland(a)redhat.com>: Douglas Schilling Landgraf has submitted this change and it was merged. ( https://gerrit.ovirt.org/80922 ) Change subject: inventory: HE - Check if HA is in Global Maint ...................................................................... inventory: HE - Check if HA is in Global Maint Change-Id: Ide30c7ce8a318b304b580fcd264717a01aef3369 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M src/inventory_report/Makefile.am M src/inventory_report/produceReport/pre-upgrade-checks A src/inventory_report/produceReport/sqls/vds_statistics_query_ha_global_maintenance.sql 3 files changed, 54 insertions(+), 0 deletions(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/80922 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ide30c7ce8a318b304b580fcd264717a01aef3369 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-log-collector Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0