pub-solar/infra
6
1
Fork 3
Code Issues 17 Pull requests 7 Actions Packages Projects 1 Releases Wiki Activity

feat: grafana + prometheus + loki on flora-6 #77

Merged
teutat3s merged 15 commits from feat/grafana into main 2023-12-13 23:21:32 +00:00
Conversation 1 Commits 15 Files changed 19 +1 -1
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 10bb3295de - Show all commits

2
hosts/flora-6/apps/grafana.nix
View file

@ -68,7 +68,7 @@
auth_url = "https://auth.pub.solar/realms/pub.solar/protocol/openid-connect/auth";
token_url = "https://auth.pub.solar/realms/pub.solar/protocol/openid-connect/token";
api_url = "https://auth.pub.solar/realms/pub.solar/protocol/openid-connect/userinfo";
role_attribute_path = "contains(roles[*], 'admin') && 'GrafanaAdmin' || contains(roles[*], 'editor') && 'Editor' || 'Viewer'";
role_attribute_path = "contains(roles[*], 'admin') && 'GrafanaAdmin' || 'Viewer'";
teutat3s marked this conversation as resolved Outdated
axeman commented 2023-12-08 15:33:34 +00:00
Outdated
Review
Copy link

This looks wrong, we only have the 'admin' role in keycloack. Perhaps we should remove the editor role, doesn't seem like it make sense for our org.

This looks wrong, we only have the 'admin' role in keycloack. Perhaps we should remove the editor role, doesn't seem like it make sense for our org.
axeman commented 2023-12-08 15:33:39 +00:00
Outdated
Review
Copy link

This looks wrong, we only have the 'admin' role in keycloack. Perhaps we should remove the editor role, probably doesn't make sense for our org.

This looks wrong, we only have the 'admin' role in keycloack. Perhaps we should remove the editor role, probably doesn't make sense for our org.
teutat3s commented 2023-12-13 16:53:00 +00:00
Outdated
Review
Copy link

Adressed in 10bb3295de.

Adressed in https://git.pub.solar/pub-solar/infra/commit/10bb3295de2f8622f9360cb310a396504d614a88.
allow_assign_grafana_admin = true;
};
};