Skip to content

Commit 6a91651

Browse files
committed
netapp_manager: return svm_name and volume names
These may be used later in other workflows
1 parent 492fe83 commit 6a91651

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

python/understack-workflows/understack_workflows/netapp_manager.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,11 +75,14 @@ def create_svm(self, project_id: str, aggregate_name: str):
7575
logger.info(
7676
"SVM '%s' created successfully with NVMe protocol allowed", svm.name
7777
)
78+
return svm.name
7879
except NetAppRestError as e:
7980
logger.error("Error creating SVM: %s", e)
8081
exit(1)
8182

82-
def create_volume(self, project_id: str, volume_size: str, aggregate_name: str):
83+
def create_volume(
84+
self, project_id: str, volume_size: str, aggregate_name: str
85+
) -> str:
8386
"""Creates a new volume within a specific SVM and aggregate."""
8487
volume_name = self._volume_name(project_id)
8588
logger.info(
@@ -97,6 +100,7 @@ def create_volume(self, project_id: str, volume_size: str, aggregate_name: str):
97100
volume.post()
98101
volume.get()
99102
logger.info("Volume %s created.", volume_name)
103+
return volume_name
100104
except NetAppRestError as e:
101105
logger.error("Error creating Volume: %s", e)
102106
exit(1)

python/understack-workflows/understack_workflows/oslo_event/keystone_project.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,10 +66,10 @@ def handle_project_created(
6666
svm_name = None
6767
try:
6868
netapp_manager = NetAppManager()
69-
netapp_manager.create_svm(
69+
svm_name = netapp_manager.create_svm(
7070
project_id=event.project_id, aggregate_name=AGGREGATE_NAME
7171
)
72-
svm_name = netapp_manager.create_volume(
72+
netapp_manager.create_volume(
7373
project_id=event.project_id,
7474
volume_size=VOLUME_SIZE,
7575
aggregate_name=AGGREGATE_NAME,

0 commit comments

Comments
 (0)