Skip to content

Add methods for Manila share type get and update - #3853

Merged
kayrus merged 10 commits into
gophercloud:mainfrom
stackhpc:manila-sharetype-getset
Jul 27, 2026
Merged

kayrus merged 10 commits into
gophercloud:mainfrom
stackhpc:manila-sharetype-getset

Conversation

@TBExtent

Copy link
Copy Markdown
Contributor

Fixes #3842

Implement Manila share type get and update API methods.

API reference for share types: https://docs.openstack.org/api-ref/shared-file-system/#share-types

Unit tests:
Adds unit tests for getting and updating sharetypes

Share types are a mechanism for supporting multi-tenant access to
a common file storage service.  This commit defines share type get
and update operations, enabling callers to control the management
of Manila share types.

Signed-off-by: Stig Telfer <[email protected]>
@github-actions github-actions Bot added the edit:sharedfilesystems This PR updates sharedfilesystems code label Jul 14, 2026
Comment thread openstack/sharedfilesystems/v2/sharetypes/results.go Outdated

@winiciusallan winiciusallan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hey @TBExtent, would you mind adding some acceptance tests?

@TBExtent

Copy link
Copy Markdown
Contributor Author

Working on some, having fun with microversions

@github-actions github-actions Bot added semver:minor Backwards-compatible change backport-v2 This PR will be backported to v2 labels Jul 16, 2026
@coveralls

coveralls commented Jul 16, 2026 •

Copy link
Copy Markdown

Coverage Status

coverage: 63.567% (+0.02%) from 63.545% — stackhpc:manila-sharetype-getset into gophercloud:main

@winiciusallan winiciusallan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for adding the tests, @TBExtent. Dropped a few comments inline.

Comment thread internal/acceptance/openstack/sharedfilesystems/v2/sharetypes_test.go Outdated
Comment thread internal/acceptance/openstack/sharedfilesystems/v2/sharetypes_test.go Outdated
Comment thread openstack/sharedfilesystems/v2/sharetypes/results.go
Comment thread openstack/sharedfilesystems/v2/sharetypes/results.go
@github-actions github-actions Bot added semver:major Breaking change and removed semver:minor Backwards-compatible change backport-v2 This PR will be backported to v2 labels Jul 17, 2026
@TBExtent
TBExtent requested a review from kayrus July 21, 2026 10:11
@kayrus
kayrus added this pull request to the merge queue Jul 23, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 23, 2026
@TBExtent

Copy link
Copy Markdown
Contributor Author

Hi @kayrus, it failed to merge due to an internal error in Coveralls, please could you add it to the merge queue again?

@kayrus
kayrus added this pull request to the merge queue Jul 27, 2026
Merged via the queue into gophercloud:main with commit 1c295d7 Jul 27, 2026
70 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

edit:sharedfilesystems This PR updates sharedfilesystems code semver:major Breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Manila: add resources for share types and share type access

5 participants