tone down the warning for missing configuration file
The warning logged when we can't load a configuration file looks like an error because it has "Errno" in it. It's not really an error, though, so log a more gentle message saying that the configuration file does not exist where we expect to find it. Change-Id: I30698162615b494f4ed2d23f119309e9fcdf3edd Signed-off-by: Doug Hellmann <doug@doughellmann.com>
This commit is contained in:
@@ -9,6 +9,7 @@
|
|||||||
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
|
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
|
||||||
# License for the specific language governing permissions and limitations
|
# License for the specific language governing permissions and limitations
|
||||||
# under the License.
|
# under the License.
|
||||||
|
import errno
|
||||||
import logging
|
import logging
|
||||||
import os.path
|
import os.path
|
||||||
|
|
||||||
@@ -152,7 +153,11 @@ class Config(object):
|
|||||||
with open(self._filename, 'r') as fd:
|
with open(self._filename, 'r') as fd:
|
||||||
self._contents = yaml.safe_load(fd)
|
self._contents = yaml.safe_load(fd)
|
||||||
except IOError as err:
|
except IOError as err:
|
||||||
LOG.info('did not load config file %s: %s',
|
if err.errno == errno.ENOENT:
|
||||||
|
LOG.info('no configuration file in %s',
|
||||||
|
self._filename)
|
||||||
|
else:
|
||||||
|
LOG.warning('did not load config file %s: %s',
|
||||||
self._filename, err)
|
self._filename, err)
|
||||||
else:
|
else:
|
||||||
self.override(**self._contents)
|
self.override(**self._contents)
|
||||||
|
|||||||
Reference in New Issue
Block a user