Thanks for the report, What options are necessary and why? It seems the only uncommented value after default installation is "VIRTWHO_DEBUG=0". Maybe the file should contain some comment about that it was generated by the configuration plugin.

I'm afraid if the customers want to set the options by self, such as running hypervisor mode or running for satellite5 by /etc/sysconfig/virt-who, they don't know which options are supported any more. 

VIRTWHO_ESX=1
VIRTWHO_ESX_OWNER=
VIRTWHO_ESX_ENV=
VIRTWHO_ESX_SERVER=
VIRTWHO_ESX_USERNAME=
VIRTWHO_ESX_PASSWORD=

# Options for Satellite 5 backend
VIRTWHO_SATELLITE5=1
VIRTWHO_SATELLITE_SERVER=
VIRTWHO_SATELLITE_USERNAME=
VIRTWHO_SATELLITE_PASSWORD=

I don't think people would try to combine manual and plugin configuration. Would the warning comment on top of the file, that this file is managed by the virt-who-configure plugin, make it clearer?

If customers wants to add new esx hypervisor, they should use the plugin that creates config file in /etc/virt-who.d for it.

If users manually modify configuration that was created by the plugin, it kind of voids the warranty that the configuration is correct.

Hi Chris,

What's your opinion?

is it possible to keep only global options in /etc/sysconfig/virt-who? thus, the hypervisor modes only can be run by CLI or /etc/virt-who.d/xxx.conf

From my basic knowledge, there's no way to specify interval or debug option in virt-who.d config file. I'd be happy to avoid touching /etc/sysconfig/virt-who if that's possible.

There are three ways to set the interval:

setting VIRTWHO_INTERVAL to an int in /etc/sysconfig (as is already being done)

OR

setting the '-i' or '--interval' option when running virt-who via the CLI.

OR

setting 'interval' in the '[global]' section of /etc/virt-who.conf to an int.



They are equivalent in function.


We'll have to ask Rich Jerrido but I am under the impression that enough customers configure virt-who using the other values in /etc/sysconfig/virt-who that we should still support them.

Rich thoughts on the above?


Marek,

I would also ask, what happens when one runs two scripts generated by the virt-who satellite plugin on the same system? I would presume that the global values set in the script run last on the system would be the ones that remain.

Cheers,
Chris

In a Satellite 6.x use-case with the configuration script, I would treat the configurations that are in /etc/sysconfig/virt-who AND /etc/virt-who.d/* as being 'owned' by the configuration script.

Outside of the options listed in comment #0 (shown below), I can't think of any needed. (most of the /etc/sysconfig/virt-who options can be also/alternatively be set in /etc/virt-who.d/).

VIRTWHO_SATELLITE6=1
VIRTWHO_DEBUG=1
VIRTWHO_INTERVAL=3600

Basically, I'd be OK with us placing a notice in the config file similar to 


### This configuration managed via the virt-who plugin
### manual edits will be deleted. 



I would like to take the stance (and I think we should document it as such) that virt-who configurations that are managed via the virt-who plugin, should SOLELY be managed by the virt-who plugin. 

This is similar to any other configuration file managed by the installer/application.

Thanks Chris. So until there's a way to have different interval per worker, I think changing /etc/sysconfig/virt-who is the best option at the moment. I agree with what Rich described, so I'll add the comment in to the top of the file.

You're right about two configurations overriding the same value. Since there's no way to set different interval per hypervisor, we'll have to live with it for now. Hopefully people will use recommended value and keep the default interval. I don't see any RFE for virt-who to enable different intervals per hypervisor, should I open one against virt-who?

Created redmine issue http://projects.theforeman.org/issues/20296 from this bug

Bryan, I'm lowering the priority. As per comment 7, we should only provide a comment into the config file so people are warned if they are modifying the config file manually. Please reset if you think it's still high priority.

VERIFIED

Version tested:
Satellite 6.3 Snap 13
tfm-rubygem-hammer_cli_foreman_virt_who_configure-0.0.3-1.el7sat.noarch
tfm-rubygem-foreman_virt_who_configure-0.1.5-1.fm1_15.el7sat.noarch

* created a new virt-who config in the WebUI
* hammer virt-who-config deploy --id 1
* head -n2 /etc/virt-who.d/virt-who-config-1.conf 
### This configuration file is managed via the virt-who configure plugin
### manual edits will be deleted.
* # head -n2 /etc/sysconfig/virt-who 
### This configuration file is managed via the virt-who configure plugin
### manual edits will be deleted.

Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA.
> 
> For information on the advisory, and where to find the updated files, follow the link below.
> 
> If the solution does not work for you, open a new bug report.
> 
> https://access.redhat.com/errata/RHSA-2018:0336