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

[lgtm-distributed] Add Grafana OnCall to the chart dependencies. #3183

Merged
merged 18 commits into from
Jul 5, 2024

Conversation

Sheikh-Abubaker
Copy link
Collaborator

Fixes #2698

@Sheikh-Abubaker Sheikh-Abubaker requested a review from a team as a code owner June 22, 2024 02:46
Signed-off-by: Sheikh-Abubaker <[email protected]>
Signed-off-by: Sheikh-Abubaker <[email protected]>
Signed-off-by: Sheikh-Abubaker <[email protected]>
Signed-off-by: Sheikh-Abubaker <[email protected]>
Signed-off-by: Sheikh-Abubaker <[email protected]>
@timberhill
Copy link
Contributor

This is very nice, is there anyone else that could review this? @zalegrala

@zalegrala
Copy link
Contributor

Not sure who maintains this chart. I don't see owners in .github/CODEOWNERS

@timberhill
Copy link
Contributor

@zalegrala I have created it and asked to add myself to the list of maintainers in the chart, but nothing about codeowners. I guess it would be good to add 2+ people. Here is the initial PR, which was reviewed by @zanhsieh (who might have some thoughts on this).

I am not part of the org, so not sure where to go from here :)

@Sheikh-Abubaker
Copy link
Collaborator Author

@timberhill if maintainers are needed I could come up as one!

@Sheikh-Abubaker Sheikh-Abubaker merged commit 0cd31ea into grafana:main Jul 5, 2024
6 checks passed
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.

[lgtm-distributed] Add optional Grafana OnCall to the chart
4 participants