Skip to content

Fix Broken MultiVersionRepositoryAccessIT#74074

Merged
original-brownbear merged 1 commit intoelastic:7.xfrom
original-brownbear:fix-broken-sn-bwc-tests
Jun 14, 2021
Merged

Fix Broken MultiVersionRepositoryAccessIT#74074
original-brownbear merged 1 commit intoelastic:7.xfrom
original-brownbear:fix-broken-sn-bwc-tests

Conversation

@original-brownbear
Copy link
Copy Markdown
Contributor

This test needs to run the restore request using the low-level REST client. My bad for missing this during a back-port,
this was breaking some 6.x versions because they wouldn't understand default parameters in the request that the client adds.

This test needs to run the restore request using the low-level REST client. My bad for missing this during a back-port,
this was breaking some 6.x versions because they wouldn't understand default parameters in the request that the client adds.
@original-brownbear original-brownbear added >test Issues or PRs that are addressing/adding tests :Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs v7.14.0 labels Jun 14, 2021
@elasticmachine elasticmachine added the Team:Distributed Meta label for distributed team. label Jun 14, 2021
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed (Team:Distributed)

@original-brownbear
Copy link
Copy Markdown
Contributor Author

broke this last night in #74044

Copy link
Copy Markdown
Contributor

@fcofdez fcofdez left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@original-brownbear original-brownbear merged commit 91a9bb7 into elastic:7.x Jun 14, 2021
@original-brownbear original-brownbear deleted the fix-broken-sn-bwc-tests branch June 14, 2021 15:02
@original-brownbear
Copy link
Copy Markdown
Contributor Author

Thanks Francisco!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs Team:Distributed Meta label for distributed team. >test Issues or PRs that are addressing/adding tests v7.14.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants