Page MenuHomeFreeBSD
Differential D22705

New Port: www/nginx-vts-exporter: Simple server that scrapes Nginx vts stats and exports them via HTTP for Prometheus consumption
ClosedPublic
Actions

Authored by joneum on Dec 6 2019, 11:46 AM.
Tags
None
Referenced Files
F103182617: D22705.id65378.diff
Thu, Nov 21, 11:57 PM
Unknown Object (File)
Mon, Nov 18, 9:34 AM
Unknown Object (File)
Tue, Oct 29, 3:02 PM
Unknown Object (File)
Sat, Oct 26, 5:14 PM
Unknown Object (File)
Fri, Oct 25, 4:03 PM
Unknown Object (File)
Oct 21 2024, 6:09 PM
Unknown Object (File)
Oct 2 2024, 8:09 AM
Unknown Object (File)
Sep 30 2024, 3:30 PM
View All Files
Subscribers
mat

Details

Reviewers
tcberner
kai
Commits
rP519285: New Port: www/nginx-vts-exportr
Summary

Simple server that scrapes Nginx vts stats and exports
them via HTTP for Prometheus consumption.

WWW: https://github.com/hnlq715/nginx-vts-exporter

Test Plan

http://joneumbox.org/build.html?mastername=121amd64-ports&build=2019-12-06_12h35m10s
http://joneumbox.org/build.html?mastername=121amd64-ports&build=2019-12-06_12h35m10s

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joneum created this revision.Dec 6 2019, 11:46 AM
Herald added a subscriber: mat. · View Herald TranscriptDec 6 2019, 11:46 AM
Harbormaster completed remote builds in B28006: Diff 65310.Dec 6 2019, 11:46 AM
joneum updated this revision to Diff 65313.Dec 6 2019, 2:44 PM

Switch to the newer GO_BUILDFLAGS

Remove GO_PKGNAME, I think that's unnecessary

Harbormaster completed remote builds in B28007: Diff 65313.Dec 6 2019, 2:44 PM
tcberner added inline comments.Dec 7 2019, 8:20 AM
www/nginx-vts-exporter/Makefile
14 ↗(On Diff #65313)

^ I would sort these alphabetically

21 ↗(On Diff #65313)

^ you're only using this twice -- so maybe you could simply type it out there.

27 ↗(On Diff #65313)

^ why +=?

joneum updated this revision to Diff 65371.Dec 7 2019, 6:30 PM

Fix the suggestions from tcberner

Harbormaster completed remote builds in B28027: Diff 65371.Dec 7 2019, 6:30 PM
joneum marked 3 inline comments as done.Dec 7 2019, 6:31 PM
joneum updated this revision to Diff 65375.Dec 7 2019, 9:08 PM

fix a misstake

Harbormaster completed remote builds in B28028: Diff 65375.Dec 7 2019, 9:08 PM
joneum updated this revision to Diff 65376.Dec 7 2019, 9:13 PM

delete overflow row

Harbormaster completed remote builds in B28029: Diff 65376.Dec 7 2019, 9:13 PM
joneum updated this revision to Diff 65377.Dec 7 2019, 9:22 PM

delete unnecessary lines

Harbormaster completed remote builds in B28030: Diff 65377.Dec 7 2019, 9:22 PM
joneum updated this revision to Diff 65378.Dec 7 2019, 9:35 PM

Fix SUB_LIST=

Harbormaster completed remote builds in B28031: Diff 65378.Dec 7 2019, 9:35 PM
This revision was not accepted when it landed; it landed in state Needs Review.Dec 8 2019, 10:28 AM
Closed by commit rP519285: New Port: www/nginx-vts-exportr (authored by joneum). · Explain Why
This revision was automatically updated to reflect the committed changes.
joneum added a commit: rP519285: New Port: www/nginx-vts-exportr.

Revision Contents
Changeset List

PathSize
head/
www/
1 line
nginx-vts-exporter/
54 lines
3 lines
files/
4 lines
52 lines
6 lines
4 lines
13 lines

Diff 65402

View Options

head/www/Makefile

Loading...
View Options

head/www/nginx-vts-exporter/Makefile

Loading...
View Options

head/www/nginx-vts-exporter/distinfo

Loading...
View Options

head/www/nginx-vts-exporter/files/newsyslog.conf.in

Loading...
View Options

head/www/nginx-vts-exporter/files/nginx_vts_exporter.in

Loading...
View Options

head/www/nginx-vts-exporter/files/syslog.d.conf.in

Loading...
View Options

head/www/nginx-vts-exporter/pkg-descr

Loading...
View Options

head/www/nginx-vts-exporter/pkg-message

Loading...