openstack
/
charm-ceph-mon
Code Issues Proposed changes

Merge "Add app-name as an option during pool creation"

This commit is contained in:
Zuul 2018-12-12 14:00:57 +00:00 committed by Gerrit Code Review
parent 4ff50f1944 4b5193442d
commit c2dee24c61
2 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
actions.yaml
View File

@ -48,6 +48,9 @@ create-pool:
name:
type: string
description: The name of the pool
app-name:
type: string
description: App name to set on the newly created pool.
profile-name:
type: string
description: The crush profile to use for this pool. The ruleset must exist first.

9
actions/create-pool.py
View File

@ -25,19 +25,24 @@ from charmhelpers.contrib.storage.linux.ceph import ErasurePool, ReplicatedPool
def create_pool():
pool_name = action_get("name")
pool_type = action_get("pool-type")
app_name = action_get("app-name") or None
try:
if pool_type == "replicated":
replicas = action_get("replicas")
replicated_pool = ReplicatedPool(name=pool_name,
service='admin',
replicas=replicas)
replicas=replicas,
app_name=app_name,
)
replicated_pool.create()
elif pool_type == "erasure":
crush_profile_name = action_get("erasure-profile-name")
erasure_pool = ErasurePool(name=pool_name,
erasure_code_profile=crush_profile_name,
service='admin')
service='admin',
app_name=app_name,
)
erasure_pool.create()
else:
log("Unknown pool type of {}. Only erasure or replicated is "