Bug 244304 - net-mgmt/netdata: add godplugin option
Summary: net-mgmt/netdata: add godplugin option
Status: New
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Mahdi Mokhtari
URL:
Keywords: buildisok, patch
Depends on: 244291
Blocks:
  Show dependency treegraph
 
Reported: 2020-02-22 10:23 UTC by Dries Michiels
Modified: 2020-03-26 07:30 UTC (History)
2 users (show)

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


Attachments
netdata.diff (3.52 KB, patch)
2020-02-22 10:23 UTC, Dries Michiels
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dries Michiels 2020-02-22 10:23:12 UTC
Created attachment 211829 [details]
netdata.diff

This links the package with the godplugin. Netdata is in the process of migrating all python plugins to go (so default on option to future proof and help in the transition).

QA: portlint -AC
testport option on current amd64
Comment 1 Automation User 2020-03-08 00:31:34 UTC
Build info is available at https://gitlab.com/swills/freebsd-ports/pipelines/124236653
Comment 2 daniel.engberg.lists 2020-03-25 06:41:59 UTC
Hi Dries!

I'm a bit confused about what this is supposed to. Are you trying to provide a "simple" way of installing the go-plugins?
Comment 3 Dries Michiels 2020-03-25 14:47:02 UTC
This makes the netdata port install netdata-go as dependency. Right now the python plugin is included in the netdata repo upstream by default. Meaning it is always installed when installing netdata. This is not the case for the god.plugin. And as netdata is in the progress to convert all python plugins to go plugins, it makes sense to enable the option by default.
Comment 4 daniel.engberg.lists 2020-03-26 06:25:25 UTC
It's more consistent to follow upstream rather than diverging and there are a lot cases where you don't need the go plugins at all also go is a pretty large dependency. Until upstream decides to fully change I think it's better to display a notice like other ports does (like when python extensions are available in the tree) for addtional funtionality via other ports.
Comment 5 Dries Michiels 2020-03-26 07:30:45 UTC
We could do that! I don't really have any hidden agenda's although if we don't want to diverge from upstream we should actually enable it. On there github README, https://github.com/netdata/go.d.plugin, they explicitly state it should be shipped with netdata. "Install Shipped with Netdata." But at least I would like to have the option toggler thingy :-D, not necessarily enabled by default.