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

[backward tests] add BookKeeper 4.14.4 #2997

Merged
merged 2 commits into from
Feb 4, 2022

Conversation

nicoloboschi
Copy link
Contributor

Motivation

BK 4.14.4 has been released and we should test it in the upgrade tests

Note: for the sake of test performance, we test the upgrades only for the latest releases of each minor release

Changes

  • Replaced BK 4.14.3 with 4.14.4

@nicoloboschi
Copy link
Contributor Author

@eolivelli @dlg99 @pkumar-singh it would make sense to add this task to the release process

@nicoloboschi
Copy link
Contributor Author

run failure checks

1 similar comment
@nicoloboschi
Copy link
Contributor Author

run failure checks

Copy link
Contributor

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

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

We didn't change the wire protocol in 4.14.4 so there is no hurry to update these values.

BTW the patch LGTM

@nicoloboschi
Copy link
Contributor Author

run failure checks

@nicoloboschi
Copy link
Contributor Author

run failure checks

@eolivelli
Copy link
Contributor

@nicoloboschi sorry for late reply,
can you please resolve the conflicts ?

@dlg99 dlg99 added this to the 4.15.0 milestone Feb 4, 2022
@dlg99 dlg99 merged commit ee8c5bc into apache:master Feb 4, 2022
StevenLuMT pushed a commit to StevenLuMT/bookkeeper that referenced this pull request Feb 16, 2022
### Motivation
BK 4.14.4 has been released and we should test it in the upgrade tests

Note: for the sake of test performance, we test the upgrades only for the latest releases of each minor release 

### Changes

* Replaced BK 4.14.3 with 4.14.4


Reviewers: Enrico Olivelli <eolivelli@gmail.com>, Andrey Yegorov <None>

This closes apache#2997 from nicoloboschi/tests/add-bk-4144-backward-compat
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants