Bug 233660

Summary: net-mgmt/pushgateway: Update to 0.6.0 (BREAKING CHANGES)
Product: Ports & Packages Reporter: Mateusz Piotrowski <0mp>
Component: Individual Port(s)Assignee: Mateusz Piotrowski <0mp>
Status: Open ---    
Severity: Affects Only Me CC: 0mp, aduitsis
Priority: --- Keywords: needs-qa, patch
Version: LatestFlags: aduitsis: maintainer-feedback+
Hardware: Any   
OS: Any   
URL: https://github.com/prometheus/pushgateway/releases/tag/v0.6.0
Description Flags
[WORK IN PROGRESS] Upgrade to v0.6.0 aduitsis: maintainer-approval+

Description Mateusz Piotrowski freebsd_committer 2018-11-30 11:47:31 UTC
Created attachment 199681 [details]
[WORK IN PROGRESS] Upgrade to v0.6.0

- https://github.com/prometheus/pushgateway/releases/tag/v0.4.0
- https://github.com/prometheus/pushgateway/releases/tag/v0.5.0
- https://github.com/prometheus/pushgateway/releases/tag/v0.5.2
- https://github.com/prometheus/pushgateway/releases/tag/v0.6.0

Also, https://github.com/prometheus/pushgateway/issues/175 suggests there were some breaking changes in the storage format between 0.4 and 0.5. I am not sure how to handle it as I don't use this software in production. We probably should add a note to UPDATING.

I'm attaching a patch with the following changes:
- Update to v0.6.0.
- Drop unnecessary subshell from do-build.
- Update the rc(8) service script to use long flags (support for short flags have been dropped).
- Extra:
  - Define do-test and pre-test to make it easier to run pushgateway's test suite. Test fail at the moment, I've not got time to investigate. The do-test implementation is not ideal too.

Certainly, the patch is not ready to be committed yet. It's merely a base for future work.
Comment 1 Athanasios Douitsis 2018-12-02 21:18:02 UTC
I'm the maintainer, this seems fine and thank you most kindly for the contribution. I set flags to '+', hope this is the correct thing to do around here.

If by any chance you'd like to maintain the port, please let me know what I must do.