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

Fix indentation of OPA extraArgs in deployment.yaml #160

Merged
merged 1 commit into from
Jun 2, 2022

Conversation

jacksgt
Copy link
Contributor

@jacksgt jacksgt commented Jun 2, 2022

Hello,
thanks for keeping up the good work in this repo!

I wanted to upgrade to the latest 7.0.0 release, but came across this minor issue in Helm output:

 Error: YAML parse error on opa-kube-mgmt/templates/deployment.yaml:
error converting YAML to JSON: yaml: line 45: did not find expected '-' indicator

This is due to minor indentation misalignment - yay YAML!

@jacksgt jacksgt force-pushed the fix-opa-extra-args branch from 83850c0 to 6467aeb Compare June 2, 2022 11:40
eshepelyuk added a commit that referenced this pull request Jun 2, 2022
Resolves #160

Signed-off-by: Ievgenii Shepeliuk <[email protected]>
@eshepelyuk eshepelyuk merged commit a57edd5 into open-policy-agent:master Jun 2, 2022
@eshepelyuk
Copy link
Contributor

Thnx a lot @jacksgt !!

eshepelyuk added a commit that referenced this pull request Jun 2, 2022
Resolves #160

Signed-off-by: Ievgenii Shepeliuk <[email protected]>
eshepelyuk added a commit that referenced this pull request Jun 2, 2022
Resolves #160

Signed-off-by: Ievgenii Shepeliuk <[email protected]>
eshepelyuk added a commit that referenced this pull request Jun 2, 2022
Resolves #160

Signed-off-by: Ievgenii Shepeliuk <[email protected]>
@eshepelyuk
Copy link
Contributor

7.0.1 is almost there, give it a try :)

@jacksgt
Copy link
Contributor Author

jacksgt commented Jun 2, 2022

Wow, you're fast!
Thanks for merging and tagging a release.

@jacksgt jacksgt deleted the fix-opa-extra-args branch June 2, 2022 12:36
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