Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix binary releases, add RC service example #116

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

Kariton
Copy link

@Kariton Kariton commented Oct 14, 2023

  • fix cross install for build
  • fix freebsd build
  • fix automated binary release
  • add RC service example (OPNsense) to README.MD

I have no clue how GitHub actions work and how i could make a release in my fork.
But the artifacts are there and working (at least FreeBSD).

fixes #59
closes #91
resolves #117
resolves #118
resolves #119

p.s.: I still have no clue about github actions. but it works :)
see https://github.com/Kariton/prometheus_wireguard_exporter/releases/tag/v3.6.6

@Kariton
Copy link
Author

Kariton commented Oct 18, 2023

The initially documented "adduser" command created an user with UID > 1000 - and was unable to survive a reboot.
With an ID < 1000 e.g. 518 (needs to be unused!) it works.

i choose 518 because 5180 is the default port for wireguard; there is actually no real reason.
i just tried to find a package (common service user) that already uses this id and there was none

@Kariton
Copy link
Author

Kariton commented Oct 22, 2023

@MindFlavor i actually managed to get it working.
this will close other PRs as stated.

any chance to get this reviewed and merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

It would be nice if github releases had binaries for download
1 participant