From 0487a9ac60c5ba289b7dd7c730b014a298c5998f Mon Sep 17 00:00:00 2001 From: Andrew Schoen Date: Tue, 3 Jul 2018 06:45:24 -0500 Subject: [PATCH] ceph-volume: always ignore a missing ceph conf in main.py Now that we have a nice error message when a ceph.conf is missing and we try to use values from it, maintaining a list of commands that don't need ceph.conf isn't as helpful. We had actually missed 'simple trigger' when we first implemented this causing all our luminous tests for simple to fail when we backported. Signed-off-by: Andrew Schoen --- src/ceph-volume/ceph_volume/main.py | 21 +++++---------------- 1 file changed, 5 insertions(+), 16 deletions(-) diff --git a/src/ceph-volume/ceph_volume/main.py b/src/ceph-volume/ceph_volume/main.py index 7de605ddc3140..f351a144fbe50 100644 --- a/src/ceph-volume/ceph_volume/main.py +++ b/src/ceph-volume/ceph_volume/main.py @@ -10,12 +10,6 @@ from ceph_volume.decorators import catches from ceph_volume import log, devices, configuration, conf, exceptions, terminal -IGNORE_CEPH_CONFIG_COMMANDS = [ - "lvm list", - "lvm zap", -] - - class Volume(object): _help = """ ceph-volume: Deploy Ceph OSDs using different device technologies like lvm or @@ -155,16 +149,11 @@ Ceph Conf: {ceph_path} try: conf.ceph = configuration.load(conf.path) except exceptions.ConfigurationError as error: - is_help = "-h" in subcommand_args or "--help" in subcommand_args - if " ".join(subcommand_args[:2]) in IGNORE_CEPH_CONFIG_COMMANDS or is_help: - # we warn only here, because it is possible that the configuration - # file is not needed, or that it will be loaded by some other means - # (like reading from lvm tags) - logger.exception('ignoring inability to load ceph.conf') - terminal.red(error) - else: - terminal.red(error) - raise + # we warn only here, because it is possible that the configuration + # file is not needed, or that it will be loaded by some other means + # (like reading from lvm tags) + logger.exception('ignoring inability to load ceph.conf') + terminal.red(error) # dispatch to sub-commands terminal.dispatch(self.mapper, subcommand_args) -- 2.39.5