qa/tasks/cephadm.py: do not create rbd pool by default

rados/cephadm/smoke* does not use the install task and the adjust-ulimits
dependency is met as a part of it. create_rbd_pool needs adjust-ulimits,
so for now we will disable create_rbd_pool by default and only set it
to true for the upgrade suite.

Signed-off-by: Neha Ojha <nojha@redhat.com>

diff --git a/qa/suites/upgrade/octopus-x/0-start.yaml b/qa/suites/upgrade/octopus-x/0-start.yaml
index 387053022aef33f70fd89cea6569a84fcd7734ec..ad3ee43d38e494c179f42a85733ea7aa10119a7c 100644 (file)
--- a/qa/suites/upgrade/octopus-x/0-start.yaml
+++ b/qa/suites/upgrade/octopus-x/0-start.yaml
@@ -25,6 +25,7 @@ openstack:
     size: 10 # GB
 overrides:
   ceph:
+    create_rbd_pool: true
     conf:
       osd:
         osd shutdown pgref assert: true

diff --git a/qa/tasks/cephadm.py b/qa/tasks/cephadm.py
index c2fa76dea24b5a8cfe212787bc8e0b3a351d3b9b..54887685e40d863ea01d6c4f89e565c69020df27 100644 (file)
--- a/qa/tasks/cephadm.py
+++ b/qa/tasks/cephadm.py
@@ -1019,25 +1019,24 @@ def crush_setup(ctx, config):
 
 @contextlib.contextmanager
 def create_rbd_pool(ctx, config):
-    cluster_name = config['cluster']
-    log.info('Waiting for OSDs to come up')
-    teuthology.wait_until_osds_up(
-        ctx,
-        cluster=ctx.cluster,
-        remote=ctx.ceph[cluster_name].bootstrap_remote,
-        ceph_cluster=cluster_name,
-    )
-    if config.get('create_rbd_pool', True):
-        log.info('Creating RBD pool')
-        _shell(ctx, cluster_name, ctx.ceph[cluster_name].bootstrap_remote,
-            args=['sudo', 'ceph', '--cluster', cluster_name,
-                  'osd', 'pool', 'create', 'rbd', '8'])
-        _shell(ctx, cluster_name, ctx.ceph[cluster_name].bootstrap_remote,
-            args=[
-                'sudo', 'ceph', '--cluster', cluster_name,
+    if config.get('create_rbd_pool', False):
+      cluster_name = config['cluster']
+      log.info('Waiting for OSDs to come up')
+      teuthology.wait_until_osds_up(
+          ctx,
+          cluster=ctx.cluster,
+          remote=ctx.ceph[cluster_name].bootstrap_remote,
+          ceph_cluster=cluster_name,
+      )
+      log.info('Creating RBD pool')
+      _shell(ctx, cluster_name, ctx.ceph[cluster_name].bootstrap_remote,
+          args=['sudo', 'ceph', '--cluster', cluster_name,
+                'osd', 'pool', 'create', 'rbd', '8'])
+      _shell(ctx, cluster_name, ctx.ceph[cluster_name].bootstrap_remote,
+          args=['sudo', 'ceph', '--cluster', cluster_name,
                 'osd', 'pool', 'application', 'enable',
                 'rbd', 'rbd', '--yes-i-really-mean-it'
-            ])
+          ])
     yield
 
 @contextlib.contextmanager