-
Notifications
You must be signed in to change notification settings - Fork 2
QA-596: special hotbackup test case for cluster deployment (topology change) #533
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
base: main
Are you sure you want to change the base?
Conversation
871f1e7
to
b4a9b14
Compare
…lusterTopologyChange
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please adjust the two methods.
It also would be great if you could ctrl+c/v the clear data changes from my PR and add the invocations in your code
self.delete_all_backups() | ||
|
||
@staticmethod | ||
def run_starter_and_wait(starter): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shouldn't this be a starter-manager method?
@@ -1280,6 +1284,15 @@ def count_occurances_in_starter_log(self, substring: str): | |||
number_of_occurances = self.get_log_file().count(substring) | |||
return number_of_occurances | |||
|
|||
def stop_dbserver(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should not be a method specific to db-servers.
It should rather be
stop_instances(instance_type)
No description provided.