Fix dict related issues for Python 2/3 compatible code.

Story: 2003429
Task: 24621

Change-Id: I71d9aa429abf8abef9b70e84e7437223a1b6719a
Signed-off-by: chenyan <yan.chen@intel.com>
Signed-off-by: Sun Austin <austin.sun@intel.com>
This commit is contained in:
chenyan 2018-08-30 16:41:04 +08:00 committed by Sun Austin
parent 7777f0a7f4
commit 201b94d89f
5 changed files with 13 additions and 13 deletions

View File

@ -129,7 +129,7 @@ class PatchAPIController(object):
@expose('json') @expose('json')
def upload_dir(self, **kwargs): def upload_dir(self, **kwargs):
files = [] files = []
for key, path in kwargs.iteritems(): for key, path in kwargs.items():
LOG.info("upload-dir: Retrieving patches from %s" % path) LOG.info("upload-dir: Retrieving patches from %s" % path)
for f in glob.glob(path + '/*.patch'): for f in glob.glob(path + '/*.patch'):
if os.path.isfile(f): if os.path.isfile(f):

View File

@ -768,7 +768,7 @@ class PatchAgent(PatchService):
print("Checking for software updates...") print("Checking for software updates...")
self.query() self.query()
install_set = [] install_set = []
for pkg, version in self.to_install.iteritems(): for pkg, version in self.to_install.items():
install_set.append("%s-%s" % (pkg, version)) install_set.append("%s-%s" % (pkg, version))
install_set += self.missing_pkgs install_set += self.missing_pkgs

View File

@ -1013,7 +1013,7 @@ class PatchController(PatchService):
# Now verify the state of the required patches # Now verify the state of the required patches
req_verification = True req_verification = True
for req_patch, iter_patch_list in required_patches.iteritems(): for req_patch, iter_patch_list in required_patches.items():
if req_patch not in self.patch_data.metadata \ if req_patch not in self.patch_data.metadata \
or self.patch_data.metadata[req_patch]["repostate"] == constants.AVAILABLE: or self.patch_data.metadata[req_patch]["repostate"] == constants.AVAILABLE:
msg = "%s is required by: %s" % (req_patch, ", ".join(sorted(iter_patch_list))) msg = "%s is required by: %s" % (req_patch, ", ".join(sorted(iter_patch_list)))
@ -1105,7 +1105,7 @@ class PatchController(PatchService):
if repo_changed: if repo_changed:
# Update the repo # Update the repo
self.patch_data.gen_groups_xml() self.patch_data.gen_groups_xml()
for ver, rdir in repo_dir.iteritems(): for ver, rdir in repo_dir.items():
try: try:
output = subprocess.check_output(["createrepo", output = subprocess.check_output(["createrepo",
"--update", "--update",
@ -1201,7 +1201,7 @@ class PatchController(PatchService):
required_patches[req_patch].append(patch_iter) required_patches[req_patch].append(patch_iter)
if len(required_patches) > 0: if len(required_patches) > 0:
for req_patch, iter_patch_list in required_patches.iteritems(): for req_patch, iter_patch_list in required_patches.items():
msg = "%s is required by: %s" % (req_patch, ", ".join(sorted(iter_patch_list))) msg = "%s is required by: %s" % (req_patch, ", ".join(sorted(iter_patch_list)))
msg_error += msg + "\n" msg_error += msg + "\n"
LOG.info(msg) LOG.info(msg)
@ -1265,7 +1265,7 @@ class PatchController(PatchService):
if repo_changed: if repo_changed:
# Update the repo # Update the repo
self.patch_data.gen_groups_xml() self.patch_data.gen_groups_xml()
for ver, rdir in repo_dir.iteritems(): for ver, rdir in repo_dir.items():
try: try:
output = subprocess.check_output(["createrepo", output = subprocess.check_output(["createrepo",
"--update", "--update",
@ -1629,7 +1629,7 @@ class PatchController(PatchService):
results = self.patch_data.metadata results = self.patch_data.metadata
else: else:
# Filter results # Filter results
for patch_id, data in self.patch_data.metadata.iteritems(): for patch_id, data in self.patch_data.metadata.items():
if query_state is not None and data["repostate"] != query_state: if query_state is not None and data["repostate"] != query_state:
continue continue
if query_release is not None and data["sw_version"] != query_release: if query_release is not None and data["sw_version"] != query_release:
@ -1652,10 +1652,10 @@ class PatchController(PatchService):
if patch_id not in self.patch_data.metadata.keys(): if patch_id not in self.patch_data.metadata.keys():
results["error"] += "%s is unrecognized\n" % patch_id results["error"] += "%s is unrecognized\n" % patch_id
for patch_id, data in self.patch_data.metadata.iteritems(): for patch_id, data in self.patch_data.metadata.items():
if patch_id in patch_ids: if patch_id in patch_ids:
results["metadata"][patch_id] = data results["metadata"][patch_id] = data
for patch_id, data in self.patch_data.contents.iteritems(): for patch_id, data in self.patch_data.contents.items():
if patch_id in patch_ids: if patch_id in patch_ids:
results["contents"][patch_id] = data results["contents"][patch_id] = data
@ -1895,7 +1895,7 @@ class PatchController(PatchService):
# Update the repo # Update the repo
self.patch_data.gen_groups_xml() self.patch_data.gen_groups_xml()
for ver, rdir in repo_dir.iteritems(): for ver, rdir in repo_dir.items():
try: try:
output = subprocess.check_output(["createrepo", output = subprocess.check_output(["createrepo",
"--update", "--update",
@ -1935,7 +1935,7 @@ class PatchController(PatchService):
rc = False rc = False
self.hosts_lock.acquire() self.hosts_lock.acquire()
for ip, host in self.hosts.iteritems(): for ip, host in self.hosts.items():
if host.state == constants.PATCH_AGENT_STATE_INSTALLING: if host.state == constants.PATCH_AGENT_STATE_INSTALLING:
rc = True rc = True
break break

View File

@ -600,7 +600,7 @@ class PatchData:
write_xml_file(top, fname) write_xml_file(top, fname)
def gen_groups_xml(self): def gen_groups_xml(self):
for ver, rdir in repo_dir.iteritems(): for ver, rdir in repo_dir.items():
self.gen_release_groups_xml(ver) self.gen_release_groups_xml(ver)
def query_line(self, def query_line(self,

View File

@ -92,7 +92,7 @@ class PatchAlarmDaemon():
data = json.loads(req.text) data = json.loads(req.text)
if 'pd' in data: if 'pd' in data:
for patch_id, metadata in data['pd'].iteritems(): for patch_id, metadata in data['pd'].items():
if 'patchstate' in metadata and \ if 'patchstate' in metadata and \
(metadata['patchstate'] == 'Partial-Apply' or metadata['patchstate'] == 'Partial-Remove'): (metadata['patchstate'] == 'Partial-Apply' or metadata['patchstate'] == 'Partial-Remove'):
raise_pip_alarm = True raise_pip_alarm = True