summaryrefslogtreecommitdiff
path: root/glanceclient
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Revert "Don't update tags every time""2.5.0Jenkins2016-08-202-34/+6
|\
| * Revert "Don't update tags every time"Steve Martinelli2016-08-192-34/+6
| | | | | | | | | | | | This reverts commit e77322c17931810f4029ef339a791f702f2f4580. Change-Id: Ida826a2aa888beeb76dbe657b2ccd6cb088157ed
* | Update doc URLKATO Tomoyuki2016-08-191-1/+1
| | | | | | | | | | | | Docs team changed URL for search engine optimization. Change-Id: I68e2a5d666da55722d5ee2fa4aec2326c0de0946
* | Merge "Fix warlock model creation"2.4.0Jenkins2016-08-125-12/+19
|\ \
| * | Fix warlock model creationSabari Kumar Murugesan2016-07-065-12/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commands like glance md-namespace-show <namespace> fail because of a breaking change in warlock 1.3.0's model creation factory method. Warlock introduced a new kwarg 'resolver' in model_factory method but changed its position with the 'base_class' kwarg. Since we were calling the model_factory method with positional arg, this broke the model creation. Closes-Bug: #1596573 Change-Id: Ic7821f4fdb1b752e0c7ed2bc486299a06bf485c1
* | | Merge "Don't update tags every time"Jenkins2016-08-082-6/+34
|\ \ \ | | |/ | |/|
| * | Don't update tags every timeMike Fedosin2016-06-012-6/+34
| | | | | | | | | | | | | | | | | | | | | | | | This code removes unnecessary PATCH requests for tags updating if they were not modified. Change-Id: I8eced32f39d0c98e0b26014e7b2341ab6580ff01 Closes-bug: 1587999
* | | Merge "Remove unused openstack/common/apiclient/client"Jenkins2016-08-042-575/+0
|\ \ \
| * | | Remove unused openstack/common/apiclient/clientMatt Riedemann2016-07-272-575/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This code comes from oslo-incubator and is now frozen, and the client module specifically is not used in glanceclient. It's confusing to have in the library since glanceclient has it's own HTTPClient under glanceclient.common.http. So let's just remove this effectively dead code. Change-Id: I820b2439ce7158a63dc55553ce50a9580d63ffb3
* | | | Merge "Fix string interpolation to delayed to be handled by the logging code"Jenkins2016-07-301-1/+1
|\ \ \ \
| * | | | Fix string interpolation to delayed to be handled by the logging codehaobing12016-07-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | String interpolation should be delayed to be handled by the logging code, rather than being done at the point of the logging call. See the oslo i18n guideline. * http://docs.openstack.org/developer/oslo.i18n/guidelines.html Change-Id: If06663076e4081c6268ba88c157513723b734b31 Closes-Bug: #1596829
* | | | | Merge "Log request-id before exceptions raised"Jenkins2016-07-301-10/+10
|\ \ \ \ \
| * | | | | Log request-id before exceptions raisedAbhishek Kekane2016-07-181-10/+10
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As of now request-id is not logged if an excpetion is raised. Rearranged code so that request-id is logged even in case of an exception. Change-Id: Iee0398404ee752c0d880edf3054207c35862e71a Closes-Bug: #1603863
* | | | | Merge "py3: Fix encoding and use sys.stdin.buffer"Jenkins2016-07-286-5/+28
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | py3: Fix encoding and use sys.stdin.bufferSirushti Murugesan2016-07-256-5/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * exc.py: Encode body in response before calling replace over it. * http.py: prepend the bytes literal to the empty string or else we hit bug 1342080 again in python 3. * utils.py: Use sys.stdin.buffer in python 3. Change-Id: Ieefb8c633658e507486438e5518c5d53e819027d
* | | | | Replace OpenStack LLC with OpenStack Foundationdineshbhor2016-07-261-1/+1
|/ / / / | | | | | | | | | | | | | | | | Change-Id: I5877db07031b12d8fa7ed774ac09d24f8906ea86 Closes-Bug: #1214176
* | | | Merge "Add comment about workaround for py3"Jenkins2016-07-201-0/+2
|\ \ \ \ | |/ / / |/| | |
| * | | Add comment about workaround for py3Louis Taylor2016-07-131-0/+2
| | | | | | | | | | | | | | | | Change-Id: Ibe8720c14e8ec401bc0d595915cbb962f4021bcb
* | | | Merge "Update outdated image shema"Jenkins2016-07-161-91/+63
|\ \ \ \
| * | | | Update outdated image shemaMike Fedosin2016-06-281-91/+63
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Image schema that is located in glance client is seriously outdated, we need to updated it and bring it in line with the server version. Change-Id: I5a79a84a9c07b9ee821a71a5bd2d61cb4299ad72
* | | | | Merge "Log request-id for each api call"Jenkins2016-07-151-0/+10
|\ \ \ \ \
| * | | | | Log request-id for each api callAbhishek Kekane2016-06-271-0/+10
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added support to log 'X-Openstack-Request-Id' for each api call. If glanceclient is used from command line then following log will be logged on console if --debug flag is used. DEBUG:glanceclient.common.http:GET call to glance-api for http://172.26.88.20:9292/v2/schemas/image used request id req-e0c7c97a-8fc0-4ce3-a669-d0b1eb5d7aae If python-glanceclient is used in applications (e.g. Nova) then following log message will be logged in service logs. DEBUG glanceclient.common.http [req-be074f1e-1c17-4786-b703-2a221751c8f4 demo demo] GET call to glance-api for http://172.26.88.20:9292/v1/images/detail?is_public=none&limit=20 used request id req-9b1dd929-df30-46b2-a8f2-dfd6ffbad3fc DocImpact: To use this feature user need to set 'default_log_levels' in third party application. For example nova uses glance then in nova.conf 'default_log_levels' should be set as below: default_log_levels = glanceclient=DEBUG Implements: blueprint log-request-id Change-Id: Ib04a07bac41ad2a5e997348f3b0bccc640169dc9
* | | | | Merge "Update docs URL"Jenkins2016-07-142-3/+3
|\ \ \ \ \
| * | | | | Update docs URLKATO Tomoyuki2016-07-132-3/+3
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | Change-Id: Id8b1b97a85534c4398b3c49648c6e2f2df3ee20e Related:-Bug: #1602266
* | | | | Merge "image-download: tests to catch stray output"Jenkins2016-07-134-4/+188
|\ \ \ \ \ | |/ / / / |/| | / / | | |/ / | |/| |
| * | | image-download: tests to catch stray outputStuart McLaren2016-06-294-4/+188
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add unit tests to ensure that any stray output (eg print statements) during image-download cause a test failure. Regression test for bug 1488914. Change-Id: Ic19ba5693d059bf7c283702e7c333672a878a1a1 Partial-bug: 1488914
* | | | Merge "Replace tempest_lib with tempest.lib"Jenkins2016-06-292-2/+2
|\ \ \ \ | |/ / / |/| | |
| * | | Replace tempest_lib with tempest.libwangxiyuan2016-06-272-2/+2
| |/ / | | | | | | | | | | | | | | | | | | As the tempest_lib is deprecated and the code has been moved into tempest. We should use tempest.lib insteded of tempest_lib. Change-Id: Id22f90a47056cd88f9524fc6015607c93f7d88b5
* | | Merge "Use correct order of arguments to assertEqual"Jenkins2016-06-295-39/+35
|\ \ \ | |/ / |/| |
| * | Use correct order of arguments to assertEqualzhengyao12016-06-295-39/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The correct order of arguments to assertEqual that is expected by testtools is (expected, observed). This patch fixes the inverted usage of arguments in some places that have cropped up since the last fix of this bug. Change-Id: If8c0dcb58496bc2fcf4c635f384522a1f7d2b2af Closes-Bug: #1259292
* | | Merge "Remove unused skip_authentication decorator"Jenkins2016-06-212-18/+1
|\ \ \ | |/ / |/| |
| * | Remove unused skip_authentication decoratorkairat_kushaev2016-06-022-18/+1
| |/ | | | | | | | | | | | | | | skip_authentication is not used as decorator for glanceclient methods. So this method can be safely removed from glance codebase because it is artifact from old implementation. Change-Id: I235b4c6b835c75266d8fae1bb603685aa17ad497
* | Remove deprecated construct method from session initkairat_kushaev2016-06-092-25/+20
|/ | | | | | | | construct method is marked as deprecated and might be deleted in one of future releases. So glanceclient need to be aware of that and initialize sesssion from command line arguments directly. Change-Id: Ie81b62b7e70bb177f178caadc41554ae88e51b05
* Merge "[Trivial] Remove unnecessary executable privilege"Jenkins2016-06-011-0/+0
|\
| * [Trivial] Remove unnecessary executable privilegeZhiQiang Fan2016-05-211-0/+0
| | | | | | | | | | | | | | | | | | glanceclient/shell.py defines a main entry, however this file cannot be executed directly because we do not call the main function in this file. It is actually a module which will be used by /usr/bin/glance, hence doesn't need to be executable. Change-Id: I8b45ac7b8b88265cfc27e5ba7201e4766d9dc276
* | Fixed grammar in image-download command descriptionIhar Hrachyshka2016-06-012-2/+2
| | | | | | | | Change-Id: Ie18e1bee3376fdc01685c6f1d3a949c6934296b3
* | Merge "Fix "Codec can't encode characters""Jenkins2016-05-232-20/+17
|\ \ | |/ |/|
| * Fix "Codec can't encode characters"Darja Shakhray2016-05-192-20/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | Headers were encoded in HTTPClient, but when glance client started to use SessionClient this functionality was lost. This commit replaces static method "encode_headers" from HTTPClient and makes it a common function, that SessionClient can use when converting image meta to headers. Change-Id: If9f8020220d2a0431b4241b38b9c83c09c0d75cb Closes-bug: #1574587
* | Merge "Get endpoint if os_image_url is not set"Jenkins2016-05-192-17/+87
|\ \
| * | Get endpoint if os_image_url is not setNiall Bunting2016-05-122-17/+87
| |/ | | | | | | | | | | | | | | | | If env['OS_IMAGE_URL'] is not set then None is returned. This is then used ignoring the endpoint_type, service_type and region_name. This patch will use those values if the endpoint is None. Change-Id: I76cc527b05d2be75d3dbc33123a0d71be97fe25c Closes-bug: #1579768
* | Merge "Add last_request_id member to HTTPClient and SessionClient"Jenkins2016-05-182-0/+12
|\ \ | |/ |/|
| * Add last_request_id member to HTTPClient and SessionClientCao ShuFeng2016-03-302-0/+12
| | | | | | | | | | | | | | | | | | apiclient.base.Resource.get method requires manager.client to have last_request_id member. Otherwise get operation fails with AttributeError exception. Change-Id: I0ece85e3f61f2a7f176520ddf3ebee7792e51993 Closes-bug: 1552533
* | Corrected wrong parameter in docstringPranaliD2016-05-021-1/+1
| | | | | | | | | | | | | | | | | | | | | | In docstring of list() method of glanceclient/v1/images.py, parameter 'return_request_id' used within **kwargs is 'return_req_id', not 'return_request_id'. Changed 'return_request_id' to 'return_req_id'. Change-Id: I7f4a2a5af1b13184c67fa81be971dc5139569f8b Closes-Bug: 1573049
* | Merge "Enable hacking checks"Jenkins2016-04-207-34/+28
|\ \
| * | Enable hacking checksTin Lam2016-04-187-34/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Enabled following hacking checks from tox.ini: - H233 Python 3.x incompatible use of print operator - H303 no wildcard import - H404 multi line docstring should start with a summary Change-Id: I2553bcd3e80c00acc08d135a1d2dadfb6cda49fe Partial-Bugs: #1475054
* | | Merge "Re-enable stacktracing when --debug is used"Jenkins2016-04-183-12/+49
|\ \ \
| * | | Re-enable stacktracing when --debug is usedStuart McLaren2016-04-073-12/+49
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit 1f89beb6098f4f6a8d8c2912392b273bc068b2e3 introduced the behaviour that a stacktrace is printed if an exception is encountered. This helped make the client more supportable: $ glance --debug image-list . . . File "glanceclient/common/http.py", line 337, in get_http_client xxx NameError: global name 'xxx' is not defined global name 'xxx' is not defined The behaviour was lost at some point. This patch re-enables it. Change-Id: I25fc8624797909d606590747f54b9cf649ade079 Closes-bug: 1563830
* | | Fix typos in docstrings and commentsDao Cong Tien2016-04-114-10/+11
| | | | | | | | | | | | | | | | | | Update a comment to be more meaningful Change-Id: Ie1aa46917c1a253db92a0dc819803a1d3e795b07
* | | Merge "Update auth_token before sending request"Jenkins2016-04-052-12/+27
|\ \ \
| * | | Update auth_token before sending requestkairat_kushaev2016-03-302-12/+27
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously auth_token was initialized once in __init__ method. After that we stored token in session headers. So to refresh token users need to instantiate a new session inside http client or re-create client itself. In order to provide possibility to refresh token we need to add token header before sending the request. So users can just update auth_token attribute in the HTTPClient to refresh user token. Change-Id: Ifebe9011870bbddc46fc6d6a26563641d5559e97 Closes-Bug: #1563495