-
Notifications
You must be signed in to change notification settings - Fork 24
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
CEP for the OCI storage of conda packages & repodata #70
base: main
Are you sure you want to change the base?
Changes from all commits
438598e
5e3daed
ea109b7
0416a33
e8f19c1
9bb27ea
6ee85fa
d339ca0
c32ccd2
8cdc5f2
5c88967
193259a
6e6cd7d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
@@ -0,0 +1,156 @@ | ||||||||||||||
<table> | ||||||||||||||
<tr><td> Title </td><td> OCI registries as conda channels </td> | ||||||||||||||
<tr><td> Status </td><td> Proposed </td></tr> | ||||||||||||||
<tr><td> Author(s) </td><td> Wolf Vollprecht <[email protected]><br> Hind Montassif <[email protected]></td></tr> | ||||||||||||||
<tr><td> Created </td><td> April 12, 2024</td></tr> | ||||||||||||||
<tr><td> Updated </td><td> August 7, 2024</td></tr> | ||||||||||||||
<tr><td> Discussion </td><td> https://github.com/conda/ceps/pull/70 </td></tr> | ||||||||||||||
<tr><td> Implementation </td><td> </td></tr> | ||||||||||||||
</table> | ||||||||||||||
|
||||||||||||||
# Abstract | ||||||||||||||
|
||||||||||||||
We want to use OCI registries as a storage for conda packages. This CEP specifies how we lay out conda packages on an OCI registry. | ||||||||||||||
|
||||||||||||||
## Specification | ||||||||||||||
|
||||||||||||||
An OCI artifact consists of a manifest and a set of blobs. The manifest is a JSON document that describes the contents of the artifact. The blobs are the actual data that the manifest refers to. The manifest is stored in the registry as a blob, and the blobs are stored in the registry as blobs. | ||||||||||||||
|
||||||||||||||
The manifest consists of some metadata and a number of "layers". Each layer is a reference to a blob. | ||||||||||||||
|
||||||||||||||
Layers can have arbitrary names and mediaTypes. | ||||||||||||||
|
||||||||||||||
An OCI manifest is referenced by a name and a tag. | ||||||||||||||
|
||||||||||||||
For further details, please refer to the official [OCI Distribution spec](https://github.com/opencontainers/distribution-spec/blob/v1.0/spec.md#definitions). | ||||||||||||||
|
||||||||||||||
### Layers | ||||||||||||||
|
||||||||||||||
Each layer must be a [descriptor](https://github.com/opencontainers/image-spec/blob/main/descriptor.md#properties) containing at least the 3 required fields: | ||||||||||||||
|
||||||||||||||
- The `mediaType` of the referenced content. | ||||||||||||||
- The `digest` of the targeted content. | ||||||||||||||
- The `size` of the raw content (in bytes). | ||||||||||||||
|
||||||||||||||
### MediaTypes | ||||||||||||||
|
||||||||||||||
Global and already defined mediaTypes are described [here](https://github.com/opencontainers/image-spec/blob/main/media-types.md#oci-image-media-types). | ||||||||||||||
|
||||||||||||||
Custom mediaTypes defined for the conda channels use case are as follows: | ||||||||||||||
|
||||||||||||||
| Blob type | Content type | mediaType | | ||||||||||||||
|------------------|---------------------------|---------------------------------------------| | ||||||||||||||
| conda package | .tar.bz2 package | application/vnd.conda.package.v1 | | ||||||||||||||
| conda package | .conda package | application/vnd.conda.package.v2 | | ||||||||||||||
| package info | `info` folder as gzip | application/vnd.conda.info.v1.tar+gzip | | ||||||||||||||
| package info | `index.json` file | application/vnd.conda.info.index.v1+json | | ||||||||||||||
| repodata | `repodata.json` file | application/vnd.conda.repodata.v1+json | | ||||||||||||||
| repodata | `repodata.json.zst` file | application/vnd.conda.repodata.v1+json+zst | | ||||||||||||||
| repodata | `repodata.json.gz` file | application/vnd.conda.repodata.v1+json+gzip | | ||||||||||||||
| repodata | `repodata.json.bz2` file | application/vnd.conda.repodata.v1+json+bz2 | | ||||||||||||||
|
||||||||||||||
If needed, more mediaTypes could be specified (i.e `application/vnd.conda.info.v1.tar+zst`). | ||||||||||||||
|
||||||||||||||
Using the `mediaType` field in the manifest, we can find the layer + SHA256 hash to pull the corresponding blob. | ||||||||||||||
Each `mediaType` should only be present in one layer. | ||||||||||||||
|
||||||||||||||
## Repodata on OCI registries | ||||||||||||||
|
||||||||||||||
The `repodata.json` file is a JSON file that contains metadata about the packages in a channel. | ||||||||||||||
It is used by conda to find packages in a channel. | ||||||||||||||
|
||||||||||||||
On an OCI registry it should be stored under `<channel>/<subdir>/repodata.json`. | ||||||||||||||
The repodata file should have one entry that has the `latest` tag. This entry should point to the latest version of the repodata. | ||||||||||||||
All versions of the repodata should also be tagged with a UTC timestamp of the following format: `YYYY.MM.DD.HH.MM.SS`, e.g. `2024.04.12.07.06.32`. | ||||||||||||||
|
||||||||||||||
The mediaType for the raw `repodata.json` file is `application/vnd.conda.repodata.v1+json`. However, for large repositories it's advised to store the `zstd` encoded repodata file with the mediaType `application/vnd.conda.repodata.v1+json+zst` as an additional layer in `<channel>/<subdir>/repodata.json`. ([ref](https://github.com/opencontainers/image-spec/blob/main/layer.md#gzip-media-types)) | ||||||||||||||
|
||||||||||||||
Other encodings are also accepted: | ||||||||||||||
|
||||||||||||||
- `application/vnd.conda.repodata.v1+json+gzip` | ||||||||||||||
- `application/vnd.conda.repodata.v1+json+bz2` | ||||||||||||||
|
||||||||||||||
### Conda package artifacts on an OCI registry | ||||||||||||||
|
||||||||||||||
The manifest for a conda package on an OCI registry should look like follows. | ||||||||||||||
|
||||||||||||||
It should have a name and a tag. The name is `<channel>/<subdir>/<package-name>`. | ||||||||||||||
The tag is the version and build string of the packages, using a `-` as a separator. | ||||||||||||||
|
||||||||||||||
For example, a package like `xtensor-0.10.4-h431234.conda` would map to a OCI registry `conda-forge/linux-64/xtensor:0.10.4-h431234`. | ||||||||||||||
|
||||||||||||||
A conda package, in an OCI registry, should ship up to 3 layers: | ||||||||||||||
|
||||||||||||||
- The package data itself, as a tarball. (mandatory) | ||||||||||||||
- This can be either a `.tar.bz2` (v1) or a `.conda` (v2) file, or both as separate layers. | ||||||||||||||
- The package `info` folder as a gzipped "tar.gz" file. | ||||||||||||||
- The package `info/index.json` file as a plain JSON file. | ||||||||||||||
|
||||||||||||||
### Mapping a conda-package to the OCI registry | ||||||||||||||
|
||||||||||||||
A given conda-package is identified by a URL like `<subdir>/<package-name>-<version>-<build>.<ext>` where `<subdir>` is the platform and architecture, `<package-name>` is the name of the package, `<version>` is the version of the package, `<build>` is the build string of the package, and `<ext>` is the extension of the package file. | ||||||||||||||
|
||||||||||||||
#### Mapping the package name | ||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Are these "mappings" reverted to their original names once downloaded/extracted? I would assume so and I think that should be part of the CEP. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I added a note for that. |
||||||||||||||
|
||||||||||||||
> [!NOTE] | ||||||||||||||
> **Package names in the conda world** | ||||||||||||||
> | ||||||||||||||
> The following regex is given by [`conda/schemas`](https://github.com/conda/schemas/blob/473708ac97283708d6664cbd89b8049ad1623489/common-1.schema.json#L58-L82) for a valid package name: `^[a-z0-9_](?!_)[._-]?([a-z0-9]+(\.|-|_|$))*$` | ||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It would be good to not rely on conda/schema alone, and verify with the conda code base about the regex, in case the schema isn't up-to-date. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. All of these schemas are probably wrong in some way. |
||||||||||||||
> | ||||||||||||||
> That means, a package can start with an alphanumeric character or a _single_ underscore (not multiple), and can contain dots, dashes, and underscores. It also has to end with a alphanumeric character (cannot end with a dot, dash, or underscore). | ||||||||||||||
|
||||||||||||||
To store this package on an OCI registry, we need to map it to a name and tag. The name is `<channel>/<subdir>/<package-name>`. The tag is `<version>-<build>`. There are some special rules for OCI registry names and tags for which we need some mapping. The regex for valid names is as follows: | ||||||||||||||
|
||||||||||||||
`[a-z0-9]+((\.|_|__|-+)[a-z0-9]+)*(\/[a-z0-9]+((\.|_|__|-+)[a-z0-9]+)*)*` ([ref](https://github.com/opencontainers/distribution-spec/blob/main/spec.md#pulling-manifests)) | ||||||||||||||
|
||||||||||||||
The regex expresses that names can only start with an alphanumeric letter. | ||||||||||||||
|
||||||||||||||
In `conda`, names can start with an underscore and it is used by conda-forge (e.g. `_libgcc_mutex`). For this reason, we prepend packages with a leading underscore with the string `zzz`. The name would thus be changed to `zzz_libgcc_mutex`. | ||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm afraid we might be late here. ~74 packages are mirrored like this already. We would need to delete and re-mirror? Or can can we just copy images and labels? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Also, how can we safely remove There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. See my comment below on making this not ambiguous by merging this part of the spec with other encodings. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
The mirrors are proofs of concept from the conda community's perspective (I think from conda-forge as well), so I don't think these matter. Re-mirroring 74 packages seems a small price to pay for a better spec. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 100% agreed @jezdez readding 74 packages is simple enough. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What happens if there is also a package called zzz_libgcc_mutex? This seems like a possible attack vector. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is similar to symbol name mangling I guess? Perhaps we should prefix all packages with a common prefix and a special one for underscores? |
||||||||||||||
|
||||||||||||||
#### Mapping the tag | ||||||||||||||
|
||||||||||||||
The tag is the version and build string of the packages, using a `-` as a separator. However, a OCI tag has to conform to the following regex: | ||||||||||||||
|
||||||||||||||
`[a-zA-Z0-9_][a-zA-Z0-9._-]{0,127}` | ||||||||||||||
|
||||||||||||||
Some characters that are used in the conda-forge repository as part of the build string are not allowed in the OCI registry. For this reason, we use the following mapping: | ||||||||||||||
|
||||||||||||||
- `+` is replaced by `__p__` | ||||||||||||||
- `!` is replaced by `__e__` | ||||||||||||||
- `=` is replaced by `__eq__` | ||||||||||||||
Comment on lines
+118
to
+120
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. That list seems to be a little random, it looks like Python dunder methods, but not quite valid, how about this instead?
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Yes! There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is there a standard encoding scheme we can rely on here instead of inventing our own? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Also, if we insist of a special encoding, we can add underscores themselves as There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Are conda package names case sensitive? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Based on your findings do you want to change anything with regard to the name mangling suggested above? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Conda package names are always lower case, so presumably case insensitive. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @baszalmstra we might as well have everything lowercase since that is what the OCI registry will look like. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Case insensitive filesystems are usually case preserving though. |
||||||||||||||
|
||||||||||||||
> [!IMPORTANT] | ||||||||||||||
> | ||||||||||||||
> The mapping of the package names and tags, as previously outlined, is solely intended for internal storage within an OCI registry. | ||||||||||||||
> | ||||||||||||||
> From the user's perspective, or in contexts outside of the OCI registry—beyond the middleware that interacts with it—, the package names and tags will remain unchanged, preserving the original strings. | ||||||||||||||
|
||||||||||||||
#### Authentication | ||||||||||||||
|
||||||||||||||
Pulling a public image from a Container registry can be done anonymously ([ref](https://docs.github.com/en/packages/learn-github-packages/about-permissions-for-github-packages#visibility-and-access-permissions-for-packages)). | ||||||||||||||
|
||||||||||||||
A token can be requested with `pull` scope, using the following URL: | ||||||||||||||
`https://ghcr.io/token?scope=repository:<org>/<channel-name>/<subdir>/<package-name-or-repodata.json>:pull` | ||||||||||||||
|
||||||||||||||
Note that in the case of pulling repodata, the name `repodata.json` is always used in the URL regardless of the encoding. | ||||||||||||||
|
||||||||||||||
#### Implementation (conda / mamba / rattler) | ||||||||||||||
|
||||||||||||||
##### mamba | ||||||||||||||
|
||||||||||||||
In order to fetch packages from an OCI registry, the corresponding URL should be used as a channel (i.e `oci://ghcr.io/channel-mirrors/conda-forge`). | ||||||||||||||
|
||||||||||||||
When a user requests installing a package, a set of requests to fetch `repodata.json` are first performed as follows: | ||||||||||||||
|
||||||||||||||
- A token is requested to anonymously pull `repodata.json` using the following URL:\ | ||||||||||||||
`https://ghcr.io/token?scope=repository:channel-mirrors/conda-forge/<subdir>/repodata.json:pull` | ||||||||||||||
- The manifest is then pulled using `https://ghcr.io/v2/channel-mirrors/conda-forge/<subdir>/repodata.json/manifests/<reference>`.\ | ||||||||||||||
`<reference>` is always set to `latest` in `mamba`.\ | ||||||||||||||
This is also where the repodata file encoding is handled (checking `mediaType` field in the layers).\ | ||||||||||||||
In `mamba`, `zstd` encoding has priority if present, otherwise, raw `repodata.json` is picked, and the corresponding SHA256 hash is set for the next step. | ||||||||||||||
- Repodata blob is then downloaded using:\ | ||||||||||||||
`https://ghcr.io/v2/channel-mirrors/conda-forge/<subdir>/repodata.json/blobs/sha256:<HASH>` | ||||||||||||||
|
||||||||||||||
Then, to fetch the package itself, and using the same token, the corresponding blob is downloaded using: | ||||||||||||||
`https://ghcr.io/v2/channel-mirrors/conda-forge/<subdir>/<package-name>/blobs/sha256:<HASH>` | ||||||||||||||
where <HASH> is the SHA256 hash of the requested package, retrieved from `repodata.json`. |
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.
The distribution-spec contains useful definitions for these terms:
https://github.com/opencontainers/distribution-spec/blob/v1.0/spec.md#definitions
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.
I linked it :)