Bug 253314

Summary: textproc/kibana7: fix memleak and some port improvements
Product: Ports & Packages Reporter: Boris Korzun <drtr0jan>
Component: Individual Port(s)Assignee: Juraj Lutter <otis>
Status: Closed FIXED    
Severity: Affects Some People CC: feld, gettoknowmi, otis
Priority: --- Keywords: patch
Version: LatestFlags: bugzilla: maintainer-feedback? (elastic)
Hardware: Any   
OS: Any   
URL: https://github.com/elastic/kibana/issues/90274
See Also: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=236243
Attachments:
Description Flags
kibana7.diff
none
kibana7.diff none

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.
Comment 3 commit-hook freebsd_committer freebsd_triage 2021-07-06 20:46:19 UTC 
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=1457ecce2ae6702e6600617902b3b3a8698c416e

commit 1457ecce2ae6702e6600617902b3b3a8698c416e
Author:     Juraj Lutter <otis@FreeBSD.org>
AuthorDate: 2021-07-05 09:09:43 +0000
Commit:     Juraj Lutter <otis@FreeBSD.org>
CommitDate: 2021-07-06 20:44:38 +0000

    textproc/kibana7: Update to 7.13.2

    - Update to 7.13.2
    - kibana since 7.13.0 ships with re2.node shared library
      built for Linux, Windows or Darwin. Build a native one
      for FreeBSD as a part of the port build. This involves:
        - Build node-nan
        - Build node-re2
    - Fix memleak [1]

    PR:             253314 [1]

 textproc/kibana7/Makefile                          | 71 +++++++++++++++++++---
 textproc/kibana7/distinfo                          | 10 ++-
 .../files/extra-node-re2-binding-gyp.patch (new)   | 51 ++++++++++++++++
 textproc/kibana7/files/kibana.in                   | 56 ++++++++++++-----
 4 files changed, 163 insertions(+), 25 deletions(-)