summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Check zone_id in notification_handler to avoid UUID format issue.stable/victoriajunbo2023-05-042-0/+11
| | | | | | | Closes-Bug: #1977826 Change-Id: I81a4883fd48653c3b0d60ac07210fc0de55ac12a (cherry picked from commit 42dac06e0cdef9eccabc7085d0ff9da47563b318) (cherry picked from commit a64811671c5a5e35d8614d5ed4e50d44b27333a6)
* Use ids when removing sink managed recordsErik Olof Gunnar Andersson2023-04-143-13/+111
| | | | | | | | | If the record status changes during the removal process, the current implementation will fail. Closes-Bug: 2015762 Change-Id: Iebe609e5f365d03e99f2a4580671175b4642763c (cherry picked from commit 2cb42ac9f6e5273ad64e697919b62e029466078f)
* Removed unsupported test B309 from banditErik Olof Gunnar Andersson2023-04-131-1/+1
| | | | | | | | B309 was removed with this patch and breaks the CI. https://github.com/PyCQA/bandit/commit/130a467434ff6b8f9bbd83100e409bc1fcb6f407 Change-Id: I15864a76c8913f151154e2398ae86aea71883498 (cherry picked from commit 758956d5b9321c72d63cbf3d8a1987d5dc6e6bb3)
* CI: Move queue configuration into project-templateDr. Jens Harbott2022-10-231-2/+1
| | | | | | | | | | | | | | | With this it can be reused by the other designate repos and we can reduce the replication everywhere. Conflicts: .zuul.yaml Change-Id: I1fe73145e4a4699939244bc857600efa264f2340 (cherry picked from commit c907cfa1e68249c21e8cad9f58ff3dc3d9cea6df) (cherry picked from commit 96d788c6733b3e3f78fab96ebc5e0aae2203842a) (cherry picked from commit 5f921b78896e68e473ab0c139d534f02a4105840) (cherry picked from commit 019b1ae9cb54f06f7b754024687e9f0fdfad9534) (cherry picked from commit 12453a34b385646bfc0dd16cac7203c4c587798a)
* Update zuul queue configurationMichael Johnson2022-09-191-4/+2
| | | | | | | | | Zuul is changing how queues are declared[1]. This patch updates Designate for the new way to define the queue. [1] https://lists.zuul-ci.org/pipermail/zuul-discuss/2022-May/001801.html Change-Id: I94324104a7ed89e757d48bb250d8b1132026ce46 (cherry picked from commit fe9fb45c21a7be2f88600e1f29fd37f56244cc1a)
* Fix zone update when adding new Bind9 target to pool.Dmitry Galkin2022-06-243-1/+43
| | | | | | | | | | | | | | | When a new Bind9 is added to the pool, Designate will trigger 'modzone' rndc command that will fail, unless zone is already present on the backend. This change will first verify that the zone is present on the backend, before an update attempt. If zone is not present - zone creation will be triggered. Closes-Bug: 1958409 Co-authored-by: Kiran Pawar <kinpaa@gmail.com> Change-Id: I782bf38a68f24a2e7133ff2afad7c96e2ae6b7f0 (cherry picked from commit 66cc876eadd9a4ded3afa35aed675c15c60f149d)
* Pin designate-tempest-plugin for stable/victoria jobsGhanshyam Mann2022-05-281-2/+4
| | | | | | | | | | | | | | | | | | As stable/victoria are in EM phase, and QA team pinning tempest 26.1.0 for stable/victoria - https://review.opendev.org/c/openstack/devstack/+/843295 To have compatible designate-tempest-plugin version with Tempest 26.1.0, we need to pin designate-tempest-plugin 0.9.0 version. master designate-tempest-plugin is not comaptbile with Tempest 26.1.0 - https://zuul.opendev.org/t/openstack/build/01dd10e9ef6c4e8385434ad2239fd2ca/logs Depends-On: https://review.opendev.org/c/openstack/devstack/+/843295 Depends-On: https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/838053 Change-Id: If7a5311291ec9729c9fdee4dc88aedbf30537330
* Checks for invalid denylist regex patternsvictoria-em11.0.2dekehn2022-04-064-2/+85
| | | | | | | | | Adds new field check method DenyListFields to validate the pattern string. in addition, check for a zero length string as well. Closes-Bug: #1934252 Change-Id: I2b69025fc11125bb73a4e0f8c0dedad951399cbf
* Make grenade non-voting for stable/victoriaMichael Johnson2022-04-051-2/+2
| | | | | | | | This patch makes the designate-grenade-pdns4 job non-voting on the stable/victoria branch in preparation for the victoria branch to go into extended maintenance. Change-Id: I1d98198ce5a1f32ec724e4509092ebae8a4dcbd0
* Fix recordset_records quota enforcementzhouhenglc2022-02-126-14/+122
| | | | | | | | | | | | | | First I set recordset_records=3, and I can create recordset with 4 records successfully. Now enforce record quota by count records in database. when create recordset the number in database is 0., and quotas will not work no matter how much quota recordset_records are set. And once the excess quota is created successfully, it can not be updated. Unless quotas are updated. Closes-Bug: #1847200 Change-Id: If8bc6043d95f52f67899a5ac69a2f72c8fd4de17 (cherry picked from commit df10ff5b5c76657e4287b36fbf525ad91091dd34)
* Fix race condition in the sink when deleting records11.0.1Erik Olof Gunnar Andersson2021-11-204-51/+336
| | | | | | | | | | | | | Updated the sink to behave closer to how we handle this type of operations in designate.api.v2. - Added object validation to all requests. - Better test coverage. - Use recordset update / delete instead of just record delete. Closes-Bug: #1947765 Change-Id: I867600eb48a3e30a4d17471ab794ca717706823d (cherry picked from commit 4807c23228fbb23af3087ee072bf73d7fc43aff5)
* Support filtering on zone import/export listMichael Chapman2021-11-202-7/+15
| | | | | | | | | | | | | List operations should add the task_type criterion to the user provided criterion. This patch also fixes the neutron-tempest-plugin-designate-scenario job to use the ussuri version neutron-tempest-plugin-designate-scenario-ussuri. This is required for this patch to merge. Change-Id: I983d930f975c109cce24a9587bba6db563b3f19f Launchpad: 1926058 (cherry picked from commit fb9f0b33d338037365f5c9dae20f4f795b119d13)
* Merge "Fix pool update with duplicate masters" into stable/victoriaZuul2021-05-172-5/+45
|\
| * Fix pool update with duplicate mastersMark Goddard2020-09-302-5/+45
| | | | | | | | | | | | | | | | | | | | | | | | If any zones exist that belong to a pool that has any duplicate masters among all targets, currently updating the pool will fail. This change fixes the issue by ensuring the pool target master list does not contain duplicates. Change-Id: I72bae79b4f8e0bab739875d6f783abdb4868950b Closes-Bug: #1897936 (cherry picked from commit 4e86bc2a58f00371a2f22217cb8a79e5ac33677c)
* | Drop lower-constraints testNicolas Bock2021-02-153-172/+0
| | | | | | | | | | | | | | This change drops the lower-constraints test. Change-Id: I2b32cec8d6570e990d19046dbbbd764cb4c4e5cc Signed-off-by: Nicolas Bock <nicolas.bock@canonical.com>
* | Fix lower-constraints test not constraining properlyErik Olof Gunnar Andersson2020-10-151-0/+4
| | | | | | | | | | | | | | | | | | | | When running the lower-constraints test we are currently using pip install -e . to install the source. This caused python libraries to unintentionally getting upgraded beyond the intended constraints. Change-Id: Ie05efab2f52fe3e1641c9863f6f85c813158e748 (cherry picked from commit 568b559b67b0d9a663172d681fe887bf06feea0a)
* | Increase lower-constraint for kombuNicolas Bock2020-10-151-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | There is no module named 'vine' on kombu==4.1.0 version [1]. Bumping to kombu-4.2.1 (and not kombu-4.2.0) [2]. [1] https://github.com/celery/kombu/issues/851 [2] https://docs.celeryproject.org/projects/kombu/en/stable/changelog.html#version-4-2-1 Change-Id: I57a7649fde1cceda787622aa1975f3a7100aeb29 Signed-off-by: Nicolas Bock <nicolas.bock@canonical.com> (cherry picked from commit a0d20514c6ebbfe6b42b989b6192ae77312d3ba7)
* | Fix pdns4 devstack pluginJens Harbott2020-09-301-21/+1
|/ | | | | | | | | | | | | We no longer need to deal with xenial or older distro versions, so we can assume that we can always install pdns4 from the distro directly and don't need the distro version check any longer. Drop a config option that was removed in pdns 4.2.x. Use mysql credentials correctly. Change-Id: I90af3a092296f943509833608f25522b6f8e9ab6 (cherry picked from commit 6f17fe10bb37d71944e16a422606ccf0f78c6771)
* Update TOX_CONSTRAINTS_FILE for stable/victoriaOpenStack Release Bot2020-09-251-4/+4
| | | | | | | | | | | | Update the URL to the upper-constraints file to point to the redirect rule on releases.openstack.org so that anyone working on this branch will switch to the correct upper-constraints list automatically when the requirements repository branches. Until the requirements repository has as stable/victoria branch, tests will continue to use the upper-constraints list on master. Change-Id: Ie606998216a12a536031cfc1f8ff4cc648c0e80c
* Update .gitreview for stable/victoriaOpenStack Release Bot2020-09-251-0/+1
| | | | Change-Id: I751d6ddcef281dd5aa83940147018c9a9a86458c
* Merge "fix database sync always get the wrong value"11.0.0.0rc111.0.0Zuul2020-09-211-1/+3
|\
| * fix database sync always get the wrong valuewu.chunyang2020-09-111-1/+3
| | | | | | | | | | | | | | | | | | | | | | we create a default pool when exec database sync, and get the default_pool_id from designate.conf, but we did not register this opt before using it, so we always get the default value which is wrong Change-Id: I500c932a502db24332aad95c54713b7056bc1df4 Closes-Bug: 1881277 Related-Bug: https://bugs.launchpad.net/kolla-ansible/+bug/1855877
* | Move back to mod_wsgi for devstackErik Olof Gunnar Andersson2020-09-181-1/+1
| | | | | | | | | | | | | | | | | | There is an issue with the unauthed tests and uwsgi causing the CI to frequently fail. Lets move back to mod_wsgi until we can figure that out. Change-Id: I9fd558cec85da87c8daea36f7a9e2666948e72f1
* | Merge "Native Zuul v3 designate-grenade-pdns4 job"Zuul2020-09-184-97/+36
|\ \
| * | Native Zuul v3 designate-grenade-pdns4 jobLuigi Toscano2020-08-234-97/+36
| | | | | | | | | | | | | | | | | | | | | It matches the feature of the existing legacy job. Depends-On: https://review.opendev.org/747579 Change-Id: I306599c0321ea40334750db57f39d9e3a237f13e
* | | Fix l-c testing for ubuntu focalwu.chunyang2020-09-162-17/+17
| |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | As per victoria cycle testing runtime and community goal[1] we need to migrate upstream CI/CD to Ubuntu Focal(20.04). - Bump the lower constraints for required deps which added python3.8 support in their later version. Story: #2007865 Task: #40206 [1] https://governance.openstack.org/tc/goals/selected/victoria/migrate-ci-cd-jobs-to-ubuntu-focal.html Change-Id: Ib99c13c8c4bbb21dd72836dfb1316f92cdc8aabe
* | Merge "Remove Babel from lower-constraints"Zuul2020-08-251-1/+0
|\ \ | |/ |/|
| * Remove Babel from lower-constraintsNicolas Bock2020-07-281-1/+0
| | | | | | | | | | | | | | See Id54c22f820b1aa6315d721e354ce150c657bd4fd Change-Id: I37eed6862a39c5821ac9d89b867822a110426056 Signed-off-by: Nicolas Bock <nicolas.bock@canonical.com>
* | Fix multi messages AXFR with TSIGhamalq2020-07-302-14/+108
| | | | | | | | | | | | | | | | | | | | TSIG sign for multi messages AXFR was using the wrong dnspython signing function which does not support sequence messages in response which cause the AXFR error below WARNING -- Some TSIG could not be validated Change-Id: I7ce84ac9cf5bc5f6fca47dc79c51fe0becf96ac6 Closes-Bug: 1886685
* | Merge "Remove translation sections from setup.cfg"Zuul2020-07-283-22/+0
|\ \ | |/ |/|
| * Remove translation sections from setup.cfggugug2020-06-203-22/+0
| | | | | | | | | | | | | | These translation sections are not needed anymore, Babel can generate translation files without them. Change-Id: Id54c22f820b1aa6315d721e354ce150c657bd4fd
* | Merge "Add a /healthcheck middleware"Zuul2020-07-281-0/+6
|\ \
| * | Add a /healthcheck middlewareThomas Goirand2020-07-241-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The /healthcheck URL, as provided by oslo.middleware, is very useful for operators to setup haproxy and to do monitoring. Such feature is already in many services like Keystone, Glance, and others. The same type of patch has been approved in Neutronm, Heat, and Cinder. It's under discussion in Nova, where they want to check more than just the wsgi app, though the pricinple to add it has been accepted already. Change-Id: I8f24431db0ec88385b4b916e181d6da3e598c764
* | | Update zones masters using pool target masters.Jorge Niedbalski2020-07-275-0/+147
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change enforces the update of the zone masters for all zones that belongs to a particular pool, using the pool's defined target(s) masters and forcing a update_zone call. This change also, moves the backend base class update_zone method as an abstract method, allowing to each backend implementation to create its own update logic. For the case of bind9 its extended to allow running a rndc modzone with the new given masters for the zone fixing the behavior exposed on LP: #1879798. Fixes-Bug: #1879798 Change-Id: I9dddd4130a0cbb29311eeb52e077e216c8c03f3a Signed-off-by: Jorge Niedbalski <jorge.niedbalski@canonical.com>
* | Merge "Fix inspect module deprecation warning"Zuul2020-07-142-3/+3
|\ \
| * | Fix inspect module deprecation warningmichaeltchapman2020-07-102-3/+3
| | | | | | | | | | | | | | | | | | | | | DeprecationWarning: inspect.getargspec() is deprecated since Python 3.0, use inspect.signature() or inspect.getfullargspec() Change-Id: I3707546e77acb9dec56ec33cbabb3e6228525f9c
* | | Imported Translations from ZanataOpenStack Proposal Bot2020-07-111-7/+4
| | | | | | | | | | | | | | | | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: Iab16f8aefddee7e50799f20f97f35bee78451997
* | | Merge "Stop to use the __future__ module."Zuul2020-07-103-4/+0
|\ \ \
| * | | Stop to use the __future__ module.Hervé Beraud2020-06-053-4/+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: I0c61adcc18a6c46e94df681f5adbd2cf1fad8963
* | | | Merge "Remove broken and outdated related doc page"Zuul2020-07-103-57/+3
|\ \ \ \ | |_|/ / |/| | |
| * | | Remove broken and outdated related doc pagemichaeltchapman2020-06-293-57/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The ubuntu PPA was for havana and there appears to be no more modern equivalent The nova and neutron designate plugins both link to github repositories that are now deleted. The chef cookbook, debian packages and example plugin have no updates in the last 7 years The bug and blueprint links are current and are already on the getting-started page The git repo has been added to the getting started page as that was the only remaining link Change-Id: I5d93ffdbeac54a396fc34d13a96636b23c5845e1
* | | | Merge "Use unittest.mock instead of mock"Zuul2020-07-091-1/+1
|\ \ \ \ | |/ / / |/| | |
| * | | Use unittest.mock instead of mockHervé Beraud2020-06-091-1/+1
| |/ / | | | | | | | | | | | | | | | | | | | | | 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: I59926bc1c116ec6fa0aef270a18001d27dfd3bb3
* | | Imported Translations from ZanataOpenStack Proposal Bot2020-06-201-4/+4
| |/ |/| | | | | | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: I3e3fcff958044d0e3557025fa6af0031b4068aca
* | Merge "Imported Translations from Zanata"Zuul2020-06-171-4/+4
|\ \
| * | Imported Translations from ZanataOpenStack Proposal Bot2020-06-171-4/+4
| |/ | | | | | | | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: I623feaeabbf730edb8c6e625050a88c2c1b66620
* | Fix uwsgi pathJens Harbott2020-06-131-1/+1
|/ | | | | | | | | | Devstack is switching back to using distro installed uwsgi, which will not reside in the same directory as the python binaries. Make sure to use the right path for it. See also [0]. [0] https://review.opendev.org/577779 Change-Id: Ic1b67c9d0ebfbf444f5ee096d6478fbe0c073329
* Merge "Use unittest.mock instead of third party mock"Zuul2020-06-0362-180/+209
|\
| * Use unittest.mock instead of third party mockSean McGinnis2020-05-2962-180/+209
| | | | | | | | | | | | | | Now that we no longer support py27, we can use the standard library unittest.mock module instead of the third party mock lib. Change-Id: I4bd0de2f553c43a805670a424c44455d691891ac
* | Merge "Switch to uwsgi as default for devstack"Zuul2020-06-021-1/+1
|\ \
View Lorry Controller Status