search

xddxdd / bird-lg-go

BIRD looking glass in Go, for better maintainability, easier deployment & smaller memory footprint
GNU General Public License v3.0
141 stars 27 forks source link

Include mtr in the `bird-lgproxy-go` docker image #100

Closed nervous-inhuman closed 8 months ago

nervous-inhuman commented 9 months ago

Currently, the bird-lgproxy-go ships with the ability to use mtr, however, the container image does not contain it. Would it be possible to add mtr to the container, as it is currently being done with tracerotue?

Thanks!

xddxdd commented 8 months ago

I just added a xddxdd/bird-lgproxy-go:develop-mtr docker image. Can you try it out? I will make a new release once you confirm it's working.

I'm creating a new image because adding mtr to the existing image will change the default behavior.

nervous-inhuman commented 8 months ago

@xddxdd It seems to be picked up, and is usable within the birg-lg frontend

birdlg_proxy  | main.settingType{birdSocket:"/var/run/bird/bird.ctl", listen:"8000", allowedNets:[]*net.IPNet{}, tr_bin:"", tr_flags:[]string{}, tr_raw:false}
birdlg_proxy  | Warning on reading config: Config File "bird-lgproxy" Not Found in "[/ /etc/bird-lg]"
birdlg_proxy  | Traceroute autodetect success: mtr -w -c1 -Z1 -G1 -b 127.0.0.1
birdlg_proxy  | Listening on 8000...
xddxdd commented 8 months ago

Thank you for testing. New version has been released. You can switch to xddxdd/bird-lgproxy-go:latest-mtr for the stable version once the CI build is complete.