Skip to content

docs: Update metrics.md to add webhook controller metrics examples#1321

Merged
aramase merged 8 commits intoAzure:mainfrom
kristianjaeger:patch-1
Mar 27, 2025
Merged

docs: Update metrics.md to add webhook controller metrics examples#1321
aramase merged 8 commits intoAzure:mainfrom
kristianjaeger:patch-1

Conversation

@kristianjaeger
Copy link
Contributor

Adding a few webhook controller metric examples. Thanks.

Reason for Change:

Adds examples of useful metrics for users of workload identity.

No.

No.

Requirements

  • [X ] squashed commits
  • [X ] included documentation
  • [X ] added unit tests and e2e tests (if applicable).

Issue Fixed:

Please answer the following questions with yes/no:

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?

  • yes
  • [X ] no

Notes for Reviewers:
Thanks for considering.

Adding a few webhook controller metric examples. Thanks.
@kristianjaeger kristianjaeger changed the title Update metrics.md Update metrics.md to add webhook controller metrics examples Apr 12, 2024
@kristianjaeger kristianjaeger changed the title Update metrics.md to add webhook controller metrics examples feat: Update metrics.md to add webhook controller metrics examples Apr 12, 2024
@aramase aramase changed the title feat: Update metrics.md to add webhook controller metrics examples docs: Update metrics.md to add webhook controller metrics examples Mar 27, 2025
kristianjaeger and others added 2 commits March 27, 2025 15:32
Co-authored-by: Anish Ramasekar <anish.ramasekar@gmail.com>
kristianjaeger and others added 4 commits March 27, 2025 15:56
Co-authored-by: Anish Ramasekar <anish.ramasekar@gmail.com>
Co-authored-by: Anish Ramasekar <anish.ramasekar@gmail.com>
Adding link for metrics reference
@kristianjaeger kristianjaeger requested a review from aramase March 27, 2025 23:15
@aramase
Copy link
Member

aramase commented Mar 27, 2025

/azp run

@azure-pipelines
Copy link

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

Copy link
Member

@aramase aramase left a comment

Choose a reason for hiding this comment

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

Thank you for the PR!

LGTM!

@aramase aramase merged commit 10168bc into Azure:main Mar 27, 2025
8 checks passed
@kristianjaeger
Copy link
Contributor Author

Thanks, @aramase ! Cheers.

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.

3 participants