[stale] add OpenBSD support for generating the machine id if /etc/machine-id …

[rnagy]

…is unavailable

use the hw.uuid sysctl on openbsd to get the hardware UUID in case /etc/machine-id
is not available because /proc is linux-only

[AlteredCoder]

Hello @rnagy , thanks for your contribution!

Can you also push the go.mod please ? The CI failed because of this i think :/

[rnagy]

The go.mod I have has no diffs to the one in HEAD.

[AlteredCoder]

Do you have a link to the documentation of the syscall module that you use ?
In the official documentation (https://golang.org/pkg/syscall/) , the Sysctl function doesn't exist.
What is your go version also please ?

[rnagy]

go version go1.15.2 openbsd/amd64

Anyway it seems that syscall is deprecated, so let me get back to you with another solution.

[AlteredCoder]

Ok @rnagy , thanks !

[cuddytrippin]

T

[rnagy]

?

[buixor]

@rnagy : I reported @cuddytrippin for spam on other issues as well :/

[AlteredCoder]

@rnagy it seems that the build still doesn't work :/, let me know if you need any help on this

[sbz]

The library used for getting the host uuid machineid does not support OpenBSD yet, I'm suggesting instead of patching crowdsec to provide the OpenBSD support in machineid

Currently machineid for the 'BSD' use /etc/hostid which is valid on FreeBSD and fallback using kenv which is not available on OpenBSD. I propose you add for OpenBSD the following:

Read from /etc/machine-id if it exists as proposed by issues#9 and fallback using sysctl hw.uuid like that the usage of machine.ID() call should be able to handle the OpenBSD case 🤝

The fallback strategy is implemented here for the BSD in machineid

For the context, I started to do the same patch with https://gist.github.com/sbz/3c23c52d6817246b3682922944a9c3ba but I realized after it's not needed.

[rnagy]

The library used for getting the host uuid machineid does not support OpenBSD yet, I'm suggesting instead of patching crowdsec to provide the OpenBSD support in machineid

Currently machineid for the 'BSD' use /etc/hostid which is valid on FreeBSD and fallback using kenv which is not available on OpenBSD. I propose you add for OpenBSD the following:

Read from /etc/machine-id if it exists as proposed by issues#9 and fallback using sysctl hw.uuid like that the usage of machine.ID() call should be able to handle the OpenBSD case

The fallback strategy is implemented here for the BSD in machineid

For the context, I started to do the same patch with https://gist.github.com/sbz/3c23c52d6817246b3682922944a9c3ba but I realized after it's not needed.

That project seems dead, but it is worth a try... i've submitted a pull request. This ecosystem is really not good.

[AlteredCoder]

Hello @rnagy , we will wait to see if they accept your PR. If not, we will integrate those changes in crowdsec

[AlteredCoder]

Hello @rnagy , your PR in the machineid project is still pending .. If it still the case at the end of the week, we will add your code to crowdsec directly next week.

[rnagy]

Hello @rnagy , your PR in the machineid project is still pending .. If it still the case at the end of the week, we will add your code to crowdsec directly next week.

What if I fork that project and then you can point the build system to use that repo?

[AlteredCoder]

Hello @rnagy, sorry for the delay

Yes that might be a good idea :)