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

chore: postgres-fine-tune #2147

Merged
merged 14 commits into from
Oct 28, 2024
Merged

chore: postgres-fine-tune #2147

merged 14 commits into from
Oct 28, 2024

Conversation

mishraomp
Copy link
Collaborator

@mishraomp mishraomp commented Oct 27, 2024 •
edited by github-actions bot
Loading

Postgres fine tuning based on below
https://vladmihalcea.com/postgresql-performance-tuning-settings/

Thanks for the PR!

Deployments, as required, will be available below:

Please create PRs in draft mode. Mark as ready to enable:

After merge, new images are deployed in:

@mishraomp
fix: misc
4145b71 
1. Helm Upgrade of crunchy
2. Memory adjustment for flyway to get rid of OOMKilled

Signed-off-by: OMPRAKASH MISHRA <[email protected]>
@mishraomp
Merge branch 'main' of https://github.com/bcgov/quickstart-openshift …
c6603b0 
…into fix/misc
@mishraomp
chore: postgres fine-tuning.
49f1497 
Signed-off-by: OMPRAKASH MISHRA <[email protected]>
@mishraomp
make max_db_connections int
1d93742 
Signed-off-by: OMPRAKASH MISHRA <[email protected]>
@mishraomp
make max_db_connections string
fcbdbd8 
Signed-off-by: OMPRAKASH MISHRA <[email protected]>
@mishraomp mishraomp marked this pull request as draft October 27, 2024 19:13
@mishraomp mishraomp marked this pull request as draft October 27, 2024 19:13
@mishraomp mishraomp marked this pull request as ready for review October 27, 2024 23:40
@mishraomp
fix: swagger docs path
8dd5a5e 
Signed-off-by: OMPRAKASH MISHRA <[email protected]>
@mishraomp
Copy link
Collaborator Author

need to look into this,

2024-10-27 21:21:24,948 WARNING: postgresql parameter max_connections=20 failed validation, defaulting to 100
2024-10-27 21:21:25,003 INFO: Changed effective_io_concurrency from 1 to 20
2024-10-27 21:21:25,004 INFO: Changed shared_buffers from 16384 to 51Mi (restart might be required)
2024-10-27 21:21:25,004 INFO: Changed wal_buffers from 512 to 1Mi (restart might be required)
2024-10-27 21:21:27,308 WARNING: Removing integer parameter=shared_buffers from the config due to the invalid value=51Mi
2024-10-27 21:21:28,104 WARNING: Removing integer parameter=wal_buffers from the config due to the invalid value=1Mi

@mishraomp mishraomp marked this pull request as draft October 27, 2024 23:51
@mishraomp
Copy link
Collaborator Author

need to look into this,

2024-10-27 21:21:24,948 WARNING: postgresql parameter max_connections=20 failed validation, defaulting to 100
2024-10-27 21:21:25,003 INFO: Changed effective_io_concurrency from 1 to 20
2024-10-27 21:21:25,004 INFO: Changed shared_buffers from 16384 to 51Mi (restart might be required)
2024-10-27 21:21:25,004 INFO: Changed wal_buffers from 512 to 1Mi (restart might be required)
2024-10-27 21:21:27,308 WARNING: Removing integer parameter=shared_buffers from the config due to the invalid value=51Mi
2024-10-27 21:21:28,104 WARNING: Removing integer parameter=wal_buffers from the config due to the invalid value=1Mi

This is now fixed

DerekRoberts
DerekRoberts approved these changes Oct 28, 2024
View reviewed changes
Copy link
Member

@DerekRoberts DerekRoberts left a comment

Choose a reason for hiding this comment

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

You do not slow down! Good job. :)

@mishraomp mishraomp marked this pull request as ready for review October 28, 2024 14:46
@mishraomp
chore: update wal storage
1202f52 
Signed-off-by: OMPRAKASH MISHRA <[email protected]>
@mishraomp mishraomp marked this pull request as draft October 28, 2024 15:26
@mishraomp mishraomp marked this pull request as ready for review October 28, 2024 17:52
@mishraomp mishraomp merged commit cd6d7ca into main Oct 28, 2024
29 checks passed
@mishraomp mishraomp deleted the chore/postgres-fine-tune branch October 28, 2024 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DerekRoberts DerekRoberts DerekRoberts approved these changes

@franTarkenton franTarkenton Awaiting requested review from franTarkenton

@paulushcgcj paulushcgcj Awaiting requested review from paulushcgcj

@cberg-aot cberg-aot Awaiting requested review from cberg-aot

Assignees
No one assigned
Labels
None yet
Projects
DevOps (NR)
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mishraomp @DerekRoberts