-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
chore(NA): setup config folder with env var on fpm os packages #67433
chore(NA): setup config folder with env var on fpm os packages #67433
Conversation
Pinging @elastic/kibana-operations (Team:Operations) |
@elasticmachine merge upstream |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@elasticmachine merge upstream |
@jbudz had you the chance to test that approach both for |
@elasticmachine merge upstream |
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
many hours later (sorry!) - yes. we can remove "-c /etc/config/kibana.yml" from /etc/system.d/system/kibana.service but otherwise LGTM.
Thanks @jbudz ! I'll do this in another PR |
…th fpm (elastic#67433) Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
…th fpm (elastic#67433) Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
When `(KBN|KIBANA)_PATH_CONF` was introduced, we removed the fallback lookup to `/etc/kibana/kibana.yml` in elastic#67433. This environment variable was added to `/etc/default/kibana` and made available, scoped to the Kibana service for deb and rpm packages. This environment setting, however, is not available when running binaries directly and caused the regression noted in This re-adds a fallback lookup to `/etc/kibana/kibana.yml` when searching for a kibana.yml configuration. Testing 1) Install a .deb or .rpm package 2) `sudo -u kibana /usr/share/bin/kibana --version` Closes elastic#101249
That pr setup the ENV var
KIBANA_PATH_CONF
in order to have it pointing to the correct location of the config path. That has been discussed on #62468 and it is required for that PR.@jbudz could you help me making sure it works as we expected? From what I was able to test what I'm doing here would do what we want.