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

ERR parsing policy #614

Open
stdmje opened this issue Oct 28, 2023 · 1 comment
Open

ERR parsing policy #614

stdmje opened this issue Oct 28, 2023 · 1 comment
Assignees
Labels
bug Something isn't working question Further information is requested

Comments

@stdmje
Copy link

stdmje commented Oct 28, 2023

Hello! I am getting a parse error when trying to use the following policy:

    ecrOptions:
      accessPolicy: |
        {
          "Statement": [
            {
              "Sid": "AllowCrossAccountPull",
              "Effect": "Allow",
              "Principal": {
                "AWS": "*"
              },
              "Action": [
                "ecr:GetDownloadUrlForLayer",
                "ecr:BatchGetImage",
                "ecr:BatchCheckLayerAvailability"
              ],
              "Condition": {
                "StringEquals": {
                  "aws:PrincipalOrgID": "o-XXXX"
                }
              }
            }
          ],
          "Version": "2008-10-17"
        }

Logs:

k8s-image-swapper-576dbbb84b-4zrkd k8s-image-swapper 11:09AM ERR parsing image swap policy failed error="unknown image swap policy string: '', defaulting to exists" policy=
k8s-image-swapper-576dbbb84b-4zrkd k8s-image-swapper 11:09AM ERR parsing image copy policy failed error="unknown image copy policy string: '', defaulting to delayed" policy=

Please note that i am upgrading from 1.1.0 helm chart version to 1.8.0 and in the previous version everything was working fine.

Thanks

@estahn
Copy link
Owner

estahn commented Feb 13, 2024

@stdmje Is this still an issue?

Is this snippet from the configmap or the helm chart? If it is from the helm chart values can you check if the configmap is properly propagated?

@estahn estahn self-assigned this Feb 13, 2024
@estahn estahn added bug Something isn't working question Further information is requested labels Feb 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants