Cleanup #57957
cephadm: rename "extra_container_args"
Status:
New
Priority:
Low
Assignee:
-
Category:
-
Target version:
-
% Done:
0%
Tags:
Backport:
Reviewed:
Affected Versions:
Pull request ID:
Description
we ideally want a different name that makes it more clear that these are arguments for the podman/docker run command itself and NOT args to be added to the entrypoint/whatever is running within the container. So far the idea mentioned was "runtime_args". Whenever we have a new name we should probably alias extra_container_args to that new name to not break existing deployments while converting all documentation to use the new term.
History
#2 Updated by John Mulligan 3 months ago
Additionally, the "args" field specific to Custom Containers. I think that "args" is an attractively named option for someone looking to add args to the entrypoint. Doubly so for someone with a kubernetes background because that's what it does in k8s.