From 54dd203b93a776b1a51e83bff7e466972486e427 Mon Sep 17 00:00:00 2001
From: Lucy Fu <lufu@redhat.com>
Date: Mon, 12 Jun 2023 09:40:42 -0400
Subject: [PATCH] Fixes #36499 - correct configuration for libvirt

---
 .../javascripts/foreman_virt_who_configure/config_edit.js     | 4 ++--
 app/models/foreman_virt_who_configure/config.rb               | 4 ++--
 app/models/foreman_virt_who_configure/output_generator.rb     | 2 ++
 3 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/app/assets/javascripts/foreman_virt_who_configure/config_edit.js b/app/assets/javascripts/foreman_virt_who_configure/config_edit.js
index a7d54d3d..e70e5bc1 100644
--- a/app/assets/javascripts/foreman_virt_who_configure/config_edit.js
+++ b/app/assets/javascripts/foreman_virt_who_configure/config_edit.js
@@ -28,13 +28,13 @@ function virt_who_update_listing_mode() {
 function virt_who_update_hypervisor_fields() {
   selected_type = $('#foreman_virt_who_configure_config_hypervisor_type').val();
   var element = $('#foreman_virt_who_configure_config_hypervisor_username');
-  element.closest('.form-group').toggle(selected_type != 'kubevirt');
+  element.closest('.form-group').toggle(selected_type != 'libvirt' && selected_type != 'kubevirt');
   var element = $('#foreman_virt_who_configure_config_hypervisor_password');
   element.closest('.form-group').toggle(selected_type != 'libvirt' && selected_type != 'kubevirt');
   var element = $('#foreman_virt_who_configure_config_kubeconfig_path');
   element.closest('.form-group').toggle(selected_type == 'kubevirt');
   var element = $('#foreman_virt_who_configure_config_hypervisor_server');
-  element.closest('.form-group').toggle(selected_type != 'kubevirt');
+  element.closest('.form-group').toggle(selected_type != 'libvirt' && selected_type != 'kubevirt');
   var element = $('#foreman_virt_who_configure_config_ahv_internal_debug');
   element.closest('.form-group').toggle(selected_type == 'ahv');
   virt_who_update_listing_mode();
diff --git a/app/models/foreman_virt_who_configure/config.rb b/app/models/foreman_virt_who_configure/config.rb
index 4e62a848..080624af 100644
--- a/app/models/foreman_virt_who_configure/config.rb
+++ b/app/models/foreman_virt_who_configure/config.rb
@@ -106,8 +106,8 @@ class Config < ActiveRecord::Base
               :presence => true
     validates :name, :uniqueness => { :scope => :organization_id }
     validates :hypervisor_password, :presence => true, :if => Proc.new { |c| c.hypervisor_type != 'libvirt' && c.hypervisor_type != 'kubevirt' }
-    validates :hypervisor_username, :presence => true, :if => Proc.new { |c| c.hypervisor_type != 'kubevirt' }
-    validates :hypervisor_server, :presence => true, :if => Proc.new { |c| c.hypervisor_type != 'kubevirt' }
+    validates :hypervisor_username, :presence => true, :if => Proc.new { |c| c.hypervisor_type != 'libvirt' && c.hypervisor_type != 'kubevirt' }
+    validates :hypervisor_server, :presence => true, :if => Proc.new { |c| c.hypervisor_type != 'libvirt' && c.hypervisor_type != 'kubevirt' }
     validates :kubeconfig_path, :presence => true, :if => Proc.new { |c| c.hypervisor_type == 'kubevirt' }
     validates :hypervisor_type, :inclusion => HYPERVISOR_TYPES.keys
     validates :hypervisor_id, :inclusion => HYPERVISOR_IDS
diff --git a/app/models/foreman_virt_who_configure/output_generator.rb b/app/models/foreman_virt_who_configure/output_generator.rb
index 4ad614b0..f566fadf 100644
--- a/app/models/foreman_virt_who_configure/output_generator.rb
+++ b/app/models/foreman_virt_who_configure/output_generator.rb
@@ -159,6 +159,8 @@ def virt_who_output(format = nil)
     def connection_details
       if config.hypervisor_type == 'kubevirt'
         ""
+      elsif config.hypervisor_type == 'libvirt'
+        "\nserver=qemu:///system"
       else
         "\nserver=#{cr_server}
 username=#{cr_username}