openstackgerrit | Merged opendev/system-config master: Switch back to docker for gerrit and nodepool-builder https://review.opendev.org/713101 | 00:02 |
---|---|---|
openstackgerrit | Merged zuul/zuul-jobs master: wait-for-pods: Wait for all pods to become Ready https://review.opendev.org/713107 | 00:50 |
openstackgerrit | Merged zuul/zuul-jobs master: Keep doc/source/roles.rst sorted https://review.opendev.org/713128 | 00:59 |
openstackgerrit | Merged zuul/zuul-jobs master: install-docker: add option to use buildset registry https://review.opendev.org/713115 | 01:00 |
openstackgerrit | Ian Wienand proposed openstack/project-config master: Remove secret import from vexxhost https://review.opendev.org/713368 | 03:36 |
openstackgerrit | Merged openstack/project-config master: vexxhost: load openstack/tempest jobs https://review.opendev.org/713339 | 03:43 |
*** factor has quit IRC | 03:56 | |
*** factor has joined #opendev | 03:56 | |
AJaeger | config-core, please review https://review.opendev.org/712547. | 05:51 |
AJaeger | frickler: could you review https://review.opendev.org/713368 , please? | 05:51 |
*** DSpider has joined #opendev | 07:11 | |
*** mnasiadka has joined #opendev | 07:12 | |
*** lpetrut has joined #opendev | 07:51 | |
frickler | AJaeger: mnaser: tempest jobs need this whole bunch os oslo projects, too, it seems https://zuul.opendev.org/t/vexxhost/config-errors | 07:53 |
openstackgerrit | Merged openstack/project-config master: Remove secret import from vexxhost https://review.opendev.org/713368 | 08:06 |
openstackgerrit | Merged opendev/system-config master: Added updates dir for Fedora 31 release https://review.opendev.org/713169 | 08:37 |
frickler | ianw: seems we also need to bindmount /opt/cache(/files) into the nb container? https://zuul.opendev.org/t/openstack/build/0a597a481b414ac58fa738f6668d3056/console | 08:43 |
AJaeger | frickler: mnaser will have fun ;) | 09:13 |
AJaeger | mnaser, once this is all in, let's revisit this experiment ;) | 09:13 |
AJaeger | frickler, ianw, could you review this zuul-jobs change to update ansible-lint testing, please? https://review.opendev.org/#/c/712547/ | 09:17 |
zbr | AJaeger: 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 |
zbr | it 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 |
mnaser | frickler, AJaeger: what an exercise this is so far. Hopefully it helps people in the future lol | 12:12 |
openstackgerrit | Mohammed Naser proposed openstack/project-config master: vexxhost: add oslo required projects for devstack https://review.opendev.org/713431 | 12:55 |
mnaser | frickler, AJaeger: round #3.. hopefully the last, i think it should be because i am not adding more jobs | 12:56 |
openstackgerrit | Thierry Carrez proposed opendev/irc-meetings master: Consolidate meeting channels https://review.opendev.org/712656 | 12:56 |
openstackgerrit | Andreas Jaeger proposed openstack/project-config master: vexxhost: add oslo required projects for devstack https://review.opendev.org/713431 | 13:17 |
mnaser | AJaeger: thank you | 13:25 |
openstackgerrit | Merged openstack/project-config master: vexxhost: add oslo required projects for devstack https://review.opendev.org/713431 | 13:36 |
openstackgerrit | Sorin Sbarnea proposed zuul/zuul-jobs master: DNM: test bindep.txt touching effects https://review.opendev.org/713445 | 13:42 |
openstackgerrit | Maciej Józefczyk proposed openstack/project-config master: Add publish-to-pypi template for ovn-octavia-provider https://review.opendev.org/713446 | 13:47 |
mnaser | AJaeger, frickler: yay. it's done. no more config errors | 14:00 |
mnaser | where would be a good place o document it ? | 14:01 |
fungi | probably in the devstack repo, maybe the readme for the relevant devstack job? | 14:02 |
fungi | there's a parent devstack job, if memory serves, so having it document the required-projects there would make sense | 14:04 |
mnaser | infra-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 message | 14:28 |
corvus | mnaser: 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#L2 | 14:28 |
mnaser | ah | 14:29 |
corvus | mnaser, mordred, fungi: that's where rethinking the config-project(s) for this new-ish way we're using tenants may be worthwhile | 14:30 |
corvus | mnaser: i'll check the logs | 14:30 |
mnaser | it'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 it | 14:30 |
openstackgerrit | Thierry Carrez proposed opendev/irc-meetings master: Consolidate meeting channels https://review.opendev.org/712656 | 14:32 |
corvus | mnaser: http://paste.openstack.org/show/790797/ | 14:37 |
mnaser | corvus: yay that's useful | 14:40 |
openstackgerrit | Sorin Sbarnea proposed zuul/zuul-jobs master: Improve ensure-tox role https://review.opendev.org/708642 | 15:07 |
openstackgerrit | Daniel Pawlik proposed zuul/zuul-jobs master: Add phoronix-test-suite job https://review.opendev.org/679082 | 15:10 |
*** lpetrut has quit IRC | 15:14 | |
openstackgerrit | Merged zuul/zuul-jobs master: Use a zuul_* and add an .ansible-lint file https://review.opendev.org/712547 | 15:40 |
*** AJaeger has quit IRC | 16:22 | |
*** AJaeger has joined #opendev | 16:22 | |
mordred | infra-root: do we have a better highlight phrase to use over here? or still just infra rootr | 16:51 |
clarkb | I can't think of a better oen off the top of my head | 16:52 |
fungi | wfm | 16:52 |
clarkb | btw my plan is to approve the opendev documentation change in the infra meeting today | 16:52 |
fungi | good plan | 16:53 |
mordred | cool | 16:56 |
clarkb | also while I'm no longer driving to arizona friday I feel like I need the mental equivalent brain checkout | 16:57 |
clarkb | drive a virtual car around on the tv or something all day :) | 16:57 |
fungi | you definitely should | 17:02 |
openstackgerrit | Merged opendev/system-config master: nodepool-builder: put container configs in /etc https://review.opendev.org/713148 | 17:50 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Use version 2 for docker-compose for review https://review.opendev.org/713502 | 18:03 |
mordred | infra-root: ^^ | 18:03 |
mordred | review-dev hit that with the new code - and it's because our testing had a flaw | 18:03 |
clarkb | mordred: note we could install from pip then use latest docker-compose if we need to | 18:06 |
clarkb | (like if features make sense for that) | 18:06 |
mordred | yeah - I don't think we need to in this case | 18:07 |
mordred | clarkb: https://docs.docker.com/compose/compose-file/compose-versioning/#version-3 | 18:08 |
corvus | version 2 is sufficient, and i think we should specify the lowest version that works | 18:09 |
mordred | yah | 18:10 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Drop version specifier for nodepool-builder compose https://review.opendev.org/713503 | 18:12 |
mordred | corvus: ^^ that one got cargo-culted in from gerrit, so fixed it too | 18:12 |
mordred | although it's likely fine in prod | 18:12 |
mordred | clarkb: ^^ | 18:13 |
corvus | clarkb: do you have a virtual car you particularly like to virtually drive on the tv? | 18:13 |
clarkb | corvus: I dont, but I can easily download a game full of them for the ps4 | 18:15 |
corvus | i briefly owned a ps3, but decided i had better things to do than watch it crash all the time so returned it. | 18:16 |
clarkb | however, I haven't played http://kentuckyroutezero.com/ yet so may give that a go. Less simulated driving and more adventure | 18:20 |
clarkb | I think it will run on linux too | 18:20 |
fungi | certainly sounds more fun than donning a suite of armor and battling in the grocery aisles for control of a case of toilet paper | 18:34 |
fungi | (not just a suit of armor mind you, an entire suite of armor) | 18:35 |
clarkb | we've got old cloth diapers if it comes to that :) | 18:35 |
clarkb | no battling necessary | 18:35 |
clarkb | also the trees will have leaves soon | 18:36 |
corvus | coast live oaks are evergreen, but, um, have small prickly leaves. | 18:37 |
fungi | our live oaks aren't prickly, but the leaves are small and rather stiff | 18:38 |
fungi | think bay leaves | 18:38 |
clarkb | around here there are plenty of maples | 18:39 |
fungi | but the alternative here is long leaf pine | 18:39 |
clarkb | though none directly adjacent to my home | 18:39 |
mordred | we have many vegetations here | 18:40 |
fungi | our fig trees will have leaves soon, thanks early spring | 18:40 |
mordred | all of them have leaves | 18:40 |
fungi | oh, technically prickly pear is evergreen here. also not recommended as a toilet tissue substitute | 18:40 |
mordred | although the oaks are actually dropping old leaves because it's spring | 18: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 |
zbr | indeed, going to the supermarket today reminded me of the zombie games | 18:44 |
clarkb | zbr: around here its highly dependent on the store. The larger less specialized ones are way worse than the smaller shops | 18:44 |
zbr | ouch, i was to the biggest local one(morrisons), and it was quite "clean", even stuff that nobody usually buys started to disappear | 18:46 |
zbr | good that i have 24 jumpo-rolls of paper that an repurpose when in... need | 18:47 |
* mordred is sad he hadn't gotten around to ordering those japanese toilet seats | 18:50 | |
mordred | still might | 18:50 |
fungi | they'll be in quarantine on a shipping dock somewhere until your need is no longer urgent | 18:51 |
openstackgerrit | Merged opendev/system-config master: Use version 2 for docker-compose for review https://review.opendev.org/713502 | 19:02 |
openstackgerrit | Sorin Sbarnea proposed zuul/zuul-jobs master: Improve ensure-tox role https://review.opendev.org/708642 | 19:06 |
openstackgerrit | Merged opendev/system-config master: Drop version specifier for nodepool-builder compose https://review.opendev.org/713503 | 19:14 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Update to gitea 1.11.3 https://review.opendev.org/713517 | 19:17 |
openstackgerrit | Merged openstack/project-config master: Cleanup #openstack-infra gerritbot https://review.opendev.org/711106 | 19:21 |
openstackgerrit | Merged openstack/project-config master: Move diskimage-builder to #opendev https://review.opendev.org/711412 | 19:22 |
ianw | mnaser: "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 kubernetes | 19:28 | |
mnaser | ianw: aha :) | 19:29 |
*** openstackgerrit has quit IRC | 19:33 | |
*** openstackgerrit has joined #opendev | 19:36 | |
openstackgerrit | Mohammed Naser proposed zuul/zuul-jobs master: DNM: Add support for installing python with pyenv https://review.opendev.org/704266 | 19:36 |
openstackgerrit | Mohammed Naser proposed zuul/zuul-jobs master: DNM: Add support for installing python with pyenv https://review.opendev.org/704266 | 19:37 |
openstackgerrit | Mohammed Naser proposed zuul/zuul-jobs master: DNM: Add support for installing python with pyenv https://review.opendev.org/704266 | 19:40 |
openstackgerrit | Merged zuul/zuul-jobs master: Improve ensure-tox role https://review.opendev.org/708642 | 19:46 |
AJaeger | mnaser:btw. is it correct that https://review.opendev.org/678440 is still WIP? | 19:47 |
mnaser | AJaeger: 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 first | 19:48 |
mnaser | it took us weeks to clean up so it's a little hard right now | 19:48 |
AJaeger | mnaser: wow - understood. | 19:48 |
mnaser | ..and what i meant by bot spam as in new reviews / things merging / etc | 20:01 |
fungi | right, it's nice to have those filtered out, though maybe our reimplementation of the bot could also filter comments from specific nics? | 20:02 |
fungi | i see clean meeting logs more as a side effect of having the meeting in a channel with no noisy bots | 20:03 |
openstackgerrit | Clark Boylan proposed opendev/system-config master: Make Advisory Board a proper noun https://review.opendev.org/713523 | 20:05 |
openstackgerrit | Clark Boylan proposed opendev/system-config master: Add a comma to docs https://review.opendev.org/713524 | 20:05 |
clarkb | I've approved the project docs update and then pushed ^ to address some minor nits that were made without affecting quorum and consensu | 20:05 |
clarkb | I'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 |
ianw | clarkb: 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 |
frickler | ianw: did you see the devstack fedora failure I mentioned earlier? | 20:09 |
ianw | frickler: ahh no, sorry. scrolling back ... | 20:09 |
clarkb | ianw: oh I see the confusion now | 20:10 |
clarkb | ianw: 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 |
clarkb | ianw: I was ultiamtely confused by thinking the yaml native data was the same as the env stuff passed in | 20:12 |
AJaeger | mordred: is it time to abandon https://review.opendev.org/707202 ? | 20:13 |
ianw | frickler: if still around, sorry i'm not seeing what you mentioned about fedora? | 20:13 |
ianw | clarkb: 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 |
clarkb | ianw: ya that would work too | 20:14 |
clarkb | infra-root https://etherpad.openstack.org/p/gerrit-downtime-march20 email for gerrit downtiem | 20:16 |
clarkb | I need to pop out for a few but will be back to hopefully send all the emails | 20:17 |
frickler | ianw: seems we also need to bindmount /opt/cache(/files) into the nb container? https://zuul.opendev.org/t/openstack/build/0a597a481b414ac58fa738f6668d3056/console | 20:19 |
frickler | didn't mention either devstack or fedora in that message, sorry | 20:20 |
frickler | and with that I'll really EOD now, cu | 20:21 |
ianw | frickler: 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 covered | 20:21 |
ianw | ttyl! | 20:21 |
clarkb | frickler: ianw no /opt/cache/files is a construct within the image build chroot | 20:22 |
clarkb | it shouldn't cross into the host vs container realm | 20:22 |
openstackgerrit | Merged opendev/system-config master: Update project doc to reflect OpenDev changes https://review.opendev.org/703488 | 20:22 |
clarkb | (there was confusiong at one point around that though so good to clarify it with more than one input (me)) | 20:22 |
ianw | clarkb: 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 regard | 20:23 |
ianw | we should keep an eye on this in the new builder | 20:23 |
clarkb | mordred: corvus fungi in particular should probably look over https://etherpad.openstack.org/p/gerrit-downtime-march20 since they've volunteered | 20:24 |
mordred | lookjng | 20:25 |
mordred | looks good | 20:25 |
mordred | clarkb: review-dev is back up and running under docker now | 20:30 |
mordred | clarkb: it was done via the normal ansible pulse | 20:30 |
clarkb | mordred: 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 in | 20:30 |
mordred | ++ | 20:30 |
clarkb | chances are we want to manually trigger replication after that happens though | 20:30 |
mordred | oh - right | 20:31 |
mordred | hrm | 20:31 |
mordred | maybe I'll land it in a bit then | 20:31 |
openstackgerrit | Mohammed Naser proposed zuul/zuul-jobs master: DNM: Add support for installing python with pyenv https://review.opendev.org/704266 | 20:33 |
*** JayF has joined #opendev | 20:36 | |
fungi | clarkb: still busy with $meal but will check out etherpad in a bit | 20:37 |
clarkb | fungi: thanks | 20:38 |
clarkb | I'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 |
fungi | clarkb: gerrit outage notice lgtm! | 20:46 |
fungi | thanks for writing it up | 20:46 |
openstackgerrit | Mohammed Naser proposed zuul/zuul-jobs master: DNM: Add support for installing python with pyenv https://review.opendev.org/704266 | 20:46 |
clarkb | fungi: thanks! I'll give corvus a bit longer to chime in | 20:48 |
*** jbryce has joined #opendev | 20:49 | |
openstackgerrit | Mohammed Naser proposed zuul/zuul-jobs master: DNM: Add support for installing python with pyenv https://review.opendev.org/704266 | 20:56 |
openstackgerrit | Mohammed Naser proposed zuul/zuul-jobs master: Add support for installing python with pyenv https://review.opendev.org/704266 | 20:56 |
openstackgerrit | Mohammed Naser proposed zuul/zuul-jobs master: Add support for installing python with pyenv https://review.opendev.org/704266 | 20:57 |
*** mtreinish has joined #opendev | 20:59 | |
*** rajinir has joined #opendev | 21:01 | |
openstackgerrit | Mohammed Naser proposed zuul/zuul-jobs master: Add support for installing python with pyenv https://review.opendev.org/704266 | 21:04 |
clarkb | corvus: I'll probably send it out at 2:15pm local time if you haven't responded by then | 21:07 |
clarkb | I've got two ACKs so its probably fine | 21:07 |
corvus | oh sorry | 21:08 |
corvus | lgtm | 21:08 |
corvus | i looked but forgot to say | 21:08 |
clarkb | great emails should be outbound shortly | 21:08 |
clarkb | fungi: 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 there | 21:10 |
clarkb | and now I get on the bike | 21:13 |
openstackgerrit | Mohammed Naser proposed zuul/zuul-jobs master: Add support for installing python with pyenv https://review.opendev.org/704266 | 21:16 |
*** tbarron has joined #opendev | 21:24 | |
fungi | clarkb: they haven't really started pushing any changes to their repo yet so probably fine if we don't | 21:26 |
*** diablo_rojo has joined #opendev | 21:26 | |
openstackgerrit | Mohammed Naser proposed zuul/zuul-jobs master: Add support for installing python with pyenv https://review.opendev.org/704266 | 21:28 |
*** johnsom has joined #opendev | 21:28 | |
openstackgerrit | Mohammed Naser proposed zuul/zuul-jobs master: Add support for installing python with pyenv https://review.opendev.org/704266 | 21:32 |
clarkb | and I'm back | 22:23 |
clarkb | fungi: thanks for confirming. I wasn't sure what state they were in at this point | 22:23 |
*** tosky has joined #opendev | 22:34 | |
*** fdegir has joined #opendev | 22:38 | |
clarkb | infra-root https://review.opendev.org/#/c/711544/ has me wondering if we should go ahead and give mnaser and noonedeadpunk +2 on lodgeit | 22:56 |
clarkb | are 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 |
mordred | clarkb: I think it's a great idea | 22:57 |
mordred | clarkb: and I think rather than unpinning we should just update our deployment to be ansible+container | 22:57 |
clarkb | mordred: that works too :) | 22:57 |
mordred | that way we'll be closer to what mnaser and noonedeadpunk are doing | 22:57 |
fungi | i'm 100% in favor of letting mnaser and noonedeadpunk help maintain lodgeit, yes | 22:58 |
fungi | and also leveraging the containerized deployment solution they've worked out | 22:58 |
mnaser | cool. I can clear fix the unpins | 23:01 |
mnaser | oh 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 |
mordred | mnaser: doh | 23:01 |
*** hashar has joined #opendev | 23:03 | |
*** hashar has quit IRC | 23:04 | |
clarkb | https://review.opendev.org/#/admin/groups/740,members has been updated | 23:05 |
clarkb | mordred: re https://review.opendev.org/#/c/713517/ I can probably land that tomorrow if you want to focus on the review stuff | 23:09 |
mordred | clarkb: cool | 23:09 |
corvus | ++ lodgeit plan | 23:09 |
clarkb | mordred: its the perfect sort of thing to monitor while also running the smoker :) | 23:13 |
openstackgerrit | Mohammed Naser proposed opendev/lodgeit master: Tuning pool_recycle to avoid MySQL server gone away https://review.opendev.org/218224 | 23:18 |
openstackgerrit | Merged opendev/lodgeit master: Fix sort comparison function https://review.opendev.org/711544 | 23:20 |
openstackgerrit | Ian Wienand proposed openstack/project-config master: nodepool: use job inheritance https://review.opendev.org/713158 | 23:30 |
*** seongsoocho has joined #opendev | 23:38 | |
*** tosky has quit IRC | 23:38 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!