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

Server failed to start on recent Ubuntu due to too low max_connections #14

Closed
dev-zero opened this issue May 7, 2019 · 4 comments · Fixed by #15
Closed

Server failed to start on recent Ubuntu due to too low max_connections #14

dev-zero opened this issue May 7, 2019 · 4 comments · Fixed by #15

Comments

@dev-zero
Copy link

dev-zero commented May 7, 2019

It seems that the most recent version of PostgreSQL in the ubuntu:rolling image increased the required amount of connections even more. In that image startup fails with the following message:

postgres: superuser_reserved_connections (3) plus max_wal_senders (10) must be less than max_connections (11)

My guess is that the number of superuser_reserved_connections got increased to 3, which makes to total amount of required connections now 13.

I'd recommend to increase it to 15.

@ltalirz fyi

@ltalirz
Copy link
Collaborator

ltalirz commented May 7, 2019

@jamesnunn By the way, is there a good reason for specifying a default number for the max allowed connections in pgtest?
What about not specifying it by default and simply letting the respective postgres version take care of it?

@dev-zero
Copy link
Author

dev-zero commented May 7, 2019

Btw, it would help a lot for debugging if in case of an error the self._log_file is read and appended to the output.

@ltalirz
Copy link
Collaborator

ltalirz commented May 7, 2019

Btw, it would help a lot for debugging if in case of an error the self._log_file is read and appended to the output.

@dev-zero I'm sure @jamesnunn would happily accept a PR for this

@ltalirz
Copy link
Collaborator

ltalirz commented May 27, 2019

@jamesnunn I'm going to prepare a PR for this - I'll be removing the max allowed connections default unless otherwise instructed

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 a pull request may close this issue.

2 participants