-
Notifications
You must be signed in to change notification settings - Fork 241
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
feat: add monitoring configuration to acryl-datahub-actions subchart #473
base: master
Are you sure you want to change the base?
feat: add monitoring configuration to acryl-datahub-actions subchart #473
Conversation
410f585
to
2bd2e47
Compare
This PR is stale. We will close it in 30 days if there is no comment or activity. If you want feedback but not able to get it on github please head to #contribute channel in slack at https://slack.datahubproject.io. |
+1 I'm also interested on this! |
2bd2e47
to
99d94bb
Compare
This PR is stale. We will close it in 30 days if there is no comment or activity. If you want feedback but not able to get it on github please head to #contribute channel in slack at https://slack.datahubproject.io. |
To make sure that this PR is not automatically closed - here’s a comment. |
This PR is stale. We will close it in 30 days if there is no comment or activity. If you want feedback but not able to get it on github please head to #contribute channel in slack at https://slack.datahubproject.io. |
dac7948
to
7156a2d
Compare
This PR is stale. We will close it in 30 days if there is no comment or activity. If you want feedback but not able to get it on github please head to #contribute channel in slack at https://slack.datahubproject.io. |
7156a2d
to
bfba889
Compare
This PR enables the monitoring endpoint for the acryl-datahub-actions deployment using a new environment variable (see here: acryldata/datahub-actions#118) in case monitoring is enabled globally.
This PR should not be merged until the other PR in the datahub-actions repository has been merged and version v0.0.17 of the image is available.
Checklist