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

Update telemetry.sh with threshold config #14615

Merged
merged 4 commits into from
Apr 18, 2023

Conversation

zbud-msft
Copy link
Contributor

Why I did it

Threshold is a new config field passed to telelemetry.go as parameter

How I did it

Add check for threshold

How to verify it

Modify telemetry.sh, systemctl restart telemetry, telemetry process has threshold of 100

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@zbud-msft zbud-msft requested a review from qiluo-msft April 18, 2023 17:17
@zbud-msft zbud-msft marked this pull request as ready for review April 18, 2023 21:24
@qiluo-msft qiluo-msft merged commit e9a9c9e into sonic-net:master Apr 18, 2023
anamehra pushed a commit to anamehra/sonic-buildimage that referenced this pull request Apr 19, 2023
#### Why I did it

Threshold is a new config field passed to telelemetry.go as parameter

#### How I did it

Add check for threshold

#### How to verify it

Modify telemetry.sh, systemctl restart telemetry, telemetry process has threshold of 100
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