Skip to content

Commit

Permalink
Revert "Dev: ui_configure: Deprecate configure erase sub-command" (bs…
Browse files Browse the repository at this point in the history
…c#1228713) (#1501)

Revert #1107, since `configure erase` is still used by the user, it
shouldn't be deprecated
  • Loading branch information
liangxin1300 authored Aug 22, 2024
2 parents 49aeacf + 40d8b2b commit 9ee6ca4
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 9 deletions.
3 changes: 0 additions & 3 deletions crmsh/ui_configure.py
Original file line number Diff line number Diff line change
Expand Up @@ -885,9 +885,6 @@ def do_rename(self, context, old_id, new_id):
@command.completers(compl.choice(['nodes']))
def do_erase(self, context, nodes=None):
"usage: erase [nodes]"
if not options.regression_tests:
logger.warning("`crm configure erase` is deprecated. The replacement could be `crm cluster remove [node]`")
return True
cib_factory.ensure_cib_updated()
if nodes is None:
return cib_factory.erase()
Expand Down
6 changes: 0 additions & 6 deletions doc/crm.8.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -2875,12 +2875,6 @@ renamed. Please use the `rename` command instead.
[[cmdhelp_configure_erase,erase the CIB]]
==== `erase`

.Deprecation note
****************************
`crm configure erase` is deprecated.
The replacement could be `crm cluster remove [node]`
****************************

The `erase` clears all configuration. Apart from nodes. To remove
nodes, you have to specify an additional keyword `nodes`.

Expand Down

0 comments on commit 9ee6ca4

Please sign in to comment.