-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Conversation
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]>
Signed-off-by: Sheikh-Abubaker <[email protected]>
Signed-off-by: Sheikh-Abubaker <[email protected]>
Signed-off-by: Sheikh-Abubaker <[email protected]>
…into oncall-pr
…into oncall-pr
This is very nice, is there anyone else that could review this? @zalegrala |
Not sure who maintains this chart. I don't see owners in |
@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 :) |
@timberhill if maintainers are needed I could come up as one! |
…into oncall-pr
Fixes #2698