]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
python-common: reformat ServiceSpec class level service type lists
authorJohn Mulligan <jmulligan@redhat.com>
Thu, 14 Dec 2023 00:20:45 +0000 (19:20 -0500)
committerAdam King <adking@redhat.com>
Mon, 15 Apr 2024 15:01:30 +0000 (11:01 -0400)
commitaec7acd24ee4a78318e445906be9749fffe141a0
tree48d79eed3fc8fc876fb43be4c9c51d0469b71bee
parent6bfd46b96e45a41410c37195a8903f01e056fcab
python-common: reformat ServiceSpec class level service type lists

Reformat the ServiceSpec classes properties KNOWN_SERVICE_TYPES and
REQUIRES_SERVICE_ID. These were previously strings that were converted
to lists via a call to split. With a string there's very little a human
or a tool can do to validate the content. Changing these into proper
lists in the source code brings clarity of intent and the ability to
analyze the code. Because there's no semantic difference what services
are listed where (this means the type could probably be a set - a quest
for another day) I also took the opportunity to sort the contents of the
lists and add some basic comments for what these lists are for.

It also removes the use of (ugly, IMO) line continuations. The downside
is that it makes more total lines, but if that bugs you - use code
folding :-).

Signed-off-by: John Mulligan <jmulligan@redhat.com>
(cherry picked from commit 4fc2697fb1e4dc71b480db275aa4e54c2b66d018)
src/python-common/ceph/deployment/service_spec.py