Skip to content

Conversation

@axos88
Copy link

@axos88 axos88 commented Sep 1, 2014

This PR adds support for

  • installing from default debian repositories, rather than the postgresql repos
  • backward compatibility down to 9.1

Postgresql repositories do NOT have binaries for ARM architectures.
I need to use the packages supplied by debian themselves for this. The debian repos have only 9.1 installed.

@axos88 axos88 mentioned this pull request Sep 8, 2014
@guillaumededrie
Copy link

Hi @axos88,
I've been testing this PR on Debian 7.7, and when i try to start postgresql (or the Restart PostgreSQL task), i get this:

Starting PostgreSQL 9.1 database server: mainThe PostgreSQL server failed to start. Please check the log output. ... failed!

Do you have the same problem?

Regards.

@axos88
Copy link
Author

axos88 commented Dec 29, 2014

it was working on debian 7.5 for me. What's the actual error you get? (int he logs)

@guillaumededrie
Copy link

Hi @axos88,
Using Debian repo and PostgreSQL 9.1, i've got:

 PST LOG:  database system was shut down at 2014-12-30 02:42:43 PST
 PST LOG:  database system is ready to accept connections
 PST LOG:  autovacuum launcher started
 PST LOG:  incomplete startup packet
 PST LOG:  received fast shutdown request
 PST LOG:  aborting any active transactions
 PST LOG:  autovacuum launcher shutting down
 PST LOG:  shutting down
 PST LOG:  database system is shut down

I've got another issue, using PostgreSQL repo and 9.1 (seems not related imho).

Thx!

@guillaumededrie
Copy link

Ok, I tested and it was a problem with shared_buffers value, which was too high. Not really related to this PR.

Sry.

@guillaumededrie
Copy link

Hum, seems we have to be careful: http://leopard.in.ua/2013/09/05/postgresql-sessting-shared-memory/ (the PostgreSQL 9.3 section).

Choose a reason for hiding this comment

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

As Ansible handle mutliple booleans forms (True/yes/…) you should remove the == True part.

Copy link
Author

Choose a reason for hiding this comment

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

Nice catch.

Choose a reason for hiding this comment

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

@axos88, sry i missed this one …

@pjan
Copy link
Contributor

pjan commented Jan 28, 2015

What's the status on this? I'm short on time to do a proper review on a different architectures.

Sorry for the delay as well. Same reason.

@clonedagain
Copy link
Contributor

👎 now that #63 is merged, the part regarding the choice of repo is obsolete, and #65 would a be cleaner way to disable the apt tasks if you want to.

By the way, I don't see what in this PR is ARM or Debian specific...

@axos88 axos88 closed this Aug 3, 2017
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.

4 participants