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

added 'drop_warn_threshold' config in tile #300

Merged
merged 1 commit into from
Oct 14, 2021
Merged

Conversation

kashyap-splunk
Copy link
Collaborator

added 'drop_warn_threshold' config in tile yml

Copy link
Collaborator

@luckyj5 luckyj5 left a comment

Choose a reason for hiding this comment

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

LGTM

@luckyj5 luckyj5 merged commit e8be734 into develop Oct 14, 2021
luckyj5 added a commit that referenced this pull request Oct 14, 2021
luckyj5 added a commit that referenced this pull request Oct 14, 2021
* update docs (#276)

* Update Go version and migrate to Go modules (#287)

* Adding go.mod and go.sum to migrate to Go modules from Glide

* Removed old Vendor folder after migrating to Go modules

* Adding new Vendor folder after migrating to Go mudules

* Removed Glide files, references in readme and updated CI/CD

* Migrated from bolt to bbolt for cache module to resolve race issue

* Added tags from the envelope to the events

* Updated tile.yml with new config

* Updated README

* Updated migration script to prevent unnecessary migration (#288)

* Updated cache.GetApp logic to retrieve app info from boltdb database as well

* Bugfix/disconnect due to slow consumer event drop (#289)

* dropping events when the queue is full

* Corrected the newly added test case with data race condition

* minor change in logging

* added event drop warning threshold in config

* minor update in the logging

* Updated comments and log info

* Trimmed white space for ApiEndPoint and SplunkHost

* Updated docs with required field & optional value

* Added default value in README

* Updated events doc

* Removed SPLUNK_VERSION config as not needed anymore

* Removed SplunkVersion from eventsink.splunk_test

* Updated README with app level index routing (#292)

* Updated README with app level index routing

* Removed unnecessary changes

* added required config to enable index routing

* updated index routing doc

* added warning in index routing docs

* upgraded module version

* upgraded gogo/protobuf version

* executed go mod tidy

* upgraded indirect dependency

* updated version in a build example in readme

* Corrected index routing docs

* Added more details (#299)

* Updated docs around

- Add Tags: Performance impact of adding tags in events
- How to detect slow downstream related issues and event drops

* added 'drop_warn_threshold' config in tile (#300)

Co-authored-by: Matthew Heidemann <[email protected]>
Co-authored-by: Shubham Jain <[email protected]>
Co-authored-by: harshit-splunk <[email protected]>
Co-authored-by: harshit-splunk <[email protected]>
@kashyap-splunk kashyap-splunk deleted the add-drop-threshold-tile branch December 2, 2021 05:37
luckyj5 pushed a commit that referenced this pull request Dec 20, 2021
* added 'drop_warn_threshold' config in tile (#300)

* Upgrading the OAuth dependency to resolve the auth issue

* Added a troubleshooting item for the auth issue with special characters

* Drain the response buffer to reuse the connection with HEC

* Changed eventwriter to write events on stdout in debug mode

* Added a test for debug mode

* Added/updated testcases to increase coverage

* updated eventsink tests

* fixed race condition in eventsink test

* Updated the nozzle version set in HEC req header

* Corrected version in test

* Updated version in readme

Co-authored-by: harshit-splunk <[email protected]>
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