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

Update pleaserun and system-install #6660

Closed
wants to merge 1 commit into from

Conversation

untergeek
Copy link
Member

This will now put ENV variables from the startup.options file into /etc/default/logstash or /etc/sysconfig/logstash (or whatever service name you chose), and use the updated pleaserun to ensure these are honored at start time for whichever init system you use (systemd, upstart, SysV).

fixes #6482

This will now put ENV variables from the `startup.options` file into `/etc/default/logstash` or `/etc/sysconfig/logstash` (or whatever service name you chose), and use the updated pleaserun to ensure these are honored at start time for whichever init system you use (systemd, upstart, SysV).

fixes elastic#6482
@jordansissel
Copy link
Contributor

Testing manually, I made this change to the startup.options:

-LS_HOME=/usr/share/logstash
+LS_HOME=/somewhere/over/the/rainbow

Running with this patch:

% sudo bin/system-install config/startup.options upstart
...
% grep LS_HOME /etc/default/logstash
LS_HOME="/somewhere/over/the/rainbow"

Looking good. The rest of the file (/etc/default/logstash) has many variables set -- all of which I expect to have been forwarded from startup.options based on this PR.

Testing master (without this patch):

% sudo bin/system-install config/startup.options upstart
% wc -l /etc/default/logstash
0 /etc/default/logstash

LGTM!

Copy link
Contributor

@jordansissel jordansissel left a comment

Choose a reason for hiding this comment

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

Tested manually. LGTM.

@suyograo suyograo added the v5.3.0 label Feb 9, 2017
Copy link
Contributor

@suyograo suyograo left a comment

Choose a reason for hiding this comment

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

LGTM

@elasticsearch-bot
Copy link

Aaron Mildenstein merged this into the following branches!

Branch Commits
master 0d90e12

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.

JAVACMD in startup.options not taken into account with upstart
4 participants