Skip to content

Conversation

@Artoria2e5
Copy link
Contributor

It's a bit awkward really, since that nspawn isn't the WHOLE repo on GitHub unlike the others...

@AkihiroSuda AkihiroSuda added this to the v1.2.1 (tentative) milestone Nov 29, 2024
@kolyshkin
Copy link
Contributor

AFAICS systemd-nspawn does not implement this spec.

@kolyshkin
Copy link
Contributor

AFAICS systemd-nspawn does not implement this spec.

Therefore this should be closed (unless I'm missing something).

@AkihiroSuda
Copy link
Member

AFAICS systemd-nspawn does not implement this spec.

Implements: https://github.com/systemd/systemd/blob/main/src/nspawn/nspawn-oci.c

@kolyshkin
Copy link
Contributor

My bad, indeed, systemd-nspawn implements --oci-bundle option since systemd v242.

Copy link
Contributor

@kolyshkin kolyshkin left a comment

Choose a reason for hiding this comment

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

My bad, indeed, systemd-nspawn implements --oci-bundle option since systemd v242.

So it would be nice to mention the above option and the version number, otherwise LGTM.

@AkihiroSuda
Copy link
Member

mention the above option and the version number

ping @Artoria2e5

@kolyshkin
Copy link
Contributor

ping @Artoria2e5

@rata
Copy link
Member

rata commented Feb 12, 2025

It reads a bundle (config.json, rootfs, etc.) but it doesn't seem to implement any of the subcommands (like features, list, etc.) and IIRC it's not a full spec or at the very least is an old spec what it implements (I think the former, not the full spec, of some old version of the spec).

@AkihiroSuda
Copy link
Member

Please squash the commits

@kolyshkin
Copy link
Contributor

@Artoria2e5 please squash the commits.

It's a bit awkward really, since that nspawn isn't the WHOLE repo on GitHub unlike the others...

Signed-off-by: Mingye Wang <[email protected]>
@Artoria2e5
Copy link
Contributor Author

Ah sorry! done.

@kolyshkin kolyshkin merged commit ea38318 into opencontainers:main Mar 10, 2025
4 checks passed
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.

4 participants