feat: Add Proper Pool Discovery and FlexVol Integration #1112
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
nidzrai
force-pushed
the
PUC-1104
branch
2 times, most recently
from
fa2642f to
cd33355
Compare
cardoe
reviewed
Jul 31, 2025
| CONF.register_opts(options.netapp_provisioning_opts, group="netapp_nvme") | ||
| CONF.register_opts(options.netapp_cluster_opts, group="netapp_nvme") | ||
| CONF.register_opts(options.netapp_san_opts, group="netapp_nvme") | ||
| CONF.register_opts(volume_driver.volume_opts, group="netapp_nvme") |
There was a problem hiding this comment.
So I understand this registration but if you look at the cinder contributor docs and how other drivers are implemented they abstract this using the cinder.volume.configuration module. This is fine to keep for now but as a follow on we should switch to using that.
| # Everything else from cinder.conf | ||
| hostname = self.configuration.netapp_server_hostname | ||
| username = self.configuration.netapp_login | ||
| password = self.configuration.netapp_password |
There was a problem hiding this comment.
Cause you're fiddling with the configuration pieces here.
cardoe
approved these changes
Jul 31, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
https://rackspace.atlassian.net/browse/PUC-1104
This update enhances the NetApp dynamic driver to support real FlexVol discovery and accurate pool capacity reporting, enabling more reliable volume scheduling and placement in OpenStack Cinder.
Real FlexVol Discovery -Retrieves actual FlexVol names and capacity details directly from the NetApp backend using the REST API.
Pool Name Extraction -Maps FlexVols to Cinder pool names correctly, ensuring volumes are routed to the right pool selected by the scheduler.
Enhanced Logging -Adds detailed logs during pool discovery and update operations to aid in debugging and traceability.
Scheduler Integration-Exposes real-time capacity and pool stats to the scheduler for accurate volume placement decisions.
Namespace - map namespace after volume
WIP: subsystem
WIP : self.client here is temporary . (we have to think of solution to replace it ) [done] used caching via dic {}
Added namespace table to fix add after volume