This was discussed a while ago and it's a bit complex. The suggested syntax is Puppet's but on the command line that doesn't work.

https://github.com/theforeman/kafo#hash-arguments has an example:

> bin/foreman-installer --puppet-server-git-branch-map=master:some --puppet-server-git-branch-map=development:another

When we tested this, it turns out it doesn't actually work well and the = needs to be dropped. That means the correct way is:

  satellite-installer --foreman-loggers sql:true

Thanks, I wonder how you write multiple entries. Is that:

satellite-installer --foreman-loggers sql:true,audit:true

or

satellite-installer --foreman-loggers sql:true --foreman-loggers audit:true

?

From the kafo example I suggests the latter. This is also how we treat arrays: you specify it multiple times. Haven't tried it myself.

Moving forward I think we should remove the example since it doesn't really add a lot. This is also considered an advanced parameter so I'm wondering how much more we need to invest into inline documentation. What would be considered sufficient to close this bug?

Hi Ewoud,

In BZ#1738850, I have added the command to enable a logger you shared to the 12.2. Enabling Individual Loggers [1] section. Still, it will be great if users can find the same information directly from the out[ut of `hammer --full-help`.
I suggest changing the current output, which is the following:
>    --foreman-loggers             Enable or disable specific loggers, e.g. {"sql" => true} (current: {})

To something not misleading, for example:
>    --foreman-loggers             Enable or disable specific loggers, e.g. {sql:true} (current: {})

Also, I like Lukas's suggestion to add a possibility to specify multiple entries with this command.

Thank you

[1] https://access.redhat.com/documentation/en-us/red_hat_satellite/6.6-beta/html/administering_red_hat_satellite/chap-red_hat_satellite-administering_red_hat_satellite-logging_and_reporting_problems#enabling_individual_loggers

(In reply to Sergei Petrosian from comment #9)
> In BZ#1738850, I have added the command to enable a logger you shared to the
> 12.2. Enabling Individual Loggers [1] section. Still, it will be great if
> users can find the same information directly from the out[ut of `hammer
> --full-help`.
> I suggest changing the current output, which is the following:
> >    --foreman-loggers             Enable or disable specific loggers, e.g. {"sql" => true} (current: {})
> 
> To something not misleading, for example:
> >    --foreman-loggers             Enable or disable specific loggers, e.g. {sql:true} (current: {})

I prefer a more specific logging document (as you linked), but I'll see what I can do to remove the confusing instructions from the help text.

> Also, I like Lukas's suggestion to add a possibility to specify multiple
> entries with this command.

This should already be possible. If you encounter issues with it, it's a bug we need to resolve.

> [1]
> https://access.redhat.com/documentation/en-us/red_hat_satellite/6.6-beta/
> html/administering_red_hat_satellite/chap-red_hat_satellite-
> administering_red_hat_satellite-
> logging_and_reporting_problems#enabling_individual_loggers

Thank you, Ewoud.

(In reply to Ewoud Kohl van Wijngaarden from comment #10)
> This should already be possible. If you encounter issues with it, it's a bug
> we need to resolve.
Sorry for the confusionm, I understood comment#6 incorrectly. The following command works fine: 
[root@sat65 ~]# satellite-installer --foreman-loggers sql:true --foreman-loggers ldap:true

Just for the record, others do not. This is not a bug definitely.
````
[root@sat65 ~]# satellite-installer --foreman-loggers sql:true ldap:true
ERROR: too many arguments

[root@sat65 ~]# satellite-installer --foreman-loggers sql:true,ldap:true
Resetting puppet server version param...
Parameter foreman-loggers invalid: Hash value elements are invalid: "true,ldap:true" is not a valid boolean
Error during configuration, exiting
````

Upon review of our valid but aging backlog the Satellite Team has concluded that this Bugzilla does not meet the criteria for a resolution in the near term, and are planning to close in approximately a month. If you have any concerns about this, please contact your Red Hat Account team.  Thank you.

Thank you for your interest in Satellite 6. We have evaluated this request, and while we recognize that it is a valid request, we do not expect this to be implemented in the product in the foreseeable future. This is due to other priorities for the product, and not a reflection on the request itself. We are therefore closing this out as WONTFIX. If you have any concerns about this, please do not reopen. Instead, feel free to contact your Red Hat Account Team. Thank you.