diff options
| author | Kaleb S. KEITHLEY <kkeithle@redhat.com> | 2018-07-02 08:18:54 -0400 | 
|---|---|---|
| committer | Amar Tumballi <amarts@redhat.com> | 2018-07-09 03:38:36 +0000 | 
| commit | 55d1b0531a0b322a1b851fc921fd6fdf25d26331 (patch) | |
| tree | f3d8c84e4d27fb8894a9c921a9a4ed06257cf80e /extras | |
| parent | dab12ce093628860bf43c3e0c7eaf9099cf7a78f (diff) | |
core/various: python3 compat, prepare for python2 -> python3
see https://review.gluster.org/#/c/19788/,
    https://review.gluster.org/#/c/19871/,
    https://review.gluster.org/#/c/19952/,
    https://review.gluster.org/#/c/20104/,
    https://review.gluster.org/#/c/20162/,
    https://review.gluster.org/#/c/20185/,
    https://review.gluster.org/#/c/20207/,
    https://review.gluster.org/#/c/20227/,
    https://review.gluster.org/#/c/20307/,
    https://review.gluster.org/#/c/20320/,
    https://review.gluster.org/#/c/20332/, and
    https://review.gluster.org/#/c/20364/
Fixes glupy.py python2isms, iteritems -> items, and some overlooked 
print() in georep/peer_mountbroker.in
Note: Fedora packaging guidelines and SUSE rpmlint require explicit
shebangs; popular practices like #!/usr/bin/env python and #!/usr/bin/python
are not allowed; they must be #!/usr/bin/python2 or #!/usr/bin/python3
Note: Selected small fixes from 2to3 utility. Specifically apply,
basestring, funcattrs, has_key, idioms, map, numliterals, raise,
set_literal, types, urllib, and zip have already been applied. Also
version agnostic imports for urllib, cpickle, socketserver, _thread,
queue, etc., suggested by Aravinda in https://review.gluster.org/#/c/19767/1
Note: these 2to3 fixes report no changes are necessary: asserts, buffer,
exec, execfile, exitfunc, filter, getcwdu, imports2, input, intern,
itertools, metaclass, methodattrs, ne, next, nonzero, operator, paren,
raw_input, reduce, reload, renames, repr, standarderror, sys_exc, throw,
tuple_params, xreadlines.
Change-Id: Idda031c1ec975417c79323aea33e7b694e752b2a
updates: #411
Signed-off-by: Kaleb S. KEITHLEY <kkeithle@redhat.com>
Diffstat (limited to 'extras')
| -rwxr-xr-x | extras/create_new_xlator/generate_xlator.py | 2 | ||||
| -rwxr-xr-x | extras/failed-tests.py | 2 | ||||
| -rwxr-xr-x | extras/git-branch-diff.py | 4 | ||||
| -rwxr-xr-x | extras/prot_filter.py | 2 | ||||
| -rwxr-xr-x | extras/rebalance.py | 2 | ||||
| -rw-r--r-- | extras/volfilter.py | 2 | 
6 files changed, 7 insertions, 7 deletions
diff --git a/extras/create_new_xlator/generate_xlator.py b/extras/create_new_xlator/generate_xlator.py index 0a54089080c..67e2e9c6e17 100755 --- a/extras/create_new_xlator/generate_xlator.py +++ b/extras/create_new_xlator/generate_xlator.py @@ -67,7 +67,7 @@ def generate(tmpl, name, table):          sdict["@RET_TYPE@"] = string.join(ret_type, "")          sdict["@RET_VAR@"] = string.join(ret_var, "") -        for old, new in sdict.iteritems(): +        for old, new in sdict.items():                  tmpl = tmpl.replace(old, new)          # TBD: reindent/reformat the result for maximum readability.          return tmpl diff --git a/extras/failed-tests.py b/extras/failed-tests.py index 54406f91ad3..0f5ad38aa09 100755 --- a/extras/failed-tests.py +++ b/extras/failed-tests.py @@ -74,7 +74,7 @@ def print_summary(failed_builds, total_builds, html=False):              count[template],              {'failed': failed_builds, 'total': total_builds}      )) -    for k, v in summary.iteritems(): +    for k, v in summary.items():          if k == 'core':              print(''.join([TERM.red, "Found cores:", TERM.normal]))              for comp, link in zip(v[::2], v[1::2]): diff --git a/extras/git-branch-diff.py b/extras/git-branch-diff.py index 99cc707b837..3c29e35bd7b 100755 --- a/extras/git-branch-diff.py +++ b/extras/git-branch-diff.py @@ -217,7 +217,7 @@ class GitBranchDiff:          print(self.tick + " Successfully Backported changes:")          print('      {' + 'from: ' + self.s_pattern + \                '  to: '+ self.t_pattern + '}\n') -        for key, value in self.s_dict.iteritems(): +        for key, value in self.s_dict.items():              if value in self.t_dict.itervalues():                  print("[%s%s%s] %s" %(self.yello_set,                                        key, @@ -231,7 +231,7 @@ class GitBranchDiff:          else:              review_list = [] -        for key, value in self.s_dict.iteritems(): +        for key, value in self.s_dict.items():              if value not in self.t_dict.itervalues():                  if any(value in s for s in review_list):                      print("[%s%s%s] %s %s(under review)%s" %(self.yello_set, diff --git a/extras/prot_filter.py b/extras/prot_filter.py index e2b81218f40..3baa27c3f6d 100755 --- a/extras/prot_filter.py +++ b/extras/prot_filter.py @@ -91,7 +91,7 @@ def generate (graph, last, stream=sys.stdout):              sv.dumped = True      print("volume %s" % last.name, file=stream)      print("    type %s" % last.xl_type, file=stream) -    for k, v in last.opts.iteritems(): +    for k, v in last.opts.items():          print("    option %s %s" % (k, v), file=stream)      if last.subvols:          print("    subvolumes %s" % string.join( diff --git a/extras/rebalance.py b/extras/rebalance.py index 32ca067b32d..0023f9970a8 100755 --- a/extras/rebalance.py +++ b/extras/rebalance.py @@ -49,7 +49,7 @@ def generate_stanza (vf, all_xlators, cur_subvol):                  sv_list.append(sv.name)          vf.write("volume %s\n"%cur_subvol.name)          vf.write("  type %s\n"%cur_subvol.type) -        for kvpair in cur_subvol.opts.iteritems(): +        for kvpair in cur_subvol.opts.items():                  vf.write("  option %s %s\n"%kvpair)          if sv_list:                  vf.write("  subvolumes %s\n"%string.join(sv_list)) diff --git a/extras/volfilter.py b/extras/volfilter.py index da0a0e62ddb..ea4bf22a440 100644 --- a/extras/volfilter.py +++ b/extras/volfilter.py @@ -132,7 +132,7 @@ def generate (graph, last, stream=sys.stdout):  			sv.dumped = True  	print("volume %s" % last.name, file=stream)  	print("    type %s" % last.type, file=stream) -	for k, v in last.opts.iteritems(): +	for k, v in last.opts.items():  		print("    option %s %s" % (k, v), file=stream)  	if last.subvols:  		print("    subvolumes %s" % string.join(  | 
