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.

Sign up for GitHub

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

Jump to bottom

Byi deploy metrics #6

Merged
merged 3 commits into from Apr 8, 2019
Merged

Byi deploy metrics #6

merged 3 commits into from Apr 8, 2019

Conversation

bin3377
Copy link

@bin3377 bin3377 commented Apr 5, 2019 •
edited

Deploy update on both docker image and Kubernetes .yaml include:

  • Dockerfile update and unit test (make sure Docker image is built and runnable)
  • Routing metrics pipeline in fluent.conf with current 3 plugins
  • Placeholder of events/logs pipeline
  • Rename ProtobufParse to ProtobufParser otherwise fluentd will fail to find it

Need an end-to-end test with Prometheus cluster

FYI - @maimaisie @yuting-liu @frankreno

@bin3377 bin3377 merged commit 6e8137c into master Apr 8, 2019
@bin3377 bin3377 deleted the byi-deploy-metrics branch April 8, 2019 21:47
psaia pushed a commit to psaia/sumologic-kubernetes-collection that referenced this pull request May 25, 2021
@msuterski
Merge pull request SumoLogic#6 from nytimes/search
142129e 
Add collector for nytimes/search-omnibus
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rvmiller89 rvmiller89 rvmiller89 approved these changes

@samjsong samjsong samjsong approved these changes

@lei-sumo lei-sumo lei-sumo approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@bin3377 @rvmiller89 @samjsong @lei-sumo