-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test.py: don't stop cluster's site if not started #11939
Conversation
CI state |
61f0935
to
c53a1b5
Compare
Changes in c53a1b5:
|
c53a1b5
to
697e7a9
Compare
Changes in 697e7a9:
|
CI state |
Failure due to #8079 (comment). |
CI state |
697e7a9
to
bd7bd97
Compare
Changes in bd7bd97:
|
CI state |
The site member is created in ScyllaCluster.start(), for startup failure this might not be initialized, so check it's present before stop()ing it. And delete it as it's not running and proper initialization should call ScyllaCluster.start(). Signed-off-by: Alejo Sanchez <alejo.sanchez@scylladb.com>
bd7bd97
to
0920771
Compare
Unrelated failure
|
Changes in 0920771:
|
CI state |
ping |
Check
ScyllaCluster.site
is present before callingstop()
. This might happen due to bad starts whenScyllaCluster.stop()
is called butScyllaCluster.start()
failed.