diff options
| author | Joe Gordon <jogo@cloudscaling.com> | 2012-10-22 13:45:28 -0700 |
|---|---|---|
| committer | Joe Gordon <jogo@cloudscaling.com> | 2012-10-22 13:47:36 -0700 |
| commit | c39dbb53bcbc0df82d9052248ee8029613858813 (patch) | |
| tree | fe60b7195f50fff2041a07f5eb5d9e42032c815c /nova | |
| parent | 4011a767c0ba45b244e004fb9a15ad921985ead9 (diff) | |
| download | nova-c39dbb53bcbc0df82d9052248ee8029613858813.tar.gz nova-c39dbb53bcbc0df82d9052248ee8029613858813.tar.xz nova-c39dbb53bcbc0df82d9052248ee8029613858813.zip | |
Fix and enable pep8 E502, E712
pep8 E502 and E712 were temporarily disabld when upgrading to pep8 1.2
Change-Id: If5158304cb8e73ef844dea9f726df6f8fe305269
Diffstat (limited to 'nova')
5 files changed, 7 insertions, 7 deletions
diff --git a/nova/db/sqlalchemy/api.py b/nova/db/sqlalchemy/api.py index 0946d0b19..5977b9ca0 100644 --- a/nova/db/sqlalchemy/api.py +++ b/nova/db/sqlalchemy/api.py @@ -1146,7 +1146,7 @@ def fixed_ip_disassociate_all_by_timeout(context, host, time): join((models.Network, models.Network.id == models.FixedIp.network_id)).\ join((models.Instance, - models.Instance.uuid == \ + models.Instance.uuid == models.FixedIp.instance_uuid)).\ filter(host_filter).\ all() @@ -4380,7 +4380,7 @@ def _instance_type_extra_specs_get_query(context, flavor_id, subquery() return model_query(context, models.InstanceTypeExtraSpecs, session=session, read_deleted="no").\ - filter(models.InstanceTypeExtraSpecs.\ + filter(models.InstanceTypeExtraSpecs. instance_type_id.in_(t)) diff --git a/nova/db/sqlalchemy/migrate_repo/versions/089_add_volume_id_mappings.py b/nova/db/sqlalchemy/migrate_repo/versions/089_add_volume_id_mappings.py index 1dd47aec2..d878e250b 100644 --- a/nova/db/sqlalchemy/migrate_repo/versions/089_add_volume_id_mappings.py +++ b/nova/db/sqlalchemy/migrate_repo/versions/089_add_volume_id_mappings.py @@ -82,7 +82,7 @@ def upgrade(migrate_engine): if migrate_engine.name == "mysql": migrate_engine.execute("ALTER TABLE volume_id_mappings Engine=InnoDB") - migrate_engine.execute("ALTER TABLE snapshot_id_mappings "\ + migrate_engine.execute("ALTER TABLE snapshot_id_mappings " "Engine=InnoDB") volumes = Table('volumes', meta, autoload=True) diff --git a/nova/db/sqlalchemy/migrate_repo/versions/112_update_deleted_instance_data.py b/nova/db/sqlalchemy/migrate_repo/versions/112_update_deleted_instance_data.py index e491e08fe..27ad13e91 100644 --- a/nova/db/sqlalchemy/migrate_repo/versions/112_update_deleted_instance_data.py +++ b/nova/db/sqlalchemy/migrate_repo/versions/112_update_deleted_instance_data.py @@ -30,7 +30,7 @@ def upgrade(migrate_engine): meta.bind = migrate_engine instances = Table('instances', meta, autoload=True) - instance_list = list(instances.select().\ + instance_list = list(instances.select(). where(instances.c.deleted == True).execute()) for table_name in TABLES: table = Table(table_name, meta, autoload=True) @@ -55,7 +55,7 @@ def downgrade(migrate_engine): meta.bind = migrate_engine instances = Table('instances', meta, autoload=True) - instance_list = list(instances.select().\ + instance_list = list(instances.select(). where(instances.c.deleted == True).execute()) for table_name in TABLES: table = Table(table_name, meta, autoload=True) diff --git a/nova/tests/network/test_linux_net.py b/nova/tests/network/test_linux_net.py index fd6708343..72bf71674 100644 --- a/nova/tests/network/test_linux_net.py +++ b/nova/tests/network/test_linux_net.py @@ -222,7 +222,7 @@ class LinuxNetworkTestCase(test.TestCase): is_admin=True) def get_vifs(_context, instance_uuid): - return [vif for vif in vifs if vif['instance_uuid'] == \ + return [vif for vif in vifs if vif['instance_uuid'] == instance_uuid] def get_instance(_context, instance_id): diff --git a/nova/tests/test_iptables_network.py b/nova/tests/test_iptables_network.py index 166e28a5c..c8f310303 100644 --- a/nova/tests/test_iptables_network.py +++ b/nova/tests/test_iptables_network.py @@ -97,7 +97,7 @@ class IptablesManagerTestCase(test.TestCase): table.remove_rule('FORWARD', '-s 1.2.3.4/5 -j DROP') new_lines = self.manager._modify_rules(current_lines, table) self.assertTrue('[0:0] -A %s-FORWARD ' - '-s 1.2.3.4/5 -j DROP' % self.binary_name \ + '-s 1.2.3.4/5 -j DROP' % self.binary_name not in new_lines) def test_nat_rules(self): |