Feature/postgre by Pierre-Demessence · Pull Request #375 · Unity-Developer-Community/UDC-Bot

Pierre-Demessence · 2026-04-01T02:59:11Z

No description provided.

Co-authored-by: Copilot <copilot@github.com>

github-actions · 2026-04-01T02:59:19Z

🚀 Deploy this PR

To deploy this branch to the dev environment, go to the Deploy PR to Dev workflow and click Run workflow with this PR number.

This will build a Docker image from your branch, push it to GHCR, and trigger an ArgoCD sync.

Copilot

Pull request overview

This PR migrates the bot’s database layer and deployment configuration from MySQL to PostgreSQL (including updated connection strings, K8s manifests, and Docker Compose), and replaces the MySQL EVENT-based karma reset mechanism with an in-app scheduled service.

Changes:

Switched DB connectivity from MySql.Data/MySqlConnection to Npgsql + Insight.Database PostgreSQL provider and updated SQL for PostgreSQL.
Added KarmaResetService to handle weekly/monthly/yearly karma resets (replacing MySQL EVENT scheduler).
Updated dev/prod Kubernetes manifests, secrets, and docs to use PostgreSQL + pgAdmin and added PostgreSQL backup manifests.

Reviewed changes

Copilot reviewed 26 out of 26 changed files in this pull request and generated 8 comments.

Show a summary per file

File	Description
README.md	Updates documentation for PostgreSQL setup and formatting adjustments.
k8s/prod/postgresql.yaml	Replaces MySQL Deployment/Service/PVC with PostgreSQL equivalents in prod.
k8s/prod/postgresql-backup.yaml	Adds a PostgreSQL backup Deployment for prod.
k8s/prod/pgadmin.yaml	Replaces phpMyAdmin with pgAdmin in prod.
k8s/prod/mysql-backup.yaml	Removes the MySQL backup Deployment from prod.
k8s/prod/external-secrets.yaml	Renames MySQL secrets to PostgreSQL + adds pgAdmin credentials secret in prod.
k8s/prod/bot.yaml	Updates bot DB secret ref and init container to wait for PostgreSQL in prod.
k8s/prod/bot-config.yaml	Updates prod `Settings.json` connection string to PostgreSQL.
k8s/dev/postgresql.yaml	Replaces MySQL Deployment/Service/PVC with PostgreSQL equivalents in dev.
k8s/dev/postgresql-backup.yaml	Adds a PostgreSQL backup Deployment for dev.
k8s/dev/pgadmin.yaml	Replaces phpMyAdmin with pgAdmin in dev.
k8s/dev/mysql-backup.yaml	Removes the MySQL backup Deployment from dev.
k8s/dev/external-secrets.yaml	Renames MySQL secrets to PostgreSQL + adds pgAdmin credentials secret in dev.
k8s/dev/bot.yaml	Updates bot DB secret ref and init container to wait for PostgreSQL in dev.
k8s/dev/bot-config.yaml	Updates dev `Settings.json` connection string to PostgreSQL.
docs/plans/done/mysql-to-postgresql-migration.md	Adds a “done” migration plan/summary document.
docker-compose.yml	Switches local dev stack from MySQL + phpMyAdmin to PostgreSQL + pgAdmin.
DiscordBot/Settings/Settings.example.json	Updates example DB connection string for PostgreSQL.
DiscordBot/Services/KarmaResetService.cs	Adds polling-based karma reset scheduler + meta table tracking.
DiscordBot/Services/DatabaseService.cs	Switches DB connections to Npgsql and rewrites schema creation for PostgreSQL.
DiscordBot/Program.cs	Registers and eagerly resolves `KarmaResetService`.
DiscordBot/Modules/ModerationModule.cs	Removes unused BouncyCastle import and minor formatting.
DiscordBot/Extensions/UserDBRepository.cs	Updates INSERT patterns and rank ordering for PostgreSQL.
DiscordBot/Extensions/DBConnectionExtension.cs	Replaces `SHOW COLUMNS` with `information_schema` query.
DiscordBot/Extensions/CasinoRepository.cs	Updates INSERT patterns and JSON handling for PostgreSQL.
DiscordBot/DiscordBot.csproj	Removes MySql.Data and adds Insight.Database PostgreSQL provider package.

Comments suppressed due to low confidence (1)

DiscordBot/Extensions/CasinoRepository.cs:16

Same identifier-casing issue as ServerUserRepo: these queries use unquoted identifiers from CasinoProps, but DatabaseService creates the casino tables with quoted mixed-case column names. In PostgreSQL, unquoted identifiers are folded to lowercase and won't match quoted mixed-case columns; align schema + query quoting/casing consistently.

    [Sql($@"
    INSERT INTO {CasinoProps.CasinoTableName} ({CasinoProps.UserID}, {CasinoProps.Tokens}, {CasinoProps.CreatedAt}, {CasinoProps.UpdatedAt}, {CasinoProps.LastDailyReward}) 
    VALUES (@{CasinoProps.UserID}, @{CasinoProps.Tokens}, @{CasinoProps.CreatedAt}, @{CasinoProps.UpdatedAt}, @{CasinoProps.LastDailyReward})
    RETURNING *")]
    Task<CasinoUser> InsertCasinoUser(CasinoUser user);

    [Sql($"SELECT * FROM {CasinoProps.CasinoTableName} WHERE {CasinoProps.UserID} = @userId")]
    Task<CasinoUser> GetCasinoUser(string userId);

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

# Conflicts: # README.md

Pierre-Demessence · 2026-04-03T17:22:45Z

/build

github-actions · 2026-04-03T17:25:41Z

🐳 Build complete!

Image: ghcr.io/unity-developer-community/udc-bot-dev:832a135

To deploy with manifest changes:

Update k8s/dev/bot.yaml image tag to 832a135 on your branch
Switch ArgoCD dev target revision to your branch

Co-authored-by: Copilot <copilot@github.com>

… for prod

…ockercompose

…'t support ulong

Pierre-Demessence · 2026-04-03T19:59:16Z

/build

github-actions · 2026-04-03T20:01:38Z

🐳 Build complete!

Image: ghcr.io/unity-developer-community/udc-bot-dev:17fc156

To deploy with manifest changes:

Update k8s/dev/bot.yaml image tag to 17fc156 on your branch
Switch ArgoCD dev target revision to your branch

Co-authored-by: Copilot <copilot@github.com>

…ntials Co-authored-by: Copilot <copilot@github.com>

…eserve data during migration - Rename TransactionType enum to TransactionKind (avoids DB column name collision) - Replace Type (integer) column with TransactionType (varchar) to match MySQL - Replace DetailsJson (jsonb) column with Description (text) to match MySQL - Add TargetUserID column for migration compatibility - Update CasinoProps constants, repository SQL, and all consumer references

- Create mysql-to-postgresql-changes.md with full schema, code, and infra details - Update data-migration-mysql-to-postgresql.md with working pgloader config - Delete outdated mysql-to-postgresql-migration.md summary - Update docs/INDEX.md with new doc links

- Add token_transactions to table matching regex (dev + prod) - Add BEFORE LOAD: DROP+CREATE token_transactions with MySQL-compatible schema - Ensures existing PG tables with old column names are replaced before import

Pierre-Demessence · 2026-04-04T01:34:17Z

/build

github-actions · 2026-04-04T01:36:47Z

🐳 Build complete!

Image: ghcr.io/unity-developer-community/udc-bot-dev:d6223c0

To deploy with manifest changes:

Update k8s/dev/bot.yaml image tag to d6223c0 on your branch
Switch ArgoCD dev target revision to your branch

Co-authored-by: Copilot <copilot@github.com>

Pierre-Demessence and others added 2 commits April 1, 2026 03:54

migrate from MySQL to PostgreSQL

9aaacc2

Co-authored-by: Copilot <copilot@github.com>

chore(k8s): remove MySQL configurations and update PostgreSQL images

a9f1915

Copilot AI review requested due to automatic review settings April 1, 2026 02:59

Copilot started reviewing on behalf of Pierre-Demessence April 1, 2026 02:59 View session

Copilot AI reviewed Apr 1, 2026

View reviewed changes

Pierre-Demessence and others added 7 commits April 1, 2026 18:11

Merge branch 'master' into feature/postgre

cbeb692

feat(k8s): add AWS credentials for PostgreSQL backups

8b80c66

Update README.md

805bc64

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

docs(README): update database container command in instructions

9c45b01

refactor(database): simplify SQL queries in services

44caba0

chore(deps): update Insight.Database to version 8.0.6

1811d8d

Merge branch 'master' into feature/postgre

832a135

# Conflicts: # README.md

Pierre-Demessence and others added 6 commits April 3, 2026 19:27

chore(k8s): update bot image to specific version 832a135

5877953

chore(k8s): remove unnecessary middleware annotation from Ingress

6dbfa78

feat(pgadmin): add configuration for PostgreSQL connection

bec3057

Co-authored-by: Copilot <copilot@github.com>

feat(pgadmin): add PostgreSQL server configuration and init container…

20cfffa

… for prod

feat(pgadmin): add pgAdmin server configuration and credentials for d…

f0c6e41

…ockercompose

refactor(casino): change types from ulong to long since postgre doesn…

17fc156

…'t support ulong

Pierre-Demessence and others added 7 commits April 3, 2026 22:05

fix(bot): update bot image to latest version 17fc156

ef5497c

feat(adminer): add deployment, service, and ingress for adminer for dev

f30e121

Co-authored-by: Copilot <copilot@github.com>

Added adminer for prod and docker-compose

e9c7811

Co-authored-by: Copilot <copilot@github.com>

Removed pgadmin

0c0f359

feat(migration): add MySQL to PostgreSQL migration plan

1ac55cd

Co-authored-by: Copilot <copilot@github.com>

feat(migration): update pgloader script to use placeholders for crede…

38c6ce1

…ntials Co-authored-by: Copilot <copilot@github.com>

Pierre-Demessence added 2 commits April 4, 2026 03:32

feat(k8s): include token_transactions in pgloader migration configs

d6223c0

- Add token_transactions to table matching regex (dev + prod) - Add BEFORE LOAD: DROP+CREATE token_transactions with MySQL-compatible schema - Ensures existing PG tables with old column names are replaced before import

Pierre-Demessence and others added 3 commits April 4, 2026 03:37

chore(k8s): update bot image to version d6223c0

7dbc47b

rework doc

46ede2c

Co-authored-by: Copilot <copilot@github.com>

fix(docs): clarify handling of birthday column in migration

9203c79

Co-authored-by: Copilot <copilot@github.com>

Pierre-Demessence merged commit c7d4b97 into master Apr 6, 2026
5 checks passed

Pierre-Demessence deleted the feature/postgre branch April 6, 2026 16:08

Pierre-Demessence restored the feature/postgre branch April 6, 2026 16:08

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature/postgre#375

Feature/postgre#375
Pierre-Demessence merged 27 commits into
masterfrom
feature/postgre

Pierre-Demessence commented Apr 1, 2026

Uh oh!

github-actions Bot commented Apr 1, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Pierre-Demessence commented Apr 3, 2026

Uh oh!

github-actions Bot commented Apr 3, 2026

Uh oh!

Pierre-Demessence commented Apr 3, 2026

Uh oh!

github-actions Bot commented Apr 3, 2026

Uh oh!

Pierre-Demessence commented Apr 4, 2026

Uh oh!

github-actions Bot commented Apr 4, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

Pierre-Demessence commented Apr 1, 2026

Uh oh!

github-actions Bot commented Apr 1, 2026

🚀 Deploy this PR

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Pierre-Demessence commented Apr 3, 2026

Uh oh!

github-actions Bot commented Apr 3, 2026

Uh oh!

Pierre-Demessence commented Apr 3, 2026

Uh oh!

github-actions Bot commented Apr 3, 2026

Uh oh!

Pierre-Demessence commented Apr 4, 2026

Uh oh!

github-actions Bot commented Apr 4, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants