Py3.7 import failure running unit tests

[Corey Bryant]

Unit tested execution with py3.7 fails with import error: https://paste.ubuntu.com/p/By2n4dRR8R/

[Cyril Roelandt (cyril-roelandt)]

So, looking at the Changelog for Python3.7:

"The configparser.ConfigParser constructor now uses read_dict() to process the default values, making its behavior consistent with the rest of the parser. Non-string keys and values in the defaults dictionary are now being implicitly converted to strings. (Contributed by James Tocknell in bpo-23835.)"

So, this *should* work.

Now, looking at msg320899 of https://bugs.python.org/issue23835 :

$ python3.6 -c 'import configparser; configparser.ConfigParser(defaults={"a": None})'
$ python3.7 -c 'import configparser; configparser.ConfigParser(defaults={"a": None})'
...
TypeError: option values must be strings

This seems to be a regression in Python, reported a few days ago.

[Cyril Roelandt (cyril-roelandt)]

And apparently it's fixed (thanks to Thomas Hervé for pointing this out): https://bugs.python.org/issue33802

[Cyril Roelandt (cyril-roelandt)]

Well, I spoke too fast, there are still issues with "None". See the last comment on https://bugs.python.org/issue33802 .

[Cyril Roelandt (cyril-roelandt)]

I proposed https://review.openstack.org/#/c/599696/ to fix this bug.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/599696
Committed: https://git.openstack.org/cgit/openstack/glance_store/commit/?id=6ce857397d74f792eaa25eed5326c00565aec850
Submitter: Zuul
Branch: master

commit 6ce857397d74f792eaa25eed5326c00565aec850
Author: Cyril Roelandt <email address hidden>
Date: Tue Sep 4 18:47:31 2018 +0200

    Fix defaults for ConfigParser

    As of 3.7, the configparser module will not allow defaults values to be None.
    This patch replaces such values with "default".

    Change-Id: Id5a414412cd66d479fb8f8784cba5deddc628dfd
    Closes-Bug: #1785641

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/glance_store 0.28.0 release.