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 mutateInstance for SearchTemplateRequestTests #43509

Merged
merged 2 commits into from
Jun 25, 2019

Conversation

jdconrad
Copy link
Contributor

A seed was hit in (#43157) that caused mutateInstance to generate an identical instance. This change prevents that.

@jdconrad jdconrad added >test Issues or PRs that are addressing/adding tests :Search/Search Search-related issues that do not fall into other categories v8.0.0 v7.3.0 labels Jun 21, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search

@jdconrad jdconrad requested a review from jtibshirani June 21, 2019 21:52
@jdconrad
Copy link
Contributor Author

@jtibshirani Requested your review as you were the last person to work on this. Thanks in advance!

Copy link
Contributor

@hub-cap hub-cap left a comment

Choose a reason for hiding this comment

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

:shipit:

@jdconrad jdconrad removed the request for review from jtibshirani June 24, 2019 15:58
@jdconrad
Copy link
Contributor Author

@baz Thanks for the review! @jtibshirani Sorry to ping you on this - just realized that you're on vacation, so I found a different reviewer.

@jdconrad jdconrad merged commit d640fb5 into elastic:master Jun 25, 2019
@jdconrad jdconrad removed the v7.3.0 label Jun 25, 2019
mayya-sharipova pushed a commit that referenced this pull request Jul 4, 2019
A seed was hit in (#43157) that caused mutateInstance to generate an identical 
instance. This change prevents that.
@mayya-sharipova
Copy link
Contributor

backported to 6.8 as well as there were failures of this test there.

jtibshirani pushed a commit that referenced this pull request Feb 8, 2021
A seed was hit in (#43157) that caused mutateInstance to generate an identical 
instance. This change prevents that.
jtibshirani pushed a commit that referenced this pull request Feb 8, 2021
A seed was hit in (#43157) that caused mutateInstance to generate an identical 
instance. This change prevents that.
jtibshirani pushed a commit that referenced this pull request Feb 8, 2021
A seed was hit in (#43157) that caused mutateInstance to generate an identical 
instance. This change prevents that.
@jtibshirani
Copy link
Contributor

Also backported to 7.x. In general we can pre-emptively backport these fixes to all branches that are actively tested.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Search/Search Search-related issues that do not fall into other categories >test Issues or PRs that are addressing/adding tests v6.8.2 v7.10.3 v7.11.1 v7.12.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants