Merge pull request #912 from untergeek/fix/874

Log version incompatibilities
elastic · Mar 27, 2017 · 323970c · 323970c
2 parents 91cf324 + 4932946
commit 323970c
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/curator/utils.py b/curator/utils.py
@@ -477,6 +477,11 @@ def check_version(client):
     )
     if version_number >= settings.version_max() \
         or version_number < settings.version_min():
+        logger.error(
+            'Elasticsearch version {0} incompatible '
+            'with this version of Curator '
+            '({0})'.format(".".join(map(str,version_number)), __version__)
+        )
         raise CuratorException(
             'Elasticsearch version {0} incompatible '
             'with this version of Curator '

diff --git a/docs/Changelog.rst b/docs/Changelog.rst
@@ -30,6 +30,11 @@ production! There `will` be many more changes before 5.0.0 is released.
   * Bumped ``urllib3`` (python module) version dependency to 1.20
   * Bumped ``elasticsearch`` (python module) version dependency to 5.2
 
+**Bug Fixes**
+
+  * Curator now logs version incompatibilities as an error, rather than just
+    raising an Exception. #874 (untergeek)
+
 
 5.0.0a1 (23 March 2017)
 -------------