Skip to content

[DEVX-727]: CLI added for CO Resources#436

Merged
srikanthbachala20 merged 19 commits intomasterfrom
DEVX-727-CLI-Support-for-CO-Resources
Nov 5, 2024
Merged

[DEVX-727]: CLI added for CO Resources#436
srikanthbachala20 merged 19 commits intomasterfrom
DEVX-727-CLI-Support-for-CO-Resources

Conversation

@srikanthbachala20
Copy link
Copy Markdown
Contributor

Why

  • To support CRUD operations on CO Resources - Compute Cluster, Nodepool, Deployment using CLI

How

  • Added CLI support for CRUD Operations of CO Resources

Tests

  • Added in tests/cli

Comment thread clarifai/cli/compute_cluster.py Outdated
Comment thread clarifai/cli/compute_cluster.py Outdated
Comment thread clarifai/cli/compute_cluster.py Outdated
Comment thread clarifai/cli/compute_cluster.py Outdated
Comment thread clarifai/cli/deployment.py Outdated
Comment thread clarifai/cli/deployment.py Outdated
Comment thread clarifai/cli/deployment.py Outdated
Comment thread clarifai/cli/compute_cluster.py Outdated
Comment thread clarifai/cli/nodepool.py Outdated
Comment thread tests/cli/test_compute_orchestration.py
Comment thread tests/cli/test_compute_orchestration.py
Comment thread tests/cli/test_compute_orchestration.py
Comment thread tests/cli/test_compute_orchestration.py
Comment thread tests/cli/test_compute_orchestration.py Outdated
Comment thread tests/test_app.py Outdated
Comment thread clarifai/cli/nodepool.py Outdated
Comment thread clarifai/cli/nodepool.py Outdated
Comment thread clarifai/cli/deployment.py Outdated
Comment thread clarifai/cli/deployment.py Outdated
Comment thread clarifai/cli/deployment.py Outdated
Comment thread clarifai/cli/deployment.py Outdated
Comment thread clarifai/cli/compute_cluster.py Outdated
Comment thread clarifai/cli/deployment.py
Comment thread clarifai/cli/nodepool.py
Comment thread clarifai/cli/base.py
@luv-bansal
Copy link
Copy Markdown
Contributor

Added comments regarding required nodepool_id and compute_cluster_id flag in create operation of deployment and nodepool respectively. Else everything seems good

Comment thread clarifai/cli/deployment.py Outdated
Comment thread clarifai/cli/compute_cluster.py
@luv-bansal
Copy link
Copy Markdown
Contributor

Push some minor changes, Now good from my side

@srikanthbachala20 srikanthbachala20 merged commit 625cf49 into master Nov 5, 2024
@srikanthbachala20 srikanthbachala20 deleted the DEVX-727-CLI-Support-for-CO-Resources branch November 5, 2024 12:49
@srikanthbachala20 srikanthbachala20 mentioned this pull request Nov 6, 2024
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