[rpm] Create default environment file at "/etc/sysconfig/kibana"#82144
Merged
jbudz merged 8 commits intoelastic:masterfrom Nov 17, 2020
Merged
[rpm] Create default environment file at "/etc/sysconfig/kibana"#82144jbudz merged 8 commits intoelastic:masterfrom
jbudz merged 8 commits intoelastic:masterfrom
Conversation
of "/etc/default/kibana". This is a standards difference. rpm based distributions generally use /etc/sysconfig. In production we're reading from both, with sysconfig second to override any historical differences. Related to elastic#36671, elastic#6423
Contributor
|
Pinging @elastic/kibana-operations (Team:Operations) |
Contributor
Author
|
@elasticmachine merge upstream |
Contributor
Author
|
@elasticmachine merge upstream |
Contributor
Author
|
@elasticmachine merge upstream |
tylersmalley
requested changes
Nov 7, 2020
Member
There was a problem hiding this comment.
Unable to start on CentOS
● kibana.service - Kibana
Loaded: loaded (/etc/systemd/system/kibana.service; enabled; vendor preset: disabled)
Active: failed (Result: exit-code) since Sat 2020-11-07 00:22:10 UTC; 1s ago
Docs: https://www.elastic.co
Process: 3030 ExecStart=/usr/share/kibana/bin/kibana --logging.dest=/var/log/kibana/kibana.log (code=exited, status=64)
Main PID: 3030 (code=exited, status=64)
Nov 07 00:22:10 tyler-centos systemd[1]: kibana.service: Service RestartSec=3s expired, scheduling restart.
Nov 07 00:22:10 tyler-centos systemd[1]: kibana.service: Scheduled restart job, restart counter is at 3.
Nov 07 00:22:10 tyler-centos systemd[1]: Stopped Kibana.
Nov 07 00:22:10 tyler-centos systemd[1]: kibana.service: Start request repeated too quickly.
Nov 07 00:22:10 tyler-centos systemd[1]: kibana.service: Failed with result 'exit-code'.
Nov 07 00:22:10 tyler-centos systemd[1]: Failed to start Kibana.
Contributor
Author
|
@elasticmachine merge upstream |
Contributor
Author
|
Do you have the logs above/have the environment available still? I just ran through upgrade and fresh install scenarios and couldn't reproduce. I'll get fresh builds up in case there's any conflicts. edit: |
Contributor
Author
|
@elasticmachine merge upstream |
Contributor
💚 Build SucceededMetrics [docs]
History
To update your PR or re-run it, just comment with: |
Member
|
LGTM - something must have been up with my test env. |
tylersmalley
approved these changes
Nov 17, 2020
Contributor
Author
|
7.x/7.11: 242eae0 |
phillipb
added a commit
to phillipb/kibana
that referenced
this pull request
Nov 18, 2020
…o-node-details * 'master' of github.com:elastic/kibana: (65 commits) update chromedriver dependency to 87 (elastic#83624) [TSVB] use new Search API for rollup search (elastic#83275) [TSVB] Y-axis has number formatting not considering all series formatters in the group (elastic#83438) [Logs UI] Update <LogStream /> internal state when its props change (elastic#83302) Add tag bulk action context menu (elastic#82816) [code coverage] adding plugin to flush coverage data (elastic#83447) [UsageCollection] Expose `KibanaRequest` to explicitly opted-in collectors (elastic#83413) Added eventBus to trigger and listen plotHandler event (elastic#83435) [Runtime fields] Editor phase 1 (elastic#81472) [Maps] Fix threshold alert issue resolving nested fields (elastic#83577) chore(NA): remove usage of unverified es snapshots (elastic#83589) [DOCS] Adds Elastic Contributor Program link (elastic#83561) Upgrade EUI to v30.2.0 (elastic#82730) Don't show loading screen during auto-reload (elastic#83376) Functional tests - fix esArchive mappings with runtime fields (elastic#83530) [deb/rpm] Create keystore after installation (elastic#76465) [rpm] Create default environment file at "/etc/sysconfig/kibana" (elastic#82144) [docker] removes workaround for missing crypto-policies-scripts subpackage (elastic#83455) [ML] Persisted URL state for the Data frame analytics jobs and models pages (elastic#83439) adds xpack.security.authc.selector.enabled setting (elastic#83551) ...
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
instead of "/etc/default/kibana".
This is a standards difference. rpm based distributions generally
use /etc/sysconfig. In production we're reading from both, with
sysconfig second to override any historical differences.
Related to #36671, #6423
Testing: