Bug #22972
closedsalt-installer fails when trying to download pgdg-keyring package from postgresql.org
Description
The error is:
Error: HTTP 404: Not Found reading /pub/repos/apt/pool/main/p/pgdg-keyring/pgdg-keyring_2018.2_all.deb
Currently we workaround-ed this issue by requesting the distro packages instead of upstream but this is not a good solution for deployments with RDS instances running PG versions that are not available on the currently used distro.
Updated by Brett Smith 9 months ago
- Target version changed from Development 2025-06-25 to Development 2025-07-09
Updated by Brett Smith 9 months ago
I checked whether this has been addressed upstream and the answer is no. There has been no development since my last PR. This is what I expected but I had to be 100% sure. We're on our own.
Updated by Lucas Di Pentima 9 months ago
22972-pgdg-keyring-fix - commit 9c894d0d (postgres-formula at github)
22972-salt-installer-pgdg-keyring - 34ac7fc6 (arvados repo)
- All agreed upon points are implemented / addressed. Describe changes from pre-implementation design.
- Yes, we switched back to using upstream PG packages by default with this change, as we're able to use apt.postgresql.org package sources again.
- Anything not implemented (discovered or discussed during work) has a follow-up story.
- Ubuntu 24.04 support was left as a future exercise. We probably won't be adding it to this version of the installer. As a result, the
test-provision-ubuntu2404job has been disabled for now.
- Ubuntu 24.04 support was left as a future exercise. We probably won't be adding it to this version of the installer. As a result, the
- Code is tested and passing, both automated and manual, what manual testing was done is described.
- Yes. I've also manually tested it by deploying a Debian 12 multi-node cluster.
- New or changed UI/UX has gotten feedback from stakeholders.
- N/A
- Documentation has been updated.
- N/A
- Behaves appropriately at the intended scale (describe intended scale).
- No changes in scale
- Considered backwards and forwards compatibility issues between client and server.
- N/A
- Follows our coding standards and GUI style guidelines.
- N/A
Changes¶
- Fixes
postgres-formulaso that it correctly downloads & installs the GPG pubkey file whereaptlooks for it and doesn't attempt to install the no longer availablepgdg-keyringpackage. - Updates the provision script to use the new PG formula version.
Updated by Brett Smith 9 months ago
Lucas Di Pentima wrote in #note-6:
22972-pgdg-keyring-fix - commit 9c894d0d (
postgres-formulaat github)
22972-salt-installer-pgdg-keyring - 34ac7fc6 (arvadosrepo)
All of this is good to merge.
- Anything not implemented (discovered or discussed during work) has a follow-up story.
- Ubuntu 24.04 support was left as a future exercise. We probably won't be adding it to this version of the installer. As a result, the
test-provision-ubuntu2404job has been disabled for now.
This is not what I expected based on our conversation. What I expected was that test-provision-ubuntu2404 would continue running with its current configuration; i.e., we would set it up to run with use_upstream_repo: false.
Let's start with the fundamentals. What is the specific issue(s) that prevents the Salt installer from working on Ubuntu 24.04?
A while back I did a branch that I apparently thought would add Ubuntu 24.04 support to the formula. Is that all we need? If you add that commit to your branch, does that get test-provision-ubuntu2404 working normally? It's literally just one line, and I've already written it, so if that's all we need that's fine to do.
Thanks.
Updated by Lucas Di Pentima 9 months ago
- Status changed from In Progress to Resolved
Applied in changeset arvados|2b67fe9a032698a8c5de59cfc5eb1001c866031a.
Updated by Lucas Di Pentima 9 months ago
- Related to Bug #23010: Salt installer fails when deploying to Ubuntu 24.04 added