puppet master should have a --test option
|Assignee:||J.D. Welch||% Done:|
|Affected Puppet version:||
puppet agent --test
puppet agent --onetime --verbose ---ignorecache \
--no-daemonize no-usecacheonfailure \
--detailed-exit-codes no-splay --show_diff
puppet master --test
puppet master --verbose --no-daemonize
Users have often found the syntax —test confusing and many users have expected in the past that it would have the functionality of —noop.
Hearing the complaint of ‘I ran puppet in —test why are things changing’ is relatively common.
We’ve also raised this point for debate in the community several times, and no-one has come up with a more appropriate name.
I’m not fussed about the name, I would just like consistent functionality across master and agent.
- Status changed from Unreviewed to Investigating
“puppet master —foreground” which is similar to other projects?
Ah you want the naming to be the same for both agent and master.
Arguably test would imply noop, but ours doesn’t.
I propose that we add
--foreground to both and deprecate
The problem with “—foreground” is that it feels like a synonym for “—no-daemonize”, and we’re trying to suggest more than that.
This means it makes sense for the master “—test” mode, but not the agent “—test” mode, which not only runs in the foreground, but only runs once, is verbose, and ignores all caches.
Well, the point of these options is so that a human can watch what’s going on, right? Let’s then name the option for the intent rather than the effect:
- Keywords set to usability
- Assignee changed from Randall Hansen to J.D. Welch
- Keywords changed from usability to backlog
- Keywords changed from backlog to ux
Also available in: Atom