Skip to content

Conversation

@2bleere
Copy link

@2bleere 2bleere commented Jul 30, 2025

Updated Dockerfile for multi-arch build for amd64 and arm64 downloading netdiscover binary for the right architecture from an alternate repo. Updated base image to bookworm slim and updated legacy ENV variables in Dockerfile.

Build with dockerbuildx:

docker buildx build --platform linux/amd64,linux/arm64 -t your-image:tag .

…g netdiscover binary for the right architecture from an alternate repo. Updated base image to bookworm slim and updated legacy ENV variables in Dockerfile. Build with dockerbuildx: docker buildx build --platform linux/amd64,linux/arm64 -t your-image:tag .
@psanders
Copy link
Member

Hi @2bleere,

Seems straightforward. I'll give it a try soon and report back. Thanks!

@psanders
Copy link
Member

psanders commented Aug 3, 2025

Since netdiscover is relatively small, I wonder if we should have it live directly in the repository in a "binary" folder. Also, it would be good to update the workflow to target the corresponding operating systems. What are your thoughts?

@2bleere
Copy link
Author

2bleere commented Aug 5, 2025

I think that works too, the build targets OS/Architecture currently only linux arm64 and amd64 as rtpengine requires linux as the OS, but if you have a different approach we can try. We can include the compiling for netdiscover in the same Dockerfile, but my guess is that the resulting image will be considerably larger as go needs to be installed in the base to compile netdiscover afterwards. There are many ways we can try, let me know which one you'd like and I can test accordingly.

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.

2 participants