[GitHub] [bookkeeper] AndreyKoltsov1997 opened a new issue #2738: BK_VERSION variable doesn't match release version in some branches.

[GitBox <gi...@apache.org>]

AndreyKoltsov1997 opened a new issue #2738:
URL: https://github.com/apache/bookkeeper/issues/2738


   **BUG REPORT**
   
   ***Describe the bug***
   
   BK_VERSION environment variable hadn't been updated within Dockerfiles on some of the branches.
   On `branch-*` branches, the last update had been made for `4.12.1`, on `release-*` branches - `4.9.0`.
   
   ***To Reproduce***
   
   Steps to reproduce the behavior:
   ****See Dockerfile for the following branches:****
   * [4.13](https://github.com/apache/bookkeeper/blob/branch-4.13/docker/Dockerfile) - `BK_VERSION=4.12.1`
   * [4.14](https://github.com/apache/bookkeeper/blob/branch-4.14/docker/Dockerfile) - `BK_VERSION=4.12.1`
   * [Release 4.10.0](https://github.com/apache/bookkeeper/blob/release-4.10.0/docker/Dockerfile) - `ARG BK_VERSION=4.9.0`
   * [Release 4.11.0](https://github.com/apache/bookkeeper/blob/release-4.11.0/docker/Dockerfile) - `ARG BK_VERSION=4.9.0`
   * [Release 4.14.0](https://github.com/apache/bookkeeper/blob/release-4.14.0/docker/Dockerfile) - `BK_VERSION=4.12.1`
   * [Release 4.14.1](https://github.com/apache/bookkeeper/blob/release-4.14.1/docker/Dockerfile) - `BK_VERSION=4.12.1`
   
   ***Expected behavior***
   
   I except the `BK_VERSION` variable to match target BookKeeper release. Otherwise, while building Docker image of a particular release, the incorrect version could be misleading for the user.
   
   ***Screenshots***
   
   The links to Dockerfiles and their associated `BK_VERSION` variables have been attached above.
   
   ***Additional context***
   
   N/A


-- 
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:
users@infra.apache.org

[GitHub] [bookkeeper] AndreyKoltsov1997 commented on issue #2738: BK_VERSION variable doesn't match release version in some branches.

[GitBox <gi...@apache.org>]

AndreyKoltsov1997 commented on issue #2738:
URL: https://github.com/apache/bookkeeper/issues/2738#issuecomment-859522189


   Hi, @eolivelli!
   Currently, I'm thinking about sending multiple PRs to all of the branches listed within the issue. 
   Would it make sense to send a PRs to all of the branches or we should rather focus on a particular ones?


-- 
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:
users@infra.apache.org