Skip to content

Commit

Permalink
refactor(cli): Improve k8s pre-processor cli args help text
Browse files Browse the repository at this point in the history
  • Loading branch information
gfieni committed Mar 27, 2024
1 parent 2d69a35 commit 581e005
Showing 1 changed file with 13 additions and 5 deletions.
18 changes: 13 additions & 5 deletions src/powerapi/cli/common_cli_parsing_manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -406,28 +406,36 @@ def __init__(self):

subparser_k8s_pre_processor = SubgroupConfigParsingManager("k8s")
subparser_k8s_pre_processor.add_argument(
"a", "api-mode", help_text="k8s api mode (local, manual or cluster)"
"a",
"api-mode",
help_text="Kubernetes API mode (local, manual or cluster)",
default_value='cluster'
)

subparser_k8s_pre_processor.add_argument(
"k",
"api-key",
help_text="API key authorization required for k8s manual configuration",
help_text="Kubernetes Bearer Token (only for manual API mode)",
)

subparser_k8s_pre_processor.add_argument(
"h",
"api-host",
help_text="host required for k8s manual configuration",
help_text="Kubernetes API host (only for manual API mode)",
)

subparser_k8s_pre_processor.add_argument(
"p",
"puller",
help_text="target puller for the pre-processor",
help_text="Name of the puller to attach the pre-processor to",
)

subparser_k8s_pre_processor.add_argument(

Check warning on line 433 in src/powerapi/cli/common_cli_parsing_manager.py

View check run for this annotation

Codecov / codecov/patch

src/powerapi/cli/common_cli_parsing_manager.py#L433

Added line #L433 was not covered by tests
"n",
"name",
help_text="Name of the pre-processor"
)

subparser_k8s_pre_processor.add_argument("n", "name", help_text="")
self.add_subgroup_parser(
subgroup_name="pre-processor",
subgroup_parser=subparser_k8s_pre_processor
Expand Down

0 comments on commit 581e005

Please sign in to comment.