Skip to content

Add support for driver_set_once_metadata config - #360

Open
kpawar-sap wants to merge 1 commit into
stable/2025.1-m3from
nfs_full_permission
Open

kpawar-sap wants to merge 1 commit into
stable/2025.1-m3from
nfs_full_permission

Conversation

@kpawar-sap

Copy link
Copy Markdown

Introduce a new config option driver_set_once_metadata (list, default empty). Metadata keys listed here may be written to a share exactly once over its lifetime. On first set the value is persisted in the database and, forwarded to the share driver via the existing update_share_from_metadata RPC. Any subsequent attempt to update or overwrite a key already present returns HTTP 400.

Change-Id: Iacb5f38db50273f8bab8fdc6721a53b9665d1014

@kpawar-sap
kpawar-sap force-pushed the nfs_full_permission branch 2 times, most recently from f165b51 to ecb6b92 Compare September 4, 2026 12:42
@kpawar-sap kpawar-sap changed the title [WIP] Add support for driver_set_once_metadata config Add support for driver_set_once_metadata config Sep 4, 2026
@kpawar-sap

kpawar-sap commented Sep 4, 2026

Copy link
Copy Markdown
Author

tested

`                                            UNIX Permissions: ---rwxrwxrwx
`
$ openstack share set  TEST_NFS_FULL --property nfs_full_permission=false
Failed to set share properties '{'nfs_full_permission': 'false'}': Metadata key 'nfs_full_permission' can only be set once and already has value 'true'. Updating set-once metadata is not allowed. (HTTP 400) (Request-ID: req-dd4a705c-71c1-89c8-ccc9-e0aa16447cd4)
One or more of the set operations failed

@kpawar-sap

Copy link
Copy Markdown
Author

Introduce a new config option driver_set_once_metadata (list, default
empty). Metadata keys listed here may be written to a share exactly once
over its lifetime. On first set the value is persisted in the database
and, forwarded to the share driver via the existing
update_share_from_metadata RPC. Any subsequent attempt to update or
overwrite a key already present returns HTTP 400.

Change-Id: Iacb5f38db50273f8bab8fdc6721a53b9665d1014
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.

1 participant