Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds 7.2.4-rc1 #8

Merged
merged 1 commit into from
Apr 9, 2024
Merged

Conversation

roshkhatri
Copy link
Member

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

Signed-off-by: Roshan Khatri <[email protected]>
@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

@@ -137,9 +137,6 @@ for version in "${versions[@]}"; do
| (
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.

@@ -7,8 +7,10 @@
Please open an Issue stating your question on [the Valkey Community](https://github.com/valkey-io/valkey-container/issues).

# 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
8 checks passed
@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