postgres-backup-s3 icon indicating copy to clipboard operation
postgres-backup-s3 copied to clipboard

Use postgres alpine images to add support for PG 17

Open miharekar opened this issue 1 year ago • 13 comments

Postgres publishes alpine versions of Docker images including for PG 17 which was recently released.

So this approach should work better than #60 where we're left to the mercy of Alpine releases 😅

miharekar avatar Oct 14 '24 13:10 miharekar

so uh @ledermann do you plan to merge this PR or that other one?

You've mentioned that this is a good way to handle things so why not move forward with this?

lesha-co avatar Nov 11 '24 22:11 lesha-co

@lesha-co I can't decide, @eeshugerman is the owner and maintainer of this repo.

ledermann avatar Nov 12 '24 05:11 ledermann

While we are waiting for this to be merged, is there a way to use it already. @miharekar Is there a public docker image with these changes applied for v17 ?

I have always used this utility for all my postgres backups, and now I have a production project running with postgres 17 just to realise later that this tool is not yet updated to support v17 yet.

ak4zh avatar Dec 02 '24 06:12 ak4zh

@ak4zh I have my own fork and I've just yesterday uploaded it with 17.2 image: https://hub.docker.com/repository/docker/mrfoto/postgres-backup-s3/general

I use it for my own needs, so no guarantees 😅 Maybe best you fork it yourself. If I managed to do it, you know it isn't that hard 😂

miharekar avatar Dec 02 '24 06:12 miharekar

@miharekar Thanks, I will see if I can figure out the docker image publishing part myself. Forking is fine but I haven’t explored the docker image publish part before, I suppose I need to signup as publisher in docker-hub or something, will give it a try today so I can reuse the image.

ak4zh avatar Dec 02 '24 06:12 ak4zh

You don't really need to go into trouble of publishing anything just for this one case.

What I did is added this repo as a submodule at commit 101949343a3eb3187e61965289b82d8e42acbd5e (i.e. this pull request's head)

then added a build (rather than an image) entry to docker-compose file

services:
  backup:
    build:
      context: ./postgres-backup-s3
      args:
        - POSTGRES_VERSION=17
    environment:
      - SCHEDULE
      - ...

and that's it.

when, if ever, this pr gets merged, I'll either just remove the submodule and use the newly available version from registry, or just change the submodule head or more likely just do nothing because why fixing something that isn't broken

lesha-co avatar Dec 02 '24 09:12 lesha-co

@lesha-co I can't decide, @eeshugerman is the owner and maintainer of this repo.

@eeshugerman can you please merge this PR?

ahmayou avatar Dec 02 '24 13:12 ahmayou

@miharekar It seems the docker hub has only arm version not for any other arch like amd64 I am trying to deploy it with caprover so can't deploy directly through docker-compose.

ak4zh avatar Dec 09 '24 08:12 ak4zh

@ak4zh yeah, like I said - I built it for my own needs. Feel free to build one yourself

miharekar avatar Dec 09 '24 08:12 miharekar

Because this repo seems to be not maintained anymore, I have created a fork and merged this PR. I have no plans to extend the tool any further. My only concern is to ensure that it works with current PostgreSQL versions.

I have set up GitHub Actions to push the Docker images to ghcr.io for arm64 and amd64.

The fork repo:

  • https://github.com/solectrus/postgres-s3-backup (Name changed to match another backup repository in my organization)

Docker images:

  • ghcr.io/solectrus/postgres-s3-backup:17
  • ghcr.io/solectrus/postgres-s3-backup:16
  • ghcr.io/solectrus/postgres-s3-backup:15
  • ghcr.io/solectrus/postgres-s3-backup:14
  • ghcr.io/solectrus/postgres-s3-backup:13

Thanks to @eeshugerman for creating this repo and to @miharekar for this PR!

ledermann avatar Jan 28 '25 14:01 ledermann

@ledermann You forked the project without any plans to extend or contribute, yet you’ve added a Ko-fi link for support—profiting off someone else's work.

TRRC62 avatar Feb 06 '25 08:02 TRRC62

@TRRC62 My plan for this fork is to keep the Docker image compatible with the latest PostgreSQL versions - something many people here have been looking for. I believe that’s a good enough reason to fork an unmaintained repo.

I created the fork under my organization @solectrus, which is an open-source project with a donation button. I didn’t realize that new repositories automatically inherit this button. That was an oversight, and I’ve now disabled it for this repo. Of course, I never intended to make money from someone else’s work.

I made the fork public because I thought it could be useful to others. If the only feedback is criticism that I have no further plans, maybe I shouldn’t have shared it at all.

ledermann avatar Feb 06 '25 09:02 ledermann

@ledermann thanks for sharing this as I'm using 17!

jspooner avatar Mar 18 '25 01:03 jspooner