Skip to content

Commit

Permalink
refactor(#703): remove mtu from client add
Browse files Browse the repository at this point in the history
  • Loading branch information
liquidiert committed Dec 5, 2024
1 parent d73c7e2 commit 1567db7
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 5 deletions.
2 changes: 0 additions & 2 deletions client/gefyra/api/clients.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ def add_clients(
client_id: str,
quantity: int = 1,
registry: Optional[str] = None,
mtu: Optional[int] = None,
kubeconfig: Optional[Path] = None,
kubecontext: Optional[str] = None,
) -> Iterable[GefyraClient]:
Expand All @@ -32,7 +31,6 @@ def add_clients(
kube_context=kubecontext,
ignore_connection=True,
registry=registry,
wireguard_mtu=mtu,
)
if quantity > 1 and client_id:
raise RuntimeError("Cannot specify both quantity > 1 and client_id")
Expand Down
4 changes: 1 addition & 3 deletions client/gefyra/cli/clients.py
Original file line number Diff line number Diff line change
Expand Up @@ -27,17 +27,15 @@ def clients(ctx):
default=1,
)
@click.option("--registry", help="The registry URL for the images", type=str)
@click.option("--mtu", help="The MTU for the Wireguard interface", type=int)
@click.pass_context
@standard_error_handler
def create_clients(ctx, client_id, quantity, registry, mtu):
def create_clients(ctx, client_id, quantity, registry):
from gefyra import api

api.add_clients(
client_id,
quantity,
registry=registry,
mtu=mtu,
kubeconfig=ctx.obj["kubeconfig"],
kubecontext=ctx.obj["context"],
)
Expand Down

0 comments on commit 1567db7

Please sign in to comment.