From 204b276693134219bd84afe084e8ef0eff30bdb2 Mon Sep 17 00:00:00 2001 From: zhufl Date: Fri, 27 Nov 2020 17:01:52 +0800 Subject: [PATCH] Fix missing self argument in instances_no_attached instances_no_attached should have self as the first argument, this is to add it. Change-Id: I010d9d1e9ddb8790c398bcf06d0772a0d17f57ec --- watcher/decision_engine/strategy/strategies/zone_migration.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/watcher/decision_engine/strategy/strategies/zone_migration.py b/watcher/decision_engine/strategy/strategies/zone_migration.py index edecd8354..9a9e25740 100644 --- a/watcher/decision_engine/strategy/strategies/zone_migration.py +++ b/watcher/decision_engine/strategy/strategies/zone_migration.py @@ -350,7 +350,7 @@ class ZoneMigration(base.ZoneMigrationBaseStrategy): def is_in_use(self, volume): return getattr(volume, 'status') == IN_USE - def instances_no_attached(instances): + def instances_no_attached(self, instances): return [i for i in instances if not getattr(i, "os-extended-volumes:volumes_attached")]