Merge simplestreams:warn-warning into simplestreams:master

[Dan Watkins]

simplestreams: fix use of deprecated LOG.warn method

[Robert C Jennings (rcj)]

+1

review: Approve

[Server Team CI bot (server-team-bot)]

FAILED: Continuous integration, rev:d17340178a744fb00177dd35b47e101077904e7e
https://jenkins.ubuntu.com/server/job/simplestreams-ci/1/
Executed test runs:
    FAILURE: https://jenkins.ubuntu.com/server/job/simplestreams-ci/nodes=metal-amd64/1/
    SUCCESS: https://jenkins.ubuntu.com/server/job/simplestreams-ci/nodes=metal-ppc64el/1/
    SUCCESS: https://jenkins.ubuntu.com/server/job/simplestreams-ci/nodes=metal-s390x/1/

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/server/job/simplestreams-ci/1//rebuild

review: Needs Fixing (continuous-integration)

[Server Team CI bot (server-team-bot)]

FAILED: Continuous integration, rev:d17340178a744fb00177dd35b47e101077904e7e
https://jenkins.ubuntu.com/server/job/simplestreams-ci/2/
Executed test runs:
    FAILURE: https://jenkins.ubuntu.com/server/job/simplestreams-ci/nodes=metal-amd64/2/
    SUCCESS: https://jenkins.ubuntu.com/server/job/simplestreams-ci/nodes=metal-ppc64el/2/
    SUCCESS: https://jenkins.ubuntu.com/server/job/simplestreams-ci/nodes=metal-s390x/2/

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/server/job/simplestreams-ci/2//rebuild

review: Needs Fixing (continuous-integration)

[Server Team CI bot (server-team-bot)]

PASSED: Continuous integration, rev:d17340178a744fb00177dd35b47e101077904e7e
https://jenkins.ubuntu.com/server/job/simplestreams-ci/3/
Executed test runs:
    SUCCESS: https://jenkins.ubuntu.com/server/job/simplestreams-ci/nodes=metal-amd64/3/
    SUCCESS: https://jenkins.ubuntu.com/server/job/simplestreams-ci/nodes=metal-ppc64el/3/
    SUCCESS: https://jenkins.ubuntu.com/server/job/simplestreams-ci/nodes=metal-s390x/3/

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/server/job/simplestreams-ci/3//rebuild

review: Approve (continuous-integration)

diff --git a/simplestreams/mirrors/__init__.py b/simplestreams/mirrors/__init__.py
index 60d5ade..4a9593a 100644
--- a/simplestreams/mirrors/__init__.py
+++ b/simplestreams/mirrors/__init__.py
@@ -203,8 +203,8 @@ class UrlMirrorReader(MirrorReader):
                 csource.read(1024)
         except Exception as e:
             if isinstance(e, IOError) and (e.errno == errno.ENOENT):
-                LOG.warn("got ENOENT for (%s, %s), trying with trailing /",
-                         self.prefix, path)
+                LOG.warning("got ENOENT for (%s, %s), trying with trailing /",
+                            self.prefix, path)
                 self.prefix = self.prefix + '/'
             else:
                 # this raised exception, but it was sneaky to do it
diff --git a/simplestreams/mirrors/glance.py b/simplestreams/mirrors/glance.py
index c3c5ab7..cd26c06 100644
--- a/simplestreams/mirrors/glance.py
+++ b/simplestreams/mirrors/glance.py
@@ -215,7 +215,7 @@ class GlanceMirror(mirrors.BasicMirrorWriter):
                 continue
 
             if image.get('status') != "active":
-                LOG.warn("Ignoring inactive image %s with status '%s'" % (
+                LOG.warning("Ignoring inactive image %s with status '%s'" % (
                     image['id'], image.get('status')))
                 continue
 
@@ -225,7 +225,7 @@ class GlanceMirror(mirrors.BasicMirrorWriter):
             version = props.get('version_name')
             item = props.get('item_name')
             if not (version and product and item and source_content_id):
-                LOG.warn("%s missing required fields" % image['id'])
+                LOG.warning("%s missing required fields" % image['id'])
                 continue
 
             # get data from the datastore for this item, if it exists
diff --git a/simplestreams/objectstores/__init__.py b/simplestreams/objectstores/__init__.py
index 6ce8ff0..f118a92 100644
--- a/simplestreams/objectstores/__init__.py
+++ b/simplestreams/objectstores/__init__.py
@@ -175,13 +175,13 @@ class FileStore(ObjectStore):
             if not reader.check():
                 os.unlink(partfile)
                 if orig_part_size:
-                    LOG.warn(resume_msg)
+                    LOG.warning(resume_msg)
                 raise checksum_util.invalid_checksum_for_reader(reader)
         else:
             if not cksum.check():
                 os.unlink(partfile)
                 if orig_part_size:
-                    LOG.warn(resume_msg)
+                    LOG.warning(resume_msg)
                 raise checksum_util.InvalidChecksum(path=path, cksum=cksum)
         os.rename(partfile, wpath)
 
diff --git a/tools/ubuntu_versions.py b/tools/ubuntu_versions.py
index cf9fd7f..4b896c6 100644
--- a/tools/ubuntu_versions.py
+++ b/tools/ubuntu_versions.py
@@ -99,8 +99,8 @@ def get_ubuntu_info(date=None):
     try:
         devel = udi.devel(date=date)
     except distro_info.DistroDataOutdated as e:
-        LOG.warn("distro_info.UbuntuDistroInfo() raised exception (%s)."
-                 " Using stable release as devel.", e)
+        LOG.warning("distro_info.UbuntuDistroInfo() raised exception (%s)."
+                    " Using stable release as devel.", e)
         devel = udi.stable(date=date)
     ret = []