Skip to content

fix: podLabels template conversion error on deployment templates#2986

Closed
fmorgad0 wants to merge 3 commits into
open-policy-agent:masterfrom
fmorgad0:podlabels-template-fix
Closed

fix: podLabels template conversion error on deployment templates#2986
fmorgad0 wants to merge 3 commits into
open-policy-agent:masterfrom
fmorgad0:podlabels-template-fix

Conversation

@fmorgad0
Copy link
Copy Markdown

@fmorgad0 fmorgad0 commented Sep 5, 2023

What this PR does / why we need it:
Fix for podLabels template rendering issue for gatekeeper-audit-deployment and gatekeeper-controller-manager-deployment

Special notes for your reviewer:
Present issue after adding podLabels with dummy values on values.yaml and trying to rendering the template:

helm template . > template.yaml
Error: YAML parse error on gatekeeper/templates/gatekeeper-audit-deployment.yaml: error converting YAML to JSON: yaml: line 30: mapping values are not allowed in this context

Use --debug flag to render out invalid YAML

@fmorgad0 fmorgad0 force-pushed the podlabels-template-fix branch 2 times, most recently from 6a3a500 to bad0853 Compare September 5, 2023 10:51
Comment thread charts/gatekeeper/templates/gatekeeper-audit-deployment.yaml
@fmorgad0 fmorgad0 force-pushed the podlabels-template-fix branch 2 times, most recently from dfe4a96 to d0371ee Compare September 14, 2023 08:52
Signed-off-by: fmorgad0 <filipe.morgado@talkdesk.com>
@fmorgad0 fmorgad0 force-pushed the podlabels-template-fix branch from d0371ee to d90f176 Compare September 14, 2023 08:53
@fmorgad0 fmorgad0 requested a review from ritazh September 14, 2023 08:59
@JaydipGabani
Copy link
Copy Markdown
Contributor

JaydipGabani commented Sep 22, 2023

@fmorgad0 Please push the changes after running make manifests after your current changes to push the updated manifest_staging.

@JaydipGabani JaydipGabani requested a review from a team as a code owner November 21, 2023 18:51
@fmorgad0 fmorgad0 closed this Dec 11, 2023
@fmorgad0
Copy link
Copy Markdown
Author

Closing this PR as it has already been merged in #3153

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.

3 participants