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

[FLINK-34282][docker] Updates snapshot workflow configuration #180

Merged
merged 3 commits into from
Feb 12, 2024

Conversation

XComp
Copy link
Contributor

@XComp XComp commented Feb 9, 2024

This way we are aligned with what's described in the README.md again.

@XComp XComp requested a review from JingGe February 9, 2024 11:22
- flink_version: 1.18-SNAPSHOT
branch: dev-1.18
- flink_version: 1.17-SNAPSHOT
branch: dev-1.17
- flink_version: 1.16-SNAPSHOT
Copy link
Contributor

Choose a reason for hiding this comment

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

You should mean that we only maintain images of the latest three versions. I couldn't find it in the https://github.com/apache/flink-docker/blob/master/README.md you mentioned in the PR description. Does it make sense to update the PR description to make the intention of the PR a little bit more clear?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I updated the PR to include this minor README change in a hotfix PR 👍

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@JingGe can you merge the PR after approving it? I'm gonna be offline for the rest of the day. But it would be nice to have the snapshot Docker images being created over the weekend already.

preparations

- 1.15 and 1.16 are already deprecated
- 1.17 is kept because we still might do a final patch release as part
  of the 1.19.0 release
Copy link
Contributor

@JingGe JingGe left a comment

Choose a reason for hiding this comment

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

Thanks! LGTM

@XComp XComp merged commit 578731b into master Feb 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants