openstack/swift
Code Issues Proposed changes

Don't double-filter replication jobs

Browse Source 
ObjectReplicator.collect_jobs() takes and correctly applies the
various overrides, so there's no need to check the returned jobs
against the overrides.

Change-Id: I2a59b26410d1732a5f2c8f1f32e397d77550860e
This commit is contained in:
Samuel Merritt 2018-03-08 13:02:46 -08:00
parent 0a4d1b7e22
commit de38b727b4
1 changed files with 0 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
swift/obj/replicator.py
View File

@ -745,11 +745,6 @@ class ObjectReplicator(Daemon):
override_policies=override_policies)
for job in jobs:
current_nodes = job['nodes']
if override_devices and job['device'] not in override_devices:
continue
if override_partitions and \
job['partition'] not in override_partitions:
continue
dev_path = check_drive(self.devices_dir, job['device'],
self.mount_check)
if not dev_path: