Skip to content

Add deprecation check for ILM poll interval <1s #41096

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Apr 12, 2019
Merged

Add deprecation check for ILM poll interval <1s #41096

merged 1 commit into from
Apr 12, 2019

Conversation

@gwbrown
Copy link
Contributor

@gwbrown gwbrown commented Apr 10, 2019

ILM poll intervals of less than 1 second will not be allowed, so add a
deprecation check for that.

Even though I'm pretty sure zero production clusters will do this, it's
best to be thorough.

Relates to #39163

@gwbrown
Add deprecation check for ILM poll interval <1s
8e59d9d 
ILM poll intervals of less than 1 second will not be allowed, so add a
deprecation check for that.

Even though I'm pretty sure zero production clusters will do this, it's
best to be thorough.
@gwbrown gwbrown requested a review from dakrone April 10, 2019 23:10
@elasticmachine
Copy link
Collaborator

elasticmachine commented Apr 10, 2019

Pinging @elastic/es-core-features

dakrone
dakrone approved these changes Apr 11, 2019
View reviewed changes
Copy link
Member

@dakrone dakrone left a comment

Choose a reason for hiding this comment

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

LGTM

@gwbrown gwbrown merged commit ef31088 into elastic:7.x Apr 12, 2019
@jrodewig jrodewig mentioned this pull request Sep 14, 2021
Merged
jrodewig added a commit that referenced this pull request Sep 21, 2021
@jrodewig
[DOCS] Add deprecation docs for low ILM poll intervals (#77733)
49494aa 
We deprecated ILM poll intervals of less than 1 second in 7.2 with PR #41096
However, we didn't add a related item to the 7.2 deprecation docs.
This adds the missing item.

Relates to #41095.
elasticsearchmachine pushed a commit that referenced this pull request Sep 21, 2021
@jrodewig
[DOCS] Add deprecation docs for low ILM poll intervals (#77733) (#78084)
8b447cd 
We deprecated ILM poll intervals of less than 1 second in 7.2 with PR #41096
However, we didn't add a related item to the 7.2 deprecation docs.
This adds the missing item.

Relates to #41095.
elasticsearchmachine pushed a commit that referenced this pull request Sep 21, 2021
@jrodewig
[DOCS] Add deprecation docs for low ILM poll intervals (#77733) (#78083)
52feb11 
We deprecated ILM poll intervals of less than 1 second in 7.2 with PR #41096
However, we didn't add a related item to the 7.2 deprecation docs.
This adds the missing item.

Relates to #41095.
elasticsearchmachine pushed a commit that referenced this pull request Sep 21, 2021
@jrodewig
[DOCS] Add deprecation docs for low ILM poll intervals (#77733) (#78085)
0c6e26f 
We deprecated ILM poll intervals of less than 1 second in 7.2 with PR #41096
However, we didn't add a related item to the 7.2 deprecation docs.
This adds the missing item.

Relates to #41095.
elasticsearchmachine pushed a commit that referenced this pull request Sep 21, 2021
@jrodewig
[DOCS] Add deprecation docs for low ILM poll intervals (#77733) (#78090)
3d8f98e 
We deprecated ILM poll intervals of less than 1 second in 7.2 with PR #41096
However, we didn't add a related item to the 7.2 deprecation docs.
This adds the missing item.

Relates to #41095.
elasticsearchmachine pushed a commit that referenced this pull request Sep 21, 2021
@jrodewig
[DOCS] Add deprecation docs for low ILM poll intervals (#77733) (#78088)
0975c5b 
We deprecated ILM poll intervals of less than 1 second in 7.2 with PR #41096
However, we didn't add a related item to the 7.2 deprecation docs.
This adds the missing item.

Relates to #41095.
elasticsearchmachine pushed a commit that referenced this pull request Sep 21, 2021
@jrodewig
[DOCS] Add deprecation docs for low ILM poll intervals (#77733) (#78086)
0a2bfaf 
We deprecated ILM poll intervals of less than 1 second in 7.2 with PR #41096
However, we didn't add a related item to the 7.2 deprecation docs.
This adds the missing item.

Relates to #41095.
elasticsearchmachine pushed a commit that referenced this pull request Sep 21, 2021
@jrodewig
[DOCS] Add deprecation docs for low ILM poll intervals (#77733) (#78089)
72bc576 
We deprecated ILM poll intervals of less than 1 second in 7.2 with PR #41096
However, we didn't add a related item to the 7.2 deprecation docs.
This adds the missing item.

Relates to #41095.
elasticsearchmachine pushed a commit that referenced this pull request Sep 21, 2021
@jrodewig
[DOCS] Add deprecation docs for low ILM poll intervals (#77733) (#78087)
a1e564a 
We deprecated ILM poll intervals of less than 1 second in 7.2 with PR #41096
However, we didn't add a related item to the 7.2 deprecation docs.
This adds the missing item.

Relates to #41095.
jrodewig added a commit that referenced this pull request Sep 21, 2021
@jrodewig
[DOCS] Add deprecation docs for low ILM poll intervals (#77733) (#78091)
f5a0b8d 
We deprecated ILM poll intervals of less than 1 second in 7.2 with PR #41096
However, we didn't add a related item to the 7.2 deprecation docs.
This adds the missing item.

Relates to #41095.
jrodewig added a commit that referenced this pull request Sep 21, 2021
@jrodewig
[DOCS] Add deprecation docs for low ILM poll intervals (#77733) (#78092)
749f498 
We deprecated ILM poll intervals of less than 1 second in 7.2 with PR #41096
However, we didn't add a related item to the 7.2 deprecation docs.
This adds the missing item.

Relates to #41095.
jrodewig added a commit that referenced this pull request Sep 21, 2021
@jrodewig
[DOCS] Add deprecation docs for low ILM poll intervals (#77733) (#78093)
f18fa37 
We deprecated ILM poll intervals of less than 1 second in 7.2 with PR #41096
However, we didn't add a related item to the 7.2 deprecation docs.
This adds the missing item.

Relates to #41095.
jrodewig added a commit that referenced this pull request Sep 21, 2021
@jrodewig
[DOCS] Add deprecation docs for low ILM poll intervals (#77733) (#78094)
1b4b12b 
We deprecated ILM poll intervals of less than 1 second in 7.2 with PR #41096
However, we didn't add a related item to the 7.2 deprecation docs.
This adds the missing item.

Relates to #41095.
jrodewig added a commit that referenced this pull request Sep 21, 2021
@jrodewig
[DOCS] Add deprecation docs for low ILM poll intervals (#77733) (#78095)
aac5777 
We deprecated ILM poll intervals of less than 1 second in 7.2 with PR #41096
However, we didn't add a related item to the 7.2 deprecation docs.
This adds the missing item.

Relates to #41095.
jrodewig added a commit that referenced this pull request Sep 21, 2021
@jrodewig
[DOCS] Add deprecation docs for low ILM poll intervals (#77733) (#78097)
47aa001 
We deprecated ILM poll intervals of less than 1 second in 7.2 with PR #41096
However, we didn't add a related item to the 7.2 deprecation docs.
This adds the missing item.

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

Reviewers

@dakrone dakrone dakrone approved these changes

Assignees

No one assigned

Labels

>enhancement v7.2.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@gwbrown @elasticmachine @dakrone