Make the polling rate configurable for job status #27
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The 0.1 second polling rate is overly intrusive for some users, and can exacerbate problems on an already loaded server. This sets the default rate to 1 second and makes it configurable.
|
Addresses part of the issue mentioned #25 |
|
Any chance to see this merged @manderson26 @jtimberman please ? |
|
LGTM 👍 |
markan
added a commit
that referenced
this pull request
Jun 17, 2015
Make the polling rate configurable for job status
|
On side note, related to cookbooks, how does someone outside Chef can become reviewer @jtimberman (not talking about opening PRs, but reviewing/testing/merging etc) ? I believe it's possible since the move to community-maintained cookbooks, but disregard if I've been dreaming 🍶 I'd like to help out 👓 |
|
@scalp42 if you'd like to be involved, check out this RFC: https://github.com/chef/chef-rfc/blob/master/rfc030-maintenance-policy.md |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The 0.1 second polling rate is overly intrusive for some users, and can
exacerbate problems on an already loaded server. This sets the default
rate to 1 second and makes it configurable.
@jkeiser If you have a chance, would you take a look at this?