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: prevent 404 on ECR during image creation #100

Merged
merged 2 commits into from
Apr 25, 2022
Merged

Conversation

tillt
Copy link
Contributor

@tillt tillt commented Apr 23, 2022

When using the inspect function on ECR repositories, it fails with an ominous 404.
containers/skopeo#1230

This skips all extended tag listing -- we are happy with the one we asked for which IIUC should not be any problem for the functionality of mindthegap.

@tillt tillt added the bug Something isn't working label Apr 23, 2022
@tillt tillt self-assigned this Apr 23, 2022
@tillt tillt changed the title Till/ecr exclusion fix: prevent 404 on ECR during image creation Apr 23, 2022
@github-actions github-actions bot added fix and removed fix labels Apr 23, 2022
@tillt tillt requested review from lukeogg and jimmidyson April 23, 2022 00:52
@tillt
Copy link
Contributor Author

tillt commented Apr 23, 2022

Validated with a Kaptain image list;

[...]
 ✓ Copying gcr.io/tfx-oss-public/ml_metadata_store_server:1.0.0 (platforms: [linux/amd64])
 ✓ Copying mcr.microsoft.com/onnxruntime/server:v1.0.0 (platforms: [linux/amd64])
 ✓ Copying nvcr.io/nvidia/tritonserver:20.08-py3 (platforms: [linux/amd64])
 ✓ Copying public.ecr.aws/j1r0q0g6/notebooks/tensorboard-controller:v1.4 (platforms: [linux/amd64])
 ✓ Copying public.ecr.aws/j1r0q0g6/notebooks/tensorboards-web-app:v1.4 (platforms: [linux/amd64])
 ✓ Copying public.ecr.aws/j1r0q0g6/notebooks/volumes-web-app:v1.4 (platforms: [linux/amd64])
 ✓ Copying public.ecr.aws/j1r0q0g6/training/training-operator:a1a0c188de17e0914bd7adfa79d16052276bffb1 (platforms: [linux/amd64])
 ✓ Copying quay.io/aipipeline/kserve-component:v0.7.0 (platforms: [linux/amd64])
[...]

Overall the performance increased by about 30%.

@tillt tillt requested a review from kaiwalyajoshi April 23, 2022 00:54
@jimmidyson jimmidyson enabled auto-merge (squash) April 25, 2022 09:06
@jimmidyson jimmidyson merged commit e000ef7 into main Apr 25, 2022
@jimmidyson jimmidyson deleted the till/ecr-exclusion branch April 25, 2022 09:28
github-actions bot pushed a commit that referenced this pull request Apr 25, 2022
* feat: introduces no-tags option for inspect

* feat: ignore tags when inspecting

(cherry picked from commit e000ef7)
@github-actions
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
release/v0.6

Questions ?

Please refer to the Backport tool documentation

jimmidyson pushed a commit that referenced this pull request Apr 25, 2022
* feat: introduces no-tags option for inspect

* feat: ignore tags when inspecting

(cherry picked from commit e000ef7)
Signed-off-by: Jimmi Dyson <[email protected]>
jimmidyson pushed a commit that referenced this pull request Apr 25, 2022
* feat: introduces no-tags option for inspect

* feat: ignore tags when inspecting

(cherry picked from commit e000ef7)
Signed-off-by: Jimmi Dyson <[email protected]>
jimmidyson pushed a commit that referenced this pull request Apr 25, 2022
* feat: introduces no-tags option for inspect

* feat: ignore tags when inspecting

(cherry picked from commit e000ef7)
Signed-off-by: Jimmi Dyson <[email protected]>

Co-authored-by: Till Toenshoff <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants