8000 fix: give grafana root access by tedim52 · Pull Request #2706 · kurtosis-tech/kurtosis · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix: give grafana root access #2706

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

Merged
merged 3 commits into from
Apr 15, 2025
Merged

fix: give grafana root access #2706

merged 3 commits into from
Apr 15, 2025

Conversation

tedim52
Copy link
Collaborator
@tedim52 tedim52 commented Apr 9, 2025

Description

A user reported the grafana container not having access to the bindmounted datasource yaml within the container. This was due to varying permissions from the original tmp file created by the CLI and inherited by the bindmounted file. To get around this, this change gives the grafana container root user which fixed their issue.

Is this change user facing?

YES

References

#2696 (comment)

@tedim52 tedim52 requested a review from mieubrisse April 9, 2025 17:04
@tedim52 tedim52 enabled auto-merge April 15, 2025 21:58
@tedim52 tedim52 added this pull request to the merge queue Apr 15, 2025
Merged via the queue into main with commit b8e2fab Apr 15, 2025
51 checks passed
@tedim52 tedim52 deleted the tedi/grafloki-perms-fix branch April 15, 2025 22:22
github-merge-queue bot pushed a commit that referenced this pull request Apr 18, 2025
🤖 I have created a release *beep* *boop*
---


##
[1.7.0](1.6.0...1.7.0)
(2025-04-17)


### Features

* add grafloki to kurtosis config
([#2707](#2707))
([b6794e1](b6794e1))
* export service logs (Kubenertes)
([#2693](#2693))
([c00f357](c00f357))
* grafloki start/stop
([#2696](#2696))
([53d823a](53d823a))
* service update
([#2689](#2689))
([4cda391](4cda391))


### Bug Fixes

* **backend/kubernetes:** use default KUBECONFIG resolution
([#2672](#2672))
([379e49b](379e49b))
* docker auth wasn't being used by engine and API
([#2699](#2699))
([4eea787](4eea787))
* give grafana root access
([#2706](#2706))
([b8e2fab](b8e2fab))
* remove k8s ingress on stop user services
([#2715](#2715))
([e31c0d4](e31c0d4))
* return empty deployment if not found
([#2716](#2716))
([451c70e](451c70e))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
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
0