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

[DE597167] Fix Gateway Helm installation failure due to incorrect indentation in secret.yaml file #304

Closed
wants to merge 41 commits into from

Conversation

snagarjunas
Copy link

Description of the change

When the User has additionalAnnotations custom_values, the Gateway Helm installation fails due to incorrect indentation in secret.yaml file.

Annotations are enabled when "additionalAnnotations" are passed from custom_values. By default "additionalAnnotations" are not set. When additional annotations are passed, the annotations section is enabled and fails due to incorrect indentation.

Benefits

Users can successfully add "additionalAnnotations" as required

kiran vaddadi and others added 30 commits January 27, 2023 15:01
[charts/gateway] Jm665334 gw pdb
Scope to add dynamic system properties
Scope to add dynamic system properties
[charts/gateway] - Adding provision to add additional system properties
jmmAVGO and others added 11 commits September 15, 2023 15:06
shared with the rest of the gw pods. bump chart
yaml.
# Conflicts:
#	charts/gateway/Chart.yaml
#	charts/gateway/README.md
#	charts/gateway/production-values.yaml
#	charts/gateway/templates/otk-install-job.yaml
#	charts/gateway/values.yaml
charts/gateway redis support, ingress update, restart on config change

Co-authored-by: Gazza7205 <[email protected]>
[charts/gateway] Independent pmtagger PDB values
@APIM-KiranVaddadi APIM-KiranVaddadi changed the base branch from develop/gateway to kv664763_otk_patch April 2, 2024 04:34
@APIM-KiranVaddadi APIM-KiranVaddadi changed the base branch from kv664763_otk_patch to stable April 2, 2024 04:35
@Gazza7205
Copy link
Contributor

Covered in #303

@Gazza7205 Gazza7205 closed this Apr 2, 2024
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.

9 participants