Ticket/2.7.x/10940 deprecate apply for catalog#231
Merged
ccaum merged 1 commit intopuppetlabs:2.7.xfrom Dec 8, 2011
kelseyhightower:ticket/2.7.x/10940_deprecate_apply_for_catalog
Merged
Ticket/2.7.x/10940 deprecate apply for catalog#231ccaum merged 1 commit intopuppetlabs:2.7.xfrom kelseyhightower:ticket/2.7.x/10940_deprecate_apply_for_catalog
ccaum merged 1 commit intopuppetlabs:2.7.xfrom
kelseyhightower:ticket/2.7.x/10940_deprecate_apply_for_catalog
Conversation
Without this patch, the `puppet apply` command uses the `--apply` option
to reference a Puppet catalog. This can cause a bit of confusion as both
the subcommand and option have the same name.
This patch improves the usability of the `puppet apply` command by
adding a new `--catalog` option. This patch also adds a deprecation
warning whenever the `--apply` option is used, which advises end-users
to use the `--catalog` option instead.
This patch updates the in-line help documentation by adding help output
for the new `--catalog` option, and adding a deprecation warning for the
`--apply` option.
The plan is to deprecate the `--apply` option in the next 2.7.x release,
and removal in the future. External documentation should be updated to
reflect the new preference towards using the `--catalog` option in-place
of `--apply`.
Preferred method of applying a Puppet catalog using `puppet apply`:
$ puppet apply --catalog <catalog>
This patch also includes the spec tests covering the changes in
behavior.
ccaum
added a commit
that referenced
this pull request
Dec 8, 2011
…cate_apply_for_catalog Ticket/2.7.x/10940 deprecate apply for catalog Reviewed by Carl Caum <carl@puppetlabs.com>
melissa
pushed a commit
to melissa/puppet
that referenced
this pull request
Mar 30, 2018
(maint) Improve output if SSL config test fails
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Without this patch, the
puppet applycommand uses the--applyoptionto reference a Puppet catalog. This can cause a bit of confusion as both
the subcommand and option have the same name.
This patch improves the usability of the
puppet applycommand byadding a new
--catalogoption. This patch also adds a deprecationwarning whenever the
--applyoption is used, which advises end-usersto use the
--catalogoption instead.This patch updates the in-line help documentation by adding help output
for the new
--catalogoption, and adding a deprecation warning for the--applyoption.The plan is to deprecate the
--applyoption in the next 2.7.x release,and removal in the future. External documentation should be updated to
reflect the new preference towards using the
--catalogoption in-placeof
--apply.Preferred method of applying a Puppet catalog using
puppet apply:This patch also includes the spec tests covering the changes in
behavior.