Tuesday, 2020-03-17

openstackgerritMerged opendev/system-config master: Switch back to docker for gerrit and nodepool-builder  https://review.opendev.org/71310100:02
openstackgerritMerged zuul/zuul-jobs master: wait-for-pods: Wait for all pods to become Ready  https://review.opendev.org/71310700:50
openstackgerritMerged zuul/zuul-jobs master: Keep doc/source/roles.rst sorted  https://review.opendev.org/71312800:59
openstackgerritMerged zuul/zuul-jobs master: install-docker: add option to use buildset registry  https://review.opendev.org/71311501:00
openstackgerritIan Wienand proposed openstack/project-config master: Remove secret import from vexxhost  https://review.opendev.org/71336803:36
openstackgerritMerged openstack/project-config master: vexxhost: load openstack/tempest jobs  https://review.opendev.org/71333903:43
*** factor has quit IRC03:56
*** factor has joined #opendev03:56
AJaegerconfig-core, please review https://review.opendev.org/712547.05:51
AJaegerfrickler: could you review https://review.opendev.org/713368 , please?05:51
*** DSpider has joined #opendev07:11
*** mnasiadka has joined #opendev07:12
*** lpetrut has joined #opendev07:51
fricklerAJaeger: mnaser: tempest jobs need this whole bunch os oslo projects, too, it seems https://zuul.opendev.org/t/vexxhost/config-errors07:53
openstackgerritMerged openstack/project-config master: Remove secret import from vexxhost  https://review.opendev.org/71336808:06
openstackgerritMerged opendev/system-config master: Added updates dir for Fedora 31 release  https://review.opendev.org/71316908:37
fricklerianw: seems we also need to bindmount /opt/cache(/files) into the nb container? https://zuul.opendev.org/t/openstack/build/0a597a481b414ac58fa738f6668d3056/console08:43
AJaegerfrickler: mnaser will have fun ;)09:13
AJaegermnaser, once this is all in, let's revisit this experiment ;)09:13
AJaegerfrickler, ianw, could you review this zuul-jobs change to update ansible-lint testing, please? https://review.opendev.org/#/c/712547/09:17
zbrAJaeger: a hint re ansible-lint, i am aware of several important bugs in ansible-lint and i started working with galaxy team to fix them, we will have a new bug-fix release this month.09:28
zbrit could make sense to avoid bumps to 4.2.0 for the moment, but fixing reported issues is clearly a good idea at any time.09:29
mnaserfrickler, AJaeger: what an exercise this is so far. Hopefully it helps people in the future lol12:12
openstackgerritMohammed Naser proposed openstack/project-config master: vexxhost: add oslo required projects for devstack  https://review.opendev.org/71343112:55
mnaserfrickler, AJaeger: round #3.. hopefully the last, i think it should be because i am not adding more jobs12:56
openstackgerritThierry Carrez proposed opendev/irc-meetings master: Consolidate meeting channels  https://review.opendev.org/71265612:56
openstackgerritAndreas Jaeger proposed openstack/project-config master: vexxhost: add oslo required projects for devstack  https://review.opendev.org/71343113:17
mnaserAJaeger: thank you13:25
openstackgerritMerged openstack/project-config master: vexxhost: add oslo required projects for devstack  https://review.opendev.org/71343113:36
openstackgerritSorin Sbarnea proposed zuul/zuul-jobs master: DNM: test bindep.txt touching effects  https://review.opendev.org/71344513:42
openstackgerritMaciej Józefczyk proposed openstack/project-config master: Add publish-to-pypi template for ovn-octavia-provider  https://review.opendev.org/71344613:47
mnaserAJaeger, frickler: yay.  it's done.  no more config errors14:00
mnaserwhere would be a good place o document it ?14:01
fungiprobably in the devstack repo, maybe the readme for the relevant devstack job?14:02
fungithere's a parent devstack job, if memory serves, so having it document the required-projects there would make sense14:04
mnaserinfra-root: zuul returned "Unable to freeze job graph: 'NoneType' object has no attribute 'decrypt'" on a job where i did not supply the secret but referenced one in https://review.opendev.org/#/c/713104/ -- i'd like to ask if it's possible for someone to check the zuul logs if there's a traceback so i can try and fix this inside zuul to give a better error message14:28
corvusmnaser: the reason vexxhost thinks that devstack is in the vexxhost check pipeline is this: https://opendev.org/opendev/project-config/src/branch/master/zuul.d/projects.yaml#L214:28
mnaserah14:29
corvusmnaser, mordred, fungi: that's where rethinking the config-project(s) for this new-ish way we're using tenants may be worthwhile14:30
corvusmnaser: i'll check the logs14:30
mnaserit's something i've struggled with too on the hosted ci side of things, modeling a clean multitenant system but with an "authoritative" tenant in all of it14:30
openstackgerritThierry Carrez proposed opendev/irc-meetings master: Consolidate meeting channels  https://review.opendev.org/71265614:32
corvusmnaser: http://paste.openstack.org/show/790797/14:37
mnasercorvus: yay that's useful14:40
openstackgerritSorin Sbarnea proposed zuul/zuul-jobs master: Improve ensure-tox role  https://review.opendev.org/70864215:07
openstackgerritDaniel Pawlik proposed zuul/zuul-jobs master: Add phoronix-test-suite job  https://review.opendev.org/67908215:10
*** lpetrut has quit IRC15:14
openstackgerritMerged zuul/zuul-jobs master: Use a zuul_* and add an .ansible-lint file  https://review.opendev.org/71254715:40
*** AJaeger has quit IRC16:22
*** AJaeger has joined #opendev16:22
mordredinfra-root: do we have a better highlight phrase to use over here? or still just infra rootr16:51
clarkbI can't think of a better oen off the top of my head16:52
fungiwfm16:52
clarkbbtw my plan is to approve the opendev documentation change in the infra meeting today16:52
fungigood plan16:53
mordredcool16:56
clarkbalso while I'm no longer driving to arizona friday I feel like I need the mental equivalent brain checkout16:57
clarkbdrive a virtual car around on the tv or something all day :)16:57
fungiyou definitely should17:02
openstackgerritMerged opendev/system-config master: nodepool-builder: put container configs in /etc  https://review.opendev.org/71314817:50
openstackgerritMonty Taylor proposed opendev/system-config master: Use version 2 for docker-compose for review  https://review.opendev.org/71350218:03
mordredinfra-root: ^^18:03
mordredreview-dev hit that with the new code - and it's because our testing had a flaw18:03
clarkbmordred: note we could install from pip then use latest docker-compose if we need to18:06
clarkb(like if features make sense for that)18:06
mordredyeah - I don't think we need to in this case18:07
mordredclarkb: https://docs.docker.com/compose/compose-file/compose-versioning/#version-318:08
corvusversion 2 is sufficient, and i think we should specify the lowest version that works18:09
mordredyah18:10
openstackgerritMonty Taylor proposed opendev/system-config master: Drop version specifier for nodepool-builder compose  https://review.opendev.org/71350318:12
mordredcorvus: ^^ that one got cargo-culted in from gerrit, so fixed it too18:12
mordredalthough it's likely fine in prod18:12
mordredclarkb: ^^18:13
corvusclarkb: do you have a virtual car you particularly like to virtually drive on the tv?18:13
clarkbcorvus: I dont, but I can easily download a game full of them for the ps418:15
corvusi briefly owned a ps3, but decided i had better things to do than watch it crash all the time so returned it.18:16
clarkbhowever, I haven't played http://kentuckyroutezero.com/ yet so may give that a go. Less simulated driving and more adventure18:20
clarkbI think it will run on linux too18:20
fungicertainly sounds more fun than donning a suite of armor and battling in the grocery aisles for control of a case of toilet paper18:34
fungi(not just a suit of armor mind you, an entire suite of armor)18:35
clarkbwe've got old cloth diapers if it comes to that :)18:35
clarkbno battling necessary18:35
clarkbalso the trees will have leaves soon18:36
corvuscoast live oaks are evergreen, but, um, have small prickly leaves.18:37
fungiour live oaks aren't prickly, but the leaves are small and rather stiff18:38
fungithink bay leaves18:38
clarkbaround here there are plenty of maples18:39
fungibut the alternative here is long leaf pine18:39
clarkbthough none directly adjacent to my home18:39
mordredwe have many vegetations here18:40
fungiour fig trees will have leaves soon, thanks early spring18:40
mordredall of them have leaves18:40
fungioh, technically prickly pear is evergreen here. also not recommended as a toilet tissue substitute18:40
mordredalthough the oaks are actually dropping old leaves because it's spring18:40
fungi(before i moved here i did not realize prickly pear cactus is indigenous to the barrier islands, i thought folks had just brought it from desert climes)18:42
zbrindeed, going to the supermarket today reminded me of the zombie games18:44
clarkbzbr: around here its highly dependent on the store. The larger less specialized ones are way worse than the smaller shops18:44
zbrouch, i was to the biggest local one(morrisons), and it was quite "clean", even stuff that nobody usually buys started to disappear18:46
zbrgood that i have 24 jumpo-rolls of paper that an repurpose when in... need18:47
* mordred is sad he hadn't gotten around to ordering those japanese toilet seats18:50
mordredstill might18:50
fungithey'll be in quarantine on a shipping dock somewhere until your need is no longer urgent18:51
openstackgerritMerged opendev/system-config master: Use version 2 for docker-compose for review  https://review.opendev.org/71350219:02
openstackgerritSorin Sbarnea proposed zuul/zuul-jobs master: Improve ensure-tox role  https://review.opendev.org/70864219:06
openstackgerritMerged opendev/system-config master: Drop version specifier for nodepool-builder compose  https://review.opendev.org/71350319:14
openstackgerritMonty Taylor proposed opendev/system-config master: Update to gitea 1.11.3  https://review.opendev.org/71351719:17
openstackgerritMerged openstack/project-config master: Cleanup #openstack-infra gerritbot  https://review.opendev.org/71110619:21
openstackgerritMerged openstack/project-config master: Move diskimage-builder to #opendev  https://review.opendev.org/71141219:22
ianwmnaser: "Situations where a PhD in k8s clusterology is infeasible" ... well i like that :)19:28
* fungi has an associate degree in clusterology but no specialization in kubernetes19:28
mnaserianw: aha :)19:29
*** openstackgerrit has quit IRC19:33
*** openstackgerrit has joined #opendev19:36
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: DNM: Add support for installing python with pyenv  https://review.opendev.org/70426619:36
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: DNM: Add support for installing python with pyenv  https://review.opendev.org/70426619:37
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: DNM: Add support for installing python with pyenv  https://review.opendev.org/70426619:40
openstackgerritMerged zuul/zuul-jobs master: Improve ensure-tox role  https://review.opendev.org/70864219:46
AJaegermnaser:btw. is it correct that https://review.opendev.org/678440 is still WIP?19:47
mnaserAJaeger: honestly i have not explored it yet and the last time this occured it was a very major issue for us so i'd like to take a bit of time to review things first19:48
mnaserit took us weeks to clean up so it's a little hard right now19:48
AJaegermnaser: wow - understood.19:48
mnaser..and what i meant by bot spam as in new reviews / things merging / etc20:01
fungiright, it's nice to have those filtered out, though maybe our reimplementation of the bot could also filter comments from specific nics?20:02
fungii see clean meeting logs more as a side effect of having the meeting in a channel with no noisy bots20:03
openstackgerritClark Boylan proposed opendev/system-config master: Make Advisory Board a proper noun  https://review.opendev.org/71352320:05
openstackgerritClark Boylan proposed opendev/system-config master: Add a comma to docs  https://review.opendev.org/71352420:05
clarkbI've approved the project docs update and then pushed ^ to address some minor nits that were made without affecting quorum and consensu20:05
clarkbI'm not sure the second one is correct, but I'll be the first to admit english isn't what I'd call a strong skill for me :)20:05
ianwclarkb: i responded on https://review.opendev.org/#/c/713157/4 .. do you want to double check what i said about the testing flag before i upload any other revisions and confuse anything further?20:07
fricklerianw: did you see the devstack fedora failure I mentioned earlier?20:09
ianwfrickler: ahh no, sorry.  scrolling back ...20:09
clarkbianw: oh I see the confusion now20:10
clarkbianw: its the env vars we check. Should we upate https://review.opendev.org/#/c/713157/4/nodepool/tests/fixtures/node_diskimage_parents.yaml line 84 to abstract-intermediate to avoid confusion with the yaml data?20:11
clarkbianw: I was ultiamtely confused by thinking the yaml native data was the same as the env stuff passed in20:12
AJaegermordred: is it time to abandon https://review.opendev.org/707202 ?20:13
ianwfrickler: if still around, sorry i'm not seeing what you mentioned about fedora?20:13
ianwclarkb: maybe _IMAGE is too overloaded a term here.  maybe PARENT_TEST_FLAG makes it clear this is just something we're passing to fake-dib?20:14
clarkbianw: ya that would work too20:14
clarkbinfra-root https://etherpad.openstack.org/p/gerrit-downtime-march20 email for gerrit downtiem20:16
clarkbI need to pop out for a few but will be back to hopefully send all the emails20:17
fricklerianw: seems we also need to bindmount /opt/cache(/files) into the nb container? https://zuul.opendev.org/t/openstack/build/0a597a481b414ac58fa738f6668d3056/console20:19
fricklerdidn't mention either devstack or fedora in that message, sorry20:20
fricklerand with that I'll really EOD now, cu20:21
ianwfrickler: oohhh, right!  might have to do with clarkb's refactor and our very bespoke current builder.  thanks will look into it and make sure it's covered20:21
ianwttyl!20:21
clarkbfrickler: ianw no /opt/cache/files is a construct within the image build chroot20:22
clarkbit shouldn't cross into the host vs container realm20:22
openstackgerritMerged opendev/system-config master: Update project doc to reflect OpenDev changes  https://review.opendev.org/70348820:22
clarkb(there was confusiong at one point around that though so good to clarify it with more than one input (me))20:22
ianwclarkb: yeah, i think it's likely the image we got out in the midst of fixing up the sourcing of the files might not be right in this regard20:23
ianwwe should keep an eye on this in the new builder20:23
clarkbmordred: corvus fungi in particular should probably look over https://etherpad.openstack.org/p/gerrit-downtime-march20 since they've volunteered20:24
mordredlookjng20:25
mordredlooks good20:25
mordredclarkb: review-dev is back up and running under docker now20:30
mordredclarkb: it was done via the normal ansible pulse20:30
clarkbmordred: cool. I also +2'd the gitea change but did not approve as I'm trying to write all the emails and then get a bike ride in20:30
mordred++20:30
clarkbchances are we want to manually trigger replication after that happens though20:30
mordredoh - right20:31
mordredhrm20:31
mordredmaybe I'll land it in a bit then20:31
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: DNM: Add support for installing python with pyenv  https://review.opendev.org/70426620:33
*** JayF has joined #opendev20:36
fungiclarkb: still busy with $meal but will check out etherpad in a bit20:37
clarkbfungi: thanks20:38
clarkbI'm gonna prep for a bike ride but will wait for actually doing that on getting the email out (and I think your input is important so can wait)20:38
fungiclarkb: gerrit outage notice lgtm!20:46
fungithanks for writing it up20:46
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: DNM: Add support for installing python with pyenv  https://review.opendev.org/70426620:46
clarkbfungi: thanks! I'll give corvus a bit longer to chime in20:48
*** jbryce has joined #opendev20:49
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: DNM: Add support for installing python with pyenv  https://review.opendev.org/70426620:56
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: Add support for installing python with pyenv  https://review.opendev.org/70426620:56
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: Add support for installing python with pyenv  https://review.opendev.org/70426620:57
*** mtreinish has joined #opendev20:59
*** rajinir has joined #opendev21:01
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: Add support for installing python with pyenv  https://review.opendev.org/70426621:04
clarkbcorvus: I'll probably send it out at 2:15pm local time if you haven't responded by then21:07
clarkbI've got two ACKs so its probably fine21:07
corvusoh sorry21:08
corvuslgtm21:08
corvusi looked but forgot to say21:08
clarkbgreat emails should be outbound shortly21:08
clarkbfungi: maybe you can forward to http://lists.opendev.org/cgi-bin/mailman/listinfo/openinfralabs or resend if you think that is appropriate? I've unfortunately not caught up there21:10
clarkband now I get on the bike21:13
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: Add support for installing python with pyenv  https://review.opendev.org/70426621:16
*** tbarron has joined #opendev21:24
fungiclarkb: they haven't really started pushing any changes to their repo yet so probably fine if we don't21:26
*** diablo_rojo has joined #opendev21:26
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: Add support for installing python with pyenv  https://review.opendev.org/70426621:28
*** johnsom has joined #opendev21:28
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: Add support for installing python with pyenv  https://review.opendev.org/70426621:32
clarkband I'm back22:23
clarkbfungi: thanks for confirming. I wasn't sure what state they were in at this point22:23
*** tosky has joined #opendev22:34
*** fdegir has joined #opendev22:38
clarkbinfra-root https://review.opendev.org/#/c/711544/ has me wondering if we should go ahead and give mnaser and noonedeadpunk +2 on lodgeit22:56
clarkbare there any objections to that? they seem motivated to do maintenance there and we are pinned anyway (can unpin that as soon as the churn is gone)22:56
mordredclarkb: I think it's a great idea22:57
mordredclarkb: and I think rather than unpinning we should just update our deployment to be ansible+container22:57
clarkbmordred: that works too :)22:57
mordredthat way we'll be closer to what mnaser and noonedeadpunk are doing22:57
fungii'm 100% in favor of letting mnaser and noonedeadpunk help maintain lodgeit, yes22:58
fungiand also leveraging the containerized deployment solution they've worked out22:58
mnasercool. I can clear fix the unpins23:01
mnaseroh it would have been nice if we were still in the same tenant, I could have ran our helm jobs against the image builds there. :)23:01
mordredmnaser: doh23:01
*** hashar has joined #opendev23:03
*** hashar has quit IRC23:04
clarkbhttps://review.opendev.org/#/admin/groups/740,members has been updated23:05
clarkbmordred: re https://review.opendev.org/#/c/713517/ I can probably land that tomorrow if you want to focus on the review stuff23:09
mordredclarkb: cool23:09
corvus++ lodgeit plan23:09
clarkbmordred: its the perfect sort of thing to monitor while also running the smoker :)23:13
openstackgerritMohammed Naser proposed opendev/lodgeit master: Tuning pool_recycle to avoid MySQL server gone away  https://review.opendev.org/21822423:18
openstackgerritMerged opendev/lodgeit master: Fix sort comparison function  https://review.opendev.org/71154423:20
openstackgerritIan Wienand proposed openstack/project-config master: nodepool: use job inheritance  https://review.opendev.org/71315823:30
*** seongsoocho has joined #opendev23:38
*** tosky has quit IRC23:38

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