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

wrong copy-circle-of-trust-certificates imagePullPolicy configuration #1881

Closed
Kebree opened this issue May 13, 2024 · 6 comments · Fixed by #1884
Closed

wrong copy-circle-of-trust-certificates imagePullPolicy configuration #1881

Kebree opened this issue May 13, 2024 · 6 comments · Fixed by #1884
Labels
bug Something isn't working

Comments

@Kebree
Copy link

Kebree commented May 13, 2024

Version of Helm and Kubernetes:
All

Which chart:
Since 107.84.10

Which product license (Enterprise/Pro/oss):
All

What happened:

Following the release of the latest update, the ha statefulset went from :

    - name: copy-circle-of-trust-certificates
       image: {{ include "artifactory-ha.getImageInfoByValue" (list . "initContainers") }}

to :

 - name: copy-circle-of-trust-certificates
       image: "{{ .Values.initContainerImage }}"
       imagePullPolicy: c

What you expected to happen:

The image pull policy cannot be 'c'

@chukka
Copy link
Collaborator

chukka commented May 13, 2024

@Kebree can you please share reproducible steps ? .Values.initContainerImage is depreciated in 107.84.10 version

@Kebree
Copy link
Author

Kebree commented May 13, 2024

Sorry, wrong paste from my part, the stable/artifactory-ha/templates/artifactory-primary-service.yaml file went from

 - name: copy-circle-of-trust-certificates
       image: "{{ .Values.initContainerImage }}"

to

 - name: copy-circle-of-trust-certificates
       image: {{ include "artifactory-ha.getImageInfoByValue" (list . "initContainers") }}
       imagePullPolicy: c

but anyway, the issue is not on the "image" part but on the "imagePullPolicy" part.

The reproducible step is setting the artifactory.circleOfTrustCertificatesSecret value.

@chukka chukka added the bug Something isn't working label May 13, 2024
@chukka
Copy link
Collaborator

chukka commented May 13, 2024

Thanks

@chukka
Copy link
Collaborator

chukka commented May 13, 2024

This has been fixed internally and will be available for next patch release

@chukka
Copy link
Collaborator

chukka commented May 17, 2024

@Kebree Please try 107.84.11 chart for the fix and share feedback

@Kebree
Copy link
Author

Kebree commented May 17, 2024

Services are starting now. Thank you for the resolution of this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants