Skip to content

Add ontapSource field to Netapp Backup #17555

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.

Sign up for GitHub

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

Merged
shuyama1 merged 1 commit into from
May 14, 2026
+26 −0
Merged

Add ontapSource field to Netapp Backup #17555

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
26 changes: 26 additions & 0 deletions mmv1/products/netapp/Backup.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -146,6 +146,32 @@ properties:
Format: `projects/{{projectId}}/locations/{{location}}/volumes/{{volumename}}/snapshots/{{snapshotname}}``
required: false
diff_suppress_func: 'tpgresource.ProjectNumberDiffSuppress'
- name: 'ontapSource'
type: NestedObject
description: |
Details of the ONTAP source volume and snapshot.
required: false
immutable: true
min_version: beta
properties:
- name: 'storagePool'
type: String
description: |
Name of the storage pool. This must be specified for creating backups for ONTAP mode volumes.
Format: `projects/{{project}}/locations/{{location}}/storagePools/{{storage_pool_id}}`
required: true
immutable: true
- name: 'volumeUuid'
type: String
description: |
The UUID of the ONTAP source volume.
required: true
immutable: true
- name: 'snapshotUuid'
type: String
description: |
The UUID of the ONTAP source snapshot.
Comment on lines +157 to +173
Copy link
Copy Markdown
Member

@shuyama1 shuyama1 May 14, 2026

Choose a reason for hiding this comment

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

Are these subfields also immutable? If so, they'll need immutable: true explicitly

Copy link
Copy Markdown
Contributor Author

@ravisankardanda99 ravisankardanda99 May 14, 2026

Choose a reason for hiding this comment

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

Added immutable: true to subfields as well.

As mentioned in #17555 (comment), it is not possible to add an acceptance test for this change.

immutable: true
- name: 'volumeRegion'
type: String
description: |
Expand Down
Loading