Bug 261979 - net-mgmt/netdata: Update to v1.33.1
Summary: net-mgmt/netdata: Update to v1.33.1
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Neel Chauhan
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-02-16 01:22 UTC by Namkhai B.
Modified: 2022-02-19 00:07 UTC (History)
0 users

See Also:
nc: maintainer-feedback+


Attachments
net-mgmt/netdata: Update to v1.33.1 (9.39 KB, patch)
2022-02-16 01:22 UTC, Namkhai B.
no flags Details | Diff
net-mgmt/netdata: Remove claim script when CLOUD=off (1.38 KB, patch)
2022-02-16 23:59 UTC, Namkhai B.
no flags Details | Diff
net-mgmt/netdata: Depend on protobuf when CLOUD=on (800 bytes, patch)
2022-02-18 23:18 UTC, Namkhai B.
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Namkhai B. 2022-02-16 01:22:59 UTC
Created attachment 231849 [details]
net-mgmt/netdata: Update to v1.33.1

As with other updates, source code for the dashboard patch can be found on https://github.com/forkbomb9/dashboard/tree/freebsd

A change I wanted feedback on (haven't done it yet) is whether netdata-claim.sh should be only installed with CLOUD=on, as it's only related to the cloud functionality and might be broken with CLOUD=off.
Comment 1 Neel Chauhan freebsd_committer freebsd_triage 2022-02-16 02:27:39 UTC
Committed!
Comment 2 commit-hook freebsd_committer freebsd_triage 2022-02-16 02:27:51 UTC
A commit in branch main references this bug:

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

commit f15b83bfe6170aed4eab052f30573e7a6b49d545
Author:     Namkhai B <namkhai.n3@protonmail.com>
AuthorDate: 2022-02-16 01:19:20 +0000
Commit:     Neel Chauhan <nc@FreeBSD.org>
CommitDate: 2022-02-16 02:27:11 +0000

    net-mgmt/netdata: Update to v1.33.1

    PR:     261979

 net-mgmt/netdata/Makefile  |   6 +--
 net-mgmt/netdata/distinfo  |  10 ++---
 net-mgmt/netdata/pkg-plist | 104 ++++++++++++++++++++++-----------------------
 3 files changed, 60 insertions(+), 60 deletions(-)
Comment 3 Namkhai B. 2022-02-16 02:29:33 UTC
Woha, that was fast!

What do you think about removing netdata-claim.sh with CLOUD=off?
Comment 4 Neel Chauhan freebsd_committer freebsd_triage 2022-02-16 02:36:33 UTC
It's a good idea IMO.
Comment 5 Namkhai B. 2022-02-16 16:43:07 UTC
What would be the best way to do it? Remove it when CLOUD=off or list it in pkg-plist as %%CLOUD%%? Or both?
Comment 6 Neel Chauhan freebsd_committer freebsd_triage 2022-02-16 16:53:57 UTC
Do both.
Comment 7 Namkhai B. 2022-02-16 23:59:53 UTC
Created attachment 231879 [details]
net-mgmt/netdata: Remove claim script when CLOUD=off

How's this looking?
Comment 8 Namkhai B. 2022-02-18 23:18:47 UTC
Created attachment 231936 [details]
net-mgmt/netdata: Depend on protobuf when CLOUD=on

Netdata started rolling out their new ACLK and cloud nodes not supporting protobuf will stop reporting after March 1st.

Building with Protobuf is enough to enable support for it.

Depends on it only when cloud is enabled, as it's useless without cloud support.
Comment 9 commit-hook freebsd_committer freebsd_triage 2022-02-19 00:07:33 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=322d74cac2358470a0ba1aa5e93f2e54971397c9

commit 322d74cac2358470a0ba1aa5e93f2e54971397c9
Author:     Namkhai B <namkhai.n3@protonmail.com>
AuthorDate: 2022-02-16 23:40:25 +0000
Commit:     Neel Chauhan <nc@FreeBSD.org>
CommitDate: 2022-02-19 00:06:35 +0000

    net-mgmt/netdata: Remove claim script when CLOUD=off

    PR:     261979

 net-mgmt/netdata/Makefile  | 4 ++++
 net-mgmt/netdata/pkg-plist | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)
Comment 10 commit-hook freebsd_committer freebsd_triage 2022-02-19 00:07:34 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=5a36d25faa5863fc893353a36034c897d9b23b97

commit 5a36d25faa5863fc893353a36034c897d9b23b97
Author:     Namkhai B <namkhai.n3@protonmail.com>
AuthorDate: 2022-02-18 23:10:20 +0000
Commit:     Neel Chauhan <nc@FreeBSD.org>
CommitDate: 2022-02-19 00:07:05 +0000

    net-mgmt/netdata: Depend on protobuf when CLOUD=on

    PR:     261979

 net-mgmt/netdata/Makefile | 1 +
 1 file changed, 1 insertion(+)