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

fix: Updated doc for metric kyverno_policy_results_total to kyverno_policy_results #1325 #1326

Merged
merged 1 commit into from
Aug 13, 2024

Conversation

praddy26
Copy link
Contributor

@praddy26 praddy26 commented Aug 13, 2024

Related issue

Fixes #1325

Checklist

  • I have read the contributing guidelines.
  • I have inspected the website preview for accuracy.
  • I have signed off my issue.

Copy link

welcome bot commented Aug 13, 2024

Thanks for opening your first Pull Request here! Please check out our Contributing guidelines and confirm that you Signed off.

@realshuting
Copy link
Member

Please add close keyword ("close") to auto-close the issue.

@praddy26
Copy link
Contributor Author

Please add close keyword ("close") to auto-close the issue.

Where should it be added?

In the PR title?
In the comment?

@realshuting
Copy link
Member

Please add close keyword ("close") to auto-close the issue.

Where should it be added?

In the PR title? In the comment?

In the PR description.

@realshuting
Copy link
Member

You need to sign your commits https://github.com/kyverno/website/pull/1326/checks?check_run_id=28701366038.

@praddy26 praddy26 force-pushed the 1325-update-metric-doc branch from 0522bd4 to 0495d70 Compare August 13, 2024 10:47
Copy link
Member

@realshuting realshuting left a comment

Choose a reason for hiding this comment

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

/lgtm

@realshuting realshuting merged commit 301a004 into kyverno:main Aug 13, 2024
6 checks passed
Copy link

welcome bot commented Aug 13, 2024

Congratulations! 🎉

Great job merging your first Pull Request here! How awesome! If you are new to this project, feel free to join our Slack community.

200w

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.

[Bug] update Policy and Rule Execution metric's name
2 participants