From 0b23cb2ddf7a8f86681ff97905a25a815777ae2c 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 (cherry picked from commit 0487a9ac60c5ba289b7dd7c730b014a298c5998f) --- 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 561a95b5a6f7..b3543b1a21d6 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 @@ -150,16 +144,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.47.3