Docker container

[luc122c]

Hi there. I found myself wanting to use this on a number of machines which all had docker installed. I quickly build your script into a distroless docker container. I've also setup a Github action to publish it as a package so it's available more widely.

[KNawm]

Hey, thanks for the contribution but I'm not sure if it would be very useful, mind sharing your opinion?

[luc122c]

As I mentioned, this is useful when you want to use this application on a system which may not have node/npm installed, but does have a docker runtime. You can easily use this in a single command without having to install any software. With the --rm flag, there is no extra cleanup required (uninstalling etc). The same command can be used on multiple systems with the exact same results, regardless of the environment.

[apalfrey]

Was hoping for something like this, tested it on my server and it works great!

[agneevX]

Can Multiarch support be added?

[luc122c]

I used a distroless image as a base for performance and security. There is a discussion about multi-arch support here: GoogleContainerTools/distroless#346. Once this happens, it should filter upstream to this image.

[CappyT]

This is perfect ffor speedtesting K8s on-prem nodes and my swarm of Flatcar linux vms

[daluu]

I see someone posted up a docker version of this at https://hub.docker.com/r/p3terx/speed-cloudflare

[daluu]

if/when ready to merge, this probably needs to be renamed for the target repo this links to?

[luc122c]

The workflow file pushes to github.repository so currently, it publishes to my fork. You're correct that when merging, it would need updating to the new location.