Skip to content

Commit

Permalink
feat: deprecate attach/auto-attach modules
Browse files Browse the repository at this point in the history
This patch adds a deprecation notice
to the attach/auto-attach command descriptions.
This patch also moves the attach command to the
'Other Modules' section in the sub-man help text
from the 'Primary Modules' section.

The attach command already displays an error
message when attempting to attach subscriptions
with an SCA registered system, so no changes
are required to implement this.

Signed-off-by: Jason Jerome <[email protected]>
  • Loading branch information
DuckBoss committed Jun 7, 2024
1 parent 2a097ba commit e7b1dcd
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 6 deletions.
7 changes: 2 additions & 5 deletions src/subscription_manager/cli_command/attach.py
Original file line number Diff line number Diff line change
Expand Up @@ -94,16 +94,13 @@ def _read_pool_ids(self, f):
self.options.pool.append(pool)

def _short_description(self):
return _(
"Attach a specified subscription to the registered system, when system does not use "
"Simple Content Access mode"
)
return _("Deprecated, this command will be removed from the future major releases")

def _command_name(self):
return "attach"

def _primary(self):
return True
return False

def _validate_options(self):
if self.options.pool or self.options.file:
Expand Down
2 changes: 1 addition & 1 deletion src/subscription_manager/cli_command/autoheal.py
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ class AutohealCommand(CliCommand):
def __init__(self):
self.uuid = inj.require(inj.IDENTITY).uuid

shortdesc = _("Set if subscriptions are attached on a schedule (default of daily)")
shortdesc = _("Deprecated, this option will be removed from the future major releases")
self._org_help_text = _("specify whether to enable or disable auto-attaching of subscriptions")
super(AutohealCommand, self).__init__("auto-attach", shortdesc, False)

Expand Down

0 comments on commit e7b1dcd

Please sign in to comment.