Skip to content

update flag-description for --cgroup-parent - #4516

Merged
thaJeztah merged 1 commit into
docker:masterfrom
thaJeztah:cgroup_parent_description
Aug 23, 2023
Merged

update flag-description for --cgroup-parent#4516
thaJeztah merged 1 commit into
docker:masterfrom
thaJeztah:cgroup_parent_description

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

relates to:

This attempts to make it clearer that the --cgroup-parent option is only used for the containers used during build. Instead of mentioning "build container", I opted for using "RUN instructions" (to match the --network description), although this may not be ideal (as it assumes the "Dockerfile" front-end, which of course may not be the case).

- What I did

- How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

This attempts to make it clearer that the --cgroup-parent option is only used
for the containers used during build. Instead of mentioning "build container",
I opted for using "RUN instructions" (to match the --network description),
although this may not be ideal (as it assumes the "Dockerfile" front-end, which
of course may not be the case).
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztahthaJeztah added this to the 25.0.0 milestone Aug 23, 2023
@thaJeztah

Copy link
Copy Markdown
MemberAuthor

Perhaps we need a backport for this one as well, because (I think) the buildx docs are taken from master, so because that one's merged, the flag descriptions would currently not match.

@thaJeztah
thaJeztah merged commit c7a3a30 into docker:masterAug 23, 2023
@thaJeztah
thaJeztah deleted the cgroup_parent_description branch August 23, 2023 11:53
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@thaJeztah@rumpl@dvdksn