search

tds-fdw / tds_fdw

A PostgreSQL foreign data wrapper to connect to TDS databases (Sybase and Microsoft SQL Server)
Other
381 stars 102 forks source link

Document RHEL and Clones, add Rocky Linux 8 and PostgreSQL to the matrix #299

Closed juliogonzalez closed 2 years ago

juliogonzalez commented 2 years ago

Fixes tds-fdw/tds_fdw#265 and tds-fdw/tds_fdw#297

The reasoning for the renaming of InstallCentOS.md is simple: we can't add CentOS 8, as it goes EoL by the end of this year, and by using the RHEL name, we can write common instructions for 7 and 8, that then can be used for every other maintained clone, such as CentOS or Oracle Linux 7, Rocky Linux or AlmaLinux 8, and many others.

The markdown files are better reviewed as a whole, instead of using the diff.

Needless to say, the badges for Rocky Linux 8 won't work until we merge this PR and the CI runs, but https://jenkins.juliogonzalez.es/job/tds_fdw-build-free/31/ was able to run the tests for Rocky Linux 8 without any problem).

juliogonzalez commented 2 years ago

It seems I am putting way too much load on the Jenkins worker #2, and they are not able to download an extract all the images, as they stay for minutes at:

21:16:51 698d50db42d0: Pull complete
21:16:53 8ce6ff6761d7: Pull complete
21:16:55 182e31163e59: Verifying Checksum
21:16:55 182e31163e59: Download complete

I will decrease the number of executors there fro 10 to 8, to see if the problem gets finally fixed.

juliogonzalez commented 2 years ago

Retest this, please

jenkins-juliogonzalez commented 2 years ago

Test PASSed.

juliogonzalez commented 2 years ago

All yours @GeoffMontee

I'd merge it on my own, but since we I am changing the doc, I'd prefer to have your approval :-)

GeoffMontee commented 2 years ago

Thanks for all your work on this, @juliogonzalez! This PR has been merged.