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 pagerduty and prometheus integration info to support chart secrets #5234

Merged

Conversation

sgibson91
Copy link
Member

This information could be used by prometheus alertmanager to send automatic alerts

Set up following these instructions: https://www.pagerduty.com/docs/guides/prometheus-integration-guide/

related to #5062

This information could be used by prometheus alertmanager to send automatic alerts
Copy link

github-actions bot commented Dec 5, 2024

Merging this PR will trigger the following deployment actions.

Support and Staging deployments

Cloud Provider Cluster Name Upgrade Support? Reason for Support Redeploy Upgrade Staging? Reason for Staging Redeploy
gcp leap Yes Support helm chart has been modified No
aws gridsst Yes Support helm chart has been modified No
kubeconfig queensu Yes Support helm chart has been modified No
gcp 2i2c-uk Yes Support helm chart has been modified No
aws maap Yes Support helm chart has been modified No
aws nasa-cryo Yes Support helm chart has been modified No
gcp 2i2c Yes Support helm chart has been modified No
gcp awi-ciroh Yes Support helm chart has been modified No
aws jupyter-health Yes Support helm chart has been modified No
gcp dubois Yes Support helm chart has been modified No
gcp hhmi Yes Support helm chart has been modified No
aws earthscope Yes Support helm chart has been modified No
kubeconfig utoronto Yes Support helm chart has been modified No
aws smithsonian Yes Support helm chart has been modified No
aws victor Yes Support helm chart has been modified No
aws openscapes Yes Support helm chart has been modified No
aws kitware Yes Support helm chart has been modified No
aws nmfs-openscapes Yes Support helm chart has been modified No
aws nasa-ghg Yes Support helm chart has been modified No
aws catalystproject-africa Yes Support helm chart has been modified No
aws 2i2c-aws-us Yes Support helm chart has been modified No
gcp catalystproject-latam Yes Support helm chart has been modified No
aws strudel Yes Support helm chart has been modified No
aws projectpythia Yes Support helm chart has been modified No
gcp cloudbank Yes Support helm chart has been modified No
aws jupyter-meets-the-earth Yes Support helm chart has been modified No
gcp pangeo-hubs Yes Support helm chart has been modified No
aws ubc-eoas Yes Support helm chart has been modified No
aws nasa-veda Yes Support helm chart has been modified No
aws opensci Yes Support helm chart has been modified No

Production deployments

No production hub upgrades will be triggered

@sgibson91 sgibson91 added the deployer:skip-deploy Skips deployment of anything (support, staging, prod) label Dec 5, 2024
@sgibson91 sgibson91 merged commit 974676f into 2i2c-org:main Dec 5, 2024
38 checks passed
@sgibson91 sgibson91 deleted the pagerduty-prometheus-integration branch December 5, 2024 10:47
Copy link

github-actions bot commented Dec 5, 2024

🎉🎉🎉🎉

Monitor the deployment of the hubs here 👉 https://github.com/2i2c-org/infrastructure/actions/runs/12178194167

@yuvipanda
Copy link
Member

I think this needs a possible addition to the json schema as well - trying to run deploy-support nasa-veda fails with:

Error: UPGRADE FAILED: values don't meet the specifications of the schema(s) in the following chart(s):
support:
- (root): Additional property pagerduty-prometheus-integration is not allowed

@sgibson91
Copy link
Member Author

@yuvipanda it's getting changed in another PR where Tarashish is actually turning on alert manager

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployer:skip-deploy Skips deployment of anything (support, staging, prod)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants