Friday, 2021-02-12

*** ircuser-1 has joined #openstack-dns04:12
*** lbragstad_ has joined #openstack-dns15:46
*** lbragstad has quit IRC15:50
*** lbragstad_ is now known as lbragstad16:25
nicolasbockGood morning johnsom ! Looking at the centralized lock change again. I don't think we ever releases Train with this fix, did we?16:46
nicolasbockIt's in the stable brnach16:46
nicolasbockbut the latest Train release is 9.0.116:46
nicolasbockAnd that change comes after 9.0.116:46
johnsomNope, it is only in 10 and on16:48
nicolasbockShould we cut a 9.0.2 release?16:55
nicolasbockI'll deploy a Train system and patch it to include the lock16:56
nicolasbockAnd then verify that this fixes (or not) the recordset issue we discussed yesterday16:56
nicolasbockBut if it fixes it then I think a 9.0.2 release might be good16:56
nicolasbockAnd potentially a backport to Stein16:56
*** gmann is now known as gmann_afk17:40
johnsomYeah, I have not looked at where we are on backport releases. (shame on me) I will put that on my list for next week.18:24
johnsomI have been pretty focused on the TripleO work for Designate recently.18:25
johnsomHaving that tested as a proof point would actually be a good case for cutting some stable releases.18:26
*** hamalq has joined #openstack-dns18:36
*** gmann_afk is now known as gmann18:52
nicolasbock+119:00
nicolasbockSince i had a Stein env up already I looked into backporting to Stein. That won't be easy since there are a few things missing in coordination in Stein that this change needs19:01
nicolasbockI am checking out Train right now19:02
nicolasbockWill let you know how that goes19:02
nicolasbockIn case you run into cloudnull say hi from me :)19:03
nicolasbockjohnsom: I patched my Train env and cannot reproduce the problem. In other words the global lock seems to have fixed the issue.I notice quite a performance impact though. The creation of the recordsets is taking longer now (but they are all created which is a plus :) ).19:47
johnsomYeah, so that makes sense all the way around.19:48
johnsomMystery solved. I still think the serial number generation can be improved however.19:48
nicolasbockI have a patch I can submit19:48
nicolasbockIn the meantime I took the liberty to submit: https://review.opendev.org/c/openstack/releases/+/77544219:49
johnsomYeah, I saw that.19:49
nicolasbockI think we still need to tag Stein with 9.0.219:49
nicolasbockNot sure how that works though19:49
nicolasbocksorry Train19:49
johnsomIt looks like we have one open backport for train that we could get in: https://review.opendev.org/c/openstack/designate/+/74361119:51
nicolasbockYes, I agree19:52
nicolasbockIf you +2 then we are good to put that one in19:52
nicolasbockA potential Stein backport would be quite involved since the global lock change depends on a few other changes that haven't landed in Stein (some refactorinig). What's your take? Skip Stein?19:52
johnsomYeah, backports are all about managing risk. If it has a lot of dependencies that aren't met, that risk may be too high for the benefit.19:53
nicolasbockI can take a closer look to see exactly what's invovled19:53
johnsomStein is extended maintenance too, so it'19:53
johnsoms really up to if someone needs it.19:54
nicolasbockWe have a customer that ran into this bug on Stein19:54
johnsomlol, well, there you go. grin19:54
nicolasbockIt's whether they are willing to consider to upgrade or not19:54
nicolasbock:)19:54
nicolasbockOnce we release 9.0.2 I can go to them and tell them the good news that once they upgrade it will be fixed :)19:55
johnsomI see you rebased the Ussuri version of that zone masters patch, any idea why zuul isn't happy? https://review.opendev.org/c/openstack/designate/+/74361019:55
nicolasbockHmm, the lower-constraints job is kiling me19:56
nicolasbockIt's broken more often than not19:56
nicolasbockI'll check19:56
johnsomThanks19:56
*** cvstealth has joined #openstack-dns22:11
*** hamalq has quit IRC23:51

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