Skip to content

[deb/rpm] Fix SysV init scripts - #95018

Merged
jbudz merged 4 commits into
elastic:7.xfrom
jbudz:fix/sysv
Mar 29, 2021
Merged

jbudz merged 4 commits into
elastic:7.xfrom
jbudz:fix/sysv

Conversation

@jbudz

@jbudz jbudz commented Mar 19, 2021 •

Copy link
Copy Markdown
Contributor

This includes a few fixes:

  1. Default values for $chdir, $chroot, and $nice
  2. Redirects stdout to startup.log. We're using the
    kibana logger to write to kibana.log Previously, these were both
    writing to the same file which would cause permission conflicts between
    the startup script (user: root) and kibana server (user: kibana)
  3. Interpolation between startup args and bin

Testing

Closes #94931

Release note
Fixes issues preventing the SysV service from starting Kibana.

This includes a few fixes:
1) Default values for $chdir, $chroot, and $nice
2) Redirects stdout to startup.log.  We're using the
kibana logger to write to kibana.log  Previously, these were both
writing to the same file which would cause permission conflicts between
the startup script (user: root) and kibana server (user: kibana)
3) Interpolation between startup args and bin
@elasticmachine

Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@jbudz

jbudz commented Mar 22, 2021

Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@jbudz jbudz added v7.12.1 and removed v7.12.0 labels Mar 24, 2021
@jbudz

jbudz commented Mar 24, 2021

Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@jbudz

jbudz commented Mar 29, 2021

Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@tylersmalley
tylersmalley requested a review from a team March 29, 2021 17:39

@mistic mistic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@kibanamachine

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@jbudz jbudz added the auto-backport Deprecated - use backport:version if exact versions are needed label Mar 29, 2021
@jbudz
jbudz merged commit 0e9aa23 into elastic:7.x Mar 29, 2021
@jbudz
jbudz deleted the fix/sysv branch March 29, 2021 20:26
jbudz added a commit to jbudz/kibana that referenced this pull request May 24, 2021
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@jbudz jbudz added v7.12.2 and removed auto-backport Deprecated - use backport:version if exact versions are needed v7.12.1 labels May 24, 2021
jbudz added a commit that referenced this pull request May 24, 2021
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
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.

4 participants