Feat/CCT-467: Add explanatory info about srcs of overrides in repo-override cmd #3474
Conversation
man/subscription-manager.8
Outdated
| @@ -604,7 +604,8 @@ option | |||
| .B --remove=NAME | |||
| Removes a named override from the repositories specified with the | |||
| .B --repo | |||
| option | |||
| option. | |||
| Source of the override that is attempted to be removed may be one of the environment(s) the system belongs in. It can only be removed by either: removing the override from the environment it originates from or removing the system from the environment the override originates from. | |||
There was a problem hiding this comment.
The first sentence could be a bit more straightforward, like: The override will not be removed if its source is one of the environments the system belongs in. It can only...
| @@ -604,7 +604,8 @@ option | |||
| .B --remove=NAME | |||
| Removes a named override from the repositories specified with the | |||
| .B --repo | |||
| option | |||
| option. | |||
| Source of the override that is attempted to be removed may be one of the environment(s) the system belongs in. It can only be removed by either: removing the override from the environment it originates from or removing the system from the environment the override originates from. | |||
|
|
|||
| .TP | |||
| .B --remove-all | |||
There was a problem hiding this comment.
Let's also extend the info on this command, by adding: ... --repo option. Overrides inherited from environments the system belongs in will not be removed. They can only be removed by either: removing them from the environment they originate from or removing the system from those environments.
man/subscription-manager.8
Outdated
| @@ -616,7 +617,8 @@ option | |||
| .B --list | |||
| Lists all overrides from repositories specified with the | |||
| .B --repo | |||
| option | |||
| option. Source of the listed overrides is a union of the override specified at the system itself and all the overrides set in the environment(s) system belongs in. | |||
There was a problem hiding this comment.
minor correction: there is a missing the between the words environment(s) and system
…epo-overrides command The requrements are to put statements containing additional info about `subscription-manager repo-override --list` and about `subscription-manager repo-override --remove` to respective parts of man pages. CARD CCT-467
grunwmar
force-pushed
the
feat/cct-467
branch
from
ec1b520 to
631e9b1
Compare
| option | ||
|
|
||
| option. Overrides inherited from environments the system belongs in will not be removed. They can only be removed by either: removing them from the environment they originate from or removing the system from those environments. | ||
| The source will not be removed if its source is one of the environment(s) the system belongs in. It can only be removed by either: removing the override from the environment it originates from or removing the system from the environment the override originates from. |
There was a problem hiding this comment.
I think you added the text I proposed for the --remove-all command into the --remove command. If you check my other comment, it was done against line 610 (where the --remove-all command description is)
There was a problem hiding this comment.
Ahh yes, Niko is right, I didn't notice as well before, the text is correct but should be moved to --remove-all. Thanks for pointing that out @nikosmoum :)
The requrements are to put statements containing additional info about
subscription-manager repo-override --listand aboutsubscription-manager repo-override --removeto respective parts of man pages.Additional info for --list:
Additional info for --remove:
CARD CCT-467