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

Improve Docs #298

Merged
merged 7 commits into from
Nov 14, 2019
Merged

Improve Docs #298

merged 7 commits into from
Nov 14, 2019

Conversation

frankreno
Copy link
Contributor

Description

Fill in your description here.
Improve non-helm install docs with more detailed steps and move filter/trim/relabel/custom metrics docs to standalone documentation.

Testing performed
  • ci/build.sh
  • Redeploy fluentd and fluentd-events pods
  • Confirm events, logs, and metrics are coming in

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.

Thanks for this Frank, unfortunately there's some merge conflicts so we should fix those before merging. Otherwise content LGTM!


Install `prometheus-operator` by generating the yaml files using Helm:

```bash
$ helm template stable/prometheus-operator --name prometheus-operator --set dryRun=true -f prometheus-overrides.yaml > prometheus.yaml
<<<<<<< HEAD
Copy link
Contributor

Choose a reason for hiding this comment

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

There's some merge conflicts here, I see we have the other helm fetch command on line 189 above that conflicts with line 214 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.

sigh, something is really messed up with my Git. Thanks @samjsong

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

@@ -219,6 +236,12 @@ kubectl -n sumologic logs prometheus-prometheus-operator-prometheus-0 prometheus

At this point setup is complete and metrics data is being sent to Sumo Logic.

<<<<<<< HEAD
__NOTE__ You can also [send custom metrics](additional_prometheus_configuration.md#custom-metrics)to Sumo Logic from Prometheus.
Copy link
Contributor

Choose a reason for hiding this comment

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

Merge conflicts here as well

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


If you would like to collect other metrics that are not listed in `prometheus-overrides.yaml`, you can add a new section to the file.
<<<<<<< HEAD
Copy link
Contributor

Choose a reason for hiding this comment

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

Merge conflicts here as well

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

@frankreno frankreno merged commit d7a4b06 into SumoLogic:master Nov 14, 2019
frankreno added a commit to frankreno/sumologic-kubernetes-collection that referenced this pull request Nov 14, 2019
Merge pull request SumoLogic#298 from frankreno/master
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.

None yet

3 participants