MariaDB instead of MySQL #19

Closed
opened 2020-09-02 13:36:39 +00:00 by Torpi · 4 comments
Torpi commented 2020-09-02 13:36:39 +00:00 (Migrated from gitea.com)

Hello,

do you plan or would you accept a PR to add (or replace) MySQL by MariaDB? Or do you recommend PostgreSQL as the way to go instead?

Thanks.
Torpi

Hello, do you plan or would you accept a PR to add (or replace) MySQL by MariaDB? Or do you recommend PostgreSQL as the way to go instead? Thanks. Torpi
lunny commented 2020-09-02 14:59:59 +00:00 (Migrated from gitea.com)

We could support more databases.

We could support more databases.
luhahn commented 2020-09-03 06:44:15 +00:00 (Migrated from gitea.com)

We can easily add more databases as built in variants. However it is possbile to add other databases as external databases at the moment.

The examples in the README will be updated with: #18

gitea:
 database:
  builtIn:
   postgresql:
    enabled: false

 config:
  database:
   DB_TYPE: mysql
   HOST: 127.0.0.1:3306
   NAME: gitea
   USER: root
   PASSWD: gitea
   SCHEMA: gitea
We can easily add more databases as built in variants. However it is possbile to add other databases as external databases at the moment. The examples in the README will be updated with: #18 ```yaml gitea: database: builtIn: postgresql: enabled: false config: database: DB_TYPE: mysql HOST: 127.0.0.1:3306 NAME: gitea USER: root PASSWD: gitea SCHEMA: gitea ```
luhahn commented 2020-09-29 07:06:11 +00:00 (Migrated from gitea.com)

Added MariaDB support as builtIn in #33

Added MariaDB support as builtIn in #33
Torpi commented 2020-09-29 18:12:08 +00:00 (Migrated from gitea.com)

Wow! Thank you so much, I was about to do a PR for this one but unfortunately got impromptu personal work that delayed my initiative.

Wow! Thank you so much, I was about to do a PR for this one but unfortunately got impromptu personal work that delayed my initiative.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: lunny/helm-chart#19
No description provided.