Bug #6917

puppet master should have a --test option

Added by Nigel Kersten about 2 years ago. Updated 8 months ago.

Status:InvestigatingStart date:03/31/2011
Priority:NormalDue date:
Assignee:J.D. Welch% Done:

0%

Category:-
Target version:-
Affected Puppet version: Branch:
Keywords:ux

Description

just like:

puppet agent --test

means

puppet agent --onetime --verbose ---ignorecache \
--no-daemonize no-usecacheonfailure \
--detailed-exit-codes no-splay --show_diff

puppet master --test

should mean

puppet master --verbose --no-daemonize

Related issues

Related to Puppet - Feature #17095: Improve puppet's command line interaction Investigating

History

#1 Updated by Dan Bode about 2 years ago

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.

#2 Updated by Nigel Kersten about 2 years ago

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.

#3 Updated by Ben Hughes about 2 years ago

  • 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.

#4 Updated by Randall Hansen about 2 years ago

I propose that we add --foreground to both and deprecate --test.

#5 Updated by Nigel Kersten about 2 years ago

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.

#6 Updated by Randall Hansen about 2 years ago

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: --monitor.

#7 Updated by Randall Hansen about 2 years ago

  • Keywords set to usability

#8 Updated by Randall Hansen 9 months ago

  • Assignee changed from Randall Hansen to J.D. Welch

#9 Updated by J.D. Welch 8 months ago

  • Keywords changed from usability to backlog

#10 Updated by eric sorenson 8 months ago

  • Keywords changed from backlog to ux

Also available in: Atom PDF