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

[ECR Public] [request]: please support the "data" field in content descriptors #2023

Closed
tianon opened this issue May 5, 2023 · 8 comments
Closed
Assignees
Labels
Proposed Community submitted issue Shipped This feature request was delivered.

Comments

@tianon
Copy link

tianon commented May 5, 2023

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment

Tell us about your request
I want to push some OCI images, but they have the optional data field in their config descriptors, so I'm getting "405 Method Not Allowed" instead of a successfully pushed image.

Which service(s) is this request for?
ECR Public

Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
I'm trying to push some verbatim OCI-spec-compliant images to ECR Public, but I can't push them as-is without modifying them (which would then change their digests and make them different images).

Are you currently working around this issue?
Nope! I've been apparently suffering from this problem for two months now, but only just today figured out why, and as a result, I likely won't fix it (I'll just stop pushing to ECR Public instead 🙈).

@tianon tianon added the Proposed Community submitted issue label May 5, 2023
@jlbutler
Copy link

Hi @tianon! 👋 We're looking into this now and should be able to give a timeline for addressing it soon. Thanks for the issue!

@jlbutler jlbutler self-assigned this Jul 12, 2023
@rukumar333
Copy link

@jlbutler Checking in if there was any update on this?

@gauthamsam
Copy link

Hello, the work to support this ask in ECR Public is already underway. We plan to rollout the changes by 09/08.

@gauthamsam
Copy link

Hello, we have now launched support for "data" field in content descriptors in ECR Public. Please note that the launch also includes support for this field in layer and subject descriptors.

@tianon
Copy link
Author

tianon commented Sep 16, 2023

Nice, can confirm it appears to be working! Thank you!

@adrianmace
Copy link

Just a heads up @jlbutler, the description in the Public ECR page for this image still says the releases are blocked on this issue.

image

Screenshot captured 27 Mar 2024 @ 09:41AM GMT+11

@tianon
Copy link
Author

tianon commented Mar 26, 2024

Oh that's my bad, sorry! I'll update that ASAP

@tianon
Copy link
Author

tianon commented Mar 27, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Proposed Community submitted issue Shipped This feature request was delivered.
Projects
None yet
Development

No branches or pull requests

5 participants