-
Notifications
You must be signed in to change notification settings - Fork 22
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
Conversation
Signed-off-by: Roshan Khatri <[email protected]>
push: | ||
workflow_dispatch: | ||
workflow_run: | ||
workflows: [GitHub CI] |
There was a problem hiding this comment.
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("-") |
There was a problem hiding this comment.
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
There was a problem hiding this 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": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's this for?
There was a problem hiding this comment.
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) |
There was a problem hiding this comment.
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 ?
it is done by |
Adds 7.2.4-rc1 and updates docker hub descriptions with new tags