Skip to content

Conversation

roshkhatri
Copy link
Member

Adds 7.2.4-rc1 and updates docker hub descriptions with new tags

Signed-off-by: Roshan Khatri <rvkhatri@amazon.com>
@roshkhatri roshkhatri requested review from hpatro and madolson April 9, 2024 20:54
push:
workflow_dispatch:
workflow_run:
workflows: [GitHub CI]
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will only run once we push the images to dockerhub

startswith(env.rcVersion + ".")
or startswith(env.rcVersion + "-")
) and (
index("-")
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This script did not allow -rc versions to get published

Copy link
Contributor

@hpatro hpatro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For each os type I presume the Dockerfile and docker-entrypoint.sh file is auto generated.

"alpine": {
"version": "3.19"
},
"gosu": {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's this for?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Its golang super user https://github.com/tianon/gosu
It is used to step down from root during container startup, specially for entrypoint.

# Supported tags and respective `Dockerfile` links
- [`unstable`](https://github.com/valkey-io/valkey-container/blob/master/unstable/debian/Dockerfile)
- [`unstable-alpine`](https://github.com/valkey-io/valkey-container/blob/master/unstable/alpine/Dockerfile)
- [`unstable`, `unstable-bookworm`](https://github.com/valkey-io/valkey-container/blob/master/unstable/debian/Dockerfile)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any reason why debian doesn't need to be added as suffix ?

@roshkhatri
Copy link
Member Author

For each os type I presume the Dockerfile and docker-entrypoint.sh file is auto generated.

it is done by apply-templates.sh using Docker.template

@roshkhatri roshkhatri merged commit bcc8adb into valkey-io:mainline Apr 9, 2024
@roshkhatri roshkhatri deleted the add-7.2.4-rc1 branch April 9, 2024 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants