Skip to content

Commit

Permalink
fix(docker-backend): do not start scylla-manager installation
Browse files Browse the repository at this point in the history
If scylla-manager is not disabled in a test config when running a test on docker
backend, the test will try to install the manager and eventually fail (this is
due to the fact that SCT installs scylla-manager server on monitor node and docker
backend doesn't support dedicated monitoring node docker instance).

The change adds a check for this situation, to be able to fail fast and
do not continue with manager installation od docker backend.

Closes: #9028
  • Loading branch information
dimakr committed Nov 21, 2024
1 parent 51856ad commit 83c049d
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions sdcm/cluster.py
Original file line number Diff line number Diff line change
Expand Up @@ -5497,6 +5497,8 @@ def node_startup(self, node, **kwargs): # pylint: disable=unused-argument
pass

def install_scylla_manager(self, node):
if node.is_docker():
raise ScyllaManagerError("Scylla Manager is not supported in docker environment")
if self.params.get("scylla_repo_m"):
scylla_repo = self.params.get("scylla_repo_m")
else:
Expand Down

0 comments on commit 83c049d

Please sign in to comment.