Commit Graph

26 Commits

Author SHA1 Message Date
Andrey Kurilin 2e5576dafc Remove all code related to HTTPClient
In previous patch we switched to use SessionClient in all cases. It means that all
HTTPClient code is redundant now.

Change-Id: I5a0da970fd82c79db3d88f1b49279133bbdba639
2016-12-16 12:56:18 +00:00
Andrey Kurilin ae6c39397e Enable check for E127
E127 continuation line over-indented for visual indent

Change-Id: I9dfd489565ee2d1469b2048a930c3a8384dd245d
2014-10-23 14:28:50 +00:00
Andrey Kurilin 8de2237cc4 Enable check for E126
E126 continuation line over-indented for hanging indent

Change-Id: Id64edce8278d96c1b2d4a6ca536e9fc8fbef478c
2014-10-23 17:28:25 +03:00
Sushil Kumar 953734d727 Enables H403 pep8 rules
Updates tox.ini to reduce ignored rules.

Updates code for H403 violation.

Change-Id: Iee7b34a27c62ce8cb0f26166b3c16e3386e2fecd
2013-12-14 15:16:20 +00:00
Dirk Mueller 67c80558b1 Start using Hacking and PyFlakes
Only blacklisting those warnings that are numerous,
so that in principle Hacking and PyFlakes warnings
are tested. Fix the easy ones alongway.

Change-Id: I571f51ebf570ac114509f2dcd71cdce281e7c70a
2013-06-01 15:49:51 +02:00
Davanum Srinivas c5b579926f Fix Copyright Headers from LLC to Foundation
follow the lead from nova and oslo projects

Change-Id: I270c5f1e4eefa4b72e292bfb4a4c60de0c3f6e4a
2013-03-13 20:12:23 -04:00
Vishvananda Ishaya 1e4a778bf8 Store tenant_id from keystone and use for quotas
Some calls in nova require a tenant_id when it could be interpreted
from the current authentication data, so save the tenant id and
use it in the quotas command if tenant_id is not specified.

Change-Id: I89647cfe9da73bc474ef80a61a5678db42a5571c
2013-01-31 14:38:21 -08:00
Chmouel Boudjnah 0e7df4c9fc Make region case insensitive.
- Match region in case insensitive way.

Change-Id: I4d97372e7804acd6b735275ed279b78332ba4eef
2012-08-29 13:16:50 +02:00
Ziad Sawalha ac43f6389f Filter out v1.0 endpoints
- Addresses bug 1014860
- v1.0 endpoints should not be considered valid endpoints
  by python-novaclient
- also ignores other unkown versions (for when 3 comes out)
- includes updated catalog tests

Change-Id: I73bd9b0dbede74ee0d975caa86145219e30262fc
2012-06-18 17:08:34 -05:00
Chuck 636f32b00a refactored --service_name to only work with compute calls and added
--volume_service_name for volume calls

Change-Id: I2b1188fb57f9576daebfaceaddc6eea44a47b4ee
2012-05-09 11:11:38 -05:00
Vishvananda Ishaya 03f54c57e1 Makes novaclient use the volumes endpoint
* Depends on https://review.openstack.org/#change,4479
 * Adds support to change service type including tests
 * Adds decorator for methods that need to use another service type
 * Changes volume and snapshots to use the volume endpoint
 * These extensions will move into the volume client once it exists
 * Fixes bug 940017

Change-Id: I683e4ca6c67e278d8aa8a9acec3dc0f1872f43f2
2012-02-26 06:11:12 +00:00
Josh Kearney 3f9c3a6f1a Properly handle KeyErrors.
Change-Id: I350d737950a64881ed01e87e3d8fb2b6011f0bcc
2012-02-09 14:34:12 -06:00
Ziad Sawalha 38bc7ea570 Handle Ambiguous Endpoints Correctly
- Added --service_name argument to allow selecting
  endpoints by service name
- Renamed endpoint_name argument to endpoint_type (this breaks
  compatibility)
- Return AmbiguousEndpoints error if more than one endpoint
  matches filter
- Also addresses bug 924052

Use case:
  $ nova --projectid xxx --version 1.1 --password xxx --username xxx --url https://identity.openstackcloud.com/ image-list
  Found more than one valid endpoint. Use a more restrictive filter
  AmbiguousEndpoints: [
    {'serviceName': 'New Cloud', 'region': 'Test', 'publicURL': 'https://test.openstackcloud.com/v1.1/tttt', 'tenantId': 'tttt'},
    {'serviceName': 'Old Cloud', 'publicURL': 'https://servers.openstackcloud.com/v1.0/tttt', 'tenantId': 'tttt'}]

  $ nova --projectid tttt --version 1.1 --password xxx --username xxx --url https://identity.openstackcloud.com/ --service_name 'New Cloud' image-list
  +--------------------------------------+-----------------------------+--------+--------+
  |                  ID                  |             Name            | Status | Server |
  +--------------------------------------+-----------------------------+--------+--------+
  | 346f4039-a81e-4444-9223-4a3d13592a07 | Debian Squeeze (6.0)        | ACTIVE |        |
  | ac8985ea-c09e-4544-82af-eb459a02a6b2 | Fedora 15                   | ACTIVE |        |
  | ddddc02e-92fa-4f44-b36f-55b39bf66a67 | CentOS 5.6                  | ACTIVE |        |
  +--------------------------------------+-----------------------------+--------+--------+

Change-Id: I9a10b9ad5e5b9cf6e762659013496a93a79774db
2012-01-31 18:37:36 -06:00
Sandy Walsh 0a3eda6cdd minor tweaks and long overdue pep8 2011-10-25 09:34:23 -07:00
Sandy Walsh 881427de9d new service catalog semantics 2011-10-25 06:28:30 -07:00
Gabriel Hurley aba2ced0fb Fixes #133 -- Keystone Client fetches correct service type and endpoint.
Made ServiceCatalog.url_for more configurable by allowing keyword arguments to control which service type is fetched and which endpoint is returned instead of having those values hard-coded to "compute" and "publicURL" respectively. The fix to keystone.client.Client allows the openstackx Account API to be deprecated in openstack-dashboard.
2011-10-17 14:42:23 -07:00
Rick Harris 8ef6268115 Don't filter endpoints when filter_value is non-truthy.
Fixes #126
2011-10-07 17:12:05 +00:00
Sandy Walsh 9cd9635c42 removed unicode casts 2011-09-26 12:58:03 -07:00
Sandy Walsh 4a53b06e0e cleaned up exception handling 2011-09-26 12:53:29 -07:00
Sandy Walsh 8491c76027 new service catalog implementation. 2011-09-26 12:28:43 -07:00
Sandy Walsh f932961252 removed debugging 2011-09-15 18:15:59 -07:00
Sandy Walsh acd1e0370f token support 2011-09-15 18:12:16 -07:00
Sandy Walsh cfab90bddf fixed unknown service 2011-09-13 19:43:57 -07:00
Sandy Walsh 55a09cd935 service catalog with multiple endpoints per service 2011-09-07 13:02:50 -07:00
Chmouel Boudjnah 9fe54984ed Fix extra # char as noticed by jk0 2011-08-26 16:06:51 -07:00
Chmouel Boudjnah bf2ef4d9ac Add piston service_catalog. 2011-08-26 16:02:08 -07:00