Skip to content

Conversation

@pgomulka
Copy link
Contributor

@pgomulka pgomulka commented Nov 21, 2019

This test no longer relies on jdk version, so the assume should be removed
relates #48209

@pgomulka pgomulka added the :Core/Infra/Core Core issues without another label label Nov 21, 2019
@pgomulka pgomulka self-assigned this Nov 21, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (:Core/Infra/Core)

@pgomulka pgomulka changed the title Fix/enable log expiration warning test Enable LicenceServiceTests for all jdks Nov 21, 2019
@pgomulka pgomulka requested a review from tvernum November 22, 2019 08:55
Copy link
Contributor

@dliappis dliappis left a comment

Choose a reason for hiding this comment

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

LGTM

@pgomulka pgomulka merged commit 7a1720c into elastic:master Nov 28, 2019
pgomulka added a commit to pgomulka/elasticsearch that referenced this pull request Nov 28, 2019
This test no longer relies on jdk version, so the assume should be removed
relates elastic#48209
pgomulka added a commit that referenced this pull request Nov 28, 2019
This test no longer relies on jdk version, so the assume should be removed
relates #48209
@polyfractal polyfractal added the >test Issues or PRs that are addressing/adding tests label Jan 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Core/Infra/Core Core issues without another label >test Issues or PRs that are addressing/adding tests v7.6.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants