Skip to content

Conversation

@prabhu
Copy link
Contributor

@prabhu prabhu commented Aug 23, 2025

  • Defined Requirements: The name_definition is now correctly marked as "required", and the version_definition is marked as "optional".
  • Added Case Sensitivity and Normalization: Both the namespace_definition and name_definition now include case_sensitive: false and a normalization_rules field to enforce that their values must be lowercased.
  • Improved Descriptions: The native_name for name_definition and version_definition has been updated for better clarity.
  • Added Qualifiers Definition: A qualifiers_definition has been added to define the optional keys repository_url, arch, and os, which are used for specifying custom registries and multi-platform images.

NOTE: Is it possible we are misusing the repository_url qualifier to represent registry. What if the repository and registry are different urls?

Signed-off-by: Prabhu Subramanian <[email protected]>
@prabhu prabhu marked this pull request as draft August 24, 2025 20:14
"requirement": "optional",
"note": "The namespace is the registry/user/organization if present.",
"requirement": "optional"
"native_name": "registry/user/organization",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When hosted on Docker Hub, this is a user or organization, but hosted anywhere else you can't assume that level of meaning without knowing about where the image is hosted.

Comment on lines +15 to +18
"case_sensitive": false,
"normalization_rules": [
"It is not case sensitive and must be lowercased."
]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think the namespace or name is case insensitive. The spec says that the name is lowercase, and at least some implementations consider uppercase to be invalid. Uppercase is not normalized to lowercase by native OCI tools I've tried.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants