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 latest grafana 7.5.x version #299

Closed
wants to merge 1 commit into from
Closed

Add latest grafana 7.5.x version #299

wants to merge 1 commit into from

Conversation

Syphdias
Copy link

@Syphdias Syphdias commented Oct 7, 2022

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
  • If updating an existing entry, verify the SOURCE is still accurate and upstream hasn't been migrated to a new regitry or repo (if they've migrated, a new repo request to EIO is needed to comply with the SOURCE DESTINATION TAG pattern)
  • 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)

Types of Change

This version fixes an issue in Grafana (grafana/grafana#54535) that makes it impossible to use any input fields in certain browsers with certain versions.

Linked Issues

grafana/grafana#54535

Additional Notes

I will follow up with a PR to the monitoring chart in the chars repository. But this needs to be merged and executed first, of course. I will make a not in the other PR.

Final Checks after the PR is merged

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

PS: I would appreciate if you could label this PR hacktoberfest-accepted.

This version fixes an issue in Grafana (grafana/grafana#54535) that
makes it impossible to use any input fields in certain browsers with
certain versions.
@Syphdias
Copy link
Author

Hey, friendly reminder that this exists. 🙂
Maybe someone could take a look at the connected rancher/charts PRs as well.

@superseb
Copy link
Contributor

Requested review from responsible team in rancher/charts#2144 (comment)

@Syphdias
Copy link
Author

Syphdias commented Nov 8, 2022

Closing due to rancher/charts#2144 (comment) (rancher/charts#2162)

@Syphdias Syphdias closed this Nov 8, 2022
@Syphdias Syphdias deleted the add-latest-grafana-7.5.x branch November 8, 2022 12:21
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.

None yet

3 participants