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

Fix up BWC for serialization of GetIndexResponse #56274

Conversation

DaveCTurner
Copy link
Contributor

Completes the backport of #56022

Closes #56265

@DaveCTurner DaveCTurner added the :Data Management/Data streams Data streams and their lifecycles label May 6, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features (:Core/Features/Data streams)

@elasticmachine elasticmachine added the Team:Data Management Meta label for data/management team label May 6, 2020
@DaveCTurner DaveCTurner added >test Issues or PRs that are addressing/adding tests v8.0.0 labels May 6, 2020
@DaveCTurner
Copy link
Contributor Author

@elasticmachine test this please

Copy link
Contributor

@danhermann danhermann left a comment

Choose a reason for hiding this comment

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

LGTM. Sorry for missing this one.

@DaveCTurner DaveCTurner merged commit f4d970a into elastic:master May 6, 2020
@DaveCTurner DaveCTurner deleted the 2020-05-06-get-index-response-serialization branch May 6, 2020 14:51
@DaveCTurner
Copy link
Contributor Author

Thanks @danhermann, no problem, it's easily done.

hendrikmuhs pushed a commit that referenced this pull request May 7, 2020
the transform upgrade tests broke due to #56238, but got fixed with #56274

fixes #56269
fixes #56250
hendrikmuhs pushed a commit that referenced this pull request May 8, 2020
the transform upgrade tests broke due to #56238, but got fixed with #56274

fixes #56269
fixes #56250
hendrikmuhs pushed a commit that referenced this pull request May 8, 2020
the transform upgrade tests broke due to #56238, but got fixed with #56274

fixes #56269
fixes #56250
hendrikmuhs pushed a commit that referenced this pull request May 8, 2020
add a very basic rolling upgrade test for get index, post mortem action of #56274
hendrikmuhs pushed a commit that referenced this pull request May 8, 2020
add a very basic rolling upgrade test for get index, post mortem action of #56274
hendrikmuhs pushed a commit to hendrikmuhs/elasticsearch that referenced this pull request May 8, 2020
add a very basic rolling upgrade test for get index, post mortem action of elastic#56274
hendrikmuhs pushed a commit that referenced this pull request May 11, 2020
add a very basic rolling upgrade test for get index, post mortem action of #56274
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Data Management/Data streams Data streams and their lifecycles Team:Data Management Meta label for data/management team >test Issues or PRs that are addressing/adding tests v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RecoveryIT.testCloseIndexDuringRollingUpgrade failures
4 participants