-
Notifications
You must be signed in to change notification settings - Fork 702
Added a plugin for Netbird #4531
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
base: master
Are you sure you want to change the base?
Conversation
Thank you for doing this—was considering taking a hack at a netbird plugin since I also packaged/maintain it for openSUSE..but I'm very glad someone else did it. :) Really really hope it gets merged soon. |
@hrfried actually we're making it a requirement to have it included in FreeBSD ports first opnsense/ports#218 (comment) If you know some ins and outs for maintaining a package maybe you can help out here too :) |
I reached out to the Netbird team to see if they have any objections on me adding the port to FreeBSD. No answer yet. |
From experience, authors do not maintain ports and packages for varying reasons and there should be no harm to go ahead with it indeed as it is in the interest of the authors, too. Cheers, |
Just do it, I was also maintaining Cacti port for long time not beeing the main dev over there :) |
Ok, out of nowhere the Netbird now submitted Netbird to the FreeBSD Ports. But they have chosen security/netbird, Tailscale is also in security/. I was using net/ because I oriented on Zerotier, which is there. I think it needs to be consistent, right? Port is security/netbird, then the plugin would also be security/netbird? I would need to recreate the pull request. Did you maybe already had a chance to review the code for bigger issues? I would be happy to fix them. |
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=284877 looks like a netbird employee, oh well :) yes, let's align with security/netbird then. it will save a lot of confusion later. |
Update: the port request was accepted. How can we proceed here? Do we need to change the category from network to security like in the ports tree? |
It's now in our tree via opnsense/ports@9521b10081 -- let me make a quick review here and then the process is to add the port to ports.conf in tools.git and get the plugin merged too |
@@ -0,0 +1,30 @@ | |||
<?php | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
copyrights are important! bsd licensed of course
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As this plugin is done by multiple people, I need to ask if they want to be mentioned in the copyright note. This needs to be the header in all files containing PHP code?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, I added the copyright and license to all files containing PHP code and the Volt files. Please review. Thank you!
Start with 0.1 Co-authored-by: Franco Fichtner <[email protected]>
Co-authored-by: Franco Fichtner <[email protected]>
Co-authored-by: Franco Fichtner <[email protected]>
Hi. i have followed this thread for some time now, and have tried it on some test vm's, to see how well it works. |
To be honest, this process is running for a longer time now. The FreeBSD port needed to be added first. Will try the current plugin with the current port on a fresh installation. Could you share more information about the changes you need to do to the rc.d script? |
I replaced it with the one you had made for the port, but i believe the least that need to be changed or added is like below. rcvar=netbird_enable --config /usr/local/etc/netbird/config.json Also the os.release service is still being run, as it are in the template for "/etc/rc.conf.d/netbird" |
@KeenanFalcon thank you for clarifying. Interesting, I did not see that the port uses a different rc.d script. I tried to ship my version of the rc.d script with the plugin, but it raised a conflict during installation.
maybe someone has a better idea? The version from FreeBSD ports may be perfect for a standard FreeBSD system, but for the OPNsense we should run our own version of the rc.d file. This would be my version:
Maybe @fichtner or @mimugmail have an idea for this issue? |
@Gauss23 i tried to install it with an embedded rc script file too, but faced same issue so i renamed it to "os-netbird". My version of the script, it needs a little more polish, but maybe worth a pull-request.
I tried to add your repo, after i forked the master from opnsense/plugins, but didn't got it done quite right. |
@Gauss23 @KeenanFalcon hey guys, I'm the maintainer of the FreeBSD port of NetBird. @KeenanFalcon is the last rc.d that you've shared works fine? If yes, I will test it in a pure freebsd and commit to our freebsd port. Is os-release still an issue? What could be the alternative so I can update the codebase? |
@hakansa i made an comment about a posible fix here netbirdio/netbird#2200 (comment) The rc.d i posted here, i have worked some more on, as it didn't worked correctly with opnsense, and i have made an PR here freebsd/freebsd-ports#404 About the os-release issue i worked on a possible fix here netbirdio/netbird@main...KeenanFalcon:netbird:opnsense-devel but i havn't made any pull requests for this one |
I've created a submission for the rc.conf hooks. Also I'm going to check os-release thing, it should not be blocker for now. |
This is an initial version of a plugin for Netbird.
I've also created a pull request for the Netbird port, as a small patch is currently needed.
https://netbird.io/