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

unmute EvilLoggerTests#testDeprecatedSettings #38743

Merged
merged 1 commit into from
Feb 11, 2019
Merged

Conversation

talevy
Copy link
Contributor

@talevy talevy commented Feb 11, 2019

unmuting test to determine whether this is still an issue on
the current branch. If it is found that this test does
not fail in CI anymore, then this commit will remain and
test will stay tested.

related to #35990.

unmuting test to determine whether this is still an issue on
the current branch. If it is found that this test does
not fail in CI anymore, then this commit will remain and
test will stay tested.

related to elastic#35990.
@talevy talevy added >test Issues or PRs that are addressing/adding tests :Core/Infra/Logging Log management and logging utilities labels Feb 11, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

Copy link
Member

@jasontedor jasontedor left a comment

Choose a reason for hiding this comment

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

LGTM.

@talevy
Copy link
Contributor Author

talevy commented Feb 11, 2019

thanks Jason!

@talevy talevy merged commit f2b4dd5 into elastic:6.7 Feb 11, 2019
@talevy talevy deleted the unmute-dep-67 branch February 11, 2019 21:31
@talevy talevy added the v6.7.0 label Feb 11, 2019
dimitris-athanasiou added a commit that referenced this pull request Mar 5, 2019
@dimitris-athanasiou
Copy link
Contributor

Since this test failed more times I have reverted this in be10934

jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Mar 6, 2019
* 6.7: (39 commits)
  Remove beta label from CCR (elastic#39722)
  Rename retention lease setting (elastic#39719)
  Add Docker build type (elastic#39378)
  Use any index specified by .watches for Watcher (elastic#39541) (elastic#39706)
  Add documentation on remote recovery (elastic#39483)
  fix typo in synonym graph filter docs
  Removed incorrect ML YAML tests (elastic#39400)
  Improved Terms Aggregation documentation (elastic#38892)
  Fix Fuzziness#asDistance(String) (elastic#39643)
  Revert "unmute EvilLoggerTests#testDeprecatedSettings (elastic#38743)"
  Mute TokenAuthIntegTests.testExpiredTokensDeletedAfterExpiration (elastic#39690)
  Fix security index auto-create and state recovery race (elastic#39582)
  [DOCS] Sorts security APIs
  Check for .watches that wasn't upgraded properly (elastic#39609)
  Assert recovery done in testDoNotWaitForPendingSeqNo (elastic#39595)
  [DOCS] Updates API in Watcher transform context (elastic#39540)
  Fixing the custom object serialization bug in diffable utils. (elastic#39544)
  mute test
  SQL: Don't allow inexact fields for MIN/MAX (elastic#39563)
  Update release notes for 6.7.0
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/Logging Log management and logging utilities >test Issues or PRs that are addressing/adding tests v6.7.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants