Skip to content

feat(grafana)!: readd#200

Merged
dargmuesli merged 1 commit intobetafrom
feat/grafana/readd
May 13, 2025
Merged

feat(grafana)!: readd#200
dargmuesli merged 1 commit intobetafrom
feat/grafana/readd

Conversation

@dargmuesli
Copy link
Copy Markdown
Member

We had the Grafana integration here before but it has been removed due to some logical inaccuracies back then (#107). This PR readds the Grafana database and role so that it configures access permissions properly right away, meaning you can just start maevsi/stack and have working dashboard in the grafana service.

Test with maevsi/stack#200 (which targets beta too).

@dargmuesli dargmuesli changed the title feat(grafana): readd feat(grafana)!: readd May 13, 2025
@dargmuesli dargmuesli force-pushed the feat/grafana/readd branch from 28f6138 to 0fbae08 Compare May 13, 2025 04:11
@dargmuesli dargmuesli requested a review from sthelemann May 13, 2025 04:14
@dargmuesli dargmuesli merged commit 49aba67 into beta May 13, 2025
8 checks passed
@dargmuesli dargmuesli deleted the feat/grafana/readd branch May 13, 2025 04:14
@maevsi-bot
Copy link
Copy Markdown

🎉 This PR is included in version 8.0.0-beta.9 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@maevsi-bot
Copy link
Copy Markdown

🎉 This PR is included in version 8.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

SELECT 'CREATE DATABASE grafana OWNER "' || :'role_service_grafana_username' || '";'
WHERE NOT EXISTS (
SELECT FROM pg_database WHERE datname = 'grafana'
)\gexec
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cleverly done

Copy link
Copy Markdown
Contributor

@sthelemann sthelemann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants