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

Simplify Metrics FluentD Pipeline #180

Merged
merged 11 commits into from
Sep 10, 2019
Merged

Simplify Metrics FluentD Pipeline #180

merged 11 commits into from
Sep 10, 2019

Conversation

frankreno
Copy link
Contributor

simplify fluentD routing, make it easier to add custom metrics and update steps in readme with details on how to do this with a seperate HTTP source.

…date steps in readme with details on how to do this with a seperate HTTP source.
Copy link
Contributor

@samjsong samjsong left a comment

Choose a reason for hiding this comment

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

LGTM, thanks Frank! Minor nit though, can we change the tab size from 4 to 2 for the stuff within the <label @DATAPOINT> block?

@frankreno
Copy link
Contributor Author

LGTM, thanks Frank! Minor nit though, can we change the tab size from 4 to 2 for the stuff within the <label @DATAPOINT> block?

Good catch! fixed

deploy/README.md Outdated Show resolved Hide resolved
deploy/README.md Outdated Show resolved Hide resolved
deploy/README.md Outdated Show resolved Hide resolved
deploy/README.md Outdated Show resolved Hide resolved
deploy/README.md Outdated Show resolved Hide resolved
deploy/README.md Outdated Show resolved Hide resolved
deploy/README.md Outdated Show resolved Hide resolved
deploy/README.md Outdated Show resolved Hide resolved
deploy/README.md Outdated
- [Step 2: Setup a service monitor so that Prometheus pulls the data](#step-2-setup-a-service-monitor-so-that-prometheus-pulls-the-data)
- [Step 3: Update the prometheus-overrides.yaml file to forward the metrics to Sumo.](#step-3-update-the-prometheus-overridesyaml-file-to-forward-the-metrics-to-sumo)
- [Step 2: Set up a service monitor so that Prometheus pulls the data](#step-2-set-up-a-service-monitor-so-that-prometheus-pulls-the-data)
- [Step 3: Step 3: Create a new HTTP source in Sumo Logic.](#step-3-create-a-new-http-source-in-sumo-logic)
Copy link
Contributor

Choose a reason for hiding this comment

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

Step 3: is written twice here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

Copy link
Contributor

Choose a reason for hiding this comment

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

did you commit the change? still written twice

Copy link
Contributor Author

Choose a reason for hiding this comment

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

hrmm its weird, i don't see this locally

deploy/README.md Outdated Show resolved Hide resolved
Copy link
Contributor

@rvmiller89 rvmiller89 left a comment

Choose a reason for hiding this comment

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

Approving to unblock merge, but I think L31 still needs fix

@frankreno frankreno merged commit c9769ae into master Sep 10, 2019
@perk-sumo perk-sumo deleted the simplify-pipeline branch May 21, 2020 16:31
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.

5 participants