x/deb-python-tempest-lib
Code Issues Proposed changes

Fix the links to api-site doc

Browse Source 
The api-site doc contained invalid links which were duplicated in the
doc. We did put TODOs for fixing the links after the api-site changes.
Now these links have been fixed on the api-site, so this patch fixes
them in tempest-lib.

Change-Id: I8bbfc2933d5400b8ae35331bc63f128421486d02
This commit is contained in:
Ken'ichi Ohmichi
2015-12-06 05:18:46 +00:00
parent e880ff6d7c
commit 3201673322
3 changed files with 4 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
tempest_lib/services/compute/aggregates_client.py
View File

@@ -97,12 +97,8 @@ class AggregatesClient(rest_client.RestClient):
"""Remove a host from the given aggregate. """Remove a host from the given aggregate.
Available params: see http://developer.openstack.org/ Available params: see http://developer.openstack.org/
api-ref-compute-v2.1.html#removeaggregate api-ref-compute-v2.1.html#removehost
""" """
# TODO(oomichi): We can see the API doc of the above method with
# the above link, but the link is wrong because the link is not for
# host api. That is api-site problem. After fixing api-site, we will
# fix the above link also.
post_body = json.dumps({'remove_host': kwargs}) post_body = json.dumps({'remove_host': kwargs})
resp, body = self.post('os-aggregates/%s/action' % aggregate_id, resp, body = self.post('os-aggregates/%s/action' % aggregate_id,
post_body) post_body)

6
tempest_lib/services/compute/flavors_client.py
View File

@@ -90,12 +90,8 @@ class FlavorsClient(rest_client.RestClient):
"""Set extra Specs to the mentioned flavor. """Set extra Specs to the mentioned flavor.
Available params: see http://developer.openstack.org/ Available params: see http://developer.openstack.org/
api-ref-compute-v2.1.html#updateflavor api-ref-compute-v2.1.html#updateFlavorExtraSpec
""" """
# TODO(oomichi): We can see the API doc of the above method with
# the above link, but the link is wrong because the link is not for
# flavor-extraspac api. That is api-site problem.
# After fixing api-site, we will fix the above link also.
post_body = json.dumps({'extra_specs': kwargs}) post_body = json.dumps({'extra_specs': kwargs})
resp, body = self.post('flavors/%s/os-extra_specs' % flavor_id, resp, body = self.post('flavors/%s/os-extra_specs' % flavor_id,
post_body) post_body)

8
tempest_lib/services/compute/security_group_default_rules_client.py
View File

@@ -26,13 +26,9 @@ class SecurityGroupDefaultRulesClient(rest_client.RestClient):
"""Create security group default rule. """Create security group default rule.
Available params: see http://developer.openstack.org/ Available params: see http://developer.openstack.org/
api-ref-compute-v2.1.html#createSecGroupRule api-ref-compute-v2.1.html
#createSecGroupDefaultRule
""" """
# TODO(oomichi): We can see the API doc of the above method with
# the above link, but the link is wrong because the link doesn't
# contain "Default" and the link is duplicated to non default sg.
# After fixing api-site, we will fix the above link also.
# LP: https://bugs.launchpad.net/openstack-api-site/+bug/1521826
post_body = json.dumps({'security_group_default_rule': kwargs}) post_body = json.dumps({'security_group_default_rule': kwargs})
url = 'os-security-group-default-rules' url = 'os-security-group-default-rules'
resp, body = self.post(url, post_body) resp, body = self.post(url, post_body)