From 3c035346e700394e2ca628a25d8a3c102f72522c Mon Sep 17 00:00:00 2001
From: =?utf8?q?C=C3=A9dric=20Ollivier?= <cedric.ollivier@orange.com>
Date: Wed, 27 May 2020 14:05:53 +0200
Subject: [PATCH] Skip manage_snapshot in CNTT
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

As detected by Orange's RC Field trial, Ceph doesn't support this
feature before Rocky [1].
"AttributeError: 'RBDDriver' object has no attribute 'unmanage_snapshot'"

manage_snapshot testing is fully passing in Functest SUT because CNTT
is verified vs OpenStack Rocky and newer.

Ceph is selected by RI as backend and is widely reused by the ecosytem.
It's similar to [2].

[1] https://bugs.launchpad.net/cinder/+bug/1645288
[2] https://gerrit.opnfv.org/gerrit/c/functest/+/69888

Change-Id: I3b7f88f5bed9a47a1e26e16bbbf74fea24ca76c2
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 830d508837f3b1011eeb44c3d5af5961fbf4c701)
---
 docker/smoke-cntt/tempest_conf.yaml | 2 +-
 docker/smoke-cntt/testcases.yaml    | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/docker/smoke-cntt/tempest_conf.yaml b/docker/smoke-cntt/tempest_conf.yaml
index 0c2aa3bd5..b45164f50 100644
--- a/docker/smoke-cntt/tempest_conf.yaml
+++ b/docker/smoke-cntt/tempest_conf.yaml
@@ -72,7 +72,7 @@ volume-feature-enabled:
     backup: true
     snapshot: true
     clone: true
-    manage_snapshot: true
+    manage_snapshot: false
     manage_volume: true
     extend_attached_volume: false
     consistency_group: false
diff --git a/docker/smoke-cntt/testcases.yaml b/docker/smoke-cntt/testcases.yaml
index 7e6cc6a5a..9d201c9ba 100644
--- a/docker/smoke-cntt/testcases.yaml
+++ b/docker/smoke-cntt/testcases.yaml
@@ -160,7 +160,7 @@ tiers:
                 criteria: 100
                 blocking: false
                 deny_skipping: true
-                tests_count: 1281
+                tests_count: 1280
                 description: >-
                     The list of test cases is generated by
                     Tempest automatically and depends on the parameters of
@@ -212,6 +212,7 @@ tiers:
                           (?!.*test_routers.RoutersTest.test_create_router_set_gateway_with_fixed_ip)\
                           (?!.*test_group_snapshots.GroupSnapshotsV319Test.test_reset_group_snapshot_status)\
                           (?!.*test_multi_backend)\
+                          (?!.*test_snapshot_manage)\
                           (?!.*test_volume_retype.VolumeRetypeWithMigrationTest)\
                           (?!.*test_volume_delete_cascade.VolumesDeleteCascade.test_volume_from_snapshot_cascade_delete)\
                           (?!.*test_volumes_backup.VolumesBackupsTest.test_volume_backup_create_get_detailed_list_restore_delete)\
-- 
2.16.6