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

Elasticsearch Publishers Full warning #4443

Merged
merged 3 commits into from
Dec 1, 2020

Conversation

lancewf
Copy link
Contributor

@lancewf lancewf commented Nov 17, 2020

🔩 Description: What code changed, and why?

When all the Elasticsearch publishers are full the log message should be an error and not a warning.

⛓️ Related Resources

#3152

👍 Definition of Done

The log message when all the Elasticsearch publishers are full is an error, not a warning.

✅ Checklist

When all the Elasticsearch publishers are full the log should be
and error and not a warning.

Signed-off-by: Lance Finfrock <lfinfrock@chef.io>
@netlify
Copy link

netlify bot commented Nov 17, 2020

Deploy preview for chef-automate ready!

Built with commit c3546dd

https://deploy-preview-4443--chef-automate.netlify.app

Lance Finfrock added 2 commits November 17, 2020 11:37
When the Elasticsearch publishers are full log to the user a way to
increase the number of publishers or the number of messages a publisher
can send at a time.

Signed-off-by: Lance Finfrock <lfinfrock@chef.io>
@lancewf lancewf merged commit b7b992f into master Dec 1, 2020
@lancewf lancewf deleted the lancewf/publishers_full_message branch December 1, 2020 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants