Skip 'eject valid master' replication test6.0.0.0rc26.0.0

This test was skipped in the scenario runs however it was
not disabled in the original api tests.  Although it
doesn't fail as often there it still happens, so the
same change has been applied to it.

See: https://review.openstack.org/#/c/368230

The test will also print out the bug number during each run
as a reminder (using the new SkipKnownBug method).

Change-Id: I931d6d72a70cc93dcd8248d9840eadf1160b9bab
Related-Bug: #1622014
(cherry picked from commit e89dfbd51d4cad01a2ad701e9300ccfe6e7eac5a)

1 files changed, 7 insertions, 3 deletions

diff --git a/trove/tests/api/replication.py b/trove/tests/api/replication.py
index 44f3de3d..6596ba61 100644
--- a/trove/tests/api/replication.py
+++ b/trove/tests/api/replication.py
@@ -30,6 +30,8 @@ from trove.tests.api.instances import TIMEOUT_INSTANCE_CREATE
 from trove.tests.api.instances import TIMEOUT_INSTANCE_DELETE
 from trove.tests.api.instances import WaitForGuestInstallationToFinish
 from trove.tests.config import CONFIG
+from trove.tests.scenario import runners
+from trove.tests.scenario.runners.test_runners import SkipKnownBug
 from trove.tests.util.server_connection import create_server_connection
 
 
@@ -267,9 +269,11 @@ class TestReplicationFailover(object):
         if CONFIG.fake_mode:
             raise SkipTest("eject_replica_source not supported in fake mode")
 
-        assert_raises(exceptions.BadRequest,
-                      instance_info.dbaas.instances.eject_replica_source,
-                      instance_info.id)
+        # assert_raises(exceptions.BadRequest,
+        #               instance_info.dbaas.instances.eject_replica_source,
+        #               instance_info.id)
+        # Uncomment once BUG_EJECT_VALID_MASTER is fixed
+        raise SkipKnownBug(runners.BUG_EJECT_VALID_MASTER)
 
     @test(depends_on=[test_promote_master, test_eject_slave,
                       test_eject_valid_master])