Skip to content

metrics: add missing metrics for cilium agent api handler #10376

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Feb 28, 2020
Merged

metrics: add missing metrics for cilium agent api handler #10376

merged 1 commit into from
Feb 28, 2020

Conversation

fristonio
Copy link
Member

@fristonio fristonio commented Feb 28, 2020 •
edited
Loading

Fixes #10338

image

image

This change is Reviewable

@fristonio
metrics: add missing metrics for cilium agent api handler
22103da 
Signed-off-by: Deepesh Pathak <deepshpathak@gmail.com>
@fristonio fristonio requested a review from aanm February 28, 2020 00:33
@fristonio fristonio requested a review from a team as a code owner February 28, 2020 00:33
@maintainer-s-little-helper
Copy link

Release note label not set, please set the appropriate release note.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 45.522% when pulling 22103da on fristonio/fix-10338 into 64c4792 on master.

@aanm aanm added needs-backport/1.7 release-note/bug This PR fixes an issue in a previous release of Cilium. labels Feb 28, 2020
@aanm
Copy link
Member

aanm commented Feb 28, 2020

test-me-please

@fristonio
Copy link
Member Author

test-me-please

@aanm aanm merged commit 6414719 into master Feb 28, 2020
@aanm aanm deleted the fristonio/fix-10338 branch February 28, 2020 14:48
@brb brb mentioned this pull request Mar 2, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aanm aanm aanm approved these changes

Assignees
No one assigned
Labels
release-note/bug This PR fixes an issue in a previous release of Cilium.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

cilium_agent_api_process_time_seconds metrics is missing
5 participants
@fristonio @coveralls @aanm @brb @joestringer