*** hamalq has quit IRC | 00:43 | |
openstackgerrit | MaAoyu proposed openstack/designate master: bump py37 to py3 in tox.ini https://review.opendev.org/757375 | 00:57 |
---|---|---|
openstackgerrit | Merged openstack/designate master: Remove the unused coding style modules https://review.opendev.org/754957 | 03:11 |
*** ircuser-1 has joined #openstack-dns | 03:11 | |
openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/designate stable/ussuri: Increase lower-constraint for kombu https://review.opendev.org/758326 | 03:50 |
openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/designate master: Fix lower-constraints test not constraining properly https://review.opendev.org/758336 | 05:18 |
openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/designate stable/victoria: Fix lower-constraints test not constraining properly https://review.opendev.org/758337 | 05:19 |
openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/designate stable/ussuri: Fix lower-constraints test not constraining properly https://review.opendev.org/758338 | 05:19 |
eandersson | johnsom ^ this is why ussuri is currently failing | 05:20 |
eandersson | pip install -e . is causing libraries (e.g. amqp) to get upgraded beyond the lower constraints | 05:21 |
eandersson | I see it affects Octavia as well | 05:25 |
eandersson | https://zuul.opendev.org/t/openstack/build/49e3b5f2ddc84d309ef9e40b22746f24/log/job-output.txt#529 | 05:25 |
eandersson | https://github.com/openstack/octavia/blob/master/lower-constraints.txt#L3 | 05:26 |
openstackgerrit | Jens Harbott (frickler) proposed openstack/designate stable/ussuri: pdns4 backend: check if zone exists before attempting delete https://review.opendev.org/758265 | 05:48 |
frickler | eandersson: so https://review.opendev.org/755363 might not have been needed after all? | 05:49 |
eandersson | Yea | 05:49 |
eandersson | It's failing for a different reason now thou | 05:49 |
eandersson | That I can't reproduce locally | 05:49 |
eandersson | https://a9fb9a201a86335f8e80-7cd608bf14cafb686390b86bc06cde2a.ssl.cf5.rackcdn.com/758336/1/check/openstack-tox-lower-constraints/53d1465/job-output.txt | 05:49 |
eandersson | > stevedore.exception.NoMatches: No 'designate.network_api' driver found, looking for 'fake'" | 05:49 |
johnsom | Hmm, I will take a look in the morning if you haven’t got it by then. It is late here | 05:50 |
eandersson | No worries. No hurry. Yea 10.51 PM over here :D | 05:51 |
johnsom | Yep, same here | 05:51 |
eandersson | frickler on the other hand probably just woke up :D | 05:51 |
frickler | might need to bump l-c for stevedore instead | 05:51 |
eandersson | Yea - was thinking that, but not sure why it isn't failing locally for me. | 05:51 |
frickler | right, no breakfast yet ;) | 05:52 |
frickler | I can give it a try locally here, if I can't reproduce either, I'll hold a node to compare | 05:53 |
johnsom | Seems like we are just missing some constraints config in tox.ini, but that is purely off my head. | 05:54 |
johnsom | I am crashing for the night, but if there is something I can pick up in the morning, ping me. | 05:56 |
eandersson | night | 05:57 |
eandersson | frickler maybe a zuul change. The problem is that the package isn't being installed | 06:13 |
eandersson | which makes sense since I disabled the pip install -e ., | 06:13 |
eandersson | Trying to figure out how nova does it | 06:14 |
eandersson | https://zuul.opendev.org/t/openstack/build/4c349867f18547c7b03c3e55d1c9c6f0/log/job-output.txt#10723 | 06:15 |
frickler | eandersson: I've compared the pip freeze from your l-c job above with the one from earlier https://api.us-east.open-edge.io:8080/swift/v1/AUTH_e02c11e4e2c24efc98022353c88ab506/zuul_opendev_logs_805/758326/1/check/openstack-tox-lower-constraints/80589aa/tox/lower-constraints-3.log and it shows a weird mix of some things being upgraded and others downgraded, not sure yet what to make of that | 06:20 |
frickler | might be the lower-constraints.txt is wildly lacking a lot of things, need to doublecheck | 06:21 |
eandersson | Yea - at least what is happening before my patch is that lower-constraints gets installed, and then it does pip install -e . | 06:21 |
eandersson | which causes some to be upgraded | 06:21 |
openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/designate master: Fix lower-constraints test not constraining properly https://review.opendev.org/758336 | 06:56 |
eandersson | I think that did it | 07:08 |
openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/designate stable/ussuri: Fix lower-constraints test not constraining properly https://review.opendev.org/758338 | 07:09 |
frickler | eandersson: ah, that was the other issue mentioned in the nova patch, great. want to drop your -2 so we can merge once tests pass? | 07:18 |
eandersson | Yea | 07:18 |
openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/designate stable/victoria: Fix lower-constraints test not constraining properly https://review.opendev.org/758337 | 07:19 |
eandersson | unfortunately still breaking | 07:26 |
frickler | :( | 07:26 |
eandersson | It is still installdeps and then installing designate after | 07:28 |
openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/designate stable/ussuri: Fix lower-constraints test not constraining properly https://review.opendev.org/758338 | 07:42 |
eandersson | Gonna go to bed. Feel free to edit the changes if you come up with anything frickler | 07:53 |
openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/designate stable/ussuri: Fix lower-constraints test not constraining properly https://review.opendev.org/758338 | 07:54 |
frickler | eandersson: thanks, good night to you | 07:54 |
openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/designate stable/ussuri: Fix lower-constraints test not constraining properly https://review.opendev.org/758338 | 08:25 |
eandersson | thanks frickler!... also had to give it a last try lol | 08:25 |
frickler | yeah, I know that kind of addiction all too well ... | 08:26 |
eandersson | That worked at least, but it isn't super pretty. | 08:33 |
openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/designate stable/victoria: Fix lower-constraints test not constraining properly https://review.opendev.org/758337 | 08:34 |
openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/designate master: Fix lower-constraints test not constraining properly https://review.opendev.org/758336 | 08:35 |
eandersson | Let me know if you can come up with something... cleaner :D | 08:36 |
eandersson | Also we should merge this https://review.opendev.org/#/c/755143/1 | 08:37 |
eandersson | Ok bed time for real now! | 08:38 |
frickler | so we have a couple of open patches for stable stein, but I think it would require someone to fix the CI first in order to get them merged. c.f. https://etherpad.opendev.org/p/designate-stable-stein-em and the recent ML post, deadline is 2020-11-11. | 09:36 |
openstackgerrit | Merged openstack/designate stable/victoria: Fix pdns4 devstack plugin https://review.opendev.org/755143 | 09:39 |
*** njohnston has joined #openstack-dns | 11:38 | |
openstackgerrit | Igor Malinovskiy proposed openstack/designate master: Fix resource links in API endpoints https://review.opendev.org/752640 | 16:08 |
openstackgerrit | Igor Malinovskiy proposed openstack/designate master: Implement sharing of zones https://review.opendev.org/726334 | 16:08 |
*** hamalq has joined #openstack-dns | 17:02 | |
*** cyberde has joined #openstack-dns | 17:02 | |
openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/designate stable/victoria: Increase lower-constraint for kombu https://review.opendev.org/758471 | 17:32 |
openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/designate stable/victoria: Fix lower-constraints test not constraining properly https://review.opendev.org/758337 | 17:33 |
eandersson | frickler maybe that kombu increase was needed after all | 17:38 |
*** Vadmacs has joined #openstack-dns | 18:16 | |
openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/designate stable/victoria: Fix lower-constraints test not constraining properly https://review.opendev.org/758337 | 18:59 |
openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/designate stable/victoria: Fix lower-constraints test not constraining properly https://review.opendev.org/758337 | 18:59 |
openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/designate stable/ussuri: Fix lower-constraints test not constraining properly https://review.opendev.org/758338 | 19:01 |
*** Vadmacs has quit IRC | 20:36 | |
*** hamalq has quit IRC | 22:32 | |
*** hamalq has joined #openstack-dns | 22:33 | |
openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/designate stable/ussuri: pdns4 backend: check if zone exists before attempting delete https://review.opendev.org/758265 | 22:39 |
eandersson | johnsom, frickler I am happy enough with the lower-constraints patch set. Let me know if they look good, or if you have any alternative ideas on how to fix the lower-constraints issue | 23:11 |
johnsom | There was a discussion started in the requirements channel too | 23:13 |
eandersson | Ah interesting | 23:14 |
eandersson | ah yea I was talking a lot to smcginnis earlier about this | 23:15 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!