Zane/rename telegraf process metrics#1655
Open
zanejohnson-azure wants to merge 4 commits intoci_prodfrom
Open
Conversation
Give the telegraf instance that collects process metrics a distinct process name so it can be distinguished from the main telegraf process. - Linux: create symlink /opt/telegraf-process-metrics -> /opt/telegraf - Windows: create symlink telegraf-process-metrics.exe -> telegraf.exe - Both configs: add new procstat entry for telegraf-process-metrics while keeping the existing telegraf entry
/proc/<pid>/exe resolves symlinks, so both telegraf processes appear as exe="telegraf". Switch to pattern-based cmdline matching: - Linux: "telegraf --" for main, "telegraf-process-metrics" for 2nd - Windows: "telegraf.exe --" for main, "telegraf-process-metrics" for 2nd
Contributor
Author
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Change `pattern = "fluentd"` to `exe = "fluentd"` so it only matches the fluentd supervisor process (PID name=fluentd), not both supervisor and worker. The ruby worker is already tracked by `exe = "ruby"`. This fixes duplicate values showing for ruby and fluentd on the dashboard. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
This reverts commit bc38d8c.
Contributor
Author
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
test
dashboard: https://dataexplorer.azure.com/dashboards/c2a4c90d-51d7-4647-b3a2-64b178283009?p-_startTime=3hours&p-_endTime=now&p-clusterIdForProcessMetrics=v-%2Fsubscriptions%2F9c17527c-af8f-4148-8019-27bada0845f7%2FresourceGroups%2Fzane-rg%2Fproviders%2FMicrosoft.ContainerService%2FmanagedClusters%2Fzane-ama-logs-helm-test&p-controllerType=v-DaemonSet&p-controllerType=v-DaemonSet-Windows&p-controllerType=v-PrometheusSidecar&p-controllerType=v-ReplicaSet&p-binSize=v-1m#edff4e67-d622-4655-a2dc-d702e96a4be2