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

Update CLI reference to latest #1446

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update CLI reference to latest #1446

wants to merge 1 commit into from

Conversation

lavianalon
Copy link
Contributor

Update CLI v2 reference to latest including inference commands

Update CLI v2 reference to latest including inference commands
Copy link
Contributor

Preview environment URL: https://d161wck8lc3ih2.cloudfront.net/PR-1446/

# Describe a inference workload with a default project
runai inference describe <inference-name>

# Describe a inference workload in a specific project
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everywhere that says a inference workload should be changed to an inference workload

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@danbar2 @NaelAsbi123 can you please help me fix in all inference docs?

### SEE ALSO

* [runai](runai.md) - Run:ai Command-line Interface
* [runai inference delete](runai_inference_delete.md) - delete inference workload
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't it be inference workload in all description like delete?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's the same as the other workloads.
We can fix it, but that means fixing it for all types.
image

@@ -0,0 +1,56 @@
## runai inference list

list inference
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

List inference workloads - should add workloads to all

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's the same as the other workloads.
We can fix it, but that means fixing it for all types.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@NaelAsbi123 you can ignore my comment then


```
--activation-replicas int32 The number of replicas to run when scaling-up from zero. Defaults to minReplicas, or to 1 if minReplicas is set to 0
--annotation stringArray Set of annotations to populate into the container running the workspace
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

workspace -> workload

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants