Skip to content

Implement an ability to configure worker options #12

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

Merged
merged 1 commit into from
Jan 7, 2017
Merged

Implement an ability to configure worker options #12

merged 1 commit into from
Jan 7, 2017

Conversation

ellisv
Copy link
Contributor

@ellisv ellisv commented Dec 9, 2016

Things like worker worker_configuration_directory could also have been moved under worker_options configuration key but I wanted this pull request to be backwards compatible. If requested I could move those options there too.

@Phobetor Phobetor merged commit 36963b3 into Phobetor:master Jan 7, 2017
@Phobetor
Copy link
Owner

Phobetor commented Jan 7, 2017

Hi @ellisv

Thanks for your contribution!

Your change is released as part of version 2.0.0.

I moved the configuration option to

rabbit_mq_supervisor:
    consumer:
        general:
            worker:
                startsecs:              2       # supervisord worker option: seconds to consider program running
                autorestart:            true    # supervisord worker option: if supervisord should restarted program automatically
                stopsignal:             INT     # supervisord worker option: the signal used to kill the program
                stopasgroup:            true    # supervisord worker option: if whole process group should be stopped
                stopwaitsecs:           60      # supervisord worker option: seconds to wait after stop signal before sending kill signal

as there are some other new options that also fit there.

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.

2 participants