-
Notifications
You must be signed in to change notification settings - Fork 36
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
feat: onboard directus #639
Conversation
@@ -0,0 +1,22 @@ | |||
1. Get the application URL by running these commands: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rm this file
directus/terraform/kube/main.tf
Outdated
@@ -0,0 +1,10 @@ | |||
resource "kubernetes_namespace" "directus" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does directus not require S3 or another form of object storage?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They don't need it to get it work, but I'm planning to add a file storage to the bucket later along with postgres backend and oidc auth
We should definitely use the postgres operator to set up a proper db here as well, you should be able to see a lot of prior art using the |
Yes for sure! Might need some help for this maybe |
you can see some postgres related templating here: https://github.com/pluralsh/plural-artifacts/pull/635/files jupyterhub was weird because we had to feed it the entire postgres url, which isn't needed for directus, looks like you can just wire up these env vars: https://docs.directus.io/self-hosted/config-options.html to give it the right db configuration. You'll almost certainly need to add it looks like directus does require s3/gcs/etc so might want to revert the |
🎉 This PR is included in version 1.0.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Summary
Test Plan
Next
Checklist
plural from-grafana
to convert a grafana dashboard found via google to our CRDpluralsh/module-library
to accelerate this