| johnsom | These zone import tests are .... They need work | 00:31 |
|---|---|---|
| opendevreview | Michael Johnson proposed openstack/designate-tempest-plugin master: Improve the zone import API tests https://review.opendev.org/c/openstack/designate-tempest-plugin/+/796575 | 00:43 |
| johnsom | eandersson That fixes your zone import orphaned zones. | 00:47 |
| opendevreview | Michael Johnson proposed openstack/designate-tempest-plugin master: Fix docs reqs for new pip https://review.opendev.org/c/openstack/designate-tempest-plugin/+/769170 | 01:08 |
| opendevreview | Michael Chapman proposed openstack/designate master: Improvements to zone blacklist doc https://review.opendev.org/c/openstack/designate/+/793325 | 02:02 |
| opendevreview | Merged openstack/designate-tempest-plugin master: Fix docs reqs for new pip https://review.opendev.org/c/openstack/designate-tempest-plugin/+/769170 | 03:11 |
| eandersson | awesome! thanks johnsom | 03:28 |
| eandersson | btw can you review / merge these small changes when you have time | 03:30 |
| eandersson | 794135: Add simple scheduler permutation tests | https://review.opendev.org/c/openstack/designate/+/794135 | 03:30 |
| eandersson | 794131: Fixed a potential circular dependency | https://review.opendev.org/c/openstack/designate/+/794131 | 03:30 |
| eandersson | Confirmed it worked for me too | 03:37 |
| eandersson | Now I just need to figure out who is creating this one | 03:37 |
| eandersson | > 04e52930-c186-470d-9831-5ce67b5f9687 | 00000000-0000-0000-0000-000000000000 | 4.24.172.in-addr.arpa. | PRIMARY | 1623814353 | ACTIVE | NONE | | 03:37 |
| opendevreview | Michael Chapman proposed openstack/designate master: TLD Doc update https://review.opendev.org/c/openstack/designate/+/795735 | 04:26 |
| opendevreview | Merged openstack/designate master: Re-enable the tempest scenario jobs https://review.opendev.org/c/openstack/designate/+/796321 | 05:20 |
| opendevreview | Arkady Shtempler proposed openstack/designate-tempest-plugin master: Fix "PTR recordset" tests suite https://review.opendev.org/c/openstack/designate-tempest-plugin/+/794708 | 06:24 |
| opendevreview | Arkady Shtempler proposed openstack/designate-tempest-plugin master: Skip "test_list_all_projects_recordsets" because of Designate bug https://review.opendev.org/c/openstack/designate-tempest-plugin/+/796469 | 07:35 |
| opendevreview | wdd proposed openstack/designate master: Changed minversion in tox to 3.18.0 https://review.opendev.org/c/openstack/designate/+/796597 | 08:00 |
| opendevreview | wdd proposed openstack/designate-dashboard master: Changed minversion in tox to 3.18.0 https://review.opendev.org/c/openstack/designate-dashboard/+/796600 | 08:11 |
| opendevreview | wdd proposed openstack/designate-tempest-plugin master: Use TOX_CONSTRAINTS_FILE and update tox https://review.opendev.org/c/openstack/designate-tempest-plugin/+/792319 | 08:13 |
| *** krthnz_ is now known as krthnz | 08:35 | |
| opendevreview | Arkady Shtempler proposed openstack/designate-tempest-plugin master: Fix "PTR recordset" tests suite https://review.opendev.org/c/openstack/designate-tempest-plugin/+/794708 | 10:00 |
| opendevreview | kiran pawar proposed openstack/designate master: [WIP]Add new API for zone move https://review.opendev.org/c/openstack/designate/+/796633 | 11:58 |
| opendevreview | Michael Johnson proposed openstack/designate-tempest-plugin master: Update the constraints url https://review.opendev.org/c/openstack/designate-tempest-plugin/+/684539 | 13:24 |
| opendevreview | Michael Johnson proposed openstack/designate-tempest-plugin master: Use TOX_CONSTRAINTS_FILE and update tox https://review.opendev.org/c/openstack/designate-tempest-plugin/+/792319 | 13:25 |
| kinpaa12389 | @johnsom created WIP PR - https://review.opendev.org/c/openstack/designate/+/796633 Please check. | 14:06 |
| johnsom | kinpaa12389 Great, thank you! I will add it to my list. Sorry I have not had a chance yet to look at your updates. I hope to soon. | 14:07 |
| kinpaa12389 | Also, the newly added API is failing on zuul with 405 Error while on devstack it is not. Error 405 - method_not_allowed | 14:07 |
| johnsom | I hope the other community members can also review | 14:07 |
| kinpaa12389 | Is there any hacks to get newly added method/API to get approved by Zuul ? | 14:07 |
| johnsom | No, there is likely something wrong in the patch. We can probably help figure that out. | 14:08 |
| kinpaa12389 | sure, it was failing on devstack too. I changed designate/devstack/setting file mod_wsgi to uwsgi. But it should not be part of PR | 14:10 |
| johnsom | Yeah, we have bug in uwsgi we are working on | 14:13 |
| johnsom | kinpaa12389 It looks like you defined a POST method in the API, but testing with a PATCH | 14:17 |
| johnsom | kinpaa12389 https://github.com/openstack/designate/blob/master/designate/api/v2/controllers/zones/tasks/transfer_requests.py#L116 | 14:20 |
| johnsom | kinpaa12389 I commented on the patch for the 405 issue | 14:23 |
| opendevreview | kiran pawar proposed openstack/designate master: [WIP]Add new API for zone move https://review.opendev.org/c/openstack/designate/+/796633 | 16:00 |
| eandersson | Do you know why mod_wsgi was failing? was it code, or devstack (bash / linux) related? | 17:08 |
| johnsom | eandersson lol, that issue you fixed with the partial imports is one of my favorite python interview questions to ask. | 21:09 |
| johnsom | If you have written a lot of python code, you immediately know the answer. | 21:09 |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: Add simple scheduler permutation tests https://review.opendev.org/c/openstack/designate/+/794135 | 21:18 |
| johnsom | Ha, I have that open for review right now. | 21:18 |
| opendevreview | Merged openstack/designate master: Fixed a potential circular dependency https://review.opendev.org/c/openstack/designate/+/794131 | 22:28 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!