search

grafana / grafana-operator

An operator for Grafana that installs and manages Grafana instances, Dashboards and Datasources through Kubernetes/OpenShift CRs
https://grafana.github.io/grafana-operator/
Apache License 2.0
863 stars 384 forks source link

fix: use correct folder uid to update permissions #1602

Closed timp87 closed 2 months ago

timp87 commented 2 months ago

Currently grafanafolder controller tries to update folder permission using CR UID no matter if the folder was created by dashboard controller earlier.

CLAassistant commented 2 months ago

CLA assistant check
All committers have signed the CLA.

theSuess commented 2 months ago

Good catch! I'll merge this in after #1600 with some minor adjustments

theSuess commented 2 months ago

Moved the reassignment of the uid in the existing conditional to prevent adding another conditional & to use the correct uid in case we are using a preexisting folder