Skip to content

feat: Add idempotent network volume deployment#79

Merged
deanq merged 1 commit intomainfrom
deanq/idempotent-volume-reference
Aug 7, 2025
Merged

feat: Add idempotent network volume deployment#79
deanq merged 1 commit intomainfrom
deanq/idempotent-volume-reference

Conversation

@deanq
Copy link
Copy Markdown
Member

@deanq deanq commented Aug 7, 2025

  • Add list_network_volumes method to RunpodRestClient
  • Implement idempotent volume creation that reuses existing volumes with same name/datacenter
  • Add resource_id property for consistent volume identification
  • Increase default volume size from 10GB to 50GB
  • Add comprehensive unit tests for network volume functionality
  • Fix typo in NetworkVolume class docstring

Volumes with the same name and datacenter will now reuse existing volumes instead of creating duplicates, enabling truly idempotent deployments.

- Add list_network_volumes method to RunpodRestClient
- Implement idempotent volume creation that reuses existing volumes with same name/datacenter
- Add resource_id property for consistent volume identification
- Increase default volume size from 10GB to 50GB
- Add comprehensive unit tests for network volume functionality
- Fix typo in NetworkVolume class docstring

Volumes with the same name and datacenter will now reuse existing volumes
instead of creating duplicates, enabling truly idempotent deployments.
@deanq deanq requested a review from pandyamarut August 7, 2025 00:28
Copy link
Copy Markdown
Contributor

@pandyamarut pandyamarut left a comment

Choose a reason for hiding this comment

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

/LGTM

@deanq deanq merged commit 289d333 into main Aug 7, 2025
7 checks passed
@deanq deanq deleted the deanq/idempotent-volume-reference branch August 7, 2025 03:50
@github-actions github-actions Bot mentioned this pull request Aug 7, 2025
pandyamarut pushed a commit that referenced this pull request Sep 9, 2025
- Add list_network_volumes method to RunpodRestClient
- Implement idempotent volume creation that reuses existing volumes with same name/datacenter
- Add resource_id property for consistent volume identification
- Increase default volume size from 10GB to 50GB
- Add comprehensive unit tests for network volume functionality
- Fix typo in NetworkVolume class docstring

Volumes with the same name and datacenter will now reuse existing volumes
instead of creating duplicates, enabling truly idempotent deployments.
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.

2 participants