| *** guillaumec has quit IRC | 00:08 | |
| *** rfolco has quit IRC | 00:24 | |
| *** cdearborn has quit IRC | 00:40 | |
| *** threestrands has joined #zuul | 00:45 | |
| *** swest has quit IRC | 01:59 | |
| *** swest has joined #zuul | 02:14 | |
| *** ianychoi_ has joined #zuul | 03:08 | |
| *** ianychoi has quit IRC | 03:10 | |
| *** bhavikdbavishi has joined #zuul | 03:19 | |
| *** bhavikdbavishi1 has joined #zuul | 03:22 | |
| *** saneax has joined #zuul | 03:22 | |
| *** bhavikdbavishi has quit IRC | 03:23 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 03:23 | |
| *** bhavikdbavishi has quit IRC | 03:58 | |
| *** bhavikdbavishi has joined #zuul | 04:01 | |
| *** threestrands has quit IRC | 04:17 | |
| *** evrardjp has quit IRC | 04:35 | |
| *** evrardjp has joined #zuul | 04:35 | |
| *** sgw has quit IRC | 05:07 | |
| *** tflink has quit IRC | 05:26 | |
| *** tflink has joined #zuul | 05:27 | |
| openstackgerrit | Merged zuul/zuul master: Fix test_crd_check_unknown tests https://review.opendev.org/724191 | 05:29 |
|---|---|---|
| openstackgerrit | Merged zuul/zuul master: Simplify FakeGithubClient and FakeGithubSession https://review.opendev.org/724192 | 05:29 |
| *** timburke has joined #zuul | 05:44 | |
| *** ysandeep|away is now known as ysandeep | 05:45 | |
| *** ysandeep is now known as ysandeep|brb | 05:55 | |
| *** rpittau|afk is now known as rpittau | 06:32 | |
| *** jcapitao has joined #zuul | 06:50 | |
| *** ysandeep|brb is now known as ysandeep | 06:51 | |
| *** dustinc has quit IRC | 06:53 | |
| *** dpawlik has joined #zuul | 06:59 | |
| *** yolanda has joined #zuul | 07:19 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Add git name and email for quickstart executor https://review.opendev.org/724096 | 07:22 |
| *** guillaumec has joined #zuul | 07:24 | |
| *** tosky has joined #zuul | 07:29 | |
| *** bhavikdbavishi has quit IRC | 07:39 | |
| *** ysandeep is now known as ysandeep|lunch | 07:42 | |
| openstackgerrit | Andy Ladjadj proposed zuul/zuul master: Add new timezone selector in web interface https://review.opendev.org/722653 | 07:53 |
| *** jpena|off is now known as jpena | 07:55 | |
| *** bhavikdbavishi has joined #zuul | 07:59 | |
| openstackgerrit | Andy Ladjadj proposed zuul/zuul master: Add new timezone selector in web interface https://review.opendev.org/722653 | 08:01 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Connect executor to Zookeeper https://review.opendev.org/716262 | 08:11 |
| *** jcapitao has quit IRC | 08:24 | |
| *** yoctozepto has joined #zuul | 08:24 | |
| yoctozepto | morning | 08:24 |
| yoctozepto | is it expected that zuul does not consider job modified (and does not run it hence) when only extra vars are modified on it? | 08:24 |
| yoctozepto | https://review.opendev.org/#/c/723354/6/.zuul.d/base.yaml | 08:24 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Add git name and email for quickstart executor https://review.opendev.org/724096 | 08:26 |
| *** ysandeep|lunch is now known as ysandeep | 08:26 | |
| *** jcapitao has joined #zuul | 08:26 | |
| openstackgerrit | Andy Ladjadj proposed zuul/zuul master: Add new timezone selector in web interface https://review.opendev.org/722653 | 08:39 |
| *** avass has quit IRC | 10:06 | |
| *** avass has joined #zuul | 10:06 | |
| avass | hmm, I'm thinking that we might want to create a linter for zuul-jobs, it's easy to miss policies like namespacing loop variables otherwise | 10:07 |
| *** bhavikdbavishi has quit IRC | 10:20 | |
| *** bhavikdbavishi has joined #zuul | 10:21 | |
| *** guillaumec has quit IRC | 10:50 | |
| *** jcapitao is now known as jcapitao_lunch | 10:54 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Add git name and email for quickstart executor https://review.opendev.org/724096 | 10:55 |
| *** rpittau is now known as rpittau|bbl | 10:55 | |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add loop var policy to ansible-lint https://review.opendev.org/724281 | 11:02 |
| avass | Looks like you can defined custom ansible-lint rules :) | 11:08 |
| zbr | avass: yes you can, but that does not mean is a good idea, feel free to propose them to the linter, I can help you getting them in | 11:19 |
| zbr | i already migrated one or two custom roles I found, during the last year. | 11:20 |
| avass | zbr: isn't the loop_var policy very specific to zuul-jobs though? | 11:21 |
| zbr | indeed, in this case is more problematic | 11:21 |
| zbr | still, how are you going to apply this rule to tons of repos we have? | 11:22 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add loop var policy to ansible-lint https://review.opendev.org/724281 | 11:22 |
| zbr | i guess the solution is to implement a plugin system for ansible-lint and publish the rules as python package | 11:22 |
| avass | zbr: I'm not saying we should apply the rule to other repos, just enforce the policy we already have for zuul-jobs | 11:23 |
| zbr | it is easier than it appears | 11:23 |
| avass | zuul-jobs-main: I'd like some input on that ^ before I start updating roles to follow the policy | 11:25 |
| *** ysandeep is now known as ysandeep|coffee | 11:29 | |
| *** jpena is now known as jpena|lunch | 11:32 | |
| *** guillaumec has joined #zuul | 11:45 | |
| tristanC | avass: adding test for policy sounds great to me | 11:52 |
| tristanC | is the AnsibleLintRule interface is stable? | 11:54 |
| AJaeger | avass: I like the idea | 11:56 |
| *** rfolco has joined #zuul | 12:14 | |
| *** jcapitao_lunch is now known as jcapitao | 12:16 | |
| *** Goneri has joined #zuul | 12:22 | |
| avass | tristanC: I have no idea if it is | 12:27 |
| avass | tristanC: or I mean, I don't know how stable it is, but it's in the official documentation :https://docs.ansible.com/ansible-lint/rules/rules.html#creating-custom-rules | 12:29 |
| *** rpittau|bbl is now known as rpittau | 12:31 | |
| *** jpena|lunch is now known as jpena | 12:32 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Use speculative zuul-base-jobs in integration test https://review.opendev.org/716300 | 12:32 |
| *** sgw has joined #zuul | 12:45 | |
| zbr | few easy reviews: https://review.opendev.org/#/c/723855/ https://review.opendev.org/#/c/723856/ | 12:47 |
| openstackgerrit | Sorin Sbarnea proposed zuul/zuul master: Make task errors expandable https://review.opendev.org/723534 | 12:54 |
| zbr | tristanC: interface is stable, my only concerns where around reusabiliy of the rule on other repos, but in-repo is perfect fine for start. | 13:03 |
| zbr | avass: tristanC can you look at https://review.opendev.org/#/c/690057/ that improves ensure-tox? | 13:05 |
| zbr | is already ½ year old, got tired of number of rebases i had to do on it :D | 13:05 |
| avass | zbr: the docs looks strange, I guess you hade to solve a merge conflict there | 13:15 |
| zbr | aha, not i see, i think i missed a space, fixing... | 13:17 |
| mordred | avass: I like it | 13:20 |
| *** cdearborn has joined #zuul | 13:21 | |
| openstackgerrit | Sorin Sbarnea proposed zuul/zuul-jobs master: tox: allow tox to be upgraded https://review.opendev.org/690057 | 13:24 |
| mordred | avass: of course, that made me go read the rest of policy - which made me think that https://zuul-ci.org/docs/zuul-jobs/policy.html#support-for-multiple-operating-systems could potentially be implemented more consistently with an action plugin ... which made me think in the new world of collections, we could potentially make an action plugin that did a consistent "include_distro_tasks" so we wouldn't have to | 13:25 |
| mordred | copy-pasta that | 13:25 |
| avass | mordred: yeah | 13:29 |
| avass | mordred: I have a change that adds similar functionality to ANSIBLE_EXTRA_PACKAGES for collections: https://review.opendev.org/#/c/723071/ but I still need to add tests for it | 13:32 |
| avass | mordred: and I want to be able to validate that the collections are installed somehow, similar to this: https://review.opendev.org/#/c/724110/ | 13:32 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add loop var policy to ansible-lint https://review.opendev.org/724281 | 13:36 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add loop var policy to ansible-lint https://review.opendev.org/724281 | 13:41 |
| avass | Hopefully I didn't miss something now | 13:41 |
| avass | zbr: re 690057: just a minor thing, we probably want to be able to use constraints even though tox_upgrade is not specified | 13:47 |
| *** bhavikdbavishi has quit IRC | 13:47 | |
| zbr | true | 13:52 |
| zbr | avass: this also means that we cannot use "type tox" test anymore, we need to really run pip again. | 13:54 |
| zbr | because we may have an older version. | 13:54 |
| *** jcapitao has quit IRC | 13:55 | |
| *** jcapitao has joined #zuul | 13:56 | |
| zbr | avass: in fact i think the code is correct, take a second look. | 13:57 |
| zbr | if constraint is default and no upgrade is requested, it will install it only if is missing. | 13:58 |
| avass | zbr: ah, I missed the "tox_constraints != 'tox'" | 13:58 |
| zbr | avass: well, confused the author too. | 13:58 |
| avass | zbr: maybe we should add something to the docs about that | 14:00 |
| zbr | i doubt, i think is implied by current readme, but you are welcomed to rephrase it. | 14:00 |
| openstackgerrit | Sorin Sbarnea proposed zuul/zuul-jobs master: Add testing of fetch-sphinx-tarball role https://review.opendev.org/721584 | 14:05 |
| avass | zbr: something like "Python requirements condition tox should adhere to"? | 14:06 |
| avass | but I'm not sure "adhere" is really the word I'm looking for | 14:06 |
| fungi | filter? | 14:08 |
| fungi | constraints is basically a sieve filter | 14:08 |
| fungi | if your requirement appears in the constraints list, then install the version of it specified there | 14:09 |
| avass | zbr: +1 for now, but I think someone else should probably look at it as well | 14:10 |
| *** hashar has joined #zuul | 14:12 | |
| avass | I'm getting and Internal Server Error for zuul | 14:15 |
| avass | an* | 14:15 |
| zbr | you are not alone, i posted on infra. | 14:16 |
| fungi | yep, second oom condition for the scheduler in a little over 2 days | 14:24 |
| fungi | http://cacti.openstack.org/cacti/graph.php?action=view&local_graph_id=64792&rra_id=all | 14:25 |
| fungi | http://cacti.openstack.org/cacti/graph.php?action=view&local_graph_id=64794&rra_id=all | 14:25 |
| fungi | for the ram and swap graphs of the scheduler, respectively | 14:25 |
| *** dpawlik has quit IRC | 14:28 | |
| avass | fungi: I'll have to check if we're experiencing something like that, I'd guess not since we don't have the same load | 14:29 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Connect executor to Zookeeper https://review.opendev.org/724336 | 14:29 |
| fungi | well, we normally hover around 5gb active memory on the scheduler, and suddenly we're spiking up to 12x that | 14:30 |
| -openstackstatus- NOTICE: Zuul had to be restarted, all changes submitted or approved between 14:00 UTC to 14:30 need to be rechecked, we queued already those running at 14:00 | 14:36 | |
| avass | fungi: we're at 3GB, I would guess that's higher than usual, but we're not tracking that it seems and I'm not sure we have access to the data IT keeps | 14:40 |
| avass | fungi: at least opendev has fast startup times ;) | 14:42 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Connect executor to Zookeeper https://review.opendev.org/716262 | 14:42 |
| mordred | corvus: I mentioned earlier in #opendev that I noticed that the podman people are pointing to opensuse kubic for debian packages now ... which caused me to realize that we are not installing skopeo into our zuul-executor containers ... should we? | 14:59 |
| corvus | that's probably a good idea | 15:01 |
| mordred | kk | 15:02 |
| openstackgerrit | Monty Taylor proposed zuul/zuul master: Install skopeo in container images https://review.opendev.org/724344 | 15:09 |
| avass | zuul-jobs-maint: can we try to sync it so this is merged before some other change breaks it? https://review.opendev.org/#/c/724281 | 15:19 |
| avass | (I'll update the commit message to be a bit more descriptive) | 15:21 |
| corvus | avass: nice :) i'm usually the person who is least likely to want to expand linting tests, but this looks like a really good use. :) | 15:22 |
| avass | corvus: Yeah, I really don't like having to review and keep all the policies in my head because it's so easy to miss, so why not just automate it :) | 15:24 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add loop var policy to ansible-lint https://review.opendev.org/724281 | 15:26 |
| AJaeger | corvus: is zuul iterating config files recursively? So, are multiple levels allowed like zuul.d/level1/config.yaml ? See https://review.opendev.org/722394 | 15:38 |
| AJaeger | clarkb: thanks for the link. I just want to be sure that it goes down levels - in that case zuul-sphinx has indeed a bug. | 15:38 |
| corvus | agree with clarkb | 15:39 |
| AJaeger | corvus: ok, thanks | 15:39 |
| clarkb | ya figuring out zuul-sphinx bug is on my todo. But if someone else wants to do it go ahead :) | 15:40 |
| zbr | can we limit the "zuul-jobs" change notifications to a single irc channel? now we get them 3 channels,.... | 15:46 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Add git name and email for quickstart executor https://review.opendev.org/724096 | 15:48 |
| fungi | zbr: that's probably a question for the folks in each of those channels | 15:52 |
| fungi | i can't imagine dropping zuul-jobs change notifications from the #zuul channel, as it's one of the zuul project's repositories | 15:52 |
| *** jcapitao has quit IRC | 15:52 | |
| fungi | so this is probably not the place to be asking that question | 15:53 |
| AJaeger | zbr: I think we had zuul-jobs always in all three, we remove many other duplicates so it's more visible that thre are very few now | 15:54 |
| clarkb | I'd be ok with removing it from the other channels since I'm in here too | 15:57 |
| clarkb | but I don't know how good our overlap is and zuul-jobs is still seeing a fair bit of churn that affects open(dev|stack) | 15:57 |
| AJaeger | the question is: Do we expect openstack folks on #opendev as well? Then we can for sure remove from #openstack-infra. | 15:58 |
| AJaeger | But I would keep it between #opendev and #zuul since that are more disjunct audiences | 15:58 |
| *** jcapitao has joined #zuul | 16:04 | |
| openstackgerrit | Merged zuul/zuul master: Add DOCKER_* to passenv https://review.opendev.org/723856 | 16:09 |
| avass | zuul-jobs-maint: anyone else for: https://review.opendev.org/#/c/724281/ ? | 16:11 |
| *** ysandeep|coffee is now known as ysandeep|away | 16:12 | |
| corvus | done | 16:13 |
| openstackgerrit | Clark Boylan proposed zuul/zuul-sphinx master: Handle directories in zuul.d/ configs https://review.opendev.org/724366 | 16:13 |
| openstackgerrit | Clark Boylan proposed zuul/zuul-sphinx master: Sort dirs and filenames when os.walk()ing https://review.opendev.org/724367 | 16:13 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Connect executor to Zookeeper https://review.opendev.org/716262 | 16:13 |
| avass | corvus: thanks! | 16:13 |
| clarkb | corvus: AJaeger ^ I think those two changes will make zuul-sphinx a bit more like zuul | 16:13 |
| clarkb | I'm not sure what the best way to test that is though | 16:14 |
| clarkb | I'm going to run tox -e docs in system-config on that broken change, install my changes above itno that venv then rerun sphinx build | 16:18 |
| AJaeger | clarkb: that looks like the best way to test it. | 16:20 |
| clarkb | that seems to have worked and the roles.html file looks ok. I'm not sure what else to check | 16:20 |
| corvus | clarkb: that ought to do it :) | 16:22 |
| *** hashar has quit IRC | 16:22 | |
| zbr | corvus: re coloring on pre, you want me to remove it even from exception or only from stdout? | 16:26 |
| openstackgerrit | Sorin Sbarnea proposed zuul/zuul master: Fix pre wrapping on console and output https://review.opendev.org/723603 | 16:32 |
| openstackgerrit | Monty Taylor proposed zuul/zuul master: Install skopeo in container images https://review.opendev.org/724344 | 16:34 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Add git name and email for quickstart executor https://review.opendev.org/724096 | 16:35 |
| *** evrardjp has quit IRC | 16:35 | |
| *** evrardjp has joined #zuul | 16:35 | |
| *** rpittau is now known as rpittau|afk | 16:37 | |
| zbr | corvus: i think you are aware the current implementation is that stderr is always empty and stdout contains both stdout+stderr | 16:42 |
| openstackgerrit | Merged zuul/zuul-jobs master: Add loop var policy to ansible-lint https://review.opendev.org/724281 | 16:42 |
| openstackgerrit | Sorin Sbarnea proposed zuul/zuul master: Make task errors expandable https://review.opendev.org/723534 | 16:49 |
| corvus | zbr: i'm mostly concerned about stdout; exception i could go either way on. and yes; there's a change in progress to work through the stdout/stderr issue | 16:58 |
| zbr | corvus: ok thanks. that is only for failed tasks but i already removed the coloring. | 16:58 |
| zbr | i am aware of that change from avass, i hope it works well as it is useful to separate the streams. | 16:59 |
| zbr | apparently that after last patch create-react-app refresh is working: https://review.opendev.org/#/c/716305 | 17:02 |
| *** jcapitao has quit IRC | 17:03 | |
| zbr | do we really need build result panel top container? It adds one extra nesting level to the UI. Removing it could simplify the interface and save few pixels on the width. | 17:07 |
| zbr | Example: https://zuul.opendev.org/t/openstack/build/603d9a4eb7ac4d26a56ba6ae2bf0c758 | 17:08 |
| *** jpena is now known as jpena|off | 17:14 | |
| avass | corvus: is the design of gertty made to look like irssi? because they look so good next to eachother | 17:14 |
| corvus | avass: they're right next to each other on my desktop :) | 17:16 |
| corvus | avass: a sort of mash of irssi, mutt, gnus.... | 17:16 |
| avass | corvus: I knew it! I'm using i3wm without borders and they fit so well together | 17:17 |
| clarkb | part of that is they both use your terminal color palette | 17:17 |
| clarkb | so naturaly share a theme | 17:17 |
| clarkb | https://review.opendev.org/#/c/723764/ is a minor gerrit connection thing I discovered when debugging zuul things over the weekend | 17:18 |
| clarkb | basically I don't think we should retry when gerrit returns http 409 | 17:19 |
| corvus | sounds good | 17:19 |
| avass | clarkb: sure but the title bar is lined up perfectly | 17:21 |
| clarkb | mordred: if you have a moment can you review https://review.opendev.org/724367 and its parent? I think we need those two changes and a release to fix the docs building issue we've got in the zuul.d reorg change in opendev | 17:42 |
| mordred | clarkb: done | 18:11 |
| openstackgerrit | Merged zuul/zuul-sphinx master: Handle directories in zuul.d/ configs https://review.opendev.org/724366 | 18:20 |
| openstackgerrit | Merged zuul/zuul-sphinx master: Sort dirs and filenames when os.walk()ing https://review.opendev.org/724367 | 18:20 |
| *** y2kenny has joined #zuul | 18:23 | |
| y2kenny | If I want to have bare metal nodes to support testing and these nodes need to be prepared and clean up in some way (such as installing driver, reboot, etc.) is the recommended path to do this in nodepool or as one of the pre tasks? | 18:25 |
| avass | y2kenny: you could probably do that as a pre-task | 18:27 |
| *** armstrongs has joined #zuul | 18:28 | |
| y2kenny | avass: so essentially just use nodepool to provision an hw node | 18:28 |
| openstackgerrit | Monty Taylor proposed zuul/zuul master: Install skopeo in container images https://review.opendev.org/724344 | 18:28 |
| avass | y2kenny: yeah, using the static driver. unless you want to write a custom driver for nodepool :) | 18:29 |
| openstackgerrit | Sorin Sbarnea proposed zuul/zuul master: Update to create-react-app 3.4.1 https://review.opendev.org/716305 | 18:29 |
| openstackgerrit | Sorin Sbarnea proposed zuul/zuul master: WIP: Enable ANSI rendering on stdout/stderr https://review.opendev.org/716251 | 18:29 |
| y2kenny | the pre-task sounds right to me intuitively since I can just use ansible to do the setup. But I was wondering if there are ways to pre-run the pre-task to shorten the turn around time. | 18:30 |
| avass | y2kenny: that sounds like a cool idea | 18:31 |
| y2kenny | avass: the same thing with post. For things like uploading log and artifacts, it is part of the job. But let say one of the post task is clean up/formatting the disk or something, then IMHO, it should be part of the nodepool life cycle instead of the job's lifecycle | 18:33 |
| clarkb | I think you can run a script via nodepool now | 18:34 |
| y2kenny | clarkb: oh I didn't know about that... I will take a look | 18:35 |
| avass | clarkb: you can? huh, Il have to take a look at that | 18:35 |
| clarkb | I might be thinking of https://zuul-ci.org/docs/nodepool/configuration.html#attr-providers.[openstack].post-upload-hook which is the wrong part of the workflow for this | 18:36 |
| clarkb | ya that must be it as I'm not finding anything else similar | 18:37 |
| *** armstrongs has quit IRC | 18:37 | |
| openstackgerrit | Monty Taylor proposed zuul/zuul-jobs master: Support multi-arch image builds with docker buildx https://review.opendev.org/722339 | 18:37 |
| openstackgerrit | Monty Taylor proposed zuul/zuul-jobs master: DNM Run builder tests on expanded node https://review.opendev.org/724079 | 18:37 |
| *** y2kenny has quit IRC | 18:38 | |
| *** y2kenny has joined #zuul | 18:40 | |
| y2kenny | do folks generally use the webchat freenode interface and do you guys get disconnected sometimes? | 18:41 |
| avass | y2kenny: i use irssi | 18:42 |
| avass | and no | 18:43 |
| y2kenny | avass: ok... I should probably look into that | 18:43 |
| clarkb | I use weechat which is basically irssi but with some different features | 18:44 |
| y2kenny | cool. freenode webchat is convinient but I get disconnection quick often. | 18:45 |
| y2kenny | quite* | 18:45 |
| clarkb | you can also hook up glowing bear to weechat if you prefer to have a browser client | 18:47 |
| clarkb | irccloud is a hosted service web client too | 18:47 |
| avass | y2kenny: regarding preparing nodes, you could always have a cleanup-run: playbook to do the cleanup ;) | 18:48 |
| y2kenny | avass: um... interesting. I did not know about that. I can probably leverage it. Thanks! | 18:52 |
| openstackgerrit | Sorin Sbarnea proposed zuul/zuul master: Add convenience Makefile https://review.opendev.org/723837 | 19:02 |
| zbr | clarkb: irccloud is cool, I am using it but quite dropping connections, 3-4 per day. | 19:09 |
| zbr | if any of you can review the pre-wrap at https://review.opendev.org/#/c/723603/ it would be really useful, i found a nice and clean way to address it both locations. | 19:20 |
| *** yolanda has quit IRC | 19:36 | |
| *** hashar has joined #zuul | 19:43 | |
| openstackgerrit | Monty Taylor proposed zuul/zuul-jobs master: Support multi-arch image builds with docker buildx https://review.opendev.org/722339 | 19:45 |
| openstackgerrit | Monty Taylor proposed zuul/zuul-jobs master: DNM Run builder tests on expanded node https://review.opendev.org/724079 | 19:45 |
| *** avass has quit IRC | 20:10 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Add git name and email for quickstart executor https://review.opendev.org/724096 | 20:26 |
| openstackgerrit | Monty Taylor proposed zuul/zuul master: Use libffi7 on ubuntu focal https://review.opendev.org/724417 | 20:30 |
| mordred | corvus, clarkb: ^^ https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_5eb/723528/18/check/system-config-run-zuul/5ebfe50/bridge.openstack.org/ara-report/result/9e53d6c7-0bad-4133-8920-bfa01edee3b6/ | 20:31 |
| mordred | clarkb, corvus: the "run executor on focal" job hit that | 20:31 |
| clarkb | mordred: check my note on the ffi change I don't think its worth a -1 but wanted to have you see it before I approve | 20:33 |
| mordred | clarkb: oh - I think that's better | 20:36 |
| mordred | let me do that | 20:36 |
| *** y2kenny has quit IRC | 20:36 | |
| openstackgerrit | Monty Taylor proposed zuul/zuul master: Use libffi7 on ubuntu focal https://review.opendev.org/724417 | 20:37 |
| mordred | clarkb: I think that ^^ | 20:37 |
| mordred | corvus: ^^ | 20:37 |
| tristanC | mordred: may i ask what libffi7 is used for? | 20:55 |
| clarkb | tristanC: cryptography iirc | 20:56 |
| mordred | yeah. I thnik it's cryptography | 21:00 |
| mordred | same thing as libffi6 - just a different ABI in focal | 21:00 |
| clarkb | corvus: I think we need a zuul-sphinx release to consume those changes I made | 21:03 |
| *** guillaumec has quit IRC | 21:04 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Add git name and email for quickstart executor https://review.opendev.org/724096 | 21:04 |
| corvus | clarkb: i will make it so | 21:06 |
| corvus | clarkb: commit 264a9fca1cfb79826f2c17f6cb2cca83b1317348 (HEAD -> master, tag: 0.5.0, origin/master, origin/HEAD, refs/changes/67/724367/1) | 21:07 |
| corvus | clarkb: look good? | 21:07 |
| clarkb | looking | 21:11 |
| clarkb | yup lgtm | 21:12 |
| corvus | pushed | 21:12 |
| *** hashar has quit IRC | 21:35 | |
| openstackgerrit | Merged zuul/zuul master: Use libffi7 on ubuntu focal https://review.opendev.org/724417 | 21:39 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Add serial pipeline manager https://review.opendev.org/722981 | 21:59 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Make fake test Gerrit merger more realistic https://review.opendev.org/722982 | 22:00 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Add git name and email for quickstart executor https://review.opendev.org/724096 | 22:00 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Add more docs about pipelines https://review.opendev.org/724185 | 22:00 |
| corvus | mordred, zbr: i don't understand the commit message on https://review.opendev.org/723855 | 22:02 |
| corvus | what is the conflict with cheroot? | 22:03 |
| corvus | when tristanC asked about that, zbr just said we couldn't afford the latest version. i don't know what that means | 22:03 |
| tristanC | corvus: i don't know what that means either, it seems like both libraries install conflicting dependencies, but that doesn't tell what is the consequence for zuul itself. | 22:05 |
| mordred | tristanC, corvus : I just responded to that in #opendev ... | 22:05 |
| corvus | i'll copy that response to the change | 22:06 |
| mordred | cool | 22:06 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Fix cherrypy dependency conflict https://review.opendev.org/723855 | 22:07 |
| mordred | ++ | 22:08 |
| corvus | and i added a comment about the memory leak | 22:09 |
| corvus | we're seeing zuul-web use a lot of ram in opendev; i think pinning to an old cherrypy will help us eliminate cherrypy v18.6.0 which released recently as a possible factor | 22:10 |
| mordred | ++ | 22:12 |
| mordred | which is a good outcome in either direction | 22:13 |
| tristanC | thanks for the detail | 22:13 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Add git name and email for quickstart executor https://review.opendev.org/724096 | 22:40 |
| openstackgerrit | Merged zuul/zuul master: Fix cherrypy dependency conflict https://review.opendev.org/723855 | 23:04 |
| *** tosky has quit IRC | 23:11 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Add git name and email for quickstart executor https://review.opendev.org/724096 | 23:27 |
| *** zxiiro has joined #zuul | 23:30 | |
| jkt | is it worth checking the "port" quota in nodepool? We're using floating IPv4 addresses on our internal OpenStack cluster, and that needs 2 (for a router) + 2 * instances (one for private ipv4, other for floating ipv4) ports in total, and our config was "funny" in that I had 10 instances, but just 15 ports) | 23:31 |
| clarkb | jkt: I don't think floating IPs count against port quota | 23:37 |
| clarkb | jkt: floating IPs are just NAT rules, but yes figure at least one port per instance interface | 23:37 |
| *** rlandy has quit IRC | 23:51 | |
| mnaser | hi all | 23:56 |
| mnaser | hate to be the bearer of bad news again but we have broke zuul/zuul-jobs again :( | 23:57 |
| mnaser | https://opendev.org/zuul/zuul-jobs/commit/d0e2016592b7027c950d14a5d78c15b0719a8444 has broken promote docker image | 23:57 |
| mnaser | mainly it has renamed the variable when including a loop to zj_image => https://opendev.org/zuul/zuul-jobs/src/commit/d0e2016592b7027c950d14a5d78c15b0719a8444/roles/promote-docker-image/tasks/main.yaml#L26 | 23:57 |
| mnaser | but then it missed it in whre it was included => https://opendev.org/zuul/zuul-jobs/src/commit/d0e2016592b7027c950d14a5d78c15b0719a8444/roles/promote-docker-image/tasks/promote-retag.yaml | 23:58 |
| mnaser | and i think it missed it here too https://opendev.org/zuul/zuul-jobs/src/commit/d0e2016592b7027c950d14a5d78c15b0719a8444/roles/promote-docker-image/tasks/promote-retag-inner.yaml | 23:58 |
| clarkb | mnaser: thats probably what broke promote for zuul-web | 23:58 |
| mnaser | perhaps! let me check | 23:59 |
| mnaser | clarkb: https://zuul.opendev.org/t/zuul/builds?pipeline=promote indeed | 23:59 |
| jkt | clarkb: if that's true, then I cannot explain why I was over quota | 23:59 |
| mnaser | let me see if i can work up a patch | 23:59 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!