-
Notifications
You must be signed in to change notification settings - Fork 519
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
host-ctr: add fallback image parsing for special regions #3138
Conversation
Built a custom AMI and deployed. Successfully starts up with admin container enabled (note this is in the us-west-2 partition)
Logs for host-ctr look good and containers are being pulled:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😶🌫️
86513dc
to
641f2d5
Compare
Force pushed for less verbose commit message title / contents. |
641f2d5
to
35bf366
Compare
35bf366
to
6d30d45
Compare
Force pushed to address @bcressey's comments!
|
Re-tested on us-west-2 - all's well!
|
In order to support pulling images from ECR repositories from il-central-1, we need to generate our own canonical ECR image reference since the region is not officially supported by aws-go-sdk yet. Co-authored-by: Erikson Tung <[email protected]> Signed-off-by: John McBride <[email protected]>
6d30d45
to
feabba8
Compare
Force pushed again to address Ben's comments - re-tested in us-west-2:
|
Tests look good in other regions - Launched aws-k8s-1.26 instance in il-central-1 👍🏼 |
Issue number:
N/a
Description of changes:
This is a continuation of #3127:
ecrRefPrefixMapping
Testing done:
Launched aws-k8s-1.26 instance in
il-central-1
:All ECR container images get pulled down by host-ctr successfully
Launched in
us-west-2
:All container images that gets pulled through
host-ctr
do get pulled down:Terms of contribution:
By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.