Tuesday, 2020-08-11

*** hamalq_ has quit IRC02:21
*** zbr9 has joined #opendev-meeting03:04
*** zbr has quit IRC03:04
*** zbr9 is now known as zbr03:04
*** hamalq has joined #opendev-meeting16:10
*** hamalq_ has joined #opendev-meeting16:11
*** hamalq has quit IRC16:16
*** hamalq_ has quit IRC16:29
*** hamalq has joined #opendev-meeting16:30
*** hamalq has quit IRC17:31
* fungi peers around19:00
clarkbhello19:00
clarkbanyone else here for our weekly meeting? we'll get started momentarily19:00
corvuso/19:00
clarkb#startmeeting infra19:01
openstackMeeting started Tue Aug 11 19:01:02 2020 UTC and is due to finish in 60 minutes.  The chair is clarkb. Information about MeetBot at http://wiki.debian.org/MeetBot.19:01
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.19:01
*** openstack changes topic to " (Meeting topic: infra)"19:01
openstackThe meeting name has been set to 'infra'19:01
clarkb#link http://lists.opendev.org/pipermail/service-discuss/2020-August/000070.html Our Agenda19:01
clarkb#topic Announcements19:01
*** openstack changes topic to "Announcements (Meeting topic: infra)"19:01
clarkbI'll be out tomorrow in order to get outside and do some socially distant fishing19:01
ianwo/19:02
zbro/19:02
clarkbmay as well take advantage of the early morning schedule for opendev event and get out on the water early too :)19:02
clarkbany other announcements?19:02
fricklero/19:03
clarkb#topic Actions from last meeting19:03
*** openstack changes topic to "Actions from last meeting (Meeting topic: infra)"19:03
clarkb#link http://eavesdrop.openstack.org/meetings/infra/2020/infra.2020-08-04-19.01.txt minutes from last meeting19:03
clarkbwe didn't record any actions, but the week prior had an action for ianw to look into python wheel caching for more than openstack19:03
clarkbianw: ^ anything new to add to that?19:03
ianwi haven't pushed any changes, but yes i started playing with the manylinux docker container builders19:04
ianwhttps://github.com/pyca/cryptography/issues/5292#issuecomment-67175930619:05
ianwthat will be useful for us, but also be something that can be useful for others too.  although how it gets consumed is an open question19:05
clarkbianw: I think a good chunk of the wheels we build assume things about system packages like libvirt? but even that one may be able to be manylinux'd if it can handle many libvirt apis19:05
clarkbthat said I suspect a number are actually reduceable to many linux19:06
ianwyeah, i think think it's an "and" thing rather than an "or"19:06
ianwanyway, i got a bit sidetracked into other things as well, so didn't make great progress, but still looking at it19:07
clarkbthanks19:07
clarkb#topic Specs approval19:07
*** openstack changes topic to "Specs approval (Meeting topic: infra)"19:07
clarkb#link https://review.opendev.org/#/c/731838/ Authentication broker service19:07
clarkbI've reviewed the latest patchset for this change and it look sgood to me19:08
clarkbbut I don't think many others have reviewed it yet. Is this something we think we might put up for approval next week and hopefully do any remaining iteration this week?19:08
fungii wouldn't mind corvus and mordred taking a look through it since they're technically co-authors (i copied and pasted text from some of their e-mails from years past)19:09
corvusi'll do that this week19:10
fungithanks!19:10
clarkbthank you. I guess we'll see where we end up and possibly will have that up for approval next week19:10
clarkb#topic Priority Efforts19:10
*** openstack changes topic to "Priority Efforts (Meeting topic: infra)"19:10
fungiwould just hate to have a spec they sort of co-wrote merge without at least getting skimmed ;)19:10
clarkbfungi: ++19:10
clarkb#topic Update Config Management19:10
*** openstack changes topic to "Update Config Management (Meeting topic: infra)"19:10
clarkbI've been poking at some Gerrit related docker things recently19:11
clarkbGerritbot containerization in progress at https://review.opendev.org/#/c/745573/1 and parents19:11
clarkb#link https://review.opendev.org/#/c/745240/ in particular needs second review.19:11
clarkbif I can get a second review that is happy with ^ I'll try to land and coordinate those changes with review.o.o and eavesdrop today19:11
clarkbthe ansible group vars should all be set. Its just a matter of stopping gerritbot on review.o.o and ensuring the process that starts in docker on eavesdrop is happy19:12
clarkbI've also been trying to better understand the gerrit upgrade process which has led to a change with image fixes for gerrit19:12
clarkb#link https://review.opendev.org/745595 fixes for gerrit plugins on newer gerrit images19:12
clarkbthat change as is is a bit omnibus like. I'm happy to split it up a bit if reviewers would prefer19:13
clarkbmostly what it does is checkout valid versions of plugins so they build properly across the gerrit versions. It also addresses a javamelody is special problem with plugin building19:13
clarkbAny other config management items to bring up?19:14
fungithat reminds me i need to finish the mirror-update ansibilification for reprepro mirrors19:15
clarkb#topic OpenDev19:16
*** openstack changes topic to "OpenDev (Meeting topic: infra)"19:16
clarkb#link https://review.opendev.org/741277 Gerritlib change to support creating projects with non master HEAD19:17
clarkb#link https://review.opendev.org/741279 Can land once Gerritlib release is made with above change. This change updates jeepyb to toggle that flag19:17
clarkbFor review-test does anyone understand the state it is in?19:17
clarkbianw: I think you had to disable the upstream trakcing cron because it had filled the disk with logs?19:18
ianwyes, but the point was more that ansible wasn't completing on it so i had to do it manually19:18
clarkbdo we need to add it to the emergency file while we figure out what it needs?19:19
ianwumm, i forget now why it wasn't simple to fix19:19
clarkbok, it would probably be a good idea to see if we can keep it from interacting with other production things19:21
clarkbfrom a general upgrade perspective I've been trying to bootstrap myself on the process there so that one can be written down for tesitng on review-test19:21
ianwhttp://eavesdrop.openstack.org/irclogs/%23opendev/%23opendev.2020-08-04.log.html#t2020-08-04T21:52:3019:22
clarkbthat is how I found the docker image issues I'm trying to fix in that change above19:22
zbrclarkb: i can help with gerritbot, probably gerritlib too,  just add me s reviewer19:22
ianwAnsibleUndefinedVariable: 'gerrit_vhost_name' is undefined19:22
ianwthat's what review-test is failing on, if that rings any bells for anyone19:22
fungimaybe it's as simple as just adding a missed hostvar19:22
clarkbcorvus: one thing in particular that I've learned/discovered is that an upgrade to 2.15 should be really straightforward. Do you know why it was rejected or at least not the preferred route when you and mordred talked to gerrit?19:23
fungithough we do set gerrit_vhost_name: review-test.opendev.org in inventory/service/host_vars/review-test.openstack.org.yaml19:23
clarkb(I know it doesn't get us to fully notedb, but it is forward progress and likely could be done with minimal downtime, though I've yet to test 2.13 -> 2.15 without 2.14 in the middle)19:23
ianwfungi: yeah i think that's about where i got to :)19:24
corvusclarkb: one thing i remember is *do not use notedb in 2.15*19:24
clarkbcorvus: ya the default is notedb is only used for accounts on 2.15 iirc19:25
corvusclarkb: so since we can't actually make progress on the conversion until 2.16, would an upgrade to 2.15 get us anything?19:25
ianwfungi: yeah, that's right ... it started getting into how we deploy the host vars which got all split up and changed and *that's* why it wasn't trivial19:25
clarkbcorvus: it gets us momentum? and simplifies the testing process for us to get there as we don't have to do the full notedb profiling (disk use and performance)19:25
corvusclarkb: i'm not sure if even that is safe?  i dunno.19:26
clarkbmaybe that is something we should ask upstream. Mostly I'm thinking that we could get that done relatively quickly while still working towards the 2.16 notedb transition too19:26
corvusclarkb: i just remember them saying that notedb in 2.15 is wrong, and doing a conversion in 2.15 would prevent a correct upgrade later.19:26
corvusclarkb: like, i don't get what the purpose is?  what is easy in 2.15 that's hard in 2.16?19:27
clarkbcorvus: 2.15 doesn't notedb for changes19:27
corvusfrom my pov, 2.16 is like 2.15 but without bugs19:27
clarkbcorvus: thats not how I read their docs19:27
clarkb2.15 is no changes notedb because its buggy19:27
clarkband you keep using the system that worked previously (which is why it is easier, we know the prformance of that)19:28
corvusyou can do that with 2.16 too, right?19:28
clarkbcorvus: their docs actively recommend against that (but don't provide a reason), but yes we could go to 2.16 with no notedb then switch to notedb after potentially19:28
fungii guess it's a question of whether they're saying not to run 2.16 without notedb (so prevents us from decoupling the upgrade and db migration steps) or whether it's safe to run with everything still in mysql on 2.16 and then perform the notedb migration19:29
clarkbbut ya maybe the way to frame it is more around change in notedb rather than specific versions. I think we can relatively easily upgrade to a gerrit version without converting to changes notedb19:29
clarkbI think it is the notedb conversion itself that we want to carefully test and profile to ensure we don't run out disk, the performance remains reasonable, and to deal with any replication change requirements19:29
corvusclarkb: say you're right: let's say it's easy to upgrade to 2.15 without making any changes.  why would we do it?  it doesn't help the upgrade to anything later.19:29
corvusit's also not supported19:29
corvus(even 2.16 isn't supported at this point)19:30
corvusto me, it just feels like wasting time that could be spent working on an upgrade to 2.1619:30
clarkbI think there is value in making some progress19:30
clarkbit gets us polygerrit19:30
corvusi don't think it's progress19:30
corvusit's a bad/wrong polygerrit19:30
corvus2.16 is the polygerrit that we actually want to introduce users to19:30
zbri am inclined to believe doing both would be less work, less risks.19:31
clarkbwhat is bad or wrong about it?19:31
fungianother question. if we upgrade to 2.16 and migrate to notedb all in one shot, but subsequently discover issues with it, would we want to try to roll back to not-notedb or roll forward to a polygerrit-only 3.x and hope the issues resolved themselves?19:31
corvusi think 2.15 polygerrit is different than 2.16; 2.16 is like what's in 3.x19:31
clarkbI'm just looking at the problem in front of us and a big part of it (from my perspective) is its size. It feels huge and breaking it down if we can would help us make progress I think19:32
clarkbit isn't perfect, and we should continue to plan to upgrade further19:32
corvusclarkb: there is a significant chance (i say this based on past upgrades) that each version upgrade will cause us to burn cycles on version-specific issues.  i'd hate to spend time on a 2.15 specific issue rather than 2.16.19:32
corvusclarkb: in general, i agree with small incremental steps.  i just don't see what the incremental step between here and 2.16 is.19:33
clarkbcorvus: thats fair, maybe we should ask about the risk with not doing the notedb migration immediately on 2.16? then we can continue to split the problems up?19:33
clarkbfrom an operational perspective it would simplify the imgaes we need to manage19:34
clarkbwe could drop 2 or 3 of them19:34
corvusclarkb: luca has offered to help us upgrade, and i think that would be exactly the sort of thing he would welcome as part of that offer19:34
corvus(that question)19:34
clarkbcorvus: thats great to hear, why don't I do a bit more local investigating (there are a few scnearios I wanted to test like skipping versions) then try and compile a couple of questions like that for luca19:35
corvussorry if that wasn't clear; i'm suggesting that asking luca about running 2.16 without notedb is a good q19:35
clarkbyup19:35
corvusalso, the q about 2.1519:35
fungialso maybe he can clarify whether running the 2.15 its-storyboard plugin with 3.x is a bad idea19:35
fungisince they don't seem to have branched it past 2.1519:36
corvus(my guess is he's more likely to agree that upgrading to 2.15 is a good idea)19:36
corvus(but he doesn't have my lived experience with what it takes to perform an opendev gerrit upgrade)19:36
corvus(he upgrades continually :)19:37
*** diablo_rojo has joined #opendev-meeting19:37
clarkbI should be able to have tested our updated images by the end of the week and have a general sense for what is potentially a good idea and what isn't based on simple local testing. Then work up an email for luca19:37
clarkbhaving images that don't spit out tracebacks on startup was step 0 there :)19:38
clarkbanything else on opendev?19:38
clarkb#topic General Topics19:39
*** openstack changes topic to "General Topics (Meeting topic: infra)"19:39
clarkb#topic Bup and Borg19:39
*** openstack changes topic to "Bup and Borg (Meeting topic: infra)"19:39
clarkbianw I think this is largely a borg topic now. The borg change has the reviews it needs19:39
ianwyep, i just need to start a server and get something in to test, still on my short-term TODO sorry19:40
clarkbno worries. I think we're all pretty swamped. But wanted ot make sure you saw that19:40
clarkb#topic Github 3rd Party CI19:40
*** openstack changes topic to "Github 3rd Party CI (Meeting topic: infra)"19:40
clarkbhow is this going?19:40
ianwso OK I think, no complaints19:41
clarkbare they consuming it as a CI system yet or are we still in the water temp testing stage?19:41
fungipraise would be better, but i'll take no complaints ;)19:41
ianwnext thing for pyca/cryptography we should enable it for master commits, as well as pull requests19:41
ianwthe project has a .zuul.d directory committed, so that's good :)19:41
ianwso probably the next thing is to see if we can fit into wheel generation somewhere, as described previously19:42
ianwi got some private communication that libxml was also in need of similar arm64 resources19:43
clarkblxml or libxml?19:44
clarkbbut ya it wouldn't surprise me if there is a similar need for many of those more costly python packages we've seen end up in the openstack wheel cache19:45
clarkbanything else on this topic?19:45
clarkb#topic Open Discussion19:47
*** openstack changes topic to "Open Discussion (Meeting topic: infra)"19:47
ianwlxml sorry19:47
fungiyeah, lxml links libxml when building19:47
clarkbwe have a bit of time for any other items that are shareable19:47
fungi#link http://lists.openstack.org/pipermail/openstack-discuss/2020-August/016424.html dates for ptg have been firmed up (it's the week after the summit)19:47
clarkboh I had missed that19:49
clarkbthe 6 hours we had last time seemed to work well so I'll probably schedule a similar block.19:49
clarkbalso we'll likely want to double check meetpad is still happy a few weeks prior to that (iirc we auto update the images so we may pull in new things we need to accomodate)19:50
corvusmy patch isn't merged upstream, so the web server container is still pinned19:51
clarkbwe should also consider scaling it up again19:51
clarkbthough we probably only need ~2 extra servers this time based on load last time (really the bottleneck most people saw eemed to be in the browser)19:52
clarkbsounds like that may be it. Thank you everyone. We'll see you here next week.19:54
clarkb#endmeeting19:54
*** openstack changes topic to "Incident management and meetings for the OpenDev sysadmins; normal discussions are in #opendev"19:54
openstackMeeting ended Tue Aug 11 19:54:40 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)19:54
openstackMinutes:        http://eavesdrop.openstack.org/meetings/infra/2020/infra.2020-08-11-19.01.html19:54
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/infra/2020/infra.2020-08-11-19.01.txt19:54
openstackLog:            http://eavesdrop.openstack.org/meetings/infra/2020/infra.2020-08-11-19.01.log.html19:54
corvusclarkb: thanks!19:54
fungithanks clarkb!19:54
zbrthanks19:54
-openstackstatus- NOTICE: The openstackgerrit IRC bot (gerritbot) will be offline for a short period while we redeploy it on a new server20:51

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