| *** rlandy has quit IRC | 00:56 | |
| *** Goneri has quit IRC | 01:03 | |
| *** rlandy has joined #zuul | 01:05 | |
| *** rlandy has quit IRC | 01:11 | |
| *** swest has quit IRC | 01:31 | |
| *** swest has joined #zuul | 01:45 | |
| *** tumble has quit IRC | 01:46 | |
| *** jamesmcarthur has joined #zuul | 02:35 | |
| *** jamesmcarthur has quit IRC | 03:26 | |
| *** jamesmcarthur has joined #zuul | 03:27 | |
| *** jamesmcarthur has quit IRC | 03:35 | |
| *** ysandeep|away is now known as ysandeep | 03:38 | |
| *** jamesmcarthur has joined #zuul | 03:46 | |
| *** jamesmcarthur has quit IRC | 03:51 | |
| *** evrardjp has quit IRC | 04:33 | |
| *** evrardjp has joined #zuul | 04:33 | |
| *** saneax has joined #zuul | 04:49 | |
| *** kmalloc has quit IRC | 04:56 | |
| *** jamesmcarthur has joined #zuul | 05:13 | |
| *** jamesmcarthur has quit IRC | 05:17 | |
| *** jamesmcarthur has joined #zuul | 05:18 | |
| *** ianychoi has quit IRC | 05:34 | |
| *** jamesmcarthur has quit IRC | 05:50 | |
| *** jamesmcarthur has joined #zuul | 05:52 | |
| *** hashar has joined #zuul | 05:57 | |
| *** dpawlik has joined #zuul | 06:05 | |
| *** dpawlik has quit IRC | 06:12 | |
| *** dpawlik has joined #zuul | 06:12 | |
| *** jamesmcarthur has quit IRC | 06:22 | |
| *** jamesmcarthur has joined #zuul | 06:23 | |
| *** jamesmcarthur has quit IRC | 06:29 | |
| *** sgw has quit IRC | 06:50 | |
| *** jamesmcarthur has joined #zuul | 07:00 | |
| *** jcapitao has joined #zuul | 07:09 | |
| *** jamesmcarthur has quit IRC | 07:10 | |
| *** tosky has joined #zuul | 07:26 | |
| *** jamesmcarthur has joined #zuul | 07:46 | |
| *** jpena|off is now known as jpena | 07:51 | |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Improve typings in context of 630472 https://review.opendev.org/715247 | 07:52 |
|---|---|---|
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Required SQL reporters https://review.opendev.org/630472 | 07:52 |
| *** jamesmcarthur has quit IRC | 07:55 | |
| *** bolg has joined #zuul | 07:56 | |
| *** bolg has quit IRC | 07:59 | |
| *** bolg has joined #zuul | 08:00 | |
| *** asaleh_ has joined #zuul | 08:01 | |
| *** nils has joined #zuul | 08:08 | |
| *** stevthedev_ has joined #zuul | 08:12 | |
| *** evgenyl_ has joined #zuul | 08:12 | |
| *** guilhermesp_ has joined #zuul | 08:12 | |
| *** gundalow_ has joined #zuul | 08:13 | |
| *** mnaser_ has joined #zuul | 08:13 | |
| *** mugsie_ has joined #zuul | 08:14 | |
| *** mnaser has quit IRC | 08:20 | |
| *** evgenyl has quit IRC | 08:20 | |
| *** stevthedev has quit IRC | 08:20 | |
| *** mugsie has quit IRC | 08:20 | |
| *** guilhermesp has quit IRC | 08:20 | |
| *** gundalow has quit IRC | 08:20 | |
| *** stevthedev_ is now known as stevthedev | 08:20 | |
| *** evgenyl_ is now known as evgenyl | 08:20 | |
| *** mnaser_ is now known as mnaser | 08:20 | |
| *** guilhermesp_ is now known as guilhermesp | 08:20 | |
| *** gundalow_ is now known as gundalow | 08:20 | |
| *** rpittau|afk is now known as rpittau | 08:22 | |
| *** zenkuro has quit IRC | 08:23 | |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Improve typings in context of 630472 https://review.opendev.org/715247 | 08:30 |
| *** ysandeep is now known as ysandeep|lunch | 08:46 | |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add ensure-dnf-copr https://review.opendev.org/730743 | 08:47 |
| *** guillaumec has joined #zuul | 09:06 | |
| *** sshnaidm|afk is now known as sshnaidm | 09:26 | |
| *** ysandeep|lunch is now known as ysandeep | 09:39 | |
| *** threestrands has quit IRC | 09:39 | |
| *** harrymichal has quit IRC | 09:39 | |
| *** guillaumec has quit IRC | 09:44 | |
| *** guillaumec has joined #zuul | 09:52 | |
| *** jamesmcarthur has joined #zuul | 09:52 | |
| *** jamesmcarthur has quit IRC | 09:58 | |
| *** rpittau is now known as rpittau|bbl | 10:07 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Support per branch change queues https://review.opendev.org/718531 | 10:09 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Move queue from pipeline to project https://review.opendev.org/720182 | 10:10 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Add optional support for circular dependencies https://review.opendev.org/685354 | 10:11 |
| zbr | i seen quite neat feature on azure pipelines today, a sudden popup appeared w/o me doing enything: https://sbarnea.com/ss/Screen-Shot-2020-05-27-11-18-08.64.png | 10:19 |
| zbr | seems like something very practical, maybe we can implement it in zuul in the future. | 10:19 |
| *** jcapitao is now known as jcapitao_lunch | 10:36 | |
| *** hashar has quit IRC | 11:16 | |
| *** fbo|off is now known as fbo | 11:19 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Web UI: add "held" information, filtering in Builds page https://review.opendev.org/729265 | 11:19 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Add configure-os-mirrors role https://review.opendev.org/677578 | 11:21 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Move queue from pipeline to project https://review.opendev.org/720182 | 11:23 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Add optional support for circular dependencies https://review.opendev.org/685354 | 11:23 |
| *** harrymichal has joined #zuul | 11:24 | |
| *** harrymichal has quit IRC | 11:24 | |
| *** sshnaidm is now known as sshnaidm|afk | 11:25 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Move queue from pipeline to project https://review.opendev.org/720182 | 11:26 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Add optional support for circular dependencies https://review.opendev.org/685354 | 11:26 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Add configure-os-mirrors role https://review.opendev.org/677578 | 11:27 |
| *** jpena is now known as jpena|lunch | 11:31 | |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Add configure-os-mirrors role https://review.opendev.org/677578 | 11:34 |
| AJaeger | corvus, clarkb, mordred, I rebased and fixed the "trivial" problems ^ | 11:35 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: upload-artifactory: documentation fix https://review.opendev.org/731170 | 11:36 |
| avass | mordred: keeping subunit2html in the role seems it's more effort than just adding os-testr to test-requirements | 11:40 |
| avass | mordred: since it's also dependent on subunit | 11:40 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: fetch-subunit-output: stop using system os-testr https://review.opendev.org/730482 | 11:43 |
| avass | mordred: so how about we go with that ^ ? | 11:44 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: ensure-haskell-stack: Make sure dnf copr is installed https://review.opendev.org/731173 | 11:46 |
| *** jcapitao_lunch is now known as jcapitao | 11:47 | |
| avass | AJaeger: between https://review.opendev.org/#/c/730743/ and https://review.opendev.org/731173 what would you prefer? I don't see a reason we need a role for that so I'd say 731173 | 11:47 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Add configure-os-mirrors role https://review.opendev.org/677578 | 11:50 |
| AJaeger | avass: agree to go with 731173 | 11:51 |
| *** jamesmcarthur has joined #zuul | 11:55 | |
| openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul-jobs master: Bump ansible-lint to 4.3.0 https://review.opendev.org/702679 | 11:57 |
| *** rlandy has joined #zuul | 11:58 | |
| AJaeger | mnaser: could you check VEXXHOST CI, please? I see it failing in several places like https://review.opendev.org/#/c/731173/ | 11:59 |
| *** jamesmcarthur has quit IRC | 12:00 | |
| guilhermesp | mornings! corvus fungi regarding our conversation from yesterday about nodesets in db... i guess a way to do it for now is just to add an extra column in the buildset table so we can store the nodesets for a given buildset? | 12:04 |
| guilhermesp | AJaeger: oh the ci is failing? | 12:05 |
| guilhermesp | i will take a look at this now | 12:05 |
| *** rpittau|bbl is now known as rpittau | 12:06 | |
| AJaeger | guilhermesp: thanks | 12:07 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Add configure-os-mirrors role https://review.opendev.org/677578 | 12:17 |
| avass | AJaeger: I'll take a look at that as well, that could be my fault but I don't understand why that worked in the first place | 12:18 |
| *** marvs has joined #zuul | 12:18 | |
| AJaeger | thanks | 12:19 |
| avass | AJaeger: it worked here for some reason https://review.opendev.org/#/c/729520/ maybe it's a difference between python versions? | 12:21 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Required SQL reporters https://review.opendev.org/630472 | 12:22 |
| guilhermesp | hum i can see some errors to build subprocess32 https://zuul.vexxhost.dev/t/opendev/build/ede22823593647d2ae698de39ac8ee5b/log/job-output.txt#679 | 12:23 |
| guilhermesp | at least for one of the jobs, the others i see https://zuul.vexxhost.dev/t/opendev/build/c285c0e6b69945068c5b632c538ced89/log/job-output.txt#422 | 12:23 |
| * guilhermesp digs if issue is in our side | 12:23 | |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: tox: envlist bugfixes https://review.opendev.org/730381 | 12:32 |
| avass | guilhermesp, AJaeger: ah, I think the problem occurs if the module is executed with python2.7 | 12:33 |
| avass | let me see if I can fix that | 12:33 |
| AJaeger | mordred: could you review https://review.opendev.org/#/c/730334 again, please? and https://review.opendev.org/#/c/729348/ as well | 12:34 |
| avass | but that's a bit worrying that we don't test that | 12:35 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: siblings: python2.7 configparser has no __getitem__ https://review.opendev.org/731187 | 12:37 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: siblings: python2.7 ConfigParser has no __getitem__ https://review.opendev.org/731187 | 12:38 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: siblings: python2.7 ConfigParser has no __getitem__ https://review.opendev.org/731187 | 12:39 |
| avass | AJaeger, guilhermesp: I believe that ^ should fix it | 12:39 |
| guilhermesp | ++ avass | 12:39 |
| * AJaeger waits for test results ;) | 12:39 | |
| avass | :) | 12:40 |
| AJaeger | thanks | 12:40 |
| mnaser | avass: i wonder how this wasn't caught the first place | 12:40 |
| avass | mnaser: yeah, maybe some machines are using python2.7 instead of python3 ? | 12:41 |
| avass | mnaser: since it works for python3 | 12:42 |
| mnaser | avass: weird, the only change that happened was addition of new dib images and thats it | 12:42 |
| *** Goneri has joined #zuul | 12:42 | |
| avass | mnaser: it should also work for python2.7 if the backported configparser is installed, could that be it? | 12:43 |
| mnaser | maybe something changed in debian world | 12:43 |
| *** sshnaidm|afk is now known as sshnaidm | 12:45 | |
| tristanC | corvus: would it be possible to get a new zuul release? | 12:53 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: siblings: python2.7 ConfigParser has no __getitem__ https://review.opendev.org/731187 | 13:04 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Constrain soupsieve to <2 for python2.7 https://review.opendev.org/731195 | 13:04 |
| avass | AJaeger: looking at the logs I believe we need that ^ too: https://zuul.opendev.org/t/zuul/build/61626b13cc9543308c17962cdc5457cb | 13:05 |
| *** sgw has joined #zuul | 13:05 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: executor: add extra kubectl port-forward debug and releasenotes https://review.opendev.org/731200 | 13:09 |
| corvus | tristanC: yes, i'll work on preparing that today; tristanC, tobiash: have either of you used the zk config with tls yet? | 13:23 |
| corvus | i was hoping we would get opendev to that point, but we haven't yet (we're close though). but i agree, it's too long between releases and we shouldn't wait any more | 13:24 |
| corvus | still, would be nice to have confirmation :) | 13:24 |
| tobiash | corvus: not yet, we did a zk upgrade last week in preparation and are planning to switch to tls during one of the next few weekends | 13:24 |
| tobiash | but we tested it in test and integration deployments | 13:24 |
| tobiash | there it worked | 13:24 |
| tobiash | corvus: shall we add a release note for step 2 in https://etherpad.opendev.org/p/zuulv4 before tagging? | 13:26 |
| tobiash | or would you do that via mailing list? | 13:26 |
| corvus | tobiash: i think relnote is good for that | 13:26 |
| corvus | (and we'll also send to the list) | 13:26 |
| corvus | i'll write one up | 13:27 |
| tristanC | corvus: same as tobiash, not enabled in production yet | 13:27 |
| corvus | tobiash, tristanC: should this be the last 3.x? | 13:29 |
| tobiash | corvus: according to the plan yes | 13:30 |
| bolg | yes :) | 13:30 |
| corvus | i figure we could either announce that this will be the last 3.x (unless we need a bugfix 3.x.y), or we could wait a little bit and see if there's anything else we want to release in 3.x and do another 3.x right before 4.0 | 13:30 |
| tristanC | where is the plan etherpad again? | 13:30 |
| corvus | tristanC: https://etherpad.opendev.org/p/zuulv4 | 13:30 |
| tobiash | at least regarding stuff we're having in flight it's fine to do in a 4.0 release | 13:31 |
| tristanC | yep, makes sense to me | 13:31 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Constrain soupsieve to <2 for python2.7 https://review.opendev.org/731195 | 13:43 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: siblings: python2.7 ConfigParser has no __getitem__ https://review.opendev.org/731187 | 13:43 |
| tosky | zuul API question: I was able to retrieve the jobs and the inheritance information through the /api/tenant/{tenant}/jobs endpoing, but is there a way to know from which repository a job comes from? | 13:44 |
| tobiash | tosky: that's part of the source_context which is part of the json response of a job | 13:52 |
| tobiash | tosky: but that's part of the .../job/<name> endpoint | 13:53 |
| fungi | https://zuul.opendev.org/api/tenant/openstack/job/openstack-tox-py37 | 13:53 |
| tobiash | the jobs endpoint doesn't provide this information | 13:53 |
| fungi | 0.source_context.project there says openstack/openstack-zuul-jobs | 13:53 |
| fungi | and includes the branch and file | 13:54 |
| fungi | just remember it's going to return a list of matches, because a job can be defined on multiple branches | 13:54 |
| tosky | oh, thanks! Is there a way to make that endpoint appear on https://zuul.opendev.org/openapi ? | 13:56 |
| tosky | (I know it's written "incomplete") | 13:56 |
| *** jamesmcarthur has joined #zuul | 13:56 | |
| corvus | tobiash: yes, it needs to be added to https://opendev.org/zuul/zuul/src/branch/master/web/public/openapi.yaml | 14:01 |
| *** jamesmcarthur has quit IRC | 14:01 | |
| tobiash | I know that :D | 14:01 |
| corvus | tosky: ^ sorry :) | 14:01 |
| *** jpena|lunch is now known as jpena | 14:04 | |
| tosky | oh, no automagic extraction from some comment in the code? | 14:07 |
| *** asaleh_ has quit IRC | 14:08 | |
| *** jamesmcarthur has joined #zuul | 14:09 | |
| corvus | tosky: nope, that's hand-edited | 14:12 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Add configure-os-mirrors role https://review.opendev.org/677578 | 14:12 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Fix py27 tox https://review.opendev.org/731187 | 14:13 |
| tobiash | tosky: as far as I understood the initial discussion around that it was non-trivial to auto extract that from the code | 14:14 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Add release note about the final 3.x release https://review.opendev.org/731221 | 14:18 |
| corvus | tobiash, tristanC: ^ | 14:18 |
| openstackgerrit | Lida Liu proposed zuul/zuul master: WIP: Add commit id to Change for mqtt reporter https://review.opendev.org/722478 | 14:20 |
| tobiash | corvus: how shall we treat step 5 (Have all components connect to ZK, even if they just idle for now)? | 14:20 |
| corvus | tobiash: oh that's a good point; we should revise that relnote about that | 14:21 |
| tobiash | that is currently in the plan between 3.x and 4.0, but as far as I understood the release not would contradict what is stated there | 14:21 |
| tobiash | corvus: do you think we should do idle, optional zk connections to all components with the 3.x release as well? | 14:22 |
| tobiash | oh wait, zk configuration for all components should be enough | 14:22 |
| tobiash | so I think tweaking the release note is enougb | 14:22 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Add release note about the final 3.x release https://review.opendev.org/731221 | 14:23 |
| corvus | tobiash: ^ how's that? | 14:23 |
| corvus | tobiash: so in 4.0, we make it an error if a component can't connect to zk | 14:24 |
| tobiash | lgtm | 14:24 |
| AJaeger | mordred: could I trouble you with reviewing https://review.opendev.org/#/c/730334 again, please? and https://review.opendev.org/#/c/729348/ as well | 14:25 |
| avass | corvus: the last sentence feels strange | 14:26 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Add release note about the final 3.x release https://review.opendev.org/731221 | 14:28 |
| tristanC | corvus: next zuul release could also use this note: https://review.opendev.org/#/c/731200/ | 14:28 |
| corvus | avass: ^ how's that? | 14:28 |
| avass | corvus: should 'highly available' be 'high availability'? | 14:30 |
| tobiash | tristanC: there is a typo in the debug log | 14:30 |
| corvus | tristanC: is that referring to a previous fix? what was it? | 14:31 |
| tobiash | maybe https://review.opendev.org/715308 ? | 14:32 |
| avass | corvus: oh! got it, I was reading it wrong | 14:32 |
| corvus | avass: well, if you read it wrong it may mean i wrote it wrong :) | 14:32 |
| avass | corvus: 'highly available and scaled-out schedulers' instead of 'scale-out' maybe? | 14:33 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Add configure-os-mirrors role https://review.opendev.org/677578 | 14:35 |
| AJaeger | corvus: I'm slowly beating this into submission ^. You invented some new filters like "issuperset" or "search", hope I rewrite them correctly... | 14:36 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Add release note about the final 3.x release https://review.opendev.org/731221 | 14:36 |
| corvus | avass: ^? | 14:36 |
| *** hashar has joined #zuul | 14:38 | |
| avass | corvus: ++ | 14:39 |
| mordred | corvus: ++ | 14:39 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: executor: add extra kubectl port-forward debug and releasenotes https://review.opendev.org/731200 | 14:41 |
| bolg | corvus: does https://review.opendev.org/731221 mean we can proceeed with reviews on https://review.opendev.org/c/630472 ? | 14:44 |
| avass | so this fixes py27 for VEXXHOST but breaks for zuul since python2-dev is missing: https://review.opendev.org/#/c/731187/ | 14:48 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Fix py27 tox https://review.opendev.org/731187 | 14:49 |
| AJaeger | avass: fixed ^ | 14:49 |
| avass | AJaeger: ah :) | 14:49 |
| avass | AJaeger: isn't it python2-devel for rpm though? | 14:51 |
| AJaeger | zuul-jobs-maint, we have a few test fails on zuul-jobs right now, avass is fixing them with 731187. | 14:51 |
| AJaeger | avass: I checked with openstack/project-config/nodepool/elements/bindep-fallback.txt | 14:52 |
| corvus | mordred, tristanC: i left a suggestion on the kubectl reno patch | 14:53 |
| openstackgerrit | Lida Liu proposed zuul/zuul master: WIP: Add commit id to Change for mqtt reporter https://review.opendev.org/722478 | 14:53 |
| avass | AJaeger: fedora is python2-devel but centos etc is python-devel | 14:56 |
| AJaeger | avass: "rpm -q --provides python2-devel" - I expect it provides python-devel for compatibility | 14:57 |
| openstackgerrit | Merged zuul/zuul-jobs master: Remove unused tox_envlist in fetch-subunit-output https://review.opendev.org/729348 | 14:58 |
| AJaeger | avass: otherwise the fallback file would be broken and we would see reports for that | 14:58 |
| avass | AJaeger: the python-devel package looks like it installs python3 headers for me on fedora 31, but I'm not sure how bindep works exactly | 15:02 |
| avass | AJaeger: while the python2-devel package is needed for python2 headers | 15:03 |
| AJaeger | avass: interesting, hope ianw has an idea ^ | 15:04 |
| mordred | avass: I imagine python-devel on f31 depends-on python3-devel | 15:04 |
| mordred | we can be more specific than platform:rpm if we need to be | 15:05 |
| avass | mordred: I mean, it only installs python3-devel | 15:05 |
| mordred | yeah - I think what I'm saying is that rather than using python-dev or python-devel at all, we explicitly depend on python2-dev or python2-devel where we need them | 15:06 |
| AJaeger | avass: on centos8 it fails "No package python-devel available." | 15:06 |
| mordred | yeah. python-devel is too non-specific | 15:06 |
| avass | mordred: oh, yeah | 15:06 |
| mordred | I don't think there is any python2 on centos | 15:06 |
| mordred | I don't think there is any python2 on centos8 | 15:06 |
| AJaeger | mordred: https://opendev.org/openstack/project-config/src/branch/master/nodepool/elements/bindep-fallback.txt | 15:07 |
| AJaeger | might need some updates | 15:07 |
| clarkb | https://apps.fedoraproject.org/packages/ is super useful for sorting out fedora | 15:07 |
| AJaeger | avass: for your change, exclude fedora and CentOS8. do you know what needs to be done or shall I update? | 15:07 |
| mordred | AJaeger: yeah- or, it just needs to go away | 15:07 |
| mordred | AJaeger: that file has needed to die for quite some time | 15:08 |
| AJaeger | mordred: yeah | 15:08 |
| AJaeger | we're not adding new files anymore to it but can't delete it completely yet | 15:08 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Fix py27 tox https://review.opendev.org/731187 | 15:09 |
| avass | AJaeger: you mean like that ^ ? | 15:09 |
| mordred | avass: yeah | 15:09 |
| AJaeger | avass: maybe "!platform:centos-8" ? | 15:12 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Fix py27 tox https://review.opendev.org/731187 | 15:12 |
| avass | AJaeger: done :) | 15:12 |
| AJaeger | avass: LGTM, gave my +2 in the hope it passes ;) Thanks | 15:13 |
| *** Open10K8S has quit IRC | 15:16 | |
| *** Open10K8S has joined #zuul | 15:16 | |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Add configure-os-mirrors role https://review.opendev.org/677578 | 15:22 |
| AJaeger | tristanC: want to review this change, please? https://review.opendev.org/731173 fixes ensure-haskell-stack | 15:24 |
| openstackgerrit | Merged zuul/zuul-jobs master: Fix py27 tox https://review.opendev.org/731187 | 15:28 |
| openstackgerrit | Jeremy Stanley proposed zuul/zuul master: Docs: CPD between non-shared-queue changes https://review.opendev.org/731246 | 15:32 |
| AJaeger | yeah! Thanks, avass ! | 15:32 |
| avass | no problem :) | 15:33 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: Add missing dateutil requirement in requirements.txt https://review.opendev.org/731249 | 15:40 |
| openstackgerrit | Merged zuul/zuul master: Gitlab - avoid duplicates in the cache https://review.opendev.org/728667 | 15:44 |
| openstackgerrit | Merged zuul/zuul-jobs master: fetch-tox-output: empty envlist should behave like tox -e ALL https://review.opendev.org/730334 | 15:46 |
| AJaeger | mordred, corvus, everybody, the configure-os-mirror change https://review.opendev.org/#/c/677578/ is updated and I now need help with the failures on Ubuntu and Debian. Could anybody dig into them and advise changes, please? Feel free to fix yourself. | 15:47 |
| openstackgerrit | Merged zuul/zuul-jobs master: ensure-haskell-stack: Make sure dnf copr is installed https://review.opendev.org/731173 | 15:51 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: [WIP] web UI: add Autoholds Requests page https://review.opendev.org/729307 | 16:00 |
| tobiash | bolg: yes, after tagging | 16:02 |
| mhu | tristanC, would you mind having a look at https://review.opendev.org/#/c/729307/ ? I need some react expertise :) when I try to access a autohold request page, I get an error "remoteData is undefined" | 16:03 |
| mhu | I've tried to mimic what was done for the single buildset page but I must have missed something | 16:03 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Add configure-os-mirrors role https://review.opendev.org/677578 | 16:10 |
| AJaeger | this just syncs a bit more with configure-mirror - I'm still unclear why configure-mirror works but configure-os-mirrors fails for Ubuntu with "Destination directory /tmp/etc/apt does not exist", see https://zuul.opendev.org/t/zuul/build/2cb3bcada7fa4e26ab9d0bb3db1a60ce | 16:12 |
| corvus | bolg: oh sorry didn't mean to ignore you question earlier. i agree with tobiash's response. | 16:13 |
| *** rpittau is now known as rpittau|afk | 16:19 | |
| openstackgerrit | Merged zuul/zuul-jobs master: tox: envlist bugfixes https://review.opendev.org/730381 | 16:20 |
| *** hashar has quit IRC | 16:28 | |
| avass | AJaeger: why are they installed into /tmp/? | 16:30 |
| avass | AJaeger: should that be '/{{ zj_repo }}' instead of '/tmp/{{ zj_repo }}' ? | 16:33 |
| AJaeger | avass: indeed - how did I miss that before? | 16:35 |
| AJaeger | fixing... | 16:35 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Add configure-os-mirrors role https://review.opendev.org/677578 | 16:36 |
| *** hashar has joined #zuul | 16:36 | |
| AJaeger | avass: thanks! | 16:36 |
| *** jcapitao has quit IRC | 16:43 | |
| avass | AJaeger: I believe the other problem is that the templates are using the DEB822 format but doesn't have the .sources extension | 16:46 |
| *** ysandeep is now known as ysandeep|away | 16:49 | |
| *** yolanda has quit IRC | 16:53 | |
| corvus | mordred, tristanC, tobiash: can you look at my comment on https://review.opendev.org/731200 ? i'd like to resolve that before the release (either by removing or changing the reno) | 16:54 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: executor: add extra kubectl port-forward debug and releasenotes https://review.opendev.org/731200 | 16:56 |
| tristanC | corvus: i kept the reno because i think it's an important fix | 16:57 |
| corvus | tristanC: ok, wfm, thanks :) | 16:58 |
| tobiash | ++ | 16:58 |
| corvus | i think that's both outstanding release notes approved, so when they land, i'll tag the release | 16:59 |
| tobiash | :) | 16:59 |
| tristanC | corvus: nice, thanks! | 16:59 |
| openstackgerrit | Lida Liu proposed zuul/zuul master: WIP: Add commit id to Change for mqtt reporter https://review.opendev.org/722478 | 17:10 |
| *** jpena is now known as jpena|off | 17:10 | |
| *** fbo is now known as fbo|off | 17:11 | |
| openstackgerrit | Merged zuul/zuul master: Add release note about the final 3.x release https://review.opendev.org/731221 | 17:18 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add configure-os-mirrors role https://review.opendev.org/677578 | 17:26 |
| *** ianychoi has joined #zuul | 17:27 | |
| avass | AJaeger: that should probably work a little better ^ :) | 17:28 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add configure-os-mirrors role https://review.opendev.org/677578 | 17:42 |
| AJaeger | thanks, avass ! | 17:53 |
| *** hashar has quit IRC | 17:53 | |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add configure-os-mirrors role https://review.opendev.org/677578 | 17:53 |
| openstackgerrit | Merged zuul/zuul master: executor: add extra kubectl port-forward debug and releasenotes https://review.opendev.org/731200 | 17:54 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: Add missing python-dateutil requirement in requirements.txt https://review.opendev.org/731249 | 17:54 |
| *** harrymichal has joined #zuul | 17:55 | |
| *** dpawlik has quit IRC | 17:59 | |
| openstackgerrit | Merged zuul/zuul master: Include extra, host, and group variables in job serialization https://review.opendev.org/729753 | 18:00 |
| avass | AJaeger: looks like it's just the ubuntu plain images that are failing now | 18:07 |
| AJaeger | avass: yeah | 18:09 |
| AJaeger | the message is strange "You have broken packages" | 18:10 |
| zbr | who worked on the fake zuul_return code? | 18:13 |
| fungi | avass: AJaeger: got a link? that's usually a case of dependency version mismatches | 18:14 |
| avass | AJaeger: it's a depency problem , | 18:14 |
| avass | fungi: yeah | 18:14 |
| avass | fungi: this is what you want https://e7eb7933bc64c756eec0-cb65abe58d87a1a010092a9adcbaff91.ssl.cf5.rackcdn.com/677578/14/check/zuul-jobs-test-new-mirror-roles-ubuntu-bionic-plain/9f6c4f5/job-output.json | 18:14 |
| avass | fungi: python3-pip : Depends: python-pip-whl | 18:14 |
| zbr | i managed to propose a generic workaround at https://github.com/ansible/ansible-lint/pull/801 maybe you can help me with some feedback. | 18:15 |
| avass | zbr: ++ | 18:16 |
| avass | zbr: if you want to make it better you could make it handle it's exit with exit_json() as well | 18:17 |
| zbr | well, i am not proud of the implementation, hacking env inside __init__, maybe someone will have better ideas. | 18:17 |
| avass | zbr: like this https://review.opendev.org/#/c/728684/23/tests/fake-ansible/library/zuul_return.py | 18:17 |
| zbr | for the moment I only want to add basic faking support | 18:17 |
| *** jamesmcarthur has quit IRC | 18:17 | |
| avass | zbr: ah, I'll take a close look later then :) | 18:18 |
| *** jamesmcarthur has joined #zuul | 18:18 | |
| *** jamesmcarthur has quit IRC | 18:19 | |
| *** jamesmcarthur has joined #zuul | 18:19 | |
| openstackgerrit | Oleksandr Kozachenko proposed zuul/zuul-jobs master: Add the namespace in wait-for-pods role https://review.opendev.org/731292 | 18:38 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: WIP: Add CentOS-8 support https://review.opendev.org/731293 | 18:38 |
| AJaeger | avass: I'm bringing in CentOS-8 support but stack this on top for now, we can squeeze later. Didn't want to interfere with any edits you might have in flight ^ | 18:39 |
| avass | AJaeger: ah, I don't have any :) | 18:40 |
| AJaeger | avass: that's sad ;) | 18:40 |
| openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul-jobs master: Bump ansible-lint to 4.3.0 https://review.opendev.org/702679 | 18:41 |
| AJaeger | so, why is our job failing but configure-mirror working? Or are we just not testing that? | 18:41 |
| openstackgerrit | Lida Liu proposed zuul/zuul master: WIP: Add commit id to Change for mqtt reporter https://review.opendev.org/722478 | 18:42 |
| avass | AJaeger: trying to figure that out at the moment | 18:43 |
| *** jamesmcarthur has quit IRC | 18:43 | |
| AJaeger | thanks | 18:44 |
| openstackgerrit | Lida Liu proposed zuul/zuul master: WIP: Add commit id to Change for mqtt reporter https://review.opendev.org/722478 | 18:45 |
| fungi | avass: so looking at the error (sorry got sidetracked by something else) it's trying to install the python3-pip version from the bionic release suite while newer python-pip-whl from the bionic-updates suite is already installed and would have to be downgraded to satisfy the strict version dependency | 18:49 |
| fungi | so the question is why isn't it trying to install python3-pip from bionic-updates | 18:50 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: WIP: Add CentOS-8 support https://review.opendev.org/731293 | 18:51 |
| openstackgerrit | Oleksandr Kozachenko proposed zuul/zuul-jobs master: Add the namespace in wait-for-pods role https://review.opendev.org/731292 | 18:51 |
| fungi | avass: it looks like the answer is that we've built the image with packages from the bionic-updates suite but are only including bionic and bionic-security suites in the sources.list | 18:51 |
| clarkb | fungi: possible that we never added bionic-updates in after the repos actually existed and that it never caused a problem until now? | 18:52 |
| clarkb | iirc updates is the one that routinely gives us trouble with reprepro | 18:53 |
| fungi | clarkb: i expect it's that the ubuntu-bionic-plain images don't have pip preinstalled, while our ubuntu-bionic images do, so aren't exercising this code path | 18:53 |
| clarkb | ah | 18:53 |
| *** jamesmcarthur has joined #zuul | 18:54 | |
| fungi | also we did work out the solution to reprepro mirroring "empty" -security and -updates suites, there's a command you can run to force it to create the indices for all configured suites even if some are empty | 18:54 |
| AJaeger | avass: in general, we need to check what is done for configure-mirrors and sync... CentOS-8 support was the first thing coming to mind. | 18:54 |
| fungi | clarkb: as for why we're only seeing it for that change, i think it's because the change is incomplete (its the work for our next-generation mirror handling role) | 18:55 |
| *** jamesmcarthur has quit IRC | 18:55 | |
| *** jamesmcarthur has joined #zuul | 18:55 | |
| fungi | but i haven't looked closely enough at it yet | 18:55 |
| avass | fungi, AJaeger: ah okay | 18:56 |
| clarkb | fungi: zuul-jobs/roles/configure-mirror does appear to configure updates for ubuntu nodes | 18:56 |
| clarkb | so that repo should be available on all our bionic hosts I think | 18:56 |
| AJaeger | fungi: yeah, I'm fear it's incomplete - it's a change from September 2019 that we just barely have working ;) | 18:57 |
| fungi | AJaeger: hey, it sounds remarkably close! | 18:57 |
| AJaeger | yeah, I'm suprised how much seems to pass - I started with patch set 2 8 hours ago, now thanks to avass' help, we're at number 14 ;) | 18:58 |
| *** jamesmcarthur has quit IRC | 18:59 | |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add updates mirror https://review.opendev.org/731305 | 19:01 |
| avass | fungi: I'm guessing something like that ^ should be enough to get the tests to pass then | 19:01 |
| avass | fungi: since we're not separating the files, at least not yet | 19:02 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Web UI: add "held" information, filtering in Builds page https://review.opendev.org/729265 | 19:05 |
| *** jamesmcarthur has joined #zuul | 19:06 | |
| fungi | avass: likely, though i also noticed in the log you linked that the node was pulling package indices for both xenial and bionic rather than just bionic | 19:11 |
| fungi | probably those entries should be grouped by release, not by suite type | 19:12 |
| *** sshnaidm is now known as sshnaidm|afk | 19:12 | |
| fungi | suites: ['bionic', 'bionic-security', 'bionic-updates'] | 19:13 |
| fungi | though i'm unclear as to how the task which installs the sources.list is supposed to filter entries by release | 19:14 |
| avass | fungi: yep | 19:14 |
| *** jamesmcarthur has quit IRC | 19:15 | |
| *** jamesmcarthur has joined #zuul | 19:15 | |
| fungi | looking at the data model, maybe our zuul_site_mirror_info keys should be like debian-buster and ubuntu-bionic rather than just debian and ubuntu | 19:15 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: WIP: Add CentOS-8 support https://review.opendev.org/731293 | 19:21 |
| *** rlandy is now known as rlandy|biab | 19:23 | |
| avass | fungi: it's currently filtering it by checking ansible_distribution_release :) | 19:24 |
| fungi | avass: and matching against what in the current data structure? | 19:25 |
| fungi | avass: oh! this is a test payload | 19:26 |
| fungi | so it's trying to install multiple releases just to exercise the list generation bits | 19:26 |
| fungi | and is not representative of a real-world variable set | 19:27 |
| avass | yeah | 19:27 |
| avass | fungi: but I guess that wouldn't work otherwise since the suite doens't need to include the release right? | 19:28 |
| fungi | avass: yes, it's merely convention | 19:29 |
| fungi | avass: for some of my personal package repositories, the suite is merely "./" and the component list is empty | 19:30 |
| fungi | and that works just fine | 19:30 |
| avass | fungi: any idea why https://zuul.opendev.org/t/zuul/build/3ba6859891ec44c99b2fc271af0df1a9 fails? | 19:32 |
| fungi | i think the idea here though is that we say we want the default ubuntu bionic sources and then pass in the appropriate variables for a given set of entries | 19:32 |
| fungi | which in turn generates the entries for bionic, bionic-security and bionic-updates suites | 19:32 |
| *** jamesmcarthur has quit IRC | 19:33 | |
| fungi | but we may also add variables to declare other entries, like nodesource or something | 19:33 |
| *** jamesmcarthur has joined #zuul | 19:35 | |
| fungi | avass: that "Zuul clone something not in required-projects" task looks obsolete... we haven't supported the zuul-cloner shim in ages i didn't think (anyway it's not preinstalled on that image) | 19:35 |
| avass | fungi: yeah, I was thinking more of the os_testr import | 19:36 |
| fungi | oh, nevermind, i misread that task | 19:36 |
| *** harrymichal has quit IRC | 19:39 | |
| fungi | yeah, so /usr/os-testr-env/ seems to have a subunit2html entrypoint which can't import from os_testr.subunit2html | 19:39 |
| fungi | that's weird | 19:39 |
| avass | yeah :) | 19:39 |
| fungi | i'll pull up an image build log and see if i can tell what goes on to create that env | 19:39 |
| fungi | 2020-05-25 05:54:32.722 in https://nb01.opendev.org/ubuntu-xenial-plain-0000014730.log is where the venv is created | 19:40 |
| fungi | error: invalid command 'bdist_wheel' | 19:41 |
| fungi | the errors there make me wonder if pip or setuptools is too old | 19:41 |
| *** hashar has joined #zuul | 19:41 | |
| clarkb | bdist_wheel required the wheel package for a while right? did it ever get pulled into pip/setuptools themselves? | 19:42 |
| fungi | or it's an issue with the distutils in py35 on xenial | 19:42 |
| avass | clarkb: doesn't look like it | 19:44 |
| fungi | clarkb: oh, yeah, could be we need to explicitly include wheel in the venv under 3.5 | 19:44 |
| clarkb | fungi: ya that would be my guess | 19:44 |
| fungi | it's included in newer venvs | 19:44 |
| fungi | checking now to see how far back | 19:45 |
| fungi | huh actually not with any of my upstream python builds from 3.5 through 3.9 beta | 19:47 |
| fungi | only pip and setuptools are seeded | 19:47 |
| fungi | so i must have misremembered that | 19:47 |
| guilhermesp | hey ya! So currently ensure-tox installs tox inside a python-venv right? I think we are having issues when we are using non-system python | 19:47 |
| fungi | we should probably directly install wheel if in there if we need to install sdists | 19:47 |
| guilhermesp | https://www.irccloud.com/pastebin/CulApC95/ | 19:47 |
| fungi | guilhermesp: looks like ansible is using python 2.7 there, i wonder if pkg_resources is only installed for python 3.x | 19:49 |
| mnaser | guilhermesp: i think this is the thing where we workaround installing python3-tox | 19:50 |
| mnaser | errr, python3-venv | 19:50 |
| mnaser | ut we don't do it for python2-venv ? | 19:50 |
| fungi | we did just discuss today that apparently ansible still defaults to using python2 with legacy-auto | 19:50 |
| mnaser | it must have spurred up recently because we had no change other than just new nodepool images being uploaded | 19:51 |
| clarkb | ya I think there may have been different expectations for ansible python choice | 19:51 |
| clarkb | auto is supposed to prefer python3 if present but the default is auto legacy | 19:52 |
| clarkb | though mordred mentioned nodepool should be setting auto. Maybe we aren't doing that properly or auto in ansible is buggy? | 19:52 |
| *** jamesmcarthur has quit IRC | 19:53 | |
| *** jamesmcarthur has joined #zuul | 19:56 | |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: WIP: Sync with configure-mirrors https://review.opendev.org/731293 | 19:59 |
| AJaeger | avass: this syncs all relevant changes from configure-mirrors over, we can squash the changes once the above is fine ^ | 20:00 |
| avass | AJaeger: sure! | 20:01 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add updates mirror https://review.opendev.org/731305 | 20:01 |
| avass | I think I'm going to quit for today, have a good evening :) | 20:02 |
| corvus | clarkb, mnaser, mordred: nodepool defaults to telling zuul to tell ansible to use "auto" for the python interpreter, but if it's "auto" zuul will set it to "python2" for ansible < 2.8 since auto isn't supported in 2.7 | 20:02 |
| corvus | you can confirm what was given to ansible in the logs; eg: https://zuul.opendev.org/t/openstack/build/323025c9d3ea4a898e1ffa56b176b1f0/log/zuul-info/inventory.yaml#7 | 20:02 |
| corvus | guilhermesp: ^ | 20:03 |
| AJaeger | avass: unfortunatley all broken right now ;( | 20:03 |
| AJaeger | avass: I'll quit soon as well - good night | 20:04 |
| guilhermesp | yes i can see here for our debian-buster corvus ansible_python_interpreter: auto | 20:06 |
| mnaser | so does that mean ansible has autodetected auto to py27 on debian buster even if its not set to legacy-auto? :X | 20:07 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: WIP: Sync with configure-mirrors https://review.opendev.org/731293 | 20:08 |
| clarkb | mnaser: yes I think so. Its possible that is a bug, but may also be intended behavior | 20:09 |
| clarkb | apparently there is some lookup table ansible uses in the auto case | 20:09 |
| *** olaph has joined #zuul | 20:09 | |
| mnaser | clarkb: yeah i remember that file | 20:09 |
| *** jamesmcarthur has quit IRC | 20:11 | |
| *** harrymichal has joined #zuul | 20:12 | |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: WIP: Sync with configure-mirrors https://review.opendev.org/731293 | 20:13 |
| AJaeger | avass: stacked them all on top of each other, if those pass, let's squash all three and review ;) | 20:14 |
| * AJaeger waves good night | 20:14 | |
| *** jamesmcarthur has joined #zuul | 20:16 | |
| *** jamesmcarthur has quit IRC | 20:20 | |
| openstackgerrit | Oleksandr Kozachenko proposed zuul/zuul-jobs master: Add namespace in the collect-k8s-logs role https://review.opendev.org/731319 | 20:21 |
| *** jamesmcarthur has joined #zuul | 20:21 | |
| *** harrymichal has quit IRC | 20:23 | |
| *** jamesmcarthur has quit IRC | 20:28 | |
| *** jamesmcarthur has joined #zuul | 20:29 | |
| *** harrymichal has joined #zuul | 20:34 | |
| *** jamesmcarthur has quit IRC | 20:35 | |
| *** nils has quit IRC | 20:42 | |
| *** jamesmcarthur has joined #zuul | 20:49 | |
| corvus | clarkb, tobiash, mordred, tristanC: does https://zuul-ci.org/docs/zuul/reference/releasenotes.html look good? | 20:49 |
| corvus | clarkb, tobiash, mordred, tristanC: and does this look right? commit 6161ad75a83cd8e8c3bd347ea7f40b0e2d6b5c33 (HEAD -> master, tag: 3.19.0, origin/master, origin/HEAD) | 20:50 |
| openstackgerrit | Lida Liu proposed zuul/zuul master: WIP: Add commit id to Change for mqtt reporter https://review.opendev.org/722478 | 20:51 |
| tobiash | corvus: reading | 20:51 |
| clarkb | corvus: maybe we should note that the sql connection is only needed for scheduler and web (we note that zk will be universal) | 20:52 |
| clarkb | but ya I think it looks good and gives people a clear path forward | 20:52 |
| mordred | corvus: looks good! | 20:54 |
| tobiash | clarkb: I think that's a general good addition to the docs (imho no need to put that info into the release note) | 20:54 |
| tobiash | corvus: release notes and hash lgtm | 20:54 |
| * clarkb checks the hash now | 20:54 | |
| clarkb | 6161ad75a83cd8e8c3bd347ea7f40b0e2d6b5c33 is what I see as HEAD -> master as well. so that hash lgtm | 20:56 |
| corvus | zuul actual is go for tagging 3.19.0 | 20:58 |
| * mordred wants to know some official sounding response to that | 21:11 | |
| corvus | https://pypi.org/project/zuul/3.19.0/ exists | 21:12 |
| corvus | oh, we need to tag nodepool too | 21:13 |
| *** hashar has quit IRC | 21:14 | |
| corvus | tobiash, clarkb, mordred, tristanC: commit fa2a850cb93066df04ae8ff5bebf4a42554ca0e4 (HEAD -> master, tag: 3.13.0, origin/master, origin/HEAD) | 21:14 |
| corvus | that look right for nodepool? | 21:14 |
| tobiash | ++ | 21:15 |
| tobiash | wow there was a long time since the last release | 21:16 |
| clarkb | fa2a850cb93066df04ae8ff5bebf4a42554ca0e4 is what I see too. lgtm. Tag version looks correct too. There was at least one feature added (the inheritance of image specs) | 21:16 |
| mordred | ++ | 21:16 |
| corvus | pushed; i'll prep announcement emails | 21:16 |
| mordred | so I guess we have to start reviewing all the bolg patches now | 21:16 |
| corvus | https://pypi.org/project/nodepool/3.13.0/ exists | 21:27 |
| corvus | email sent | 21:27 |
| *** saneax has quit IRC | 21:34 | |
| *** jamesmcarthur_ has joined #zuul | 21:37 | |
| tobiash | mordred: the change queue patches and cdep would be awesome as well :) | 21:40 |
| *** jamesmcarthur has quit IRC | 21:41 | |
| corvus | i'm planning on reviewing cdep with a fresh brain tomorrow; i'm +2 on the queue | 21:44 |
| corvus | i also pointed the queue patches out to some other opendev folks to make sure they're okay with the implications (so far so good) | 21:45 |
| tobiash | corvus: thanks! | 21:45 |
| *** jamesmcarthur_ has quit IRC | 21:48 | |
| *** harrymichal has quit IRC | 21:49 | |
| *** jamesmcarthur has joined #zuul | 22:03 | |
| *** rlandy|biab is now known as rlandy | 22:04 | |
| * fungi cheers zuul 3.19.0 and nodepool 3.13.0 from the sidelines | 22:10 | |
| corvus | fungi: soon we get to cough politely when someone says "zuul v3" | 22:16 |
| fungi | the python community will be doing the same once python 4 releases and still uses /usr/bin/python3 as its executable name ;) | 22:18 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-jobs master: shake-build: add shake build system job https://review.opendev.org/731365 | 22:21 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-jobs master: shake-build: add shake build system job https://review.opendev.org/731365 | 22:22 |
| tristanC | corvus: is it me or the new timezone selector is missing from the https://zuul.opendev.org deployment? | 22:27 |
| tristanC | and congrats for the release, well done! | 22:28 |
| corvus | tristanC: we may need a container restart now (we used to deploy web near-continuously, but i think since we switched to containers, we'll need to restart) | 22:29 |
| clarkb | we can probably switch web back to restarting on container update | 22:30 |
| corvus | hrm, i just pulled and restarted and still don't see it | 22:30 |
| clarkb | its a separate config to schedyler | 22:30 |
| corvus | more investigation may be warranted :/ | 22:30 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-jobs master: WIP: shake-build: add shake build system job https://review.opendev.org/731365 | 22:34 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-jobs master: WIP: shake-build: add shake build system job https://review.opendev.org/731365 | 22:46 |
| *** guillaumec has quit IRC | 22:46 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-jobs master: shake-build: add shake build system job https://review.opendev.org/731365 | 23:04 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-jobs master: shake-build: add shake build system job https://review.opendev.org/731365 | 23:14 |
| openstackgerrit | Pierre-Louis Bonicoli proposed zuul/zuul master: last3x release note: fix a typo https://review.opendev.org/731372 | 23:16 |
| *** tosky has quit IRC | 23:30 | |
| openstackgerrit | Pierre-Louis Bonicoli proposed zuul/nodepool master: doc: openshiftpods handles python-path too https://review.opendev.org/731373 | 23:38 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-jobs master: shake-build: add shake build system job https://review.opendev.org/731365 | 23:40 |
| *** Goneri has quit IRC | 23:49 | |
| *** jamesmcarthur has quit IRC | 23:50 | |
| *** jamesmcarthur has joined #zuul | 23:54 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
