Bug 253314 - textproc/kibana7: fix memleak and some port improvements
Summary: textproc/kibana7: fix memleak and some port improvements
Status: Open
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Juraj Lutter
URL: https://github.com/elastic/kibana/iss...
Keywords: patch
Depends on:
Blocks:
 
Reported: 2021-02-06 21:50 UTC by Boris Korzun
Modified: 2021-02-11 08:15 UTC (History)
3 users (show)

See Also:
bugzilla: maintainer-feedback? (elastic)


Attachments
kibana7.diff (2.71 KB, patch)
2021-02-06 21:50 UTC, Boris Korzun
no flags Details | Diff
kibana7.diff (2.91 KB, patch)
2021-02-11 08:15 UTC, Boris Korzun
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Boris Korzun 2021-02-06 21:50:27 UTC
Created attachment 222228 [details]
kibana7.diff

There's a memleak in Kibana ( https://github.com/elastic/kibana/issues/90274 ). I think it is involved by incorrect using parameters xpack.ml.enabled and xpack.reporting.enabled added by ports r484109.

The options should be passing by config file or command line arguments, not by environmental variables (as a Docker).

I've passed the params by command line and memleak had fixed.

Also I've added some improvements to the rc-script:
- Added kibana_syslog_output_* options for writing daemon(8) stdout to syslog.
- Moved pidfile from /var/run/kibana to /var/run directory.
- Moved daemon(8) pidfile control from child (-p) to supervisor (-P). It is need for using -r/-R options (auto-restart after terminated, ex. after memory exhausting).
- Added environmental variables passing for FreeBSD 11.4 compatibility.
- Added --max-http-header-size=65536 node option (Kibana using).
Comment 1 Boris Korzun 2021-02-06 22:04:00 UTC
Also files/patch-config_kibana.yml with xpack.ml.enabled and xpack.reporting.enabled params was added at ports r484109.
But the patch was been deleted in bug #236243.
Comment 2 Boris Korzun 2021-02-11 08:15:06 UTC
Created attachment 222360 [details]
kibana7.diff

- Added "reload" extra option to rc-script.
- Removed "xpack.ml.enabled=false" as unnecessary.