Issue #16189 has been updated by Henrik Lindberg.

**cigar !**
So, there is already a global parameter defined with parsing and everything for 
the option `run_mode`. It is however made readonly.

By simply making it not read only, and translating the string to a symbol, the 
run_mode gets set as a default. The default default is still `:user` (defined 
in `Settings`). In addition to this, the only thing required is for the 
application is to use this default from `Settings` instead of a static `:user`. 

This means that applications (such as `Application::Master`) which sets the 
run_mode explicitly works as before. (As will it work for any other application 
that did the same). All others, which relied on the default `:user` will now 
either get it from the command line option, or the `:user` default in 
`Settings`.

**Problem solved**

The option `--mode` can be removed from `FaceBase`, as can the attempt in 
`FaceBase` to set the run mode to `:agent` (which does not work at all anyway).

>From a user perspective, what changed was the `--mode` changed to 
>`--run_mode`. (The `doc` applications use of `--mode` is still bad, but that 
>is a separate issue).

----------------------------------------
Bug #16189: puppet config print and --configprint disagree
https://projects.puppetlabs.com/issues/16189#change-71028

Author: Patrick Carlisle
Status: Accepted
Priority: Normal
Assignee: 
Category: 
Target version: 3.0.0
Affected Puppet version: 3.0.0-rc5
Keywords: 
Branch: 


    thinky:puppet:% puppet master --configprint ssldir                          
                                                                                
                            <3.x>
    /etc/puppet/ssl
    thinky:puppet:% puppet config print ssldir --mode master                    
                                                                                
                            <3.x>
    /home/patrick/.puppet/ssl


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-bugs?hl=en.

Reply via email to