ANXS / postgresql

Fairly full featured Ansible role for Postgresql.
http://anxs.io/
MIT License
848 stars 573 forks source link

Wrong postgresql_yum_repository_baseurl #514

Closed JorisVanEijden closed 3 years ago

JorisVanEijden commented 3 years ago

In defaults/main.yml:

postgresql_yum_repositorybaseurl: "{{ postgresqlyum_repository_url }}/{{ postgresql_version }}/{{ ansible_os_family | lower }}/rhel-{{ ansible_distribution_major_version }}-{{ ansible_architecture }}"

has to be:

postgresql_yum_repositorybaseurl: "{{ postgresqlpgdg_repository_url }}/{{ postgresql_version }}/{{ ansible_os_family | lower }}/rhel-{{ ansible_distribution_major_version }}-{{ ansible_architecture }}"

maglub commented 3 years ago

Hi @JorisVanEijden,

Could you please give a little more context to why? If I can reproduce your issue, it is easier for me to help.

I am not a CentOS guy myself, but the CI/CD tests are going through properly for both 7 and 8.

The variables you are speaking of are:

ANXS.postgresql/defaults/main.yml:postgresql_yum_repository_url: "http://yum.postgresql.org"
ANXS.postgresql/defaults/main.yml:postgresql_pgdg_repository_url: "https://download.postgresql.org/pub/repos/yum/"
JorisVanEijden commented 3 years ago

I tested again and now it appears to work without the workaround in place.

I see http://yum.postgresql.org/13/redhat/rhel-7-x86_64/ redirects to https://download.postgresql.org/pub/repos/yum/13/redhat/rhel-7-x86_64/ So I can imagine:

I'll close this for now and will keep an eye out when I provision more new machines.

maglub commented 3 years ago

Great, thanks a lot!