Skip to content

Conversation

@jaymode
Copy link
Member

@jaymode jaymode commented Mar 10, 2020

The setting, xpack.logstash.enabled, exists to enable or disable the
logstash extensions found within x-pack. In practice, this setting had
no effect on the functionality of the extension. Given this, the
setting is now deprecated in preparation for removal.

The setting, `xpack.logstash.enabled`, exists to enable or disable the
logstash extensions found within x-pack. In practice, this setting had
no effect on the functionality of the extension. Given this, the
setting is now deprecated in preparation for removal.
@jaymode jaymode added :Core/Infra/Core Core issues without another label >deprecation v8.0.0 v7.7.0 labels Mar 10, 2020
@jaymode jaymode requested a review from rjernst March 10, 2020 20:55
@elasticmachine
Copy link
Collaborator

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

Copy link
Member

@rjernst rjernst left a comment

Choose a reason for hiding this comment

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

LGTM

@jaymode jaymode merged commit c482744 into elastic:master Mar 12, 2020
@jaymode jaymode deleted the deprecate_logstash_enabled_setting branch March 12, 2020 14:41
jaymode added a commit to jaymode/elasticsearch that referenced this pull request Mar 12, 2020
The setting, `xpack.logstash.enabled`, exists to enable or disable the
logstash extensions found within x-pack. In practice, this setting had
no effect on the functionality of the extension. Given this, the
setting is now deprecated in preparation for removal.

Backport of elastic#53367
jaymode added a commit that referenced this pull request Mar 12, 2020
The setting, `xpack.logstash.enabled`, exists to enable or disable the
logstash extensions found within x-pack. In practice, this setting had
no effect on the functionality of the extension. Given this, the
setting is now deprecated in preparation for removal.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants