Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(llm_inference): change more cli fields to positional #3732

Merged
merged 3 commits into from
Apr 2, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
Add new ACL rules for a specific deployment.

USAGE:
scw llm-inference acl add [arg=value ...]
scw llm-inference acl add <deployment-id ...> [arg=value ...]

ARGS:
deployment-id ID of the deployment to add ACL rules to
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
Delete an exising ACL.

USAGE:
scw llm-inference acl delete [arg=value ...]
scw llm-inference acl delete <acl-id ...> [arg=value ...]

ARGS:
acl-id ID of the ACL rule to delete
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
List ACLs for a specific deployment.

USAGE:
scw llm-inference acl list [arg=value ...]
scw llm-inference acl list <deployment-id ...> [arg=value ...]

ARGS:
deployment-id ID of the deployment to list ACL rules for
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
Set new ACL rules for a specific deployment.

USAGE:
scw llm-inference acl set [arg=value ...]
scw llm-inference acl set <deployment-id ...> [arg=value ...]

ARGS:
deployment-id ID of the deployment to set ACL rules for
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
Create a new Endpoint related to a specific deployment.

USAGE:
scw llm-inference endpoint create [arg=value ...]
scw llm-inference endpoint create <deployment-id ...> [arg=value ...]

ARGS:
deployment-id ID of the deployment to create the endpoint for
Expand Down
10 changes: 5 additions & 5 deletions docs/commands/llm-inference.md
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ Add new ACL rules for a specific deployment.
**Usage:**

```
scw llm-inference acl add [arg=value ...]
scw llm-inference acl add <deployment-id ...> [arg=value ...]
```


Expand All @@ -59,7 +59,7 @@ Delete an exising ACL.
**Usage:**

```
scw llm-inference acl delete [arg=value ...]
scw llm-inference acl delete <acl-id ...> [arg=value ...]
```


Expand All @@ -79,7 +79,7 @@ List ACLs for a specific deployment.
**Usage:**

```
scw llm-inference acl list [arg=value ...]
scw llm-inference acl list <deployment-id ...> [arg=value ...]
```


Expand All @@ -99,7 +99,7 @@ Set new ACL rules for a specific deployment.
**Usage:**

```
scw llm-inference acl set [arg=value ...]
scw llm-inference acl set <deployment-id ...> [arg=value ...]
```


Expand Down Expand Up @@ -270,7 +270,7 @@ Create a new Endpoint related to a specific deployment.
**Usage:**

```
scw llm-inference endpoint create [arg=value ...]
scw llm-inference endpoint create <deployment-id ...> [arg=value ...]
```


Expand Down
2 changes: 1 addition & 1 deletion go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ require (
github.com/moby/buildkit v0.11.6
github.com/opencontainers/go-digest v1.0.0
github.com/pkg/errors v0.9.1
github.com/scaleway/scaleway-sdk-go v1.0.0-beta.25.0.20240329105646-c0ed2d3750f9
github.com/scaleway/scaleway-sdk-go v1.0.0-beta.25.0.20240329152759-b47741a7bf2a
github.com/skratchdot/open-golang v0.0.0-20200116055534-eef842397966
github.com/spf13/cobra v1.8.0
github.com/spf13/pflag v1.0.5
Expand Down
4 changes: 2 additions & 2 deletions go.sum
Original file line number Diff line number Diff line change
Expand Up @@ -425,8 +425,8 @@ github.com/rogpeppe/go-internal v1.11.0/go.mod h1:ddIwULY96R17DhadqLgMfk9H9tvdUz
github.com/russross/blackfriday/v2 v2.1.0/go.mod h1:+Rmxgy9KzJVeS9/2gXHxylqXiyQDYRxCVz55jmeOWTM=
github.com/sabhiram/go-gitignore v0.0.0-20210923224102-525f6e181f06 h1:OkMGxebDjyw0ULyrTYWeN0UNCCkmCWfjPnIA2W6oviI=
github.com/sabhiram/go-gitignore v0.0.0-20210923224102-525f6e181f06/go.mod h1:+ePHsJ1keEjQtpvf9HHw0f4ZeJ0TLRsxhunSI2hYJSs=
github.com/scaleway/scaleway-sdk-go v1.0.0-beta.25.0.20240329105646-c0ed2d3750f9 h1:ytq3dM17GsFlK8REuRMMr0+pwqj84824pXLw8m1ruW8=
github.com/scaleway/scaleway-sdk-go v1.0.0-beta.25.0.20240329105646-c0ed2d3750f9/go.mod h1:fCa7OJZ/9DRTnOKmxvT6pn+LPWUptQAmHF/SBJUGEcg=
github.com/scaleway/scaleway-sdk-go v1.0.0-beta.25.0.20240329152759-b47741a7bf2a h1:C1cHJWg9ko/9GwhCMD6mY8x0/R4bJd2TA5K0iMaM6eA=
github.com/scaleway/scaleway-sdk-go v1.0.0-beta.25.0.20240329152759-b47741a7bf2a/go.mod h1:fCa7OJZ/9DRTnOKmxvT6pn+LPWUptQAmHF/SBJUGEcg=
github.com/sclevine/spec v1.4.0 h1:z/Q9idDcay5m5irkZ28M7PtQM4aOISzOpj4bUPkDee8=
github.com/sclevine/spec v1.4.0/go.mod h1:LvpgJaFyvQzRvc1kaDs0bulYwzC70PbiYjC4QnFHkOM=
github.com/secure-systems-lab/go-securesystemslib v0.4.0 h1:b23VGrQhTA8cN2CbBw7/FulN9fTtqYUdS5+Oxzt+DUE=
Expand Down
10 changes: 5 additions & 5 deletions internal/namespaces/llm_inference/v1beta1/llm_inference_cli.go
Original file line number Diff line number Diff line change
Expand Up @@ -416,7 +416,7 @@ func llmInferenceEndpointCreate() *core.Command {
Short: `ID of the deployment to create the endpoint for`,
Required: true,
Deprecated: false,
Positional: false,
Positional: true,
},
{
Name: "endpoint.private-network.private-network-id",
Expand Down Expand Up @@ -534,7 +534,7 @@ func llmInferenceACLList() *core.Command {
Short: `ID of the deployment to list ACL rules for`,
Required: true,
Deprecated: false,
Positional: false,
Positional: true,
},
core.RegionArgSpec(scw.RegionFrPar, scw.Region(core.AllLocalities)),
},
Expand Down Expand Up @@ -573,7 +573,7 @@ func llmInferenceACLAdd() *core.Command {
Short: `ID of the deployment to add ACL rules to`,
Required: true,
Deprecated: false,
Positional: false,
Positional: true,
},
{
Name: "acls.{index}.ip",
Expand Down Expand Up @@ -617,7 +617,7 @@ func llmInferenceACLSet() *core.Command {
Short: `ID of the deployment to set ACL rules for`,
Required: true,
Deprecated: false,
Positional: false,
Positional: true,
},
{
Name: "acls.{index}.ip",
Expand Down Expand Up @@ -661,7 +661,7 @@ func llmInferenceACLDelete() *core.Command {
Short: `ID of the ACL rule to delete`,
Required: true,
Deprecated: false,
Positional: false,
Positional: true,
},
core.RegionArgSpec(scw.RegionFrPar),
},
Expand Down
Loading