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

Long pod name after moving image to ECR #33

Closed
shlomitsur opened this issue Apr 13, 2021 · 4 comments
Closed

Long pod name after moving image to ECR #33

shlomitsur opened this issue Apr 13, 2021 · 4 comments
Labels
bug Something isn't working resolved Problem solved selenosis

Comments

@shlomitsur
Copy link

Hello @alcounit!
To avert the rate-limit I've moved the images (selenosis, seleniferous, chrome) to ecr.
selenosis & seleniferous works well but the chrome pods are failing because the pod name is longer than 63 characters:

"failed to start browser: failed to create pod Pod \"543052680787-dkr-ecr-us-west-2-amazonaws-com-chrome-89-0-e2f3f465-a0f4-42a1-8e56-b88ca0e35f6d\" is invalid: [metadata.labels: Invalid value: \"543052680787-dkr-ecr-us-west-2-amazonaws-com-chrome-89-0-e2f3f465-a0f4-42a1-8e56-b88ca0e35f6d\": must be no more than 63 characters, spec.hostname: Invalid value: \"543052680787-dkr-ecr-us-west-2-amazonaws-com-chrome-89-0-e2f3f465-a0f4-42a1-8e56-b88ca0e35f6d\": must be no more than 63 characters]"

browser.yaml:

chrome:
  defaultVersion: "89.0"
  path: "/"
  kernelCaps:
  - SYS_ADMIN
  versions:
    '89.0':
      image: 543037328387.dkr.ecr.us-west-2.amazonaws.com/chrome:89.0
      spec:
        resources:
          requests:
            memory: 1Gi
            cpu: '1'
          limits:
            memory: 1Gi
            cpu: '1'

Thank you

@alcounit
Copy link
Owner

@shlomitsur thanks, will fix it soon.

@alcounit alcounit added the bug Something isn't working label Apr 13, 2021
alcounit added a commit that referenced this issue Apr 13, 2021
alcounit added a commit that referenced this issue Apr 13, 2021
@alcounit
Copy link
Owner

@shlomitsur please try alcounit/selenosis:v1.0.3

@shlomitsur
Copy link
Author

Thanks @alcounit for the quick fix, I'll apply it once we'll pass peak-hours in the us.

@shlomitsur
Copy link
Author

@alcounit thanks it works well

@alcounit alcounit added the resolved Problem solved label Apr 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working resolved Problem solved selenosis
Projects
None yet
Development

No branches or pull requests

2 participants