commit eb7eeec41d438076274c0192d3cf94dce80098c1
author Ken'ichi Ohmichi <oomichi@mxs.nes.nec.co.jp> Tue Jul 21 01:00:06 2015 +0000
committer Ken'ichi Ohmichi <oomichi@mxs.nes.nec.co.jp> Tue Aug 04 05:59:04 2015 +0000
tree 47bbfdfe8762d3852198e556d3b47220b4fe4d03
parent eb426a6c9405160e7f7e4bd8a4bf420bd24eeb2f

Make security_group_rules_client use **kwargs

As we discussed on
http://lists.openstack.org/pipermail/openstack-dev/2015-July/068864.html
All http POST/PUT methods need to contain **kwargs as their arguments.
This patch makes security_group_rules_client use **kwargs.

Partially implements blueprint consistent-service-method-names

Change-Id: I7ea7817c1a6a39bb509c4cd5452baa7f5fffb683

tempest/scenario/manager.py

diff --git a/tempest/scenario/manager.py b/tempest/scenario/manager.py
index 869de2d..15b0db3 100644
--- a/tempest/scenario/manager.py
+++ b/tempest/scenario/manager.py
@@ -233,14 +233,14 @@
         rulesets = [
             {
                 # ssh
-                'ip_proto': 'tcp',
+                'ip_protocol': 'tcp',
                 'from_port': 22,
                 'to_port': 22,
                 'cidr': '0.0.0.0/0',
             },
             {
                 # ping
-                'ip_proto': 'icmp',
+                'ip_protocol': 'icmp',
                 'from_port': -1,
                 'to_port': -1,
                 'cidr': '0.0.0.0/0',
@@ -248,8 +248,8 @@
         ]
         rules = list()
         for ruleset in rulesets:
-            sg_rule = _client_rules.create_security_group_rule(secgroup_id,
-                                                               **ruleset)
+            sg_rule = _client_rules.create_security_group_rule(
+                parent_group_id=secgroup_id, **ruleset)
             self.addCleanup(self.delete_wrapper,
                             _client_rules.delete_security_group_rule,
                             sg_rule['id'])