Skip to content

Commit

Permalink
Fix limit option (#2292)
Browse files Browse the repository at this point in the history
flyteorg/flyte#5086

1. removed `--limit` option in `RemoteEntityGroup`
2. set `--limit` in `RunLevelParams` to show up instead of hidden. 
Signed-off-by: novahow <[email protected]>
  • Loading branch information
novahow authored Mar 29, 2024
1 parent acf6bab commit 60c6234
Showing 1 changed file with 0 additions and 9 deletions.
9 changes: 0 additions & 9 deletions flytekit/clis/sdk_in_container/run.py
Original file line number Diff line number Diff line change
Expand Up @@ -242,7 +242,6 @@ class RunLevelParams(PyFlyteParams):
required=False,
type=int,
default=50,
hidden=True,
show_default=True,
help="Use this to limit number of entities to fetch",
)
Expand Down Expand Up @@ -661,14 +660,6 @@ def __init__(self, command_name: str):
super().__init__(
name=command_name,
help=f"Retrieve {command_name} from a remote flyte instance and execute them.",
params=[
click.Option(
["--limit", "limit"],
help=f"Limit the number of {command_name}'s to retrieve.",
default=50,
show_default=True,
)
],
)
self._command_name = command_name
self._entities = []
Expand Down

0 comments on commit 60c6234

Please sign in to comment.