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

Resolve bug in spec section #37

Open
wants to merge 2 commits into
base: trunk
Choose a base branch
from

Conversation

Merkg9
Copy link

@Merkg9 Merkg9 commented Feb 25, 2022

Error: INSTALLATION FAILED: YAML parse error on opencommerce/templates/web/web-deployment.yaml: error converting YAML to JSON: yaml: line 31: did not find expected key

@dcrdev
Copy link
Contributor

dcrdev commented Aug 8, 2022

@zenweasel I believe this can be closed as it was fixed back in march.

@Merkg9
Copy link
Author

Merkg9 commented Aug 12, 2022

@dcrdev check this file:
templates/web/web-deployment.yaml
ImagePullSecrets must be in deployment spec. Not spec.containers

@dcrdev
Copy link
Contributor

dcrdev commented Aug 12, 2022

@Merkg9 you are correct, I missed that this was in the storefront deployment rather than the api deployment

@brent-hoover
Copy link
Collaborator

@dcrdev Is this PR ok to merge?

@dcrdev
Copy link
Contributor

dcrdev commented Aug 15, 2022

@dcrdev Is this PR ok to merge?

Yup you can merge this

Copy link
Collaborator

@brent-hoover brent-hoover left a comment

Choose a reason for hiding this comment

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

Approved per @dcrdev

@brent-hoover
Copy link
Collaborator

@Merkg9 Thanks for the fix. Can you update your DCO? You can see the instructions here

Then we can go ahead and merge

brent-hoover and others added 2 commits September 18, 2022 13:57
Signed-off-by: Merkg9 <merkg9.merkg9@merkg9>
Signed-off-by: Merkg9 <merkg9@merkg9.com>
Signed-off-by: Merkg9 <merkg9.merkg9@merkg9>
Signed-off-by: Merkg9 <merkg9@merkg9.com>
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