Skip to content

chore: move committeeclash pg to host pg#556

Open
SilasPeters wants to merge 5 commits intomasterfrom
chore/move-committeeclash-postgres-to-host-pg
Open

chore: move committeeclash pg to host pg#556
SilasPeters wants to merge 5 commits intomasterfrom
chore/move-committeeclash-postgres-to-host-pg

Conversation

@SilasPeters
Copy link
Copy Markdown
Member

@SilasPeters SilasPeters commented Mar 9, 2026

Yay

@SilasPeters SilasPeters self-assigned this Mar 9, 2026
Comment on lines +34 to +38
- name: "restarting postgres"
ansible.builtin.systemd_service:
name: "postgresql"
state: "restarted"
listen: "restart postgres"
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This is intrusive though

@SilasPeters SilasPeters marked this pull request as ready for review March 23, 2026 19:59
@SilasPeters SilasPeters requested a review from KoenDR06 March 23, 2026 20:06
@SilasPeters
Copy link
Copy Markdown
Member Author

@rensvmoorsel once this PR is merged, we will require copying over all data from the containerisd postgress instance to the public postgres instance. A pg_dump could suffice, but we have to make sure not to override anything already existing in production. We could use staging as a test.

The database into which the data needs to be copied is best made by deplying the changes in this PR, meaning that there is a brief moment where committeeclash must be made unavailable through nginx (maintenance mode) because we then are manually filling the used production database which is then empty.

@KoenDR06
Copy link
Copy Markdown
Contributor

TODO: fix auth issue

@SilasPeters is this TODO resolved?

@SilasPeters
Copy link
Copy Markdown
Member Author

@KoenDR06 yes

@KoenDR06
Copy link
Copy Markdown
Contributor

While deploying the committeeclash role just now, the server seemed to not respond to any requests, I even got a 500 from koala. If this was an issue before this PR it's good to be merged in my opinion.

@SilasPeters
Copy link
Copy Markdown
Member Author

I experienced the same earlier. Not sure what causes it, I suspect a shortage of memory

Copy link
Copy Markdown
Contributor

@KoenDR06 KoenDR06 left a comment

Choose a reason for hiding this comment

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

Seems like it was already an issue, LGTM!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants