Skip to content
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

Daemons process title name #651

Merged
merged 1 commit into from
Dec 19, 2016
Merged

Daemons process title name #651

merged 1 commit into from
Dec 19, 2016

Conversation

mohierf
Copy link
Contributor

@mohierf mohierf commented Dec 16, 2016

Alignak doc updated accordingly: https://github.com/Alignak-monitoring/alignak-doc/blob/update-features/source/03_how_it_work/run_daemons.rst#alignak-processes-list


According to #614, daemons names are:

alignak   3251  0.6  1.1 867048 45052 ?        Sl   10:37   0:00 alignak-scheduler scheduler-master
alignak   3252  0.0  0.9 208640 37264 ?        S    10:37   0:00 alignak-scheduler
alignak   3308  0.4  1.0 864824 42484 ?        Sl   10:37   0:00 alignak-poller poller-master
alignak   3309  0.0  0.9 495000 37844 ?        Sl   10:37   0:00 alignak-poller
alignak   3369  0.4  1.0 864812 42376 ?        Sl   10:37   0:00 alignak-reactionner reactionner-master
alignak   3370  0.0  0.9 347540 37864 ?        Sl   10:37   0:00 alignak-reactionner
alignak   3506  1.2  1.1 867732 45144 ?        Sl   10:37   0:00 alignak-broker broker-master
alignak   3507  0.0  0.9 495276 40444 ?        Sl   10:37   0:00 alignak-broker
alignak   3585  0.5  1.0 864576 42052 ?        Sl   10:37   0:00 alignak-receiver receiver-master
alignak   3586  0.0  0.9 347940 37864 ?        Sl   10:37   0:00 alignak-receiver
alignak   3647  2.0  1.1 867596 44528 ?        Sl   10:37   0:00 alignak-arbiter
alignak   3648  0.0  0.9 211000 39292 ?        S    10:37   0:00 alignak-arbiter arbiter-master
alignak   3663  0.0  0.9 864184 40412 ?        S    10:37   0:00 alignak-poller-master worker
alignak   3672  0.0  1.0 864320 41068 ?        S    10:37   0:00 alignak-receiver-master module: nsca
alignak   3682  0.4  1.0 866752 43396 ?        S    10:37   0:00 alignak-broker-master module: backend_broker
alignak   3692  0.2  1.0 866752 43292 ?        S    10:37   0:00 alignak-broker-master module: logs
alignak   3703  0.0  1.0 864196 40536 ?        S    10:37   0:00 alignak-poller-master worker
alignak   3711  0.0  1.0 864188 40472 ?        S    10:37   0:00 alignak-reactionner-master worker

@coveralls
Copy link

Coverage Status

Coverage increased (+0.04%) to 81.633% when pulling a42db41 on daemons-name into fd6cd99 on develop.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.06%) to 81.653% when pulling a42db41 on daemons-name into fd6cd99 on develop.

@mohierf mohierf changed the title [WiP] daemons process title name Daemons process title name Dec 16, 2016
@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 81.606% when pulling 07586f7 on daemons-name into fd6cd99 on develop.

@@ -229,6 +231,7 @@ def __init__(self, name, config_file, is_daemon, do_replace, debug, debug_file):
"""
self.check_shm()

self.daemon_type = 'unknown'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not define the 'unknown' in the default value in the properties dictionary ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

with self.conf_lock:
conf = unserialize(self.new_conf, True)
logger.warning("Configuration: %s", conf)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure it's a good idea to put in logger the conf (will be big text :p)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do agree, it may be important and not as useful as expected. I remove this log :/

@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) to 81.644% when pulling 48516b6 on daemons-name into fd6cd99 on develop.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) to 81.644% when pulling 48516b6 on daemons-name into fd6cd99 on develop.

@coveralls
Copy link

coveralls commented Dec 19, 2016

Coverage Status

Coverage remained the same at ?% when pulling 7921660 on daemons-name into fd6cd99 on develop.

@ddurieux ddurieux merged commit ba57491 into develop Dec 19, 2016
@ddurieux ddurieux deleted the daemons-name branch December 19, 2016 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants