Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Refactor DB Roles #251

Draft
wants to merge 48 commits into
base: master
Choose a base branch
from
Draft

Refactor DB Roles #251

wants to merge 48 commits into from
+156 −140

Conversation

owine
Copy link
Collaborator

@owine owine commented Apr 29, 2023 •
edited
Loading

Deploy database roles with a unique database instance

Also adds multi instance to Wordpress. Closes #249

@owine owine requested a review from saltydk April 29, 2023 20:46
@saltydk
Copy link
Member

saltydk commented Apr 29, 2023

I'll try getting through it tomorrow bud.

saltydk
saltydk requested changes Apr 30, 2023
View reviewed changes
roles/bookstack/tasks/main.yml Outdated Show resolved Hide resolved
roles/invoiceninjav5/tasks/main.yml Outdated Show resolved Hide resolved
roles/rocketchat/tasks/main.yml Outdated Show resolved Hide resolved
roles/tandoor/tasks/main.yml Outdated Show resolved Hide resolved
roles/teamspeak/tasks/main.yml Outdated Show resolved Hide resolved
roles/wordpress/tasks/main2.yml Outdated Show resolved Hide resolved
saltydk
saltydk reviewed May 2, 2023
View reviewed changes
Copy link
Member

@saltydk saltydk left a comment

Choose a reason for hiding this comment

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

LGTM.

I haven't tested the roles themselves so we should do that before any actual merge.

@owine
Copy link
Collaborator Author

owine commented May 3, 2023 •
edited
Loading

Testing notes

MARIADB

  • bookstack
  • docspell
  • gitea
  • invoiceninja
  • koel
  • nextcloud
  • photoprism
  • plexshare
  • teamspeak
  • wordpress (multi)

POSTGRES

  • joplin
  • miniflux
  • n8n
  • paperless_ngx (also redis)
  • tandoor
  • wikijs

MONGODB

  • rocketchat
  • your_spotify - unable to test, I do not have a Spotify account

REDIS

  • tubearchivist
  • paperless_ngx

@owine
Copy link
Collaborator Author

owine commented May 3, 2023

Struggling with Paperless, RocketChat and TeamSpeak - if someone else can help test troubleshoot I'd appreciate it.

I am also unable to test your_spotify.

@owine owine force-pushed the master branch 3 times, most recently from f86ea16 to c115f3b Compare September 18, 2023 15:46
@saltydk saltydk force-pushed the master branch 2 times, most recently from ba4842d to 6af13e1 Compare December 25, 2023 16:03
owine and others added 29 commits January 30, 2024 16:27
@owine
Wordpress: Set MariaDB instance
880c1a2
@owine
Bookstack: Fixup DB creation
345fd7d
@owine
Gitea: Fixup DB creation
2008d56
@owine
Invoiceninja: Fixup DB creation
1f85964
@owine
Teamspeak: Fixup DB creation
74e7d47
@owine
Wordpress: Fixup DB creation
69868ce
@owine
Bookstack: Revert extra DB task
f44bc79
@owine
Gitea: Revert extra DB task
2d7e96b
@owine
Invoiceninja: Revert extra DB task
833d510
@owine
Teamspeak: Revert extra DB task
fa041fb
@owine
Wordpress: Revert extra DB task
944d673
@owine
Various: Use _name for DB envs
f9952a2
@owine
Various: Remove _name vars for db task includes
24a7269
@RaneyDazed @owine
Wordpress: Multiple instance support. Closes #249
9ca369a
@saltydk @owine
invoiceninja: fix spacing
e5257fe
@saltydk @owine
wordpress: fix spacing
7b25566
@owine
Remove unneeded default db variables
ef10061
@owine
Photoprism: DB refactor
cb386a0
@owine
Bookstack: Don't set Docker user, opt folder not owned by SB user
173749f
@owine
Photoprism: Fix env
d268364
@owine
Invoiceninja: Don't set Docker user, opt folder not owned by SB user
f60affa
@owine
Fix mariadb password env
0f86555
@owine
Wordpress: Change loop var
24e258d
@owine
Invoiceninja: Provision db
4a41385
@owine
Teamspeak: Provision db
a02ddd0
@owine
Nextcloud: Provision db
9bc2c96
@owine
Create DBs are DB user, not root
ad1d6ab
@owine
Fix usernames
d22106e
@owine
Don't create DBs
804dd74
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@saltydk saltydk saltydk requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@owine @saltydk @RaneyDazed