Thursday, 2020-10-15

*** hamalq has quit IRC00:43
openstackgerritMaAoyu proposed openstack/designate master: bump py37 to py3 in tox.ini  https://review.opendev.org/75737500:57
openstackgerritMerged openstack/designate master: Remove the unused coding style modules  https://review.opendev.org/75495703:11
*** ircuser-1 has joined #openstack-dns03:11
openstackgerritErik Olof Gunnar Andersson proposed openstack/designate stable/ussuri: Increase lower-constraint for kombu  https://review.opendev.org/75832603:50
openstackgerritErik Olof Gunnar Andersson proposed openstack/designate master: Fix lower-constraints test not constraining properly  https://review.opendev.org/75833605:18
openstackgerritErik Olof Gunnar Andersson proposed openstack/designate stable/victoria: Fix lower-constraints test not constraining properly  https://review.opendev.org/75833705:19
openstackgerritErik Olof Gunnar Andersson proposed openstack/designate stable/ussuri: Fix lower-constraints test not constraining properly  https://review.opendev.org/75833805:19
eanderssonjohnsom ^ this is why ussuri is currently failing05:20
eanderssonpip install -e . is causing libraries (e.g. amqp) to get upgraded beyond the lower constraints05:21
eanderssonI see it affects Octavia as well05:25
eanderssonhttps://zuul.opendev.org/t/openstack/build/49e3b5f2ddc84d309ef9e40b22746f24/log/job-output.txt#52905:25
eanderssonhttps://github.com/openstack/octavia/blob/master/lower-constraints.txt#L305:26
openstackgerritJens Harbott (frickler) proposed openstack/designate stable/ussuri: pdns4 backend: check if zone exists before attempting delete  https://review.opendev.org/75826505:48
fricklereandersson: so https://review.opendev.org/755363 might not have been needed after all?05:49
eanderssonYea05:49
eanderssonIt's failing for a different reason now thou05:49
eanderssonThat I can't reproduce locally05:49
eanderssonhttps://a9fb9a201a86335f8e80-7cd608bf14cafb686390b86bc06cde2a.ssl.cf5.rackcdn.com/758336/1/check/openstack-tox-lower-constraints/53d1465/job-output.txt05:49
eandersson> stevedore.exception.NoMatches: No 'designate.network_api' driver found, looking for 'fake'"05:49
johnsomHmm, I will take a look in the morning if you haven’t got it by then. It is late here05:50
eanderssonNo worries. No hurry. Yea 10.51 PM over here :D05:51
johnsomYep, same here05:51
eanderssonfrickler on the other hand probably just woke up :D05:51
fricklermight need to bump l-c for stevedore instead05:51
eanderssonYea - was thinking that, but not sure why it isn't failing locally for me.05:51
fricklerright, no breakfast yet ;)05:52
fricklerI can give it a try locally here, if I can't reproduce either, I'll hold a node to compare05:53
johnsomSeems like we are just missing some constraints config in tox.ini, but that is purely off my head.05:54
johnsomI am crashing for the night, but if there is something I can pick up in the morning, ping me.05:56
eanderssonnight05:57
eanderssonfrickler maybe a zuul change. The problem is that the package isn't being installed06:13
eanderssonwhich makes sense since I disabled the pip install -e .,06:13
eanderssonTrying to figure out how nova does it06:14
eanderssonhttps://zuul.opendev.org/t/openstack/build/4c349867f18547c7b03c3e55d1c9c6f0/log/job-output.txt#1072306:15
fricklereandersson: 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 that06:20
fricklermight be the lower-constraints.txt is wildly lacking a lot of things, need to doublecheck06:21
eanderssonYea - at least what is happening before my patch is that lower-constraints gets installed, and then it does pip install -e .06:21
eanderssonwhich causes some to be upgraded06:21
openstackgerritErik Olof Gunnar Andersson proposed openstack/designate master: Fix lower-constraints test not constraining properly  https://review.opendev.org/75833606:56
eanderssonI think that did it07:08
openstackgerritErik Olof Gunnar Andersson proposed openstack/designate stable/ussuri: Fix lower-constraints test not constraining properly  https://review.opendev.org/75833807:09
fricklereandersson: 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
eanderssonYea07:18
openstackgerritErik Olof Gunnar Andersson proposed openstack/designate stable/victoria: Fix lower-constraints test not constraining properly  https://review.opendev.org/75833707:19
eanderssonunfortunately still breaking07:26
frickler:(07:26
eanderssonIt is still installdeps and then installing designate after07:28
openstackgerritErik Olof Gunnar Andersson proposed openstack/designate stable/ussuri: Fix lower-constraints test not constraining properly  https://review.opendev.org/75833807:42
eanderssonGonna go to bed. Feel free to edit the changes if you come up with anything frickler07:53
openstackgerritErik Olof Gunnar Andersson proposed openstack/designate stable/ussuri: Fix lower-constraints test not constraining properly  https://review.opendev.org/75833807:54
fricklereandersson: thanks, good night to you07:54
openstackgerritErik Olof Gunnar Andersson proposed openstack/designate stable/ussuri: Fix lower-constraints test not constraining properly  https://review.opendev.org/75833808:25
eanderssonthanks frickler!... also had to give it a last try lol08:25
frickleryeah, I know that kind of addiction all too well ...08:26
eanderssonThat worked at least, but it isn't super pretty.08:33
openstackgerritErik Olof Gunnar Andersson proposed openstack/designate stable/victoria: Fix lower-constraints test not constraining properly  https://review.opendev.org/75833708:34
openstackgerritErik Olof Gunnar Andersson proposed openstack/designate master: Fix lower-constraints test not constraining properly  https://review.opendev.org/75833608:35
eanderssonLet me know if you can come up with something... cleaner :D08:36
eanderssonAlso we should merge this https://review.opendev.org/#/c/755143/108:37
eanderssonOk bed time for real now!08:38
fricklerso 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
openstackgerritMerged openstack/designate stable/victoria: Fix pdns4 devstack plugin  https://review.opendev.org/75514309:39
*** njohnston has joined #openstack-dns11:38
openstackgerritIgor Malinovskiy proposed openstack/designate master: Fix resource links in API endpoints  https://review.opendev.org/75264016:08
openstackgerritIgor Malinovskiy proposed openstack/designate master: Implement sharing of zones  https://review.opendev.org/72633416:08
*** hamalq has joined #openstack-dns17:02
*** cyberde has joined #openstack-dns17:02
openstackgerritErik Olof Gunnar Andersson proposed openstack/designate stable/victoria: Increase lower-constraint for kombu  https://review.opendev.org/75847117:32
openstackgerritErik Olof Gunnar Andersson proposed openstack/designate stable/victoria: Fix lower-constraints test not constraining properly  https://review.opendev.org/75833717:33
eanderssonfrickler maybe that kombu increase was needed after all17:38
*** Vadmacs has joined #openstack-dns18:16
openstackgerritErik Olof Gunnar Andersson proposed openstack/designate stable/victoria: Fix lower-constraints test not constraining properly  https://review.opendev.org/75833718:59
openstackgerritErik Olof Gunnar Andersson proposed openstack/designate stable/victoria: Fix lower-constraints test not constraining properly  https://review.opendev.org/75833718:59
openstackgerritErik Olof Gunnar Andersson proposed openstack/designate stable/ussuri: Fix lower-constraints test not constraining properly  https://review.opendev.org/75833819:01
*** Vadmacs has quit IRC20:36
*** hamalq has quit IRC22:32
*** hamalq has joined #openstack-dns22:33
openstackgerritErik Olof Gunnar Andersson proposed openstack/designate stable/ussuri: pdns4 backend: check if zone exists before attempting delete  https://review.opendev.org/75826522:39
eanderssonjohnsom, 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 issue23:11
johnsomThere was a discussion started in the requirements channel too23:13
eanderssonAh interesting23:14
eanderssonah yea I was talking a lot to smcginnis earlier about this23:15

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!