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

Regress fixes for PG13 - PG16 and simplification #251

Merged
merged 2 commits into from
May 18, 2023

Conversation

robe2
Copy link
Contributor

@robe2 robe2 commented May 18, 2023

  • Just use libgdal-dev from main repo, its 3.4 still same as ubuntugis
  • Allow read of source folder

This mostly sets the permission of the folders so PG14 and PG15 regress cleanly.
I also got rid of ubuntugis because the version shipped with latest ubuntu is 3.4.1 (not too far from ubuntugis 3.4.3). I'm mostly concerned about other things ubuntugis might bring in. You can revert if you want.

The issue with PG13 is puzzling. I tried wiping out all remnants of PostgreSQL and even setting priority of PostgreSQL install.

It seems to be actually creating a postgresql 14 folder to install things in. So this might be a bug in apt.postgresql.org instead. I'll test later on ubuntu latest 13 to see if I get the same odd behavior.

robe2 added 2 commits May 17, 2023 23:27
 - Just use libgdal-dev from main repo,
   its 3.4 still same as ubuntugis
 - Allow read of source folder
 - Remove all preinstalled postgresql,
   PG14 was installed in all of them
   Which was causing issues for PG13
 - Force all postgresql dependencies to come from apt.postgresql.org
 - Use pg_lsclusters to confirm what are running, mostly for debugging
   in future
 - Put back PG16 in testing loop
@robe2
Copy link
Contributor Author

robe2 commented May 18, 2023

Okay I managed to get them all working.
Turns out PostgreSQL 14 was being pre-installed in all of them as was evident when I ran pg_lsclusters. It was overriding PG13 partly and I think might have been what was causing issues in PG16.

I'm not sure if for whatever reason the GHA image just comes with PostgreSQL 14 installed or if it was being pulled in by GDAL.

I also changed the routine back to using pg_ctlcluster instead of your make shift pg_ctl command. I think that might have been causing issues with PG16, some extra thing you might have needed to add.

@robe2 robe2 changed the title Regress fixes for PG14 and PG15 and simplification Regress fixes for PG13 - PG16 and simplification May 18, 2023
@robe2
Copy link
Contributor Author

robe2 commented May 18, 2023

I forgot to mention I turned back on PG16 since it's now working.

@robe2 robe2 mentioned this pull request May 18, 2023
@pramsey
Copy link
Owner

pramsey commented May 18, 2023

OK, I will take this and probably trim it a little as I figured out the pg13 problem myself yesterday. GHA does ship with pg14, but if you are clever you can leave it in place and install the other versions you want and force paths to make it work. This is nice as it removes the need to correctly remove pg14 and thus the need to peg the runner version. (It's also fewer lines :)

@pramsey pramsey merged commit 6503097 into pramsey:master May 18, 2023
4 checks passed
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.

None yet

2 participants