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 Prometheus Adapter v0.8.4 #122

Merged
merged 1 commit into from
Jun 9, 2021

Conversation

aiyengar2
Copy link
Contributor

@aiyengar2 aiyengar2 commented Jun 8, 2021

Pull Request Checklist

  • Change does not remove any existing Images or Tags in the images-list file
  • Change does not remove / overwrite exiting Images or Tags in Rancher DockerHub
  • New entries are in format SOURCE DESTINATION TAG
  • New entries are added to the correct section of the list (sorted lexicographically)
  • New entries have a repo created in Rancher Dockerhub (where the image will be mirrored to)
  • Changes to scripting or CI config have been tested to the best of your ability

Types of Change

Linked Issues

Related PR: rancher/charts#1250
Related Issue: rancher/rancher#32519

Additional Notes

Final Checks after the PR is merged

  • Confirm that you can pull the new images and tags from DockerHub

@aiyengar2 aiyengar2 requested a review from cbron as a code owner June 8, 2021 07:32
@@ -37,6 +37,7 @@ curlimages/curl rancher/curlimages-curl 7.73.0
curlimages/curl rancher/mirrored-curlimages-curl 7.70.0
curlimages/curl rancher/mirrored-curlimages-curl 7.73.0
directxman12/k8s-prometheus-adapter rancher/mirrored-directxman12-k8s-prometheus-adapter v0.8.3
directxman12/k8s-prometheus-adapter rancher/mirrored-directxman12-k8s-prometheus-adapter v0.8.4
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@cbron cbron merged commit 0c5ebe6 into rancher:master Jun 9, 2021
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.

2 participants