Skip to content

netapp: fix replica create failure for multi-protocol shares - #368

Merged
Carthaca merged 1 commit into
stable/2025.1-m3from
maurice-escher-sap-fix-netapp-replica-unix-permissions
Sep 16, 2026
Merged

Carthaca merged 1 commit into
stable/2025.1-m3from
maurice-escher-sap-fix-netapp-replica-unix-permissions

Conversation

@Carthaca

Copy link
Copy Markdown
Collaborator

Multi-protocol shares get unix_permissions='0777' forced in
_get_provisioning_options_for_share. When creating a replica
(_allocate_container with replica=True), this option is combined
with volume_type='dp' for the destination volume, causing ONTAP
to reject volume-create/volume-create-async with error 13001:
'Only volumes of type "RW" can be assigned UNIX permissions.'

Change-Id: Ieb54a9e852778e27a8e307822d8768885db9f947
Signed-off-by: Maurice Escher maurice.escher@sap.com
Co-authored-by: Copilot App 223556219+Copilot@users.noreply.github.com

Multi-protocol shares get unix_permissions='0777' forced in
_get_provisioning_options_for_share. When creating a replica
(_allocate_container with replica=True), this option is combined
with volume_type='dp' for the destination volume, causing ONTAP
to reject volume-create/volume-create-async with error 13001:
'Only volumes of type "RW" can be assigned UNIX permissions.'

Change-Id: Ieb54a9e852778e27a8e307822d8768885db9f947
Signed-off-by: Maurice Escher <maurice.escher@sap.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@kpawar-sap

Copy link
Copy Markdown

so #360 also need to consider this check ?

@Carthaca

Copy link
Copy Markdown
Collaborator Author

so #360 also need to consider this check ?

yes, if this applies to replicas

@Carthaca
Carthaca enabled auto-merge (rebase) September 16, 2026 11:09
@Carthaca
Carthaca merged commit b65e3ea into stable/2025.1-m3 Sep 16, 2026
2 checks passed
@Carthaca
Carthaca deleted the maurice-escher-sap-fix-netapp-replica-unix-permissions branch September 16, 2026 11:37
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