Skip to content

Commit

Permalink
removing unnecessary cli args.
Browse files Browse the repository at this point in the history
Signed-off-by: lrangine <[email protected]>
  • Loading branch information
lokeshrangineni committed Dec 17, 2024
1 parent 1d64ebb commit 9540e7e
Showing 1 changed file with 1 addition and 11 deletions.
12 changes: 1 addition & 11 deletions sdk/python/feast/cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -1155,23 +1155,13 @@ def serve_registry_command(
show_default=False,
help="path to TLS certificate public key. You need to pass --key as well to start server in TLS mode",
)
@click.option(
"--verify_client",
"-v",
"tls_verify_client",
type=click.BOOL,
default="True",
show_default=True,
help="Verify the client or not for the TLS client certificate.",
)
@click.pass_context
def serve_offline_command(
ctx: click.Context,
host: str,
port: int,
tls_key_path: str,
tls_cert_path: str,
tls_verify_client: bool,
):
"""Start a remote server locally on a given host, port."""
if (tls_key_path and not tls_cert_path) or (not tls_key_path and tls_cert_path):
Expand All @@ -1180,7 +1170,7 @@ def serve_offline_command(
)
store = create_feature_store(ctx)

store.serve_offline(host, port, tls_key_path, tls_cert_path, tls_verify_client)
store.serve_offline(host, port, tls_key_path, tls_cert_path)


@cli.command("validate")
Expand Down

0 comments on commit 9540e7e

Please sign in to comment.