diff options
author | Devan Goodwin <dgoodwin@redhat.com> | 2016-04-15 14:53:11 -0300 |
---|---|---|
committer | Devan Goodwin <dgoodwin@redhat.com> | 2016-04-15 15:05:54 -0300 |
commit | a1bb23b8abec1e700407534ce31ded5dbe077d0e (patch) | |
tree | a1c67530edefe9e24ac04058e059a48d47c0fc78 | |
parent | dbb94c4e5461780adb74e864e6b0919153b5ffe7 (diff) | |
download | openshift-a1bb23b8abec1e700407534ce31ded5dbe077d0e.tar.gz openshift-a1bb23b8abec1e700407534ce31ded5dbe077d0e.tar.bz2 openshift-a1bb23b8abec1e700407534ce31ded5dbe077d0e.tar.xz openshift-a1bb23b8abec1e700407534ce31ded5dbe077d0e.zip |
Fix use of JSON inventory vars with raw booleans.
Thought this was fixed in recent patch but somehow backed out a critical
change to the isinstance, we need to check for basestring to also have
this work with unicode strings.
-rwxr-xr-x | roles/openshift_facts/library/openshift_facts.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/roles/openshift_facts/library/openshift_facts.py b/roles/openshift_facts/library/openshift_facts.py index 9218e12ae..54b1d9e75 100755 --- a/roles/openshift_facts/library/openshift_facts.py +++ b/roles/openshift_facts/library/openshift_facts.py @@ -1154,7 +1154,7 @@ def merge_facts(orig, new, additive_facts_to_overwrite, protected_facts_to_overw if key in inventory_json_facts: # Watchout for JSON facts that sometimes load as strings. # (can happen if the JSON contains a boolean) - if isinstance(new[key], str): + if isinstance(new[key], basestring): facts[key] = yaml.safe_load(new[key]) else: facts[key] = copy.deepcopy(new[key]) |