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

Helm chart container image related parameter (pullsecret, pullpolicy) support added. #55

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

Conversation

europ
Copy link

@europ europ commented Nov 1, 2023

Problem

  • missing image pull policy parameter
  • missing image pull secret name parameter
  • inconsistent naming convention
  • inconsistent parameter format
  • unnecessary secret object definition

Solution

Added support for currently not supported parameters:

  • image pull secret
  • image pull policy

Modified parameter:

  • image url

@europ europ changed the title Helm chart container image related parameters modified. Helm chart container image related parameter (pullsecret, pullpolicy) support added. Nov 1, 2023
@europ europ force-pushed the missing_parameter branch 2 times, most recently from 11e6f23 to 0a4c9b3 Compare November 2, 2023 07:57
@@ -25,11 +25,12 @@ spec:
fsGroup: {{ default 0 .Values.deployment.security.fsGroup }}
serviceAccountName: {{ include "mgmt-agent.serviceAccount" . }}
imagePullSecrets:
- name: {{ include "mgmt-agent.resourceNamePrefix" . }}-mgmt-agent-container-registry-key
Copy link
Member

Choose a reason for hiding this comment

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

@niravg75 , please review the Mgmt Agent changes suggested in this PR.

* image url
* image pull secret
* image pull policy
@europ europ force-pushed the missing_parameter branch from 0a4c9b3 to 3a7b13b Compare November 6, 2023 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants