mgr/cephadm: allow miscellaneous container args at service level

[adk3798]

Signed-off-by: Adam King adking@redhat.com

Fixes: https://tracker.ceph.com/issues/51566

E.g. applying a spec like

service_type: mon
service_name: mon
placement:
  count: 5
extra_container_args:
  - "--cpus=2"
  - "--quiet"

would get you all the mons would deployed with --cpus=2 --quiet

Checklist

• Tracker (select at least one)
  • References tracker ticket
  • Very recent bug; references commit where it was introduced
  • New feature (ticket optional)
  • Doc update (no ticket needed)
  • Code cleanup (no ticket needed)
• Component impact
  • Affects Dashboard, opened tracker ticket
  • Affects Orchestrator, opened tracker ticket
  • No impact that needs to be tracked
• Documentation (select at least one)
  • Updates relevant documentation
  • No doc update is appropriate
• Tests (select at least one)
  • Includes unit test(s)
  • Includes integration test(s)
  • Includes bug reproducer
  • No tests

Show available Jenkins commands

• jenkins retest this please
• jenkins test classic perf
• jenkins test crimson perf
• jenkins test signed
• jenkins test make check
• jenkins test make check arm64
• jenkins test submodules
• jenkins test dashboard
• jenkins test dashboard cephadm
• jenkins test api
• jenkins test docs
• jenkins render docs
• jenkins test ceph-volume all
• jenkins test ceph-volume tox

[adk3798]

Changed from dict to list of str for extra args, fixed unit tests, added new unit test for this change

[sebastian-philipp]

ping @adk3798 ?

[adk3798]

ping @adk3798 ?

review comments should be addressed now. This is ready for a new review and qa if no concerns

[adk3798]

jenkins test make check

[adk3798]

jenkins test make check

[adk3798]

jenkins test make check

[adk3798]

jenkins test make check

[sebastian-philipp]

https://pulpito.ceph.com/swagner-2022-01-24_16:47:41-orch:cephadm-wip-swagner-testing-2022-01-24-1121-distro-default-smithi/

• failures: ingress