Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update SIDECAR_IMAGE tag in Helm chart variable #4020

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

0xaravindh
Copy link
Contributor

What type of PR is this?

Uncomment only one /kind <> line, press enter to put that in a new line, and remove leading whitespace from that line:

/kind breaking
/kind bug

/kind cleanup

/kind documentation
/kind feature
/kind hotfix
/kind release

What this PR does / Why we need it:

Which issue(s) this PR fixes: #3979

Closes #3979

Special notes for your reviewer:

@github-actions github-actions bot added kind/cleanup Refactoring code, fixing up documentation, etc size/XS labels Oct 22, 2024
@agones-bot
Copy link
Collaborator

Build Failed 😭

Build Id: 28ed4343-6ce7-417f-b808-5499ac931a10

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@peterzhongyi
Copy link
Collaborator

Issue #3979 mentioned the specific use case for adding a separate variable to specify the sidecar image is "useful in situations where the sidecar image needs to be pulled from a private registry". Can a image from a private registry be simply specified by a tag, or does it need some sort of repository url with a tag? In other words, how would pulling from a sidecar image a private registry be done with the tag?

@0xaravindh
Copy link
Contributor Author

We are currently facing an issue with specifying the agones.sdk tag #3978. We are unable to specify the Windows tag and are only using the global tag.To address this, I have declared a specific tag for the SDK images. My question is whether we need to specify the entire URL, or if it would be sufficient to provide an option to declare the other Agones SDK tags? @peterzhongyi

@peterzhongyi
Copy link
Collaborator

@kangjk1017 Since you opened this issue, I thought it might be helpful to mention you here - do you think adding a variable to accept other Agones SDK tags will help fix the issue?

@@ -123,7 +123,7 @@ spec:
value: {{ .Values.gameservers.additionalPortRanges | toJson | quote }}
{{- end }}
- name: SIDECAR_IMAGE # overwrite the GameServer sidecar image that is used
value: "{{ .Values.agones.image.registry }}/{{ .Values.agones.image.sdk.name}}:{{ default .Values.agones.image.tag .Values.agones.image.sdk.tag }}"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think what was asking for is expose a variable to set the registry for SIDECAR_IMAGE separately rather than using agones.image.registry

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

user has provided a comment regarding the same queries. (#3979 (comment))

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/cleanup Refactoring code, fixing up documentation, etc size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Helm Chart Variable for Configuring SIDECAR_IMAGE Independently
4 participants