Saturday, 2020-03-21

mordredfungi: feel like nudging https://review.opendev.org/#/c/714171/ across the line?00:00
fungithat is a marathon change, but will try00:19
mordredfungi: it's mostly just requires/provides :)00:22
fungiyeah, but this is the time of night where quarantine era fungi breaks into the cache of rum00:23
mordredmmm. case of rum00:24
mordredfungi: I feel like Quarantine Era Fungi should be a bobblehead00:24
* fungi does basically buy by the case from the local distillery00:24
fungithough as of yesterday our distillery also started making and giving out free hand sanitizer00:25
fungithey ran out in 3 hours00:25
fungibut had more today00:25
fungifrom what i gather it's basically "151" but without bothering to fraction off the nastier esters and labeled "not for internal use"00:27
fungialso if you've ever wanted an excuse for having an open container of 151 in the car when you get stopped at a police roadblock... "it's just hand sanitizer, officer!"00:29
fungimordred: stray comment on 714171? (see inline)00:33
fungiif you confirm we can just remove that in a followup change, i'm happy to approve00:33
*** diablo_rojo__ has quit IRC02:43
*** DSpider has joined #opendev06:04
*** tristanC_ has joined #opendev06:19
*** tristanC has quit IRC06:21
AJaegerconfig-core, for the infra-manual change, please review https://review.opendev.org/#/c/713925/08:18
openstackgerritAndreas Jaeger proposed openstack/project-config master: Update infra-manual links  https://review.opendev.org/71392508:29
fungi#status log ze01 was rebooted at 04:08z and 06:26z in the process of a server migration off a problem hypervisor host12:12
openstackstatusfungi: finished logging12:12
*** tosky has joined #opendev13:50
*** tristanC_ is now known as tristanC14:07
openstackgerritMonty Taylor proposed opendev/system-config master: Cleanup a few docker image related thigns  https://review.opendev.org/71425714:31
openstackgerritDonny Davis proposed openstack/diskimage-builder master: Adding Fedora 31 to DIB_RELEASE  https://review.opendev.org/71425915:11
openstackgerritMonty Taylor proposed opendev/system-config master: Split gitea and gerrit services from manage-projects  https://review.opendev.org/71399515:16
openstackgerritMonty Taylor proposed opendev/system-config master: Add opendev-buildset-registry depends where overridden  https://review.opendev.org/71426015:16
openstackgerritMonty Taylor proposed opendev/system-config master: Remove unused gerrit puppet things  https://review.opendev.org/71400115:17
openstackgerritJames E. Blair proposed opendev/system-config master: Add meetpad server  https://review.opendev.org/71423815:20
openstackgerritMonty Taylor proposed opendev/system-config master: Cleanup a few docker image related thigns  https://review.opendev.org/71425715:21
openstackgerritMonty Taylor proposed opendev/system-config master: Add opendev-buildset-registry depends where overridden  https://review.opendev.org/71426015:21
openstackgerritMonty Taylor proposed opendev/system-config master: Split gitea and gerrit services from manage-projects  https://review.opendev.org/71399515:21
openstackgerritMonty Taylor proposed opendev/system-config master: Remove unused gerrit puppet things  https://review.opendev.org/71400115:22
mordredcorvus: I feel like that .env file might do something magic with docker-compose?15:25
corvusfungi, mordred: fungi's comment on https://review.opendev.org/714171 is addressed, should we land that now?15:25
mordredcorvus: I'd like to. also - I have a fun follow up stack for your reviewing pleasure :)15:26
corvusmordred: yes it does; it sets env variables globally15:26
mordredcorvus: neat.15:26
corvusit's convenient, and i like it, except that it's a dotfile which i don't like15:26
mordredbut why would you ever want to see what env vars are being set? impl details15:27
corvusmaybe we should add a comment to the docker-compose file letting folks know there's a dotfile it depends on15:27
mordred++15:27
mordredI like that idea15:27
corvusmordred: i like your changes... but i'm wondering whether we should have all the dependencies in the pipelines, or all in the jobs15:29
mordredI had the same thought ...15:29
corvusi'm +2 on 260 as written because either is better than both :)15:30
corvusbut we should mull it over15:30
mordredI feel like if it's a job that might get run in another repo maybe the deps in the pipeline is better?15:30
mordredbut if it's a job that's really just about system-config, then deps in the job itself might be better?15:30
corvusyep; pipelines also let you trace the dependencies easier.  but it makes the pipeline itself harder to read.15:31
mordredyeah15:31
mordredI'm also now wishing that dependencies was a dict instead of a list so that it would merge rather than overwrite15:32
mordredbut, c'est la vie15:32
fungiwouldn't a set() also allow that?15:34
mordredfungi: yeah - but yaml doesn't have a set construct15:35
mordred(good point though)15:36
AJaegerconfig-core, for the infra-manual move, please review https://review.opendev.org/#/c/713925/ - I'd like to see whether we're aligned on these locations15:36
mordredfungi: while you're in an an approving mode, feel like https://review.opendev.org/#/c/714143/ ? everybody loves ulimits15:36
corvusmordred: yeah, i've been thinking about the override situation, and i think maybe for dependencies, override is important (eg, the "other repo" case)15:39
AJaegercorvus: so, you might want to replace one job with another that e.g. also implements a registry? Yeah, that looks nice to be able to do so15:40
mordredcorvus: ah - good point15:41
openstackgerritsebastian marcet proposed opendev/puppet-openstackid master: Updated python dependencies install  https://review.opendev.org/71426215:41
corvusmordred: your stack lgtm15:43
mordredcorvus: \o/15:46
openstackgerritsebastian marcet proposed opendev/system-config master: OpenStackID 3.0.6 production deployment  https://review.opendev.org/71426315:48
openstackgerritMonty Taylor proposed opendev/system-config master: Use dev subdir on review-dev for project-config things  https://review.opendev.org/71426415:48
openstackgerritAndreas Jaeger proposed openstack/project-config master: Update third-party-check pipeline description  https://review.opendev.org/71426515:49
AJaegerfungi: here's a proposed wording ^15:49
fungiAJaeger: yep, that looks great--thanks!15:55
AJaegercool15:55
openstackgerritMonty Taylor proposed opendev/system-config master: Install utility scripts for running jeepyb commands  https://review.opendev.org/71426716:14
mordredcorvus: ^^ how's the idea look? basically, make those wrappers so that it's easy to manually run if we need to. then also cron jobs and playbooks can just run the scripts16:14
openstackgerritMerged opendev/system-config master: Base 2.13 image on gerrit-base  https://review.opendev.org/71417116:21
openstackgerritMerged opendev/puppet-openstackid master: Updated python dependencies install  https://review.opendev.org/71426216:24
openstackgerritMonty Taylor proposed opendev/system-config master: Split gitea and gerrit services from manage-projects  https://review.opendev.org/71399516:34
openstackgerritMonty Taylor proposed opendev/system-config master: Use dev subdir on review-dev for project-config things  https://review.opendev.org/71426416:34
openstackgerritMonty Taylor proposed opendev/system-config master: Install utility scripts for running jeepyb commands  https://review.opendev.org/71426716:34
openstackgerritMonty Taylor proposed opendev/system-config master: Remove unused gerrit puppet things  https://review.opendev.org/71400116:40
openstackgerritMonty Taylor proposed opendev/system-config master: Set ulimits for gerrit  https://review.opendev.org/71414316:41
openstackgerritMonty Taylor proposed opendev/system-config master: Remove unused gerrit puppet things  https://review.opendev.org/71400116:42
openstackgerritMonty Taylor proposed opendev/system-config master: Correct vars indentation error in rename playbook  https://review.opendev.org/71412716:48
mordredinfra-root: if anyone is bored over the weekend: https://review.opendev.org/#/q/topic:gerrit-docker is a fun for your reviewing pleasure!16:49
clarkbAJaeger: I think https://review.opendev.org/#/c/714229/1/modules/openstack_project/files/gerrit/usg-cla.html needs to point at the contributor guide instead?16:51
clarkbI think we can land that change as is then fix that ina followup16:51
clarkbmordred: why is that start playbook not needed?16:56
mordredclarkb: we don't use it for anything - turns out we pretty much never want to run init except on upgrade - so if we want to run start it's "docker compose up"16:57
clarkbgot it16:57
mordredclarkb: we learned some things about what init does - namely, it downloads your database library and is extracts plugins - and then runs schema migrations16:58
mordredwe updated the image build to do the download and extraction at build time16:58
AJaegerclarkb: will fix in followup...16:58
AJaegerclarkb: we have only https://docs.openstack.org/contributors/common/accounts.html#setup-your-openstack-foundation-account - I'll link to that one17:02
AJaegerclarkb: found better one - https://docs.openstack.org/contributors/common/setup-gerrit.html#signing-it17:03
openstackgerritAndreas Jaeger proposed opendev/system-config master: Update CLA link  https://review.opendev.org/71426917:04
AJaegerclarkb: ^17:04
openstackgerritMerged opendev/system-config master: Don't run gerrit init on restart  https://review.opendev.org/71411717:06
clarkbAJaeger: hrm I think we need to edit the ansible role too as well as the system cla17:06
clarkbI suspect at this point the ansible role is the one that matters not the other one17:07
clarkbalso its the weekend so feel free to ignore until monday :)17:07
AJaegerI'll fix anyhow ;)17:07
openstackgerritAndreas Jaeger proposed opendev/system-config master: Update CLA link  https://review.opendev.org/71426917:10
AJaegerclarkb: did I get them all now?17:10
clarkblooks like it17:11
AJaegerthanks17:11
AJaegerwant to +A the other one?17:11
AJaegerI mean 714229 - otherwise I'll ask on Monday again ;)17:12
clarkbmordred: ^ its safe to make changes to files in the ansible role ya? I mena we explicitly don't restart the service17:12
openstackgerritAndreas Jaeger proposed opendev/system-config master: Update CLA link  https://review.opendev.org/71426917:19
mordredclarkb: yeah. and review is still in emergency17:19
AJaegerclarkb: updated the link to  th super section17:19
AJaegermordred: no urgency to deploy it - just like to have it in repo eventually17:20
mordredyeah. it'll roll out there once we get cfgmgmt rolling again17:20
AJaegerok.17:20
AJaegerhttps://review.opendev.org/#/c/713925/ is the one I'd like to see in - will ask next week again...17:21
AJaegerhave a nice Saturday..17:21
clarkbI approved 71422917:22
clarkbmordred: ^ thanks for confirming on that one17:22
mordredclarkb: ++17:24
mordredclarkb: I imagine I'll be floating in and out over the weekend ... because it's not like there's a huge amount of other things to do :)17:24
clarkbya same here. I've basically got "entertain kids, eat, work" as options :)17:24
clarkbtoday we are going to put a tent up in the backyard and hope that occupies them for several hours17:25
mordredthat sounds like fun!17:28
mordredI'm making some bone broth - and I think later I'm going to make some tahini and then some hummus17:28
clarkboh ya we baked a hcicken last night that I should now turn into soup17:28
mordredmmm. soup17:28
openstackgerritMerged opendev/system-config master: Set ulimits for gerrit  https://review.opendev.org/71414317:30
openstackgerritMerged opendev/system-config master: Correct vars indentation error in rename playbook  https://review.opendev.org/71412717:33
openstackgerritMerged openstack/project-config master: Update infra-manual links  https://review.opendev.org/71392517:33
openstackgerritMerged openstack/project-config master: Update third-party-check pipeline description  https://review.opendev.org/71426517:33
fungiooh, i should try making tahini. i always just buy cans of it for making my hummus17:40
fungiand i do have an appropriate countertop grinder which would probably do the trick17:40
openstackgerritMerged opendev/system-config master: Update infra-manual location  https://review.opendev.org/71422918:02
AJaegerthanks!18:03
AJaegeryummy ;)18:03
openstackgerritMerged opendev/system-config master: Update CLA link  https://review.opendev.org/71426918:24
*** diablo_rojo__ has joined #opendev18:40
corvuswe're making marmalade with the meyer lemons off our tree19:12
fungiooh!19:16
fungithough my personal favorite is grapefruit marmalade19:16
fungibut i've never met a marmalade i didn't like19:16
*** diablo_rojo__ has quit IRC19:25
openstackgerritJames E. Blair proposed opendev/system-config master: Add meetpad server  https://review.opendev.org/71423819:26
mordredfungi: https://minimalistbaker.com/how-to-make-tahini/ is what I'm starting from20:15
mordredand then I'm going to make Ben Starr's hummus: https://www.youtube.com/watch?v=Z5W_CmjCtn8&feature=youtu.be - because he told me last time I saw him that he'd just made the best hummus ever and I wanna try it out20:17
fungilooks amazingly simple, yep. once i get through the current cans of tahini in my pantry/fridge (which will last me probably another 6 months at least), i'll probably tweak my hummus prep steps to start with "make enough tahini for the hummus"20:18
mordredyeah - that was kind of my thinking - this looks so easy, why would I ever nort?20:24
mordrednot?20:24
mordredalso why would I ever nort, but that's a different question altogether20:24
fungiwell, it's especially compelling for me because i have a good source for bulk sprouted sesame seeds, which saves me the trouble of sprouting them20:29
fungigotta freeze them though, or they go rancid before i get all the way through them20:30
fungialready get sprouted chickpeas from the same source which i use for the hummus. dry chickpeas are rather quick and painless in a pressure cooker20:31
mordredinfra-root: I restarted review-dev with the latest 2.13 image21:00
mordredthe gitweb links work21:00
mordredhah. next issue - jeepyb and python3 :)21:02
openstackgerritMonty Taylor proposed opendev/jeepyb master: Update ConfigParser imports for python3  https://review.opendev.org/71427621:09
fungid'oh!21:14
fungimordred: apparently the only job run on jeepyb changes is pep8, and it doesn't like you21:15
fungilikely the ordering of imports21:15
fungii didn't look closely21:15
fungihappy to quick-approve that once passing21:17
mordredfungi: cool. will fix21:19
clarkbI have astack up to add actual functional testing of  manage projects21:31
clarkbif wewant to land that it may make some of this transition work less scary21:31
openstackgerritMonty Taylor proposed opendev/jeepyb master: Update ConfigParser imports for python3  https://review.opendev.org/71427621:31
fungiclarkb: that sounds good to me. mordred: are you up for rebasing it on 704671? then we can try to just land them all together and that should hopefully also at least exercise the manage-projects code paths as a bit of a canary for the rest21:34
fungii'm reviewing the testing stack now21:34
mordredwhere's the stack?21:36
clarkbthats the 704671 change21:38
mordredoh. in jeepyb21:38
*** DSpider has quit IRC21:38
clarkbthe db thing at the bottom is probably most scary21:38
clarkbbut even that should be fine21:39
openstackgerritMonty Taylor proposed opendev/jeepyb master: Update ConfigParser imports for python3  https://review.opendev.org/71427621:42
mordredclarkb: I think it looks great!21:42
mordredand there's the pep8 thing rebased on top of your functional stack21:42
openstackgerritMerged opendev/jeepyb master: Don't use the db in manage-projects  https://review.opendev.org/70464521:44
openstackgerritMerged opendev/jeepyb master: Log errors when committing and pushing acls  https://review.opendev.org/70466021:44
corvusmarmalade complete21:49
funginow to make the scones it goes on?21:50
corvusyep!  or, probably tomorrow morning.  fortunately, we bought flour before the apocalypse21:50
ianwi have a classic cookbook that says a good housemaid can get a batch of scones on between the moment she hears the front gate swing to the knock on the door21:51
fungiwhere do i find one of those?21:53
fungioh, wait, i guess i'm supposed to be the housemaid in that parable21:53
fungiand yeah, i always buy flour like it's the apocalypse. usually 50 lbs worth in a delivery21:54
corvusianw: heh, i echoed that to li, and she said "huh, i wonder what mrs beeton has to say about scones" and headed off to dig through historic cookbooks :)21:54
corvusokay, now she's found "indian cornflour bread" and we're wondering what indian cornflour is -- is that more like masa, or more like corn meal?21:55
fungisomething tells me that housemaid worked at a manor with a nicely long front walk21:56
corvusalso, the recipe calls for 4 lbs of flour and 2 lbs of indian corn flour21:56
fungithat's a lot of cornbread21:56
corvusso, like, fungi quantities21:56
clarkbha21:57
fungiyeah, my usual order is 25 lbs sprouted whole white wheat flour and 25 lbs einkorn flour. i keep them in 5 gallon buckets and portion out however much i need for the kitchen week by week21:57
corvusfungi: from where?21:58
fungihttps://healthyflour.com/21:58
fungii get most of my dry legumes and various other grains there too21:58
fungithough for most stuff like beans i don't order more than 10 lbs at a shot21:59
fungii just tend to go through a lot of flour21:59
fungii make my sourdough sandwich loaves with 100% einkorn, and the white wheat we use in basically anything else that calls for flour, like in place of all-purpose. works just fine22:02
fungiit's not cheap, but for organically grown, sprouted, stone ground, and then delivered to your door? it's reasonable (unless you're lucky enough to live in easy reach of the grains and have your own mill)22:04
fungialso part of why i order in such large quantities is to keep the cost down22:08
mordredfungi: oh - so 25 pounds fits into a 5 gallon bucket?22:17
fungiyep, with just a little room to spar22:20
fungie22:20
clarkbif you get a 6.5 gallon bucket then you can make beer with it in 5 gallon batches too :)22:21
fungialso for a bit of added insurance, since i have a large upright deep-freezer in the garage, i stick the shipment in there for at least a few days to make sure it kills any hitchhiking grain mite eggs22:22
clarkbfungi: it was a game growing up to pick the bugs out22:23
fungii'd prefer not to have to play that game if i can help it22:24
* mordred needs a chest freezer22:31
clarkbwe are now watching frozen 2 for what must be the 50th time22:35
clarkbI'd requesthelp but I must face thid alone22:35
fungiyou have my sympathies23:21

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!