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.

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

Update start.sh #1502

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

David-Angel
Copy link

@David-Angel David-Angel commented Jan 19, 2023

This is a solution for this issue:
CrunchyData/postgres-operator#3477

It turns off hugepages in the "postgresql.conf.sample" sample file.

Checklist:

  • [x ] Have you added an explanation of what your changes do and why you'd like them to be included?
  • [x ] Have you updated or added documentation for the change, as applicable?
  • [x ] Have you tested your changes on all related environments with successful results, as applicable?

Type of Changes:

  • [x ] Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

What is the current behavior? (link to any open issues here)
CrunchyData/postgres-operator#3477

What is the new behavior (if this is a feature change)?
The new behavior would not crash the pod when initdb is called.

Other information:

This is a temporary solution for this issue:
CrunchyData/postgres-operator#3477

It turns off hugepages in the "postgresql.conf.sample" sample file.
@linouxis9
Copy link

This issue has been mentionned many times as shown here: CrunchyData/postgres-operator#3477 (comment)
It would be amazing to finally have a fix be merged, thank you @David-Angel

@David-Angel David-Angel reopened this Feb 1, 2023
@David-Angel
Copy link
Author

David-Angel commented Feb 1, 2023

@linouxis9 thanks for your assistance.
I was working too late and mistakenly closed this.

This solution will work if crunchydata accepts it.

@David-Angel
Copy link
Author

Another solution to this is to create a custom docker build that replaces the sample config file.
Be sure to set the user "USER root" before the change and "USER 26" after.

This PR is still the best solution once accepted.

@David-Angel
Copy link
Author

David-Angel commented Jun 23, 2023

This may be resolved in the latest build by Crunchydata as they updated their documentation to say it is now disabled by default.
https://access.crunchydata.com/documentation/postgres-operator/latest/guides/huge-pages/

I'll verify and close out the defect if not commented on by Crunchydata

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.

2 participants