Skip to content

fix: give grafana root access#2706

Merged
tedim52 merged 3 commits intomainfrom
tedi/grafloki-perms-fix
Apr 15, 2025
Merged

fix: give grafana root access#2706
tedim52 merged 3 commits intomainfrom
tedi/grafloki-perms-fix

Conversation

@tedim52
Copy link
Copy Markdown
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
Comment thread cli/cli/helpers/grafloki/docker_grafloki.go
@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