<#356 Dockerize buf call> Pull request opened by <...
# flyte-github
a
#356 Dockerize buf call Pull request opened by hamersaw TL;DR Dockerizing the
buf
call so users do not need to manually install
buf
to generate protos. The PR uses the existing approach of setting the user id but requires manually setting the buf cache directory. If not set it will default to the root users home (ie.
/.cache
) which the caller likely does not have permissions to access. Therefore, we set it to
/tmp/cache
which should be globally accessible. Type ☐ Bug Fix ☑︎ Feature ☐ Plugin Are all requirements met? ☑︎ Code completed ☑︎ Smoke tested ☐ Unit tests added ☑︎ Code documentation added ☐ Any pending items have an associated Issue Complete description ^^^ Tracking Issue flyteorg/flyte#3139 Follow-up issue NA flyteorg/flyteidl All checks have passed 13/13 successful checks