-
Notifications
You must be signed in to change notification settings - Fork 19
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
[WiP] - Add a multi-realms test in the load test suite #901
Conversation
1eb79fe
to
a3c94da
Compare
alignak/daemons/arbiterdaemon.py
Outdated
result = False | ||
else: | ||
logger.info("%s running (pid=%d)", | ||
daemon_name, self.my_satellites[daemon_name].pid) | ||
return result | ||
|
||
def stop_missing_daemons(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 function has many problems:
- in comment, it's written
stop the daemon
but in code it just look if stopped, not try stop it - in comment we have
True if all daemons are stopped, else False
, but in code it return True only in case all daemons are running
alignak/daemons/arbiterdaemon.py
Outdated
@@ -784,6 +816,9 @@ def do_loop_turn(self): | |||
# Main loop | |||
self.run() | |||
|
|||
if self.interrupted: | |||
self.stop_missing_daemons() |
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.
after my previous comment, we not use the return status, perhaps manage it
@ddurieux, please merge this ... |
But you not see my comments... :p |
Add a multi-realms test with undefined daemons Clean daemons launch and stop
bd6864e
to
066c83f
Compare
I deprecate this in favor of #990 |
No description provided.