zentol commented on a change in pull request #18: URL: https://github.com/apache/flink-docker/pull/18#discussion_r421327322
########## File path: README.md ########## @@ -39,24 +50,30 @@ There are additional steps required when a new Flink minor version (x.y.0) is re ### Release workflow -When a new release of Flink is available, the Dockerfiles in this repo should be updated and a new +When a new release of Flink is available, the Dockerfiles in the `master` branch should be updated and a new manifest sent to the Docker Library [`official-images`]( https://github.com/docker-library/official-images) repo. -Updating the Dockerfiles involves 3 steps: - -1. Update `add-version.sh` with the GPG key ID of the key used to sign the new release - * Commit this change with message `Add GPG key for x.y.z release` <sup>\[[example]( - https://github.com/apache/flink-docker/commit/94845f46c0f0f2de80d4a5ce309db49aff4655d0)]</sup> -2. Remove any existing Dockerfiles from the same minor version - * e.g. `rm -r 1.2`, if the new Flink version is `1.2.1` -3. Run `add-version.sh` with the appropriate arguments (`-r flink-minor-version -f - flink-full-version`) - * e.g. `./add-version.sh -r 1.2 -f 1.2.1` - * Commit these changes with message `Update Dockerfiles for x.y.z release` <sup>\[[example]( +The Dockerfiles are generated on the respective `dev-<version>` branches, and copied over to the `master` branch for +publishing. + +Updating the Dockerfiles involves the following steps: + +1. Generate the Dockerfiles + * Checkout the `dev-x.y` branch of the respective release, e.g., dev-1.9 + * Update `add-version.sh` with the GPG key ID of the key used to sign the new release Review comment: But yes, currently we need it for CI for the dev-1.x, but this would be trivial to change. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
