summaryrefslogtreecommitdiff
path: root/designateclient
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Add hacking check for line continuation backslash"Zuul2023-02-217-29/+54
|\
| * Add hacking check for line continuation backslashMichael Johnson2023-02-137-29/+54
| | | | | | | | | | | | This patch adds a hacking check for line continuation backslashes and fixes the occurences that existed in the code. Change-Id: I13cd03e16698b7f1d8036815d12a665bd1156f2f
* | Add shared zone commands5.2.0Igor Malinovskiy2023-02-137-4/+365
|/ | | | | | | Co-Authored-By: Michael Johnson <johnsomor@gmail.com> Change-Id: Iea92371176d9126205384624a18a9097acb3daef Partial-Bug: #1714088 Depends-On: https://review.opendev.org/#/c/726334/
* Provide option for delete zonefiles on zone deleteManish Honap2023-01-303-1/+29
| | | | | | | | | | This commit provides an header option (hard-delete) for deletion of zonefiles on backend as part of zone delete API. Main patch (in Designate) is: https://review.opendev.org/c/openstack/designate/+/835317 Change-Id: I0080c028466d809f4f5118fec05d9e46b2dfce84
* Include SECONDARY zones into list command outputEkaterina Chernova2022-07-141-1/+1
| | | | | | | | | | PRIMARY zone type is set by default, so SECONDARY zones were not included in the command output. Now all types are in output by default and specific type can be passed as an argument like before Closes-Bug: #1940544 Change-Id: Idbba68ebd3f74bde74f0b50ffcb233673dff2303
* Fix quota list command for non-admin users5.0.0Dr. Jens Harbott2022-07-071-2/+3
| | | | | | | | Due to a logic error the all-projects header was being set if no explicit project was being specified. Closes-Bug: 1980942 Change-Id: I3c492b4ffd1b75a42a6a152b1dea219e4986f411
* Fix missing --target-project-id for transfer reqMichael Johnson2022-06-031-0/+7
| | | | | | | | | | | | The zone transfer request set command was missing the --target-project-id option so you cannot change the target project ID of a zone transfer request after creation with the CLI. This option is supported by the API and documented in the API reference guide. This patch adds this option. Closes-Bug: #1977625 Change-Id: Idd570a5aef57c06971db2450fdfb823cf0c07bfd
* Add profiling support to designateclientkpdev2021-12-241-0/+6
| | | | | | | | | | | | | | To be able to create profiling traces for Designate, client should be able to send special HTTP header that contains trace info. Don't worry no security issue here, trace information is signed by HMAC key that is setted in api-paste.ini. So only person that knows HMAC key is able to send proper header. Main patch (in Designate) is: https://review.opendev.org/c/openstack/designate/+/773574 Change-Id: I4b84ac1a5d4f84da3a38e4e0cb0990a54c6044ef
* Add basic osc unit testsErik Olof Gunnar Andersson2021-12-0710-0/+479
| | | | | | | | We currently have no unit test coverage for the cli (openstack) implementation. This adds some very basic coverage that we can later expand upon. Change-Id: Ibf06cfcbf4f7c12957cf5aa0ba8de66e0554f4d0
* Add Designate hacking checks to the clientMichael Johnson2021-11-293-1/+166
| | | | | | | | | | This patch adds the Designate hacking checks to also run against the Designate client code to maintain consistent style checking across the Designate repositories. It also resolves one style check failure. Change-Id: I09786482b2e7fecaa3d4484c9b1a017fae04cfea
* Remove edit-managed from unsupported commandsMichael Chapman2021-11-252-2/+4
| | | | | | | | | | | | --edit-managed sets a header x-designate-edit-managed-records that indicates the current request wishes to modify managed records such as the SOA or NS record created for a zone. This is only used by the recordset update and delete API methods so can be removed elsewhere. Closes-Bug: 1928411 Change-Id: I18bfdae045c8b5e589075865eb024cee4852d8f9
* Merge "Clarify that name or ID can be used on TLDs"Zuul2021-11-241-3/+3
|\
| * Clarify that name or ID can be used on TLDsMichael Johnson2021-10-201-3/+3
| | | | | | | | | | | | | | | | | | We had a question about whether the TLD name could be used as the "id" positional argument when show/set/delete a TLD. This patch clarifies that names or IDs will work with those commands in the OpenStack client. Change-Id: I93952596f3c82ae37d903a8ea4dd8700ae8fc314
* | Stop to use the __future__ module.Hervé Beraud2021-11-231-2/+0
|/ | | | | | | | | | | | | | | | | | | | | | | | | | The __future__ module [1] was used in this context to ensure compatibility between python 2 and python 3. We previously dropped the support of python 2.7 [2] and now we only support python 3 so we don't need to continue to use this module and the imports listed below. Imports commonly used and their related PEPs: - `division` is related to PEP 238 [3] - `print_function` is related to PEP 3105 [4] - `unicode_literals` is related to PEP 3112 [5] - `with_statement` is related to PEP 343 [6] - `absolute_import` is related to PEP 328 [7] [1] https://docs.python.org/3/library/__future__.html [2] https://governance.openstack.org/tc/goals/selected/ussuri/drop-py27.html [3] https://www.python.org/dev/peps/pep-0238 [4] https://www.python.org/dev/peps/pep-3105 [5] https://www.python.org/dev/peps/pep-3112 [6] https://www.python.org/dev/peps/pep-0343 [7] https://www.python.org/dev/peps/pep-0328 Change-Id: I3dadeaa3242d0b55ba891a8eb4abb199a8d2468b
* Remove six and update lower-constraints appdirsMichael Johnson2021-03-2615-64/+45
| | | | | | | | | | This patch removes the use of "six" as the package declares only python3 support. It also updates the appdirs lower-constraint to 1.4.0 to support pip. This version is available on both centos8 and focal. It also removes the linter related packages from lower-constraints. Change-Id: I9337f1998749bc40737f2f0e2dcc406b6f3a0ddf
* Merge "Fixed zone transfer accept list command"Zuul2021-03-093-1/+10
|\
| * Fixed zone transfer accept list commandMike Durnosvystov2020-10-233-1/+10
| | | | | | | | | | Change-Id: If7ba1fc9dd3f9d67a49ad3b8ae4c0e0f9733195b Closes-Bug: 1901172
* | Add choices to `--type` command line argumentNicolas Bock2020-10-131-2/+7
|/ | | | | | | | | We only allow `PRIMARY` and `SECONDARY` zones. Instead of testing for either one of those two choices later in the code, we can offload this test to the parser. Change-Id: I8244e13303646e6686e4233e1edbd2bbc788e054 Signed-off-by: Nicolas Bock <nicolas.bock@canonical.com>
* Increase hacking versionNicolas Bock2020-10-103-6/+6
| | | | | | | | | | | Increase the hacking version in test-requirements to match the version used in the `designate` project. This change fixes a few errors that the more recent `pyflakes` flags. Change-Id: If7afefde0f33161016a27774021d27239c642eb5 Co-Authored-By: Michael Chapman <woppin@gmail.com> Signed-off-by: Nicolas Bock <nicolas.bock@canonical.com>
* Use unittest.mock instead of mockHervé Beraud2020-06-093-3/+3
| | | | | | | | The mock third party library was needed for mock support in py2 runtimes. Since we now only support py36 and later, we can use the standard lib unittest.mock module instead. Change-Id: I3be6c40ef61e0334cab860689abbee84f5d19265
* Use unittest.mock instead of third party mockSean McGinnis2020-03-131-3/+1
| | | | | | | | Now that we are py36 or later, we can use the standard library unittest.mock module instead of the third party mock lib. Change-Id: I9582d623727e5853637812083033a348b71551ce Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* Merge "Removing unused code"3.1.0Zuul2019-09-282-194/+0
|\
| * Removing unused codeErik Olof Gunnar Andersson2019-09-162-194/+0
| | | | | | | | Change-Id: I01d39252b0be77b035bde175b4045a8cdc859f65
* | Fix "recordset list --all-projects" in interactive OSCJens Harbott2019-09-181-1/+1
| | | | | | | | | | | | | | | | | | Class variables get re-used when multiple commands are executed in a row within a single OSC interactive session. So we need to take a copy before modifying in order to avoid unwanted side-effects. Change-Id: I9716014d65a3f649c23eb78faca2cb6dbf5eb5db Related-Bug: 1844028
* | Fix "zone list --all-projects" in interactive OSCJoseph M2019-09-161-1/+1
|/ | | | | | | | | | | | The current zone list function does not check to see if project_id is already a column to be output when adding it during an --all-projects list. This causes a duplicate field name to be added which is then passed to prettytables which subsequently throws an exception. Create a copy of the columns list so it is not appended to with every command. Change-Id: I5f62500086a807d5082221993c3b491f3b6a1db0 Closes-Bug: #1844028
* Removing v1 api supporttrain-em3.0.0Erik Olof Gunnar Andersson2019-09-1036-3060/+3
| | | | | | | The v1 has officially been removed from Designate since the Queens release, and was deperecated long before that. Change-Id: Ic5b44761ff939e2b319924af87849b3a79f9cb07
* Update json module to jsonutilscao.yuan2019-02-287-18/+24
| | | | | | | oslo project provide jsonutils, and the others use it in many place, this PS to update the json moudule to oslo jsonutils for consistency. Change-Id: I8cbf78b0735572f550ece31611258c3da9ae1d35
* Merge "Check item is one of six.string_types not just str"Zuul2018-08-112-3/+2
|\
| * Check item is one of six.string_types not just strGraham Hayes2018-04-162-3/+2
| | | | | | | | | | Change-Id: I343d9f6bdb3509256191a44e686f24e0ebeb5d22 Closes-Bug: #1762612
* | Merge "server-get/update show wrong values about 'id' and 'update_at'"rocky-em2.10.0Zuul2018-07-251-3/+3
|\ \
| * | server-get/update show wrong values about 'id' and 'update_at'miaoyuliang2018-06-201-3/+3
| |/ | | | | | | | | | | | | | | | | While process 'designate server-get' and 'designate server-update', the 'id' and 'update_at' values are wrong, they should be interchanged. Change-Id: Ibddf9cc9a018774ed229c0dfa81e705cd74704bc Closes-Bug: #1777576
* | Fix copy-pasta in quota command descriptionsBen Nemec2018-06-291-2/+2
|/ | | | | | | | These seem to have been copied from the blacklist command and don't make sense for quotas. They also appear in the openstackclient help output which is confusing. Change-Id: Ib5d2c0fd7da8cb18e230d84c5f344217ad853da7
* Merge "Highlight V1 API removal"Zuul2018-02-281-1/+5
|\
| * Highlight V1 API removalGraham Hayes2018-02-121-1/+5
| | | | | | | | | | | | | | Update Docs and the execption logged to point users to the openstack cli + v2 client. Change-Id: Ib08330d348a6a2f0e976ab0df26e91b2d7bd4e13
* | Improve the RecordSet Update UIGraham Hayes2018-02-121-2/+15
|/ | | | | | | | | | | This adds the logic introduced in I8fefd9d0f104170d50c5d5dc3cbcc53facda9baf to the "recordset set" command as well. This helps with the consistancy, and fixes some of the same issues in the set command. Change-Id: Ib943ed0eea436ad1a7ad0c3717f0fb6bd6b5b3f0
* Improve recordset create UIJens Harbott2017-12-071-3/+17
| | | | | | | | | | | | | | The current implementation has '--records' as a quasi-positional argument, with the nargs='+' parameter it can only be used at the end of the command, which is confusing to users and doesn't comply with the help message. Add a new option '--record' that takes only exactly one record as parameter and can be repeated when multiple records are present. Deprecate the old option so it can be removed in the future. Change-Id: I8fefd9d0f104170d50c5d5dc3cbcc53facda9baf Closes-Bug: 1736161
* Make remote error handling more robustJens Harbott2017-09-012-1/+8
| | | | | | | | | When a RemoteError is created, it is passed the complete content of the json response from the server. Thus is can happen that it contains unexpected entries, we should just ignore them. Change-Id: I2155d1d44432884f19ef926c02b9d0f99ade57d9 Closes-Bug: 1714460
* Add tsig key support to python-designateclientRudolf Vriend2017-07-078-1/+472
| | | | | | | | | | This fix adds support for the designate tsig keys api to python-designateclient. It will add tsigkey related crud commands to the openstackclient. Change-Id: I84336c3aca85ca62771fd2115481eda32ee980d2 Closes-Bug: #1702506
* Use Sphinx 1.5 warning-is-errorStephen Finucane2017-04-206-15/+22
| | | | | | | | | | | With pbr 2.0 and Sphinx 1.5, the setting for treat sphinx warnings as errors is setting warning-is-error in build_sphinx section. Migrate the setting from the old warnerrors one. This involves a couple of changes to the docs to fix the now broken build. Change-Id: I9bee7f6f9c69a9ed7bb22325ad1a0a1316dde526
* Set client module __version__Rui Chen2017-02-062-3/+23
| | | | | | | | | | According to PEP 396, the version of client library should be set in xxxclient.__version__, that help python-openstackclient command "module list" to get the right plugin module versions. Change-Id: I932287726c5b0a8638c6ad72171e98181867d88f Partial-Bug: #1662058
* Merge "Add attribute support to create zone cli"ocata-em2.6.0Jenkins2017-02-012-1/+20
|\
| * Add attribute support to create zone cliGraham Hayes2017-01-312-1/+20
| | | | | | | | | | | | also format attributes for cli display Change-Id: Ia338f607e6d4b70610132ab66e15381981beb038
* | Merge "Replace 'assertFalse(a in b)' with 'assertNotIn(a, b)'"Jenkins2017-02-011-1/+1
|\ \
| * | Replace 'assertFalse(a in b)' with 'assertNotIn(a, b)'howardlee2016-11-181-1/+1
| | | | | | | | | | | | | | | | | | Trivial fix. Change-Id: I0e678649225d58e70db09f6c0acce59d20f48004
* | | Show proper error on over quota commandsGraham Hayes2017-01-312-0/+6
| |/ |/| | | | | | | Change-Id: I811a0f0e3dcc3406ba9a07b0d54788a3241a7cd2 Closes-Bug: #1629222
* | Removes unnecessary utf-8 encodingCao Xuan Hoang2016-12-222-4/+0
| | | | | | | | | | | | | | | | | | | | The following files added utf-8 encoding but never used. So we can remove them at all. designateclient/tests/base.py designateclient/tests/test_designateclient.py Change-Id: Id11c8241544ef076c95986bb645cdc6cce652a1c
* | Ensure X-Designate-Edit-Managed-Records header value is a stringCedric Brandily2016-12-091-1/+1
| | | | | | | | | | | | | | | | Header values are strings so we should provide a string as X-Designate-Edit-Managed-Records header value instead of a boolean (which is transformed into: "<type 'bool'>" by python-requests). Change-Id: Iaf248a0e817fe65785ddcdca46663e7273a6dad0
* | Use tempest instead of Tempest-libghanshyam2016-11-229-10/+10
|/ | | | | | | | | | | tempest-lib has been deprecated since a year. All the interfaces available in tempest-lib are now available in tempest/lib. This patch make use of tempest/lib instead of tempest-lib. Also add tempest in test-requirements.txt Change-Id: I6cb50b68631391f0368e8a2e6133f94c1da52376
* Merge "Ensure X-Auth-All-Projects header value is a string"Jenkins2016-11-041-1/+1
|\
| * Ensure X-Auth-All-Projects header value is a stringCedric Brandily2016-10-281-1/+1
| | | | | | | | | | | | | | | | Header values are strings so we should provide a string as X-Auth-All-Projects header value instead of a boolean (which is transformed into: "<type 'bool'>" by python-requests) Change-Id: Id4172f424565da5afb1f310195a0b7ba37674064