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

Add ingress_controller_configuration_push_count.failure_reason label description #4500

Merged
merged 8 commits into from
Sep 29, 2022

Conversation

czeslavo
Copy link
Contributor

@czeslavo czeslavo commented Sep 23, 2022

Summary

It adds a description regarding ingress_controller_configuration_push_count.failure_reason metric label. Label was released in KIC 2.7.0 therefore it's conditionally rendered using {% if_version gte:2.7.x %}.

Reason

Kong/kubernetes-ingress-controller#2484
Depends on: Kong/kubernetes-ingress-controller#2965

Testing

https://deploy-preview-4500--kongdocs.netlify.app/kubernetes-ingress-controller/2.7.x/references/prometheus/

@czeslavo czeslavo added do not merge Issues/ PRs whose changes should not be merged at this time review:general Review for general accuracy and presentation. Does the doc work? Does it output correctly? labels Sep 23, 2022
@netlify
Copy link

netlify bot commented Sep 23, 2022

Deploy Preview for kongdocs ready!

Name Link
🔨 Latest commit b317f2a
🔍 Latest deploy log https://app.netlify.com/sites/kongdocs/deploys/633566bd6c81200008032de5
😎 Deploy Preview https://deploy-preview-4500--kongdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@czeslavo czeslavo requested a review from a team September 23, 2022 11:21
@czeslavo czeslavo marked this pull request as ready for review September 26, 2022 07:46
@czeslavo czeslavo requested a review from a team as a code owner September 26, 2022 07:46
@czeslavo czeslavo marked this pull request as draft September 26, 2022 07:46
@czeslavo czeslavo marked this pull request as ready for review September 27, 2022 13:05
@czeslavo czeslavo added team-k8s Kubernetes task tracking (including KIC,KGO) and removed do not merge Issues/ PRs whose changes should not be merged at this time labels Sep 27, 2022
@cloudjumpercat cloudjumpercat self-assigned this Sep 27, 2022
* [workqueue metrics definition](https://github.com/kubernetes/component-base/blob/release-1.20/metrics/prometheus/workqueue/metrics.go#L29)
`ingress_controller_configuration_push_count` provides the number of successful or failed configuration pushes to {{site.base_gateway}}.

This metric expects that the following parameters are configured:
Copy link
Contributor

Choose a reason for hiding this comment

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

@czeslavo I added this to each metric section because the full metric example (ex. ingress_controller_configuration_push_count[success=true|false][protocol=db-less|deck]) was too long for the page. Is this accurate to call these "parameters"? And would customers know where to put these values without the example (ex. ingress_controller_configuration_push_count[success=true|false][protocol=db-less|deck])?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

These parameters are called metric's labels. I reworded the sentences to specifically use this term (b317f2a). They are populated by us while reporting the metric, and the user can inspect them. I think that it's going to be enough to have all possible values of every label explained in the way you've structured that. 👍

Copy link
Contributor

@cloudjumpercat cloudjumpercat left a comment

Choose a reason for hiding this comment

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

LGTM!

@cloudjumpercat cloudjumpercat merged commit fa5774e into main Sep 29, 2022
@cloudjumpercat cloudjumpercat deleted the kic-push-failure-label branch September 29, 2022 13:47
rainest pushed a commit that referenced this pull request Oct 3, 2022
…description (#4500)

* Add ingress_controller_configuration_push_count.failure_reason label description

* Add network label value

* point

* drop unnecessary row

* Fix table formatting and copyedit

* Add lead in sentence to bullet points

* add types, parameters -> labels

Co-authored-by: Diana <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review:general Review for general accuracy and presentation. Does the doc work? Does it output correctly? team-k8s Kubernetes task tracking (including KIC,KGO)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants