summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* OpenDev Migration Patchocata-eolstable/ocataOpenDev Sysadmins2019-04-194-9/+9
| | | | | | | | | | | | | | | This commit was bulk generated and pushed by the OpenDev sysadmins as a part of the Git hosting and code review systems migration detailed in these mailing list posts: http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003603.html http://lists.openstack.org/pipermail/openstack-discuss/2019-April/004920.html Attempts have been made to correct repository namespaces and hostnames based on simple pattern matching, but it's possible some were updated incorrectly or missed entirely. Please reach out to us via the contact information listed at https://opendev.org/ with any questions you may have.
* Replace openstack.org git:// URLs with https://Ian Wienand2019-03-242-2/+2
| | | | | | | | | | | | | | | | | This is a mechanically generated change to replace openstack.org git:// URLs with https:// equivalents. This is in aid of a planned future move of the git hosting infrastructure to a self-hosted instance of gitea (https://gitea.io), which does not support the git wire protocol at this stage. This update should result in no functional change. For more information see the thread at http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003825.html Change-Id: I84fd6c2c4315b31f118258a15ceea3500a3a3db6
* import zuul job settings from project-configDoug Hellmann2018-09-121-0/+10
| | | | | | | | | | | | | | | | | | | | This is a mechanically generated patch to complete step 1 of moving the zuul job settings out of project-config and into each project repository. Because there will be a separate patch on each branch, the branch specifiers for branch-specific jobs have been removed. Because this patch is generated by a script, there may be some cosmetic changes to the layout of the YAML file(s) as the contents are normalized. See the python3-first goal document for details: https://governance.openstack.org/tc/goals/stein/python3-first.html Change-Id: Iad8455bff1663cff1c27cd3600a21cfe72252e18 Story: #2002586 Task: #24315
* Use generic user for both zuul v2 and v3Dai Dang Van2018-09-121-3/+3
| | | | | | | | Zuul v2 uses 'jenkins' as user, but Zuul v3 uses 'zuul'. Using $USER solves it for both cases. Change-Id: Ia3f2646bd3b5fa87c90c8dac57117fd29d73706b (cherry picked from commit 4707422377214829126f1d6352119ea08a7ec104)
* Merge "Zuul: Remove project name" into stable/ocataZuul2018-02-131-1/+0
|\
| * Zuul: Remove project nameJames E. Blair2018-02-131-1/+0
| | | | | | | | | | | | | | Zuul no longer requires the project-name for in-repo configuration. Omitting it makes forking or renaming projects easier. Change-Id: I924f8eb9cd5a249dd7ab1f0eda44c40a692c0bca
* | Avoid tox_install.sh for constraints supportAndreas Jaeger2018-02-102-38/+5
|/ | | | | | | | | | | | We do not need tox_install.sh, pip can handle constraints itself and install the project correctly. Thus update tox.ini and remove the now obsolete tools/tox_install.sh file. This follows https://review.openstack.org/#/c/508061 to remove tools/tox_install.sh. Change-Id: I7f06c0be47f57a9c1ee3cadc7328b1b1b138b3cd (cherry picked from commit aede01a0a8d21d49fc4a811650b5ae5bdcb17adc)
* Move zuulv3 jobs to project repoghanshyam2017-11-225-0/+310
| | | | | | | | | This patch moves the zuulv3 jobs for python novaclient. Needed-By: I1508933ef77669754adf8032fc3d835960f78cb7 Needed-By: I37b02be0aeffc3a0f0516616b5294444012b8dea Change-Id: I43a8435485751748ca6228f67d401945cb32652e
* Updated from global requirementsocata-em7.1.2OpenStack Proposal Bot2017-05-151-1/+1
| | | | Change-Id: Ib314316d806c594648ff79c3641b3770004d7ac8
* Merge "Tags and Metadata fields with unicode cannot be correctly displayed" ↵7.1.1Jenkins2017-05-012-3/+18
|\ | | | | | | into stable/ocata
| * Tags and Metadata fields with unicode cannot be correctly displayedKevin_Zheng2017-03-052-3/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | Currently, Tags(list) and Metadata(dict) fields of instance will firstly transformed to str using jsondump first when display. And cannot be correctly transfomed and displayed afterwards. This patch adds ensure_ascii=False to the transform function thus those fields can be correctly tranformed and displayed afterwards. Change-Id: Ib4e7a34f3b19db89280cc73053acbac8c8816f85 Closes-Bug: #1669683 (cherry picked from commit 07971152413c3aa94e87b16f562acf686b9c5f48)
* | Merge "Fix devstack python-novaclient warning" into stable/ocataJenkins2017-05-011-2/+2
|\ \
| * | Fix devstack python-novaclient warningDiana Clarke2017-04-291-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The following warning is emitted for every nova command using devstack, stable/ocata. site-packages/novaclient/client.py:278: UserWarning: The 'tenant_id' argument is deprecated in Ocata and its use may result in errors in future releases. As 'project_id' is provided, the 'tenant_id' argument will be ignored. See the following commit for related changes: Clarify meaning of project_id var 9bbe5a87b7df62a7962debba5db7c96555da6761 Change-Id: Ifa5ed16f10a43c9961e98b03fc0535e12d7977ba (cherry picked from commit 1a9bfc424a59b4f2675d607881a5f0b2605a3f06)
* | | Merge "Fix ValueError when incorrect metadata passed" into stable/ocataJenkins2017-04-302-1/+27
|\ \ \
| * | | Fix ValueError when incorrect metadata passedDinesh Bhor2017-03-102-1/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If you pass incorrect formatted metadata to the 'boot', 'rebuild' and 'image-create' apis returns following error: ERROR (ValueError): dictionary update sequence element #0 has length 1; 2 is required Caught the ValuError and raised argparse.ArgumentTypeError with proper error message. Closes-Bug: #1668549 Change-Id: I14a30b93f4a916fc04610f9e475c12eb352e38c5 (cherry picked from commit 72a0ec9adcd19a96644638e8324566fcc4af8799)
* | | | Fix cinder volume leakageTakashi NATSUME2017-04-111-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A cinder volume remains after the 'test_boot_server_with_legacy_bdm_volume_id_only' functional test. It is resource leakage. So fix it. Change-Id: I3cacbac6468b5fad1c9666892fc57906af24b169 Partial-Bug: #1613105 (cherry-picked from commit 4d956d06f8c2bd3971cd46fb7ccdab33aaa5709c)
* | | | Updated from global requirementsOpenStack Proposal Bot2017-03-221-1/+1
| |/ / |/| | | | | | | | Change-Id: Ica7319230790213b9e2b99170355c1a1103e122b
* | | Fix aggregate_update name and availability_zone clashDan Smith2017-03-173-30/+72
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The name and availability_zone arguments to aggregate update were replaced by optional parameters in change I778ab7ec54a376c60f19dcc89fe62fcab6e59e42. However, the '--name' and 'name' arguments in the parser would conflict, resulting in only the deprecated argument working. Thus, attempting to update the name on an aggregate using --name would end up doing a PUT with no new name provided. Note that there were unit tests for this, but they were not catching this problem. So, this removes those tests and adds functional tests to poke it. Change-Id: Ifef6fdc1a737dd219712a4525d4e34afd3fbd80c Closes-Bug: #1673789 (cherry picked from commit 20f00553d0d8ed791105d5f7fc8798b9ac6a6a53)
* | Update UPPER_CONSTRAINTS_FILE for stable/ocataOpenStack Release Bot2017-03-081-2/+2
| | | | | | | | Change-Id: Ia90f0eb140a318e1678a709a38dac1d1253b3072
* | Update .gitreview for stable/ocataOpenStack Release Bot2017-01-311-0/+1
|/ | | | Change-Id: I34b37ca960f533189a21dfa6f056fffd6032a343
* Merge "Add release not for fixing token auth method"7.1.0Jenkins2017-01-261-0/+5
|\
| * Add release not for fixing token auth methodAndrey Kurilin2017-01-261-0/+5
| | | | | | | | Change-Id: I0fade88d7c58f4688285ce694ab1258f2ff13c9e
* | Fix functional tests to deal with multiple networksMatt Riedemann2017-01-243-3/+20
|/ | | | | | | | | | | | | | | | | | | | | | | | | | | | There was a change in the openstacksdk 0.9.11 which python-openstackclient uses to create networks in a devstack run with Neutron. Because of this change, the admin tenant has access to both the 'public' and 'private' network setup in devstack which before used to just be the 'public' network, which exposed a bug in the novaclient functional testing where the admin user is attempting to create a server but not specify a specific network to use, but multiple networks are available to the admin (it can list multiple networks). In this case the networks are the standard public and private networks that are created in devstack. Since a network isn't specified when creating the server, the nova API fails with a 409 error because it can't determine which network to use. This patch fixes the testing in novaclient by checking to see if there are multiple networks available and if so, specifies one for the legacy BDM tests that weren't specifying a network ID (those tests don't really care about the networking). The auto-network test is skipped if there are multiple networks available because passing in a specific network would defeat the purpose of that test. Change-Id: I22ee148581a94b153cf7e733563cfafaa56b1ffd Closes-Bug: #1654806
* Merge "ListExtResource given in place of ListExtManager"Jenkins2017-01-242-1/+10
|\
| * ListExtResource given in place of ListExtManagerRadomir Dopieralski2017-01-242-1/+10
| | | | | | | | | | | | | | This patch makes it use ListExtManager again. Change-Id: I00ef7107cdf027c747806c4e5037c05b60312e21 Closes-bug: #1658963
* | Pass relevant parameters to Token based authenticationIstvan Imre2017-01-243-14/+48
|/ | | | | | | | | In case of token authentication is used pass relevant parameters to Token authenticator. Co-Authored-By: Andrey Kurilin <andr.kurilin@gmail.com> Change-Id: I9a04d89016a834fe96f1b77e91011f7fa4fdda51 Closes-Bug: #1654183
* Merge "x-openstack-request-id logged twice in logs"Jenkins2017-01-222-27/+1
|\
| * x-openstack-request-id logged twice in logsAbhishek Kekane2017-01-192-27/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the recent release of keystoneauth1 2.18.0 provision is made to log x-openstack-request-id for session client. Once this new library is synced in openstack projects, the x-openstack-request-id will be logged twice on the console if session client is used. For example, $ nova --debug list DEBUG (session:640) GET call to compute for http://10.232.48.204:8774/v2.1/servers/detail used request id req-b6aeff56-0408-44ed-80ba-cd7b950a8f21 DEBUG (client:85) GET call to compute for http://10.232.48.204:8774/v2.1/servers/detail used request id req-b6aeff56-0408-44ed-80ba-cd7b950a8f21 Above log will be logged twice on the console. Removed logging of x-openstack-request-id in case of SessionClient as it is already logged in keystoneauth1. Depends-On: I492b331ff3da8d0b91178bf0d5fe1d3702f15bd7 Closes-Bug: #1657351 Change-Id: I9ba6a9e498fbcb027f45499d27ece966c52580ba
* | Updated from global requirementsOpenStack Proposal Bot2017-01-211-1/+1
| | | | | | | | Change-Id: I226c003ea438edd6d4dec6f0b135847381ad7d07
* | Merge "Allow multiple tag add/delete from cli"Jenkins2017-01-204-6/+57
|\ \ | |/ |/|
| * Allow multiple tag add/delete from cliint32bit2017-01-174-6/+57
| | | | | | | | | | | | | | | | Our users always want to add/remove multiple tags easily, but currently we can only add/remove a single tag to a server via Nova CLI. This patch allows user to add or delete server tags in one hit. Change-Id: I4dc6e5cb30b99857970965cd180e116afba1fc7d
* | Add profiling support to novaclientBoris Pavlovic2017-01-196-5/+101
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To be able to create profiling traces for Nova, client should be able to send special HTTP header that contains trace info. This patch is also important to be able to make cross project traces. (Typical case heat calls nova via python client, if profiler is initialized in heat, nova client will add extra header, that will be parsed by special osprofiler middleware in nova api.) Security considerations: trace information is signed by one of the HMAC keys that are set in nova.conf. So only person who knows HMAC key is able to send proper header. oslo-spec: https://review.openstack.org/#/c/103825/ Based on: https://review.openstack.org/#/c/105089/ Co-Authored-By: Dina Belova <dbelova@mirantis.com> Co-Authored-By: Roman Podoliaka <rpodolyaka@mirantis.com> Co-Authored-By: Tovin Seven <vinhnt@vn.fujitsu.com> Partially implements: blueprint osprofiler-support-in-nova Depends-On: I82d2badc8c1fcec27c3fce7c3c20e0f3b76414f1 Change-Id: I56ce4b547230e475854994c9d2249ef90e5b656c
* | Merge "Make _console() public"Jenkins2017-01-184-42/+180
|\ \
| * | Make _console() publichuangtianhua2017-01-184-42/+180
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Heat wants to use 'novaclient.v2.servers. ServerManager._console' method to simplify the implementation of retrieving server console. It's better to change the method to public for public use. The patch changes: 1. add public method named get_console_url() for class 'novaclient.v2.servers.Server' 2. rename _console() to get_console_url() for class 'novaclient.v2.servers.ServerManager' Change-Id: I3d1485468d1d0a79d4002981ebe05b6cdf2332e7 Closes-Bug: #1651677
* | | Fix help stringsAndreas Jaeger2017-01-181-4/+4
|/ / | | | | | | | | | | | | Some help strings miss spaces and thus get displayed like "willbe" or "biggerthan" - add missing spaces. Change-Id: I560738dd46b3088a6e528a1a24fb2dbea495dcc7
* | Updated from global requirementsOpenStack Proposal Bot2017-01-161-1/+1
|/ | | | Change-Id: I291eb851e3d77afd4586b2dfab33cd5302a4b66d
* Enable coverage report in console outputJeremy Liu2017-01-131-1/+3
| | | | | | | | Currently, the coverage test job doesn't provide any test result to developer, we could enable coverage report in console output to let the developer know the coverage status when running coverage test job. Change-Id: I12764de112e948129e21732faae03562649522fc
* Merge "Add support for showing aggregate UUIDs (v2.41)"Jenkins2017-01-105-16/+101
|\
| * Add support for showing aggregate UUIDs (v2.41)Matt Riedemann2017-01-055-16/+101
| | | | | | | | | | | | | | | | | | | | | | | | | | This adds support for the v2.41 microversion which returns the aggregate uuid in os-aggregates responses. With this change, CLI requests with microversion >= 2.41 will show the aggregate UUID in the table output. Depends-On: I4112ccd508eb85403933fec8b52efd468e866772 Part of blueprint return-uuid-from-os-aggregates-api Change-Id: I71aeec07c7c442fe82e2bb3f75f49de6ced22e77
* | Fix "Message object do not support addition".Kevin_Zheng2017-01-082-15/+24
|/ | | | | | | | | | | | | | | | | | | | | | | | In the 7.0.0 Novaclient release we added some warning log about user should using "endpoint_type" rather than "interface" when init novaclient: https://github.com/openstack/python-novaclient/blob/master/novaclient/client.py#L312-L327 https://github.com/openstack/python-novaclient/blob/master/novaclient/client.py#L249-L272 This is now causing a lot jenkins failures acroos projects that is using novaclient with enable_lazy set to False. As in this kind of scenario the warning message is a message object instead of unicode sting and it cannot be added: http://git.openstack.org/cgit/openstack/oslo.i18n/tree/oslo_i18n/_message.py#n227 and "Message object do not support addition" will raise. Related Jenkins Error: Ceilometer: http://logs.openstack.org/29/333129/3/check/gate-ceilometer-python27-ubuntu-xenial/bd125e7/ Closes-bug: #1652414 Change-Id: I11a490f759fdac9707c1321c9659da2605196a94
* Merge "Blacklist rather than whitelist autodoc modules"Jenkins2017-01-051-11/+10
|\
| * Blacklist rather than whitelist autodoc modulesDiana Clarke2016-12-301-11/+10
| | | | | | | | | | | | | | | | Autogenerate the nova client python API module list rather than having to remember to add new modules; use exclude to blacklist modules that shouldn't be included in autodoc. Change-Id: I051a7094f3b536f1e9a939af87e3fc89554fb375
* | Merge "Use more specific asserts in tests"Jenkins2017-01-024-11/+11
|\ \ | |/ |/|
| * Use more specific asserts in testsGábor Antal2016-12-064-11/+11
| | | | | | | | | | | | | | | | | | | | | | I changed asserts with more specific assert methods. e.g.: from assertTrue(sth == None) to assertIsNone(*) or assertTrue(isinstance(inst, type)) to assertIsInstace(inst, type) or assertTrue(not sth) to assertFalse(sth). The code gets more readable, and a better description will be shown on fail. Change-Id: I8be876e0d2a1ea5d6f8454ca973e2d874d10283d
* | Merge "Fixed the __ne__ implementation in base.Resource"Jenkins2016-12-212-1/+9
|\ \
| * | Fixed the __ne__ implementation in base.ResourceAkshil Verma2016-12-192-1/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Any object of Resource class or its child class do not compare with None as expected. For example if a server has been found and is clearly not None, the test "server!=None" will be False. This was occuring because the __eq__ implementation was returning the 'NotImplemented' keyword and the __ne__ implementation was returning the not of __eq__, which in this case will return False as the expected python behavior for a the not of NotImplemented is False. Changed the __ne__ implementation to return the correct boolean value and added the test case that fails with the older implementation and passes in the current fix. Change-Id: I6bf5a6e9c9eed4bbcf6678467df19dfea560b4de Closes-Bug: #1648207
* | | Merge "Replaces uuid.uuid4 with uuidutils.generate_uuid()"Jenkins2016-12-217-26/+23
|\ \ \
| * | | Replaces uuid.uuid4 with uuidutils.generate_uuid()Luong Anh Tuan2016-12-207-26/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Openstack common has a wrapper for generating uuids. We should use that function when generating uuids for consistency. Change-Id: Ic6045a3b8dabedf3ecaa14d94707614fc4d454e4 Closes-Bug: #1082248
* | | | Clarify some release notes prior to the 7.0.0 release7.0.0Matt Riedemann2016-12-205-18/+21
|/ / / | | | | | | | | | Change-Id: I55dc81659973ab5afc60b030b44f30e62b73dbd9
* | | Merge "Microversion 2.40 - Simple tenant usage pagination"Jenkins2016-12-209-17/+414
|\ \ \