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 another example for filtering metrics #529

Merged
merged 4 commits into from
Mar 30, 2020

Conversation

spchin
Copy link
Contributor

@spchin spchin commented Mar 25, 2020

Can you please check the syntax that I put in?

Description

Fill in your description here.

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

Can you please check the syntax that I put in?
@vsinghal13 vsinghal13 requested review from vsinghal13, drduke1, samjsong and perk-sumo and removed request for vsinghal13 March 25, 2020 22:44
@perk-sumo perk-sumo added this to the v1.0 milestone Mar 26, 2020
@vsinghal13 vsinghal13 requested review from MariePMcGarry and removed request for drduke1 March 26, 2020 16:41
The above section should be added in each of the kube-state remote write blocks.
Copy link
Contributor

Choose a reason for hiding this comment

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

"the above section" should be "the section above". You could remove extra space before "kube-state"

The above section should be added in each of the kube-state remote write blocks.

Here is another example of excluding up metrics in the sumologic namespace and keep the up metrics for all other namespaces:
Copy link
Contributor

@MariePMcGarry MariePMcGarry Mar 26, 2020

Choose a reason for hiding this comment

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

suggest you change "and keep the up metrics" to "while still collecting up metrics". (unless I misunderstand)

@vsinghal13 vsinghal13 closed this Mar 27, 2020
@vsinghal13 vsinghal13 reopened this Mar 27, 2020
@vsinghal13 vsinghal13 merged commit 6f6c7b2 into SumoLogic:master Mar 30, 2020
vsinghal13 added a commit that referenced this pull request Mar 30, 2020
* Added another example for filtering metrics

Can you please check the syntax that I put in?

* Update Best_Practices.md

* Update Best_Practices.md

Co-authored-by: Vijit Singhal <56007827+vsinghal13@users.noreply.github.com>
vsinghal13 added a commit that referenced this pull request Mar 31, 2020
* Added another example for filtering metrics

Can you please check the syntax that I put in?

* Update Best_Practices.md

* Update Best_Practices.md

Co-authored-by: Vijit Singhal <56007827+vsinghal13@users.noreply.github.com>
vsinghal13 added a commit that referenced this pull request Mar 31, 2020
* Added another example for filtering metrics

Can you please check the syntax that I put in?

* Update Best_Practices.md

* Update Best_Practices.md

Co-authored-by: Vijit Singhal <56007827+vsinghal13@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants