Skip to content

Conversation

glehmann
Copy link
Member

I forgot to update them when I removed xcp-ng-dev-env-create.

@glehmann glehmann requested review from guzu, stormi and ydirson October 13, 2025 13:40
Signed-off-by: Gaëtan Lehmann <[email protected]>
@glehmann glehmann force-pushed the gln/doc-image-build-uosq branch from e49ab1a to 8c87cdd Compare October 14, 2025 09:01
@glehmann glehmann requested a review from ydirson October 14, 2025 09:03
Comment on lines +81 to +95
## Building the container image(s)

You need one container image per target version of XCP-ng.

Clone this repository (outside any container), then use `xcp-ng-dev-env-create` to
generate the images for the wanted releases of XCP-ng.
Clone this repository (outside any container), then use `./container/build.sh` to
generate the image. Adapt the version to the wanted release of XCP-ng.
Note that Docker and Podman store container images separately.

```
Usage: xcp-ng-dev-env-create {version_of_XCP_ng}
... where {version_of_XCP_ng} is a 'x.y' version such as 8.0.
Usage: ./container/build.sh [--platform PF] <version>
... where <version> is a 'x.y' version such as 8.0.
Copy link
Member

Choose a reason for hiding this comment

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

I think here we could remind first that you can download a pre-built image if all you want is to use it.

Signed-off-by: Gaëtan Lehmann <[email protected]>
@glehmann glehmann force-pushed the gln/doc-image-build-uosq branch from 8c87cdd to b85b759 Compare October 15, 2025 11:26
@stormi
Copy link
Member

stormi commented Oct 15, 2025

@glehmann I haven't seen a re-review request, but seeing the diff I suppose there was an implicit one (caution with implicit re-review requests, they don't always work :))

@glehmann glehmann merged commit 4d51f00 into master Oct 15, 2025
10 checks passed
@glehmann glehmann deleted the gln/doc-image-build-uosq branch October 15, 2025 13:16
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.

3 participants