Fix Dockerfile entrypoint to use Binary instead of Name#435
Open
Fix Dockerfile entrypoint to use Binary instead of Name#435
Conversation
The Dockerfile.std template was using `.Name` (repo name) for the entrypoint, but should use `.Binary` (the actual binary name). This caused issues for ai-studio where: - Name = ai-studio - Binary = tyk-ai-studio - PackageName = tyk-ai-studio The entrypoint was incorrectly set to /opt/tyk-ai-studio/ai-studio instead of /opt/tyk-ai-studio/tyk-ai-studio. This fix ensures the Docker container uses the correct binary path for all repositories. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
.Binaryinstead of.Namefor the entrypointProblem
The Dockerfile.std template was generating incorrect entrypoints for repositories where the binary name differs from the repo name. For example, ai-studio:
ai-studiotyk-ai-studiotyk-ai-studioThe entrypoint was incorrectly set to
/opt/tyk-ai-studio/ai-studioinstead of/opt/tyk-ai-studio/tyk-ai-studio.Solution
Changed the template to use
{{ .Binary }}instead of{{ .Name }}for the entrypoint path.Test plan
ENTRYPOINT ["/opt/tyk-ai-studio/tyk-ai-studio" ]ENTRYPOINT ["/opt/tyk-gateway/tyk" ]ENTRYPOINT ["/opt/tyk-pump/tyk-pump" ]ENTRYPOINT ["/opt/tyk-dashboard/tyk-analytics" ]🤖 Generated with Claude Code