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

Logging plugins fixes and random stuff #7

Merged
merged 2 commits into from
Aug 31, 2019
Merged

Logging plugins fixes and random stuff #7

merged 2 commits into from
Aug 31, 2019

Conversation

javuto
Copy link
Collaborator

@javuto javuto commented Aug 31, 2019

Overview

  • Added extra debug logging to the logging plugins.
  • Splunk and Graylog plugins were never initialized despite the configuration being loaded. That was throwing a panic when using any of those plugins.
  • Metadata for nodes is updated before logging, so in the case the plugins crashes, the metadata will be up to date.
  • Removing flag --tls_dump.
  • Adding flag --distributed_tls_max_attempts.
  • Random clean-up code.

@javuto javuto added this to the 0.1.6 milestone Aug 31, 2019
@javuto javuto merged commit f69c109 into master Aug 31, 2019
@javuto javuto deleted the logging-plugins branch August 31, 2019 01:18
CptOfEvilMinions added a commit that referenced this pull request Dec 28, 2021
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.

1 participant