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

Export field for xpack.enabled setting #9642

Merged
merged 1 commit into from Dec 19, 2018

Conversation

ycombinator
Copy link
Contributor

@ycombinator ycombinator commented Dec 18, 2018

The cleanup in #8308 introduced a regression which caused the xpack.enabled flag in the kibana module to stop having any effect.

This PR fixes that issue by reverting the troublesome bits from #8308, ensuring that the struct field for the parsed configuration setting is exported.

@elasticmachine
Copy link
Collaborator

Pinging @elastic/stack-monitoring

@ycombinator
Copy link
Contributor Author

jenkins, test this

@ycombinator ycombinator merged commit 8f37eec into elastic:master Dec 19, 2018
ycombinator added a commit to ycombinator/beats that referenced this pull request Dec 19, 2018
The cleanup in elastic#8308 introduced a regression which caused the `xpack.enabled` flag in the `kibana` module to stop having any effect.

This PR fixes that issue by reverting the troublesome bits from elastic#8308, ensuring that the struct field for the parsed configuration setting is exported.

(cherry picked from commit 8f37eec)
ycombinator added a commit that referenced this pull request Dec 19, 2018
Cherry-pick of PR #9642 to 6.x branch. Original message: 

The cleanup in #8308 introduced a regression which caused the `xpack.enabled` flag in the `kibana` module to stop having any effect.

This PR fixes that issue by reverting the troublesome bits from #8308, ensuring that the struct field for the parsed configuration setting is exported.
@ycombinator ycombinator deleted the bugfix-mb-kb-xpack branch December 25, 2019 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants