From 8efc63dea6e33d5cf6ea9920f5e032348f6ac113 Mon Sep 17 00:00:00 2001 From: Just_A_Anon Date: Thu, 29 Jan 2026 18:12:09 +0100 Subject: [PATCH 1/2] fix(osfamilymap): gpg keyring urls for debian and redhat --- postgres/osfamilymap.yaml | 5 +++-- postgres/upstream.sls | 21 +-------------------- 2 files changed, 4 insertions(+), 22 deletions(-) diff --git a/postgres/osfamilymap.yaml b/postgres/osfamilymap.yaml index f4e64c24..a8b09853 100644 --- a/postgres/osfamilymap.yaml +++ b/postgres/osfamilymap.yaml @@ -17,7 +17,8 @@ Debian: pkg_repo: humanname: PostgreSQL Official Repository file: /etc/apt/sources.list.d/pgdg.list - pkg_repo_keyring: 'https://download.postgresql.org/pub/repos/apt/pool/main/p/pgdg-keyring/pgdg-keyring_2018.2_all.deb' + key_url: https://download.postgresql.org/pub/repos/apt/ACCC4CF8.asc + name: deb https://apt.postgresql.org/pub/repos/apt {{ grains.get('oscodename') }}-pgdg main pkg_repo_keyid: ACCC4CF8 {% if repo.use_upstream_repo == true %} pkg_dev: '' @@ -60,7 +61,7 @@ RedHat: name: pgdg{{ release }} humanname: PostgreSQL {{ repo.version }} $releasever - $basearch gpgcheck: 1 - gpgkey: 'https://download.postgresql.org/pub/repos/yum/RPM-GPG-KEY-PGDG-{{ release }}' + gpgkey: 'https://download.postgresql.org/pub/repos/yum/keys/PGDG-RPM-GPG-KEY-RHEL' baseurl: 'https://download.postgresql.org/pub/repos/yum/{{ repo.version }}/redhat/rhel-$releasever-$basearch' pkgs_deps: diff --git a/postgres/upstream.sls b/postgres/upstream.sls index 83042be1..31b16af4 100644 --- a/postgres/upstream.sls +++ b/postgres/upstream.sls @@ -12,7 +12,7 @@ postgresql-profile: - name: /etc/profile.d/postgres.sh - user: root - group: root - - mode: 644 + - mode: '0644' - template: jinja - source: salt://postgres/templates/postgres.sh.j2 - defaults: @@ -24,22 +24,9 @@ postgresql-pkg-deps: - pkgs: {{ postgres.pkgs_deps | json }} # Add upstream repository for your distro - {% if grains.os_family == 'Debian' %} - {% if salt['pkg.version_cmp'](pg_common_version, '246') <= 0 %} -postgresql-repo-keyring: - pkg.installed: - - sources: - - pgdg-keyring: {{ postgres.pkg_repo_keyring }} - - require_in: - - pkgrepo: postgresql-repo - {%- endif %} - {%- endif %} - postgresql-repo: pkgrepo.managed: {{- format_kwargs(postgres.pkg_repo) }} - - require: - - pkg: postgresql-pkg-deps {%- else -%} @@ -51,12 +38,6 @@ postgresql-repo: - keyid: {{ postgres.pkg_repo_keyid }} {%- endif %} - {% if grains.os_family == 'Debian' %} -postgresql-repo-keyring: - pkg.removed: - - name: pgdg-keyring - {%- endif -%} - {%- endif -%} {%- elif grains.os not in ('Windows', 'MacOS',) %} From 7f00198933cb4fd0f82fdd3ec92ed1dfd318d504 Mon Sep 17 00:00:00 2001 From: Just_A_Anon Date: Tue, 18 Aug 2026 19:09:28 +0200 Subject: [PATCH 2/2] fix(codenamemap): removed signed-by --- postgres/codenamemap.yaml | 2 +- postgres/upstream.sls | 3 ++- test/integration/repo/controls/repository.rb | 24 +------------------- 3 files changed, 4 insertions(+), 25 deletions(-) diff --git a/postgres/codenamemap.yaml b/postgres/codenamemap.yaml index 26676607..c6d225e3 100644 --- a/postgres/codenamemap.yaml +++ b/postgres/codenamemap.yaml @@ -29,7 +29,7 @@ data_dir: {{ data_dir }} fromrepo: {{ fromrepo }} pkg_repo: - name: 'deb [signed-by=/usr/share/postgresql-common/pgdg/apt.postgresql.org.gpg] http://apt.postgresql.org/pub/repos/apt {{ name }}-pgdg main' + name: 'deb http://apt.postgresql.org/pub/repos/apt {{ name }}-pgdg main' pkg: postgresql-{{ version }} pkg_client: postgresql-client-{{ version }} prepare_cluster: diff --git a/postgres/upstream.sls b/postgres/upstream.sls index 31b16af4..6676ead4 100644 --- a/postgres/upstream.sls +++ b/postgres/upstream.sls @@ -27,7 +27,8 @@ postgresql-pkg-deps: postgresql-repo: pkgrepo.managed: {{- format_kwargs(postgres.pkg_repo) }} - + - require: + - pkg: postgresql-pkg-deps {%- else -%} # Remove the repo configuration (and GnuPG key) as requested diff --git a/test/integration/repo/controls/repository.rb b/test/integration/repo/controls/repository.rb index b0e9dbc6..afd26690 100644 --- a/test/integration/repo/controls/repository.rb +++ b/test/integration/repo/controls/repository.rb @@ -18,30 +18,8 @@ repo_file = os_name_repo_file[platform.name] when 'debian' - repo_keyring = '/usr/share/postgresql-common/pgdg/apt.postgresql.org.gpg' repo_file = '/etc/apt/sources.list.d/pgdg.list' - # rubocop:disable Layout/LineLength - repo_url = "deb [signed-by=#{repo_keyring}] http://apt.postgresql.org/pub/repos/apt #{system.platform[:codename]}-pgdg main" - # rubocop:enable Layout/LineLength -end - -control 'Postgresql repository keyring' do - title 'should be installed' - - only_if('Requirement for Debian family') do - os.debian? - end - - describe package('pgdg-keyring') do - it { should be_installed } - end - - describe file(repo_keyring) do - it { should exist } - it { should be_owned_by 'root' } - it { should be_grouped_into 'root' } - its('mode') { should cmp '0644' } - end + repo_url = "deb http://apt.postgresql.org/pub/repos/apt #{system.platform[:codename]}-pgdg main" end control 'Postgresql repository' do