diff options
| author | Tan Lin <tan.lin.good@gmail.com> | 2014-12-29 22:45:29 +0000 |
|---|---|---|
| committer | Lin Tan <tan.lin.good@gmail.com> | 2015-01-14 20:55:41 +0800 |
| commit | 9650c88d97c46f8d03921674d2c92cc1d0c9b0d9 (patch) | |
| tree | b03d7521f39d43e58fd5ac8cb080dff92f2304c4 /ironic/drivers/modules/fake.py | |
| parent | 892e382c21e1ba5ea079edffa4cefe536a71399e (diff) | |
| download | ironic-9650c88d97c46f8d03921674d2c92cc1d0c9b0d9.tar.gz | |
Make 'method' explicit for VendorInterface.validate()
Method should be more explicit than only having it via kwargs.
Closes-Bug: #1394470
Change-Id: I6a55e1be013629e22c384b968699980e83ae0cd3
Diffstat (limited to 'ironic/drivers/modules/fake.py')
| -rw-r--r-- | ironic/drivers/modules/fake.py | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/ironic/drivers/modules/fake.py b/ironic/drivers/modules/fake.py index 967ecd816..4451891a6 100644 --- a/ironic/drivers/modules/fake.py +++ b/ironic/drivers/modules/fake.py @@ -89,8 +89,7 @@ class FakeVendorA(base.VendorInterface): return {'A1': 'A1 description. Required.', 'A2': 'A2 description. Optional.'} - def validate(self, task, **kwargs): - method = kwargs.get('method') + def validate(self, task, method, **kwargs): if method == 'first_method': bar = kwargs.get('bar') if not bar: @@ -110,8 +109,7 @@ class FakeVendorB(base.VendorInterface): return {'B1': 'B1 description. Required.', 'B2': 'B2 description. Required.'} - def validate(self, task, **kwargs): - method = kwargs.get('method') + def validate(self, task, method, **kwargs): if method in ('second_method', 'third_method_sync'): bar = kwargs.get('bar') if not bar: |