Add Dockerfile-go#1828
Conversation
955a76f to
2476631
Compare
anthony-gomez-fastly
left a comment
There was a problem hiding this comment.
TY for your contribution! LGTM, once @kpfleming reviews we can see about merging
kpfleming
left a comment
There was a problem hiding this comment.
I'm fine with this being merged as-is, but it (of course) suffers from the same issues that the others do. The example command for using the image is ... compute serve, which means that Viceroy will have to be downloaded and that sort of defeats the purpose of using a container image since it isn't self-contained and requires Internet access at runtime (and the Viceroy binary will have to be downloaded every time, unlike a local CLI installation where it will only be downloaded once).
If the CLI had some command which could be used to trigger the download of the ancillary binaries that could be used in the Dockerfile to avoid this problem... but it doesn't :-)
|
@kpfleming #1833 adds a |
Change summary
Adds a
Dockerfile-goto the repo, mirroring the existingDockerfile-nodeandDockerfile-rustbut based on thegolang:latestimage. Like the others, this is a convenience Dockerfile that users can build to get a container with the Fastly CLI installed.Build/run usage is documented in comments at the bottom of the file:
All Submissions:
User Impact
Adds a Go-based Dockerfile option alongside the existing Node and Rust ones for users who prefer to build their own Fastly CLI container. No impact on existing functionality or users.
Are there any considerations that need to be addressed for release?
None. No breaking changes, and the image is not part of any build or release pipeline.