| @clarkb:matrix.org | I believe ttx has a couple of talks there | 00:09 |
|---|---|---|
| @chgans:matrix.org | Hi there, what are you using to connect Gerrit/Zull with matrix/element? Some custom code or is there any official plugins? (that i couldn't find) | 00:15 |
| @clarkb:matrix.org | > <@chgans:matrix.org> Hi there, what are you using to connect Gerrit/Zull with matrix/element? Some custom code or is there any official plugins? (that i couldn't find) | 00:16 |
| Its the gerritbox-matrix bot that tristanC wrote. Let me see if I can find a link to it | ||
| @clarkb:matrix.org | https://softwarefactory-project.io/r/plugins/gitiles/software-factory/gerritbot-matrix.git/ I'm not sure that is the best link but should be cloneable from there | 00:17 |
| @chgans:matrix.org | awesome, i'll have a look, thanks! | 00:18 |
| @chgans:matrix.org | wow, haskell 🤔 the documentation is sparse... :P | 00:20 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 872482: Fix race condition in pipeline change list init https://review.opendev.org/c/zuul/zuul/+/872482 | 00:33 | |
| @clarkb:matrix.org | > <@chgans:matrix.org> wow, haskell 🤔 the documentation is sparse... :P | 00:35 |
| This is opendev's ansible role to manage our deployment of it (the one running for this room) that might help as far as knowing what config to set etc https://opendev.org/opendev/system-config/src/branch/master/playbooks/roles/matrix-gerritbot | ||
| -@gerrit:opendev.org- Ian Wienand proposed: | 00:54 | |
| - [zuul/zuul-jobs] 872365: zuul-jobs-test-registry-docker-* : update to jammy nodes https://review.opendev.org/c/zuul/zuul-jobs/+/872365 | ||
| - [zuul/zuul-jobs] 872258: build-docker-image: fix change prefix https://review.opendev.org/c/zuul/zuul-jobs/+/872258 | ||
| - [zuul/zuul-jobs] 872375: container-roles-jobs: Update tests to jammy nodes https://review.opendev.org/c/zuul/zuul-jobs/+/872375 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 872485: Simplufy shouldRefresh* checks https://review.opendev.org/c/zuul/zuul/+/872485 | 01:37 | |
| @jim:acmegating.com | ianw: is that stack ready? (i've been waiting for it to settle down before taking a look) | 01:39 |
| @iwienand:matrix.org | corvus: haha it was, but ansible-lint has released and broken the gate. i'm having a look now | 01:39 |
| @jim:acmegating.com | ok. i'll leave you to it. consider me interested when it's ready :) | 01:40 |
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 871106: Require latest layout for processing mgmt events https://review.opendev.org/c/zuul/zuul/+/871106 | 01:45 | |
| -@gerrit:opendev.org- Ian Wienand proposed: | 04:08 | |
| - [zuul/zuul-jobs] 872371: linters-requirements : update Ansible to 2.12 https://review.opendev.org/c/zuul/zuul-jobs/+/872371 | ||
| - [zuul/zuul-jobs] 872365: zuul-jobs-test-registry-docker-* : update to jammy nodes https://review.opendev.org/c/zuul/zuul-jobs/+/872365 | ||
| - [zuul/zuul-jobs] 872258: build-docker-image: fix change prefix https://review.opendev.org/c/zuul/zuul-jobs/+/872258 | ||
| - [zuul/zuul-jobs] 872375: container-roles-jobs: Update tests to jammy nodes https://review.opendev.org/c/zuul/zuul-jobs/+/872375 | ||
| - [zuul/zuul-jobs] 872489: ansible-lint: Ignore some command-instead-of-module warnings https://review.opendev.org/c/zuul/zuul-jobs/+/872489 | ||
| - [zuul/zuul-jobs] 872490: ansible-lint: fix a bunch of command-instead-of-shell errors https://review.opendev.org/c/zuul/zuul-jobs/+/872490 | ||
| - [zuul/zuul-jobs] 872491: ansible-lint: add names to blocks/includes, etc. https://review.opendev.org/c/zuul/zuul-jobs/+/872491 | ||
| - [zuul/zuul-jobs] 872492: ansible-lint: ignore use of mkdir https://review.opendev.org/c/zuul/zuul-jobs/+/872492 | ||
| - [zuul/zuul-jobs] 872493: ansible-lint: use pipefail https://review.opendev.org/c/zuul/zuul-jobs/+/872493 | ||
| - [zuul/zuul-jobs] 872494: ansible-lint: ignore latest git pull https://review.opendev.org/c/zuul/zuul-jobs/+/872494 | ||
| - [zuul/zuul-jobs] 872495: ansible-lint: uncap https://review.opendev.org/c/zuul/zuul-jobs/+/872495 | ||
| -@gerrit:opendev.org- Ian Wienand proposed: | 04:09 | |
| - [zuul/zuul-jobs] 872371: linters-requirements : update Ansible to 2.12, ansible-lint <6.12.0 https://review.opendev.org/c/zuul/zuul-jobs/+/872371 | ||
| - [zuul/zuul-jobs] 872489: ansible-lint: Ignore some command-instead-of-module warnings https://review.opendev.org/c/zuul/zuul-jobs/+/872489 | ||
| - [zuul/zuul-jobs] 872490: ansible-lint: fix a bunch of command-instead-of-shell errors https://review.opendev.org/c/zuul/zuul-jobs/+/872490 | ||
| - [zuul/zuul-jobs] 872491: ansible-lint: add names to blocks/includes, etc. https://review.opendev.org/c/zuul/zuul-jobs/+/872491 | ||
| - [zuul/zuul-jobs] 872492: ansible-lint: ignore use of mkdir https://review.opendev.org/c/zuul/zuul-jobs/+/872492 | ||
| - [zuul/zuul-jobs] 872493: ansible-lint: use pipefail https://review.opendev.org/c/zuul/zuul-jobs/+/872493 | ||
| - [zuul/zuul-jobs] 872494: ansible-lint: ignore latest git pull https://review.opendev.org/c/zuul/zuul-jobs/+/872494 | ||
| - [zuul/zuul-jobs] 872495: ansible-lint: uncap https://review.opendev.org/c/zuul/zuul-jobs/+/872495 | ||
| - [zuul/zuul-jobs] 872365: zuul-jobs-test-registry-docker-* : update to jammy nodes https://review.opendev.org/c/zuul/zuul-jobs/+/872365 | ||
| - [zuul/zuul-jobs] 872258: build-docker-image: fix change prefix https://review.opendev.org/c/zuul/zuul-jobs/+/872258 | ||
| - [zuul/zuul-jobs] 872375: container-roles-jobs: Update tests to jammy nodes https://review.opendev.org/c/zuul/zuul-jobs/+/872375 | ||
| @iwienand:matrix.org | of course i'm sure that will touch a bunch of jobs that have now bitrotted ... anyway, fingers crossed | 04:10 |
| @avass:vassast.org | Apparently curl has stopped using zuul for their ci now: https://daniel.haxx.se/blog/2023/02/01/curls-use-of-many-ci-services/ | 07:14 |
| @q:fricklercloud.de | > <@avass:vassast.org> Apparently curl has stopped using zuul for their ci now: https://daniel.haxx.se/blog/2023/02/01/curls-use-of-many-ci-services/ | 09:11 |
| and with good reasoning https://mastodon.social/@bagder/109782783782628905 | ||
| @avass:vassast.org | Yep. I'm also not a huge fan of the GitHub interface for zuul (like he says all jobs show up as the same job in GitHub). And i tried to help out when they had issues but i didn't have access to the system they were using so i couldn't actually debug or fix anything. | 09:15 |
| @avass:vassast.org | They also weren't really using any zuul features since they didn't want to be locked in to the system | 09:26 |
| @mhuin:matrix.org | > <@jim:acmegating.com> ttx mhu sometimes go; pinging them for input; i won't be able to attend this year :( | 09:29 |
| Sorry, not this year ... But I do make sure to advertise the project as much as possible: https://i.postimg.cc/zfZ8wBV7/20230202-102430.jpg | ||
| @mhuin:matrix.org | > <@chgans:matrix.org> wow, haskell 🤔 the documentation is sparse... :P | 09:32 |
| it should be relatively straightforward to use but if you need some help you can ask here https://matrix.to/#/#softwarefactory-project:matrix.org | ||
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 872519: Fix serialization of ProjectContext objects https://review.opendev.org/c/zuul/zuul/+/872519 | 09:50 | |
| @tcarrez:matrix.org | I'll be at FOSDEM... but not doing a Zuul talk. I haven't seen any Zuul content | 10:15 |
| @mhuin:matrix.org | The CI track is not as loaded as in the previous years | 10:25 |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 810955: GUI: Add tenant dropdown to top menu https://review.opendev.org/c/zuul/zuul/+/810955 | 10:48 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 794854: Test zuul-client console-stream https://review.opendev.org/c/zuul/zuul/+/794854 | 10:49 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 771962: zuul-client: test "show running-jobs" command https://review.opendev.org/c/zuul/zuul/+/771962 | 10:50 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 872552: Keycloak tutorial: update to keycloak 20, add CLI example https://review.opendev.org/c/zuul/zuul/+/872552 | 12:21 | |
| @westphahl:matrix.org | Clark: replied to your comment on https://review.opendev.org/c/zuul/zuul/+/871108/ | 13:23 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 868131: Track object versions in the Buildset object https://review.opendev.org/c/zuul/zuul/+/868131 | 14:06 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 810955: GUI: Add tenant dropdown to top menu https://review.opendev.org/c/zuul/zuul/+/810955 | 14:41 | |
| @jim:acmegating.com | It looks like curl was using a very old zuul run by a commercial service. it seems a little disingenuous to say that the "zuul team" didn't respond to their issues in that situation. but i certainly understand their decision. | 14:45 |
| @buktop:matrix.org | Hi all, | 14:49 |
| We have some proposed changes for extending the openshift nodepool drivers and we would appreciate if someone can review them. | ||
| - https://review.opendev.org/c/zuul/nodepool/+/871060 | ||
| - https://review.opendev.org/c/zuul/nodepool/+/867971 | ||
| - https://review.opendev.org/c/zuul/nodepool/+/872318 | ||
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 842290: Remove zuul CLI, REST ops https://review.opendev.org/c/zuul/zuul/+/842290 | 15:23 | |
| @clarkb:matrix.org | swest: corvus the comments on those pipeline event cleanup changes make sense to me. Feel free to approve. Or I'll hit the +A button in a few minutes after distro updates reboots and logging back into gerrit | 16:14 |
| @mhuin:matrix.org | dunno if it's just me being unlucky but it seems like the zuul-build-image has been failing for the last 4 hours, RETRY_LIMITS hit for 3 very different patches (one on doc, one on GUI, one on client) | 16:39 |
| @jim:acmegating.com | mhu if it's failing on different jobs, then yeah, it's likely unluck. possibly hitting docker rate limits on re-used ip addresses. | 16:40 |
| @mhuin:matrix.org | last error is "AppArmor enabled on system but the docker-default profile could not be loaded" - is that new? | 16:40 |
| @jim:acmegating.com | oh that's different | 16:40 |
| @jim:acmegating.com | mhu: there's some talk in #opendev about that | 16:40 |
| @clarkb:matrix.org | yes, docker-ce 23.0.0 released a few hours ago and depends on the apparmor package but does not install it | 16:40 |
| @clarkb:matrix.org | I think we can/should update the role installing docker-ce to also explicitly install apparmor and that should hopefully fix things | 16:41 |
| @jim:acmegating.com | so the randomness might be due to different package indexes | 16:41 |
| @mhuin:matrix.org | there you go - thanks corvus and Clark | 16:41 |
| @jim:acmegating.com | Clark: ++ | 16:41 |
| @clarkb:matrix.org | mhu: are you interested in writing that change? If not can you link to one of the job logs so that we can be sure we update the correct location | 16:42 |
| @mhuin:matrix.org | Clark, if you know which role to update, go for it, as it would take me some time to pinpoint it | 16:43 |
| @clarkb:matrix.org | I think it is he ensure-docker role. Let me take a look | 16:44 |
| @mhuin:matrix.org | Clark: these are the RETRY_LIMITs I was impacted with https://zuul.opendev.org/t/zuul/builds?job_name=zuul-build-image&project=zuul%2Fzuul&result=RETRY_LIMIT&skip=0&limit=3 | 16:44 |
| @mhuin:matrix.org | "Task | 16:45 |
| Start the buildset registry | ||
| failed running on host | ||
| ubuntu-jammy" | ||
| @mhuin:matrix.org | is that a haiku? | 16:45 |
| @mhuin:matrix.org | (I just copy/pasted the error from one of the failed builds' status page) | 16:45 |
| @fungicide:matrix.org | a terrifying haiku | 16:47 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 872569: Install apparmor when installing upstream docker-ce https://review.opendev.org/c/zuul/zuul-jobs/+/872569 | 16:48 | |
| @mhuin:matrix.org | Clark: ++ | 16:48 |
| @clarkb:matrix.org | That may do it. I put a note in the commit message that opensuse may need a similar fix too and CI should tell us if we run ensure-docker on opensuse | 16:48 |
| @avass:vassast.org | > <@jim:acmegating.com> It looks like curl was using a very old zuul run by a commercial service. it seems a little disingenuous to say that the "zuul team" didn't respond to their issues in that situation. but i certainly understand their decision. | 16:55 |
| Yeah vexxhost was hosting it for them | ||
| @clarkb:matrix.org | Looks like `docker buildx create --name foo` is no longer valid under docker 23.0.0. I'm not yet sure what the fix for that is. The docs haven't been udpated for new docker yet... | 17:04 |
| @jim:acmegating.com | Clark: is that causing that change to fail? should we be pinning instead? | 17:07 |
| @clarkb:matrix.org | corvus: yes, in the test job that tests multiarch builds (these depend on buildx to set up the emulated build env) | 17:10 |
| @clarkb:matrix.org | I put a hold on the zuul-jobs-test-registry-docker job so that I can interact with it directly a bit more | 17:16 |
| @clarkb:matrix.org | I'm beginning to suspect that 23 doesn't include buildx in the packages like 20 did. Trying to confirm this now | 17:22 |
| @clarkb:matrix.org | ya there is a new separate package | 17:22 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 872569: Fix ensure-docker for docker-ce 23.0.0 upstream packaging https://review.opendev.org/c/zuul/zuul-jobs/+/872569 | 17:27 | |
| @clarkb:matrix.org | Ok this should hopefully get us closer. THe registry build test jobs use ensure-docker out of a trusted context which means we may need to force merge this... | 17:28 |
| @clarkb:matrix.org | yup first multiarch job has passed. I need to eat breakfast, but other zuulians (and opendev admins) should think about how we want ot handle the base-job usage of the roll. I think whether we pin or roll forward a force merge is likely | 17:35 |
| @jim:acmegating.com | Clark: thanks; i think force-merge roll-forward sounds good -- especially since we're already broken. | 17:38 |
| @fungicide:matrix.org | i've +2'd the new revision, but didn't approve it because i didn't want to cancel the check builds and possibly slow discovery of yet another problem with it | 17:38 |
| @jim:acmegating.com | if we force-merge, it might be good to push up a change that runs all the registry test jobs afterwords to double check it worked as expected. | 17:39 |
| @clarkb:matrix.org | ya a noop change to ensure-docker would do it I think | 17:44 |
| @clarkb:matrix.org | zuul-jobs-test-ensure-kubernetes-crio-ubuntu-bionic is also failing. I don't really grasp the error message | 17:45 |
| @clarkb:matrix.org | if related I don't think that one will be any worse off with the force merge though | 17:46 |
| @jim:acmegating.com | Clark: it started failing ~1 week ago | 17:49 |
| @clarkb:matrix.org | also the linters job fails but ianw has a stack to fix that and I believe it to be independent too | 17:49 |
| @jim:acmegating.com | so likely unrelated, but also likely another real problem. i believe ianw felt the crio path for that had a limited lifespan anyway. it's conceivable that the fix for that is to remove the job. | 17:50 |
| @jim:acmegating.com | * the crio error is likely unrelated, but also likely another real problem. i believe ianw felt the crio path for that had a limited lifespan anyway. it's conceivable that the fix for that is to remove the job. | 17:50 |
| @clarkb:matrix.org | cool. In that case I'm not seeing anything yet that says "force merging this is a terrible idea | 17:51 |
| @jim:acmegating.com | Clark: yeah i think maybe we've seen all the green we need/expect to see at this point | 17:51 |
| @clarkb:matrix.org | corvus: tox-py38 is the last one I expect to be green. Maybe wait on it? | 17:51 |
| @jim:acmegating.com | Clark: yep. i'm just assuming that the one in the middle isn't going to fail for a legit reason. hopefully that doesn't jinx it. | 17:55 |
| @jim:acmegating.com | (also nothing in this change should touch that) | 17:55 |
| @clarkb:matrix.org | ya good point. Looks like it is finishing up and should be successful though. Who wants to push the buttons? SHould I? | 17:55 |
| @jim:acmegating.com | Clark: i vote you :) | 17:56 |
| @clarkb:matrix.org | ok working on it. Will take a few minutes while I sort out gerrit perms and stuff | 17:56 |
| @fungicide:matrix.org | i support bypassing zuul to merge this change | 17:56 |
| @fungicide:matrix.org | thanks! | 17:56 |
| -@gerrit:opendev.org- cboylan.admin merged on behalf of Clark Boylan: [zuul/zuul-jobs] 872569: Fix ensure-docker for docker-ce 23.0.0 upstream packaging https://review.opendev.org/c/zuul/zuul-jobs/+/872569 | 18:00 | |
| @clarkb:matrix.org | Done | 18:00 |
| @clarkb:matrix.org | I'll push up a noop chagne to ensure-docker to check that the buildset registry jobs are happy now | 18:01 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 872594: DNM double checking docker 23 fixes https://review.opendev.org/c/zuul/zuul-jobs/+/872594 | 18:02 | |
| @clarkb:matrix.org | the promote jobs for the force merged change failed but that is due to no artifact being available. I believe this is expected | 18:04 |
| @clarkb:matrix.org | https://github.com/moby/moby/issues/44900 upstream is aware of the apparmor issue | 18:05 |
| @clarkb:matrix.org | we might be able to rollback apparmor installs, but also I think its fine if we don't | 18:05 |
| @clarkb:matrix.org | * we might be able to eventually rollback apparmor installs, but also I think its fine if we don't | 18:06 |
| @clarkb:matrix.org | Once I'm happy with the DNM sanity check results I guess I should review ianw's linterfixes | 18:09 |
| @clarkb:matrix.org | the buildset registry job is paused now. It wasn't able to get that far before | 18:12 |
| @clarkb:matrix.org | ok all jobs that failed due to base-jobs use of ensure-docker have passed now. mhu you should be good to recheck things | 18:26 |
| @clarkb:matrix.org | ianw: I've reviewed your stack. One change converts shell to command when using pipes so I -1'd it and then another updates the docker image tag value and I had a questions about the pipeline version of that. And I rechecked a few changes to get results back after fixing docker 23 stuff | 18:44 |
| @clarkb:matrix.org | ianw: hrm it looks like we're hitting the unsupported mime type issue in some of those jobs. I'm guessing we need to install an up to date skopeo in those contexts too (I think because we're emulating the executor in those tests to shuffle data around?) | 19:15 |
| @iwienand:matrix.org | Clark: thanks, i'll loop back on it soon | 20:11 |
| -@gerrit:opendev.org- Ian Wienand proposed: | 20:31 | |
| - [zuul/zuul-jobs] 872490: ansible-lint: fix a bunch of command-instead-of-shell errors https://review.opendev.org/c/zuul/zuul-jobs/+/872490 | ||
| - [zuul/zuul-jobs] 872491: ansible-lint: add names to blocks/includes, etc. https://review.opendev.org/c/zuul/zuul-jobs/+/872491 | ||
| - [zuul/zuul-jobs] 872492: ansible-lint: ignore use of mkdir https://review.opendev.org/c/zuul/zuul-jobs/+/872492 | ||
| - [zuul/zuul-jobs] 872493: ansible-lint: use pipefail https://review.opendev.org/c/zuul/zuul-jobs/+/872493 | ||
| - [zuul/zuul-jobs] 872494: ansible-lint: ignore latest git pull https://review.opendev.org/c/zuul/zuul-jobs/+/872494 | ||
| - [zuul/zuul-jobs] 872495: ansible-lint: uncap https://review.opendev.org/c/zuul/zuul-jobs/+/872495 | ||
| - [zuul/zuul-jobs] 872365: zuul-jobs-test-registry-docker-* : update to jammy nodes https://review.opendev.org/c/zuul/zuul-jobs/+/872365 | ||
| - [zuul/zuul-jobs] 872258: build-docker-image: fix change prefix https://review.opendev.org/c/zuul/zuul-jobs/+/872258 | ||
| - [zuul/zuul-jobs] 872375: container-roles-jobs: Update tests to jammy nodes https://review.opendev.org/c/zuul/zuul-jobs/+/872375 | ||
| @jkt_:matrix.org | so I just got biten in my rear by a typo in my commit message, I said `Depend-on` instead of `Depends-on`. What would be the easiest way of writing a super-simple job which lints the commit message? I guess I still need a VM for that, and that I cannot "just" run something on the executor, right? | 22:27 |
| @jkt_:matrix.org | I don't feel like writing a gerrit plugin for this | 22:30 |
| @clarkb:matrix.org | Jan Kundrát: it should be doable on the executor | 22:33 |
| @clarkb:matrix.org | and the origin remote should be configured to show you the merged state and the normal local branches are the speculative state so you can look at commit messages between those two points | 22:33 |
| @clarkb:matrix.org | One thing to keep in mind generally when linting commit messages is that many users don't run linters after commiting. They run them first, things go green then commit. In this case its probably fine though since its about establishing relationships between changes | 22:34 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 872482: Fix race condition in pipeline change list init https://review.opendev.org/c/zuul/zuul/+/872482 | 22:35 | |
| @clarkb:matrix.org | fungi: (or anyone else) I'm looking at startinga patch to kazoo to fix this SSLv23 deprecation warning. It looks like the things we are supposed to use is PROTOCOL_TLS_CLIENT but that enables hostname checking. We connect to our servers via IP addrs and I don't think our certs have IP addrs as altnames. Does this imply that python is basically saying you should not do ssl to something with ip addrs. Or I guess we must stick IP addrs in the altnames? | 22:53 |
| @clarkb:matrix.org | In particular kazoo is a library and if opendev's zuul has these concerns I want to be extra careful and not suddenly require everyone to change their certificate setups | 22:53 |
| @clarkb:matrix.org | Instead of writing a small patch I might need to start with an issue to begin a conversation with upstream | 22:55 |
| @clarkb:matrix.org | The other zuul warning appears to come from the garbage collector warning about network connections not being explicitly closed. I'm not sure what to do about those yet as I think our manual running of the garbage collector to look for leaked resources trips this ahead of when we might close things? This one is definitely going to be a puzzle. | 22:56 |
| @clarkb:matrix.org | hrm one option for the TLS thing may be to explicitly disable hostname checking | 22:59 |
| @clarkb:matrix.org | and make a note that maybe one day this should change | 22:59 |
| @clarkb:matrix.org | hrm any idea why our CI jobs get those extra warnings but when I run tests locally I don't get them? | 23:24 |
| @clarkb:matrix.org | making it hard to determine if the patch I've written to kazoo helps or not | 23:24 |
| @jim:acmegating.com | Clark: old local venv? | 23:27 |
| @clarkb:matrix.org | no I ran with `nox -N` | 23:27 |
| @clarkb:matrix.org | I'm wondering if it has to do with how python is configured on the systems | 23:28 |
| @clarkb:matrix.org | school run time. I think this shifted from fixing kazoo to understanding why we get warnings in CI better | 23:29 |
| @clarkb:matrix.org | will have to dig more later | 23:29 |
| @jim:acmegating.com | Clark: you talking about `DeprecationWarning: ssl.PROTOCOL_TLS is deprecated`? | 23:29 |
| @clarkb:matrix.org | yes | 23:30 |
| @jim:acmegating.com | (if so, i get those locally on ubuntu) | 23:30 |
| @clarkb:matrix.org | https://paste.opendev.org/show/b9u5dSEucSKT5ef5Ngpc/ is the patch I'm working with against kazoo 2.9.0 | 23:30 |
| @clarkb:matrix.org | ya so maybe its a suse python vs ubuntu python thing | 23:30 |
| @iwienand:matrix.org | corvus: would you have a sec for the bottom 5 changes of https://review.opendev.org/q/topic:ansible-lint-req-2.12 ... the rest are going to need some squashing and reordering to deal with the registry/docker/to be debugged changes on top, but the bottom ones are ok i think so it would help my rebasing sanity | 23:37 |
| @jim:acmegating.com | Clark: that patch appears to resolve the problem in my environment | 23:38 |
| @jim:acmegating.com | ianw: does "bottom" mean "first to land"? | 23:39 |
| @iwienand:matrix.org | corvus: ahh yep; https://review.opendev.org/c/zuul/zuul-jobs/+/872371 | 23:39 |
| @iwienand:matrix.org | up, or down, until https://review.opendev.org/c/zuul/zuul-jobs/+/872492/3 | 23:41 |
| @iwienand:matrix.org | i guess depending if you're antipodean or not :) | 23:41 |
| @jim:acmegating.com | ianw: i knew there was a joke like that in there somewhere. ;) i just opened the stack in gertty and it puts first-to-land at the top and i had a moment of ...dizzyness? :) | 23:43 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
