Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 654630 - app-misc/elasticsearch-6.2.3: init.d script ignores $LOG_DIR variable
Summary: app-misc/elasticsearch-6.2.3: init.d script ignores $LOG_DIR variable
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Tomáš Mózes
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-05-02 14:17 UTC by manwe
Modified: 2018-05-25 13:38 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---

Attachments
patch (elasticsearch.initd.patch,897 bytes, patch)
2018-05-02 14:18 UTC, manwe 		Details | Diff
patch (elasticsearch.initd.patch,927 bytes, patch)
2018-05-02 14:19 UTC, manwe 		Details | Diff
patch (es.patch,895 bytes, patch)
2018-05-02 20:11 UTC, manwe 		Details | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description manwe 2018-05-02 14:17:41 UTC 
Config conf.d/elasticsearch contains LOG_DIR variable (not set by default) that should point ES to directory for log storage. But init.d/elasticsearch always overwrites that variable with /var/log/elasticsearch/${ES_INSTANCE} 

Here's patch fixing that. 

Reproducible: Always
Comment 1 manwe 2018-05-02 14:18:02 UTC 
Created attachment 529406 [details, diff]
patch
Comment 2 manwe 2018-05-02 14:19:39 UTC 
Created attachment 529408 [details, diff]
patch

Once again, typo
Comment 3 manwe 2018-05-02 20:11:48 UTC 
Created attachment 529484 [details, diff]
patch

Third time's a charm…
Comment 4 Larry the Git Cow gentoo-dev 2018-05-25 13:14:40 UTC 
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=426ed349ab5ba38a3a3a2266a2d5199ad2d206a4

commit 426ed349ab5ba38a3a3a2266a2d5199ad2d206a4
Author:     Tomas Mozes <hydrapolic@gmail.com>
AuthorDate: 2018-05-24 12:04:07 +0000
Commit:     Aaron Bauman <bman@gentoo.org>
CommitDate: 2018-05-25 13:14:01 +0000

    app-misc/elasticsearch: allow to override LOG_DIR
    
    Closes: https://bugs.gentoo.org/654630
    Package-Manager: Portage-2.3.38, Repoman-2.3.9
    Closes: https://github.com/gentoo/gentoo/pull/8547

 app-misc/elasticsearch/files/elasticsearch.init.3 | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)
Comment 5 Tomáš Mózes 2018-05-25 13:38:22 UTC 
Thanks for the report and patch manwe.