Skip to content

Ubuntu 18.04 support#420

Closed
nchudleigh wants to merge 8 commits intoANXS:masterfrom
nchudleigh:ubuntu-1804
Closed

Ubuntu 18.04 support#420
nchudleigh wants to merge 8 commits intoANXS:masterfrom
nchudleigh:ubuntu-1804

Conversation

@nchudleigh
Copy link
Copy Markdown
Contributor

@nchudleigh nchudleigh commented Mar 13, 2019

Based on the work in #384 by @irionr

A couple things to call out here:

Vagrant IPs

I have used 192.168.88.21 for the 1804 inventory rather than shifting all the IPs of the other vagrant images. I am not sure how much we care about the order of these things or if 21 is reserved for something else. Happy to change this.

Vagrant Image

The builded image for 18.04 works, I have run it locally with no issue. But I am unsure whether or not it is required. It has just been copied from the 16.04 image and may need updates but we will see as this hits the test suite.

Vars

I noticed there is a vars/xenial.yml file that makes a few changes for Postgis packages, though I could not see it being used anywhere in the role (seems like Debian.yml is used). In the event that this is needed for 18.04 I can create a bionic.yml. Deferring to maintainers for guidance on this.

@nchudleigh
Copy link
Copy Markdown
Contributor Author

nchudleigh commented Mar 13, 2019

Build for 18.04 is stalling on Reload all conf files step

Debugging locally

@nchudleigh
Copy link
Copy Markdown
Contributor Author

image

Vagrant build works locally, going to run against Docker (builded) target next to see if that is the issue

@nchudleigh
Copy link
Copy Markdown
Contributor Author

nchudleigh commented Mar 13, 2019

Local run of ubuntu:18.04-builded image against https://github.com/ANXS/postgresql/releases/tag/v1.10.1 ran successfully
image
image

Testing against master next

@nchudleigh
Copy link
Copy Markdown
Contributor Author

Local run of ubuntu:18.04-builded against current master was successful
image

image

@nchudleigh
Copy link
Copy Markdown
Contributor Author

@gclough At this point I am out of ideas, locally the tests run cleanly on the latest code. Perhaps it is something to do with the travis environment. I tried pulling the builder image used to build the test environment travis-ci-garnet-trusty-1512502259-986baf0 but was denied access to it.

If you have any ideas on where to look next I would really appreciate a pointer.

Going to leave off here for the day and come back with fresh eyes tomorrow

@nchudleigh
Copy link
Copy Markdown
Contributor Author

@gclough Really need some help on this one! All detail is given in previous comment.

@gclough
Copy link
Copy Markdown
Collaborator

gclough commented Mar 21, 2019

@nchudleigh I've re-triggered the failed build, as sometimes Travis hangs unexpectedly. If it fails again, then I'll have to review it more in depth.

@gclough
Copy link
Copy Markdown
Collaborator

gclough commented Mar 21, 2019

:-/ Same problem. I can't understand why it's hanging during a parameter reload.

TASK [ANXS.postgresql : PostgreSQL | Reload all conf files] ********************
No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.
Check the details on how to adjust your build configuration on: https://docs.travis-ci.com/user/common-build-problems/#Build-times-out-because-no-output-was-received
The build has been terminated

@gclough
Copy link
Copy Markdown
Collaborator

gclough commented Mar 21, 2019

The code it's running is pretty benign:

- name: PostgreSQL | Reload all conf files
  service:
    name: "{{ postgresql_service_name }}"
    state: reloaded
  when: postgresql_configuration_pt1.changed or postgresql_configuration_pt2.changed or postgresql_configuration_pt3.changed or postgresql_systemd_custom_conf.changed

I've never seen a PostgreSQL database hang on a config reload. If someone can provide advice on how to capture the logs before Travis terminates the server, I'd appreciate it.

@vladp
Copy link
Copy Markdown

vladp commented May 20, 2019

I think I am also running into this problem (in both Unbuntu 18 and Debian 9)
I noticed that I have to manually run pg_ctlcluster to restart.

root@lnx-14:/var# pg_lsclusters
Ver Cluster Port  Status Owner    Data directory              Log file
11  main    14544 online postgres /var/lib/postgresql/11/main /var/log/postgresql/postgresql-11-main.log
root@lnx-14:/var# pg_c
pg_config         pg_conftool       pg_createcluster  pg_ctlcluster
root@lnx-14:/var# pg_ctlcluster 11 main restart
root@lnx-14:/var#

@nchudleigh
Copy link
Copy Markdown
Contributor Author

@vladp Thanks for the suggestion, will have to give that a shot

jasonrogena added a commit to onaio/postgresql that referenced this pull request May 21, 2019
Since the TravisCI deployments freeze-up when you use the Ansible
service module to reload config files, use the pg_ctlcluster command to
reload the configuration files.

Fix suggested by @vladp ANXS#420.

Signed-off-by: Jason Rogena <jason@rogena.me>
jasonrogena added a commit to onaio/postgresql that referenced this pull request May 21, 2019
Since the TravisCI deployments freeze-up when you use the Ansible
service module to reload config files, use the pg_ctlcluster command to
reload the configuration files.

Fix suggested by @vladp ANXS#420.

Signed-off-by: Jason Rogena <jason@rogena.me>
jasonrogena added a commit to onaio/postgresql that referenced this pull request May 21, 2019
Since the TravisCI deployments freeze-up when you use the Ansible
service module to reload config files, use the pg_ctlcluster command to
reload the configuration files.

Fix suggested by @vladp ANXS#420.

Signed-off-by: Jason Rogena <jason@rogena.me>
jasonrogena added a commit to onaio/postgresql that referenced this pull request May 21, 2019
Since the TravisCI deployments freeze-up when you use the Ansible
service module to reload config files, use the pg_ctlcluster command to
reload the configuration files.

Fix suggested by @vladp ANXS#420.

Signed-off-by: Jason Rogena <jason@rogena.me>
jasonrogena added a commit to onaio/postgresql that referenced this pull request May 21, 2019
Since the TravisCI deployments freeze-up when you use the Ansible
service module to reload config files, use the pg_ctlcluster command to
reload the configuration files.

Fix suggested by @vladp ANXS#420.

Signed-off-by: Jason Rogena <jason@rogena.me>
jasonrogena added a commit to onaio/postgresql that referenced this pull request May 21, 2019
Since the TravisCI deployments freeze-up when you use the Ansible
service module to reload config files, use the pg_ctlcluster command to
reload the configuration files.

Fix suggested by @vladp ANXS#420.

Signed-off-by: Jason Rogena <jason@rogena.me>
jasonrogena added a commit to onaio/postgresql that referenced this pull request May 21, 2019
Since the TravisCI deployments freeze-up when you use the Ansible
service module to reload config files, use the pg_ctlcluster command to
reload the configuration files.

Fix suggested by @vladp ANXS#420.

Signed-off-by: Jason Rogena <jason@rogena.me>
@nchudleigh nchudleigh closed this Aug 27, 2019
michael-ai-eng added a commit to michael-ai-eng/postgresql that referenced this pull request Mar 12, 2026
Since the TravisCI deployments freeze-up when you use the Ansible
service module to reload config files, use the pg_ctlcluster command to
reload the configuration files.

Fix suggested by @vladp ANXS/postgresql#420.

Signed-off-by: Jason Rogena <jason@rogena.me>
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