| @jim:acmegating.com | Michael Kelly: I would... use caution? it's tended to get updated in bursts recently. | 00:01 |
|---|---|---|
| @michael_kelly_anet:matrix.org | Sounds good. :) | 00:39 |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/nodepool] 853574: functional-container: use different root-disk label https://review.opendev.org/c/zuul/nodepool/+/853574 | 01:21 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 851942: docs: update console streaming docs https://review.opendev.org/c/zuul/zuul/+/851942 | 01:25 | |
| @iwienand:matrix.org | is there anything we can do about | 01:28 |
| @iwienand:matrix.org | ```ImportError: libre2.so.0a: cannot open shared object file: No such file or directory')``` | 01:28 |
| @iwienand:matrix.org | constantly running tox-ish anything on fedora. every time i remove a tox environment i have to work around it | 01:29 |
| @iwienand:matrix.org | https://meetings.opendev.org/irclogs/%23zuul/%23zuul.2020-03-13.log.html#t2020-03-13T00:23:31 | 01:31 |
| @iwienand:matrix.org | i guess it's not a new problem | 01:32 |
| @iwienand:matrix.org | ```$ ldd /home/iwienand/programs/zuul/zuul/.tox/docs/lib64/python3.10/site-packages/_re2.cpython-310-x86_64-linux-gnu.so | 01:34 |
| libre2.so.0a => not found``` | ||
| @iwienand:matrix.org | ```ldd /home/iwienand/programs/zuul/zuul/.tox/docs/lib64/python3.10/site-packages/_re2.cpython-310-x86_64-linux-gnu.so``` | 01:34 |
| @iwienand:matrix.org | ... where did this .so come from, and how did it link against libre2.so.0a is i guess the fundamental question | 01:35 |
| @iwienand:matrix.org | https://pypi.org/project/re2/#files doesn't publish wheels | 01:36 |
| @iwienand:matrix.org | ```ldd /usr/lib64/python3.10/site-packages/_re2.cpython-310-x86_64-linux-gnu.so``` is the system version which is right | 01:37 |
| @iwienand:matrix.org | oh hang on, our requirement is https://pypi.org/project/fb-re2/#description | 01:40 |
| @iwienand:matrix.org | https://pypi.org/project/google-re2/ appears the be the maintained version of this? | 01:41 |
| @clarkb:matrix.org | I've had to delete cached built wheels for that dep when my system updated python and/or libre2 | 01:44 |
| @iwienand:matrix.org | yeah, that explains it | 01:48 |
| @iwienand:matrix.org | i guess it just comes and goes as various updates happen | 01:48 |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853579: test-setup-docker: wrap call in env https://review.opendev.org/c/zuul/zuul/+/853579 | 03:17 | |
| @iwienand:matrix.org | ``` at /home/iwienand/programs/zuul/zuul/web/node_modules/react-scripts/node_modules/babel-loader/lib/index.js:59:103 { | 03:25 |
| opensslErrorStack: [ 'error:03000086:digital envelope routines::initialization error' ], | ||
| library: 'digital envelope routines', | ||
| reason: 'unsupported', | ||
| code: 'ERR_OSSL_EVP_UNSUPPORTED' | ||
| } | ||
| ``` | ||
| @iwienand:matrix.org | wtf? this is trying to run tox | 03:25 |
| @iwienand:matrix.org | when it's trying to install test-requirements.txt | 03:27 |
| @iwienand:matrix.org | it's tools/pip.sh that does it | 03:35 |
| @michael_kelly_anet:matrix.org | Does the opendev.org gerrit instance support SSH pushing? I generated an HTTP password for HTTP pushing but it doesn't seem to be happy about this | 03:43 |
| @iwienand:matrix.org | "This has been resolved in Webpack versions 5 and 6, but there are no plans to backport this fix to version 4." | 03:44 |
| @iwienand:matrix.org | > <@michael_kelly_anet:matrix.org> Does the opendev.org gerrit instance support SSH pushing? I generated an HTTP password for HTTP pushing but it doesn't seem to be happy about this | 03:45 |
| yes; the easiest way is probably to use "git review" and see the .gitreview file in projects root | ||
| @michael_kelly_anet:matrix.org | Ah, actually, looks like I'm supposed to be talking to *review.opendev.org* - this is likely the source of my problems... | 03:46 |
| -@gerrit:opendev.org- Michael Kelly proposed: [zuul/zuul-operator] 853581: Convert base64 encoded value to string https://review.opendev.org/c/zuul/zuul-operator/+/853581 | 03:48 | |
| @michael_kelly_anet:matrix.org | Crap, accidentally added my WIP :P | 03:49 |
| -@gerrit:opendev.org- Michael Kelly proposed: [zuul/zuul-operator] 853582: Convert base64 encoded value to string https://review.opendev.org/c/zuul/zuul-operator/+/853582 | 03:50 | |
| @iwienand:matrix.org | reach can't work with webpack 5 -> https://github.com/facebook/create-react-app/issues/11708 | 03:53 |
| @iwienand:matrix.org | everyone says this is node 17 issue, but afaics i'm using : Welcome to Node.js v16.14.0. | 03:56 |
| @iwienand:matrix.org | i've uninstalled the system node packages, and installed the lts version of npm with nvm and it seems to be working.. | 04:11 |
| -@gerrit:opendev.org- Michael Kelly proposed: [zuul/zuul-operator] 853585: kopf: Upgrade requirement to 1.35.6 https://review.opendev.org/c/zuul/zuul-operator/+/853585 | 04:17 | |
| -@gerrit:opendev.org- Michael Kelly proposed: [zuul/zuul-operator] 853586: Only listen for updates to known secrets https://review.opendev.org/c/zuul/zuul-operator/+/853586 | 04:30 | |
| -@gerrit:opendev.org- Michael Kelly proposed: | 04:33 | |
| - [zuul/zuul-operator] 853585: kopf: Upgrade requirement to 1.35.6 https://review.opendev.org/c/zuul/zuul-operator/+/853585 | ||
| - [zuul/zuul-operator] 853586: Only listen for updates to known secrets https://review.opendev.org/c/zuul/zuul-operator/+/853586 | ||
| -@gerrit:opendev.org- Michael Kelly proposed: | 06:36 | |
| - [zuul/zuul-operator] 853586: Only listen for updates to known secrets https://review.opendev.org/c/zuul/zuul-operator/+/853586 | ||
| - [zuul/zuul-operator] 853592: Allow the specification of storageClassName in PVCs https://review.opendev.org/c/zuul/zuul-operator/+/853592 | ||
| -@gerrit:opendev.org- Michael Kelly proposed: | 08:07 | |
| - [zuul/zuul-operator] 853586: Only listen for updates to known secrets https://review.opendev.org/c/zuul/zuul-operator/+/853586 | ||
| - [zuul/zuul-operator] 853592: Allow the specification of storageClassName in PVCs https://review.opendev.org/c/zuul/zuul-operator/+/853592 | ||
| -@gerrit:opendev.org- Michael Kelly proposed: [zuul/zuul-operator] 853592: Allow the specification of storageClassName in PVCs https://review.opendev.org/c/zuul/zuul-operator/+/853592 | 08:10 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 853144: Set remote URL after config was updated https://review.opendev.org/c/zuul/zuul/+/853144 | 08:37 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853603: zuul_stream: handle non-string msg value https://review.opendev.org/c/zuul/zuul/+/853603 | 08:38 | |
| @iwienand:matrix.org | corvus: ^ I thought that might be something to do with the failures in the ansible 6 update, but it seems like it was just a red-herring that was a problem, but not *the* problem | 08:38 |
| @iwienand:matrix.org | * corvus: ^ I thought that might be something to do with the failures in the ansible 6 update, but after i debugged it, it seems like it was just a red-herring that was a problem, but not _the_ problem | 08:39 |
| -@gerrit:opendev.org- Michael Kelly proposed: | 19:18 | |
| - [zuul/zuul-operator] 853586: Only listen for updates to known secrets https://review.opendev.org/c/zuul/zuul-operator/+/853586 | ||
| - [zuul/zuul-operator] 853592: Allow the specification of storageClassName in PVCs https://review.opendev.org/c/zuul/zuul-operator/+/853592 | ||
| - [zuul/zuul-operator] 853695: Prefix zuul-specific resources with instance name https://review.opendev.org/c/zuul/zuul-operator/+/853695 | ||
| - [zuul/zuul-operator] 853696: Prefix nodepool specific resources with instance name https://review.opendev.org/c/zuul/zuul-operator/+/853696 | ||
| @michael_kelly_anet:matrix.org | Looking at https://review.opendev.org/c/zuul/zuul-operator/+/853582/1 - Code-Review+2 and Verified+1 - what do I need to do to get Workflow+1 ? | 19:42 |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/nodepool] 853698: Update TESTING doc to reference the correct tox target https://review.opendev.org/c/zuul/nodepool/+/853698 | 19:47 | |
| @jim:acmegating.com | Michael Kelly: be patient :) we try to have at least two maintainers approve changes | 19:51 |
| @jim:acmegating.com | (i was planning on taking a look at those closer to the end of my day) | 19:52 |
| @jim:acmegating.com | Michael Kelly: okay you have some +Ws and some suggestions for changes from me and questions from tristanC on that stack. thanks! :) | 20:01 |
| @michael_kelly_anet:matrix.org | Keen. It was more of an informational question, fwiw, just not sure if there's some procedure one is expected to follow ;) | 20:05 |
| @michael_kelly_anet:matrix.org | Responded to your comment, fyi | 20:09 |
| @jim:acmegating.com | Michael Kelly: heh, you followed it :) | 20:09 |
| @michael_kelly_anet:matrix.org | Sounds good. | 20:10 |
| @michael_kelly_anet:matrix.org | fwiw, this is also a good demo for me of zuul in action. ;) | 20:12 |
| @jim:acmegating.com | we try to demo all the things :) | 20:14 |
| @michael_kelly_anet:matrix.org | For sure. Another random question for the field - how congruent is the gerrit zuul-info plugin with the newer checks-ui stuff? eg: is there an expectation to maybe migrate in that direction? | 20:15 |
| @jim:acmegating.com | (except circular dependencies, which aren't enabled on opendev) | 20:15 |
| @michael_kelly_anet:matrix.org | s/zuul-info/zuul-summary/ | 20:16 |
| @jim:acmegating.com | opendev uses the zuul-results-summary plugin. i believe it should be possible to migrate that to the checks-ui framework, but no one has started that work. | 20:16 |
| @michael_kelly_anet:matrix.org | Yea.. checks-ui is a bit rough and ready from a consumer standpoint thusfar. | 20:17 |
| @jim:acmegating.com | zuul is usable without the plugin; it's just annoyingly chatty comments | 20:17 |
| @jim:acmegating.com | (but some people are fine with that) | 20:17 |
| @michael_kelly_anet:matrix.org | Hmmm. I'm less concerned about that part, honestly, just trying to figure out roadmap for these sorts of things vs gerrit... we'd migrated a bunch of stuff to the (now legacy) checks plugin infrastructure which appears to now have been a bit of a rug pull. | 20:18 |
| @jim:acmegating.com | the zuul-results-summary plugin itself is at risk -- the gerrit devs want to remove its hook point, but they are aware of the zuul-results-summary plugin and seem to want to avoid messing that up so hopefully will find an alternative. but just thought i'd mention it as a potential risk. | 20:18 |
| @jim:acmegating.com | Michael Kelly: re checks plugin, yes indeed. zuul actually supports the checks api interface, but that support has never left the "experimental" stage thanks to the gerrit changes upstream | 20:19 |
| @jim:acmegating.com | Michael Kelly: if you want to see zuul checks plugin in action, it's actually on gerrit's gerrit. but of course don't use it since i assume its days are numbered (I just don't know what that number is) | 20:20 |
| @michael_kelly_anet:matrix.org | hah | 20:20 |
| @michael_kelly_anet:matrix.org | That' | 20:21 |
| @michael_kelly_anet:matrix.org | * I didn't happen to notice that zuul was there under the covers, but I've used the checks stuff in gerrit-review | 20:21 |
| @michael_kelly_anet:matrix.org | They seem to be using some mix of jenkins+zuul | 20:21 |
| @jim:acmegating.com | yeah, the plugins are using zuul, jenkins for core | 20:21 |
| @michael_kelly_anet:matrix.org | My gerrit contributions have been mostly just minor things... | 20:21 |
| @michael_kelly_anet:matrix.org | Ah, gotcha. My (minor) changes were in the LFS plugin and in gerrit core, so wouldn't have encountered it (LFS plugin had no verify job at the time, iirc) | 20:22 |
| @jim:acmegating.com | here's one at random: https://gerrit-review.googlesource.com/c/plugins/serviceuser/+/343060 | 20:23 |
| -@gerrit:opendev.org- Zuul merged on behalf of Michael Kelly: [zuul/zuul-operator] 853582: Convert base64 encoded value to string https://review.opendev.org/c/zuul/zuul-operator/+/853582 | 20:23 | |
| @jim:acmegating.com | Michael Kelly: ^ 🎉 | 20:23 |
| @michael_kelly_anet:matrix.org | yay | 20:23 |
| @michael_kelly_anet:matrix.org | So, all this said, putting work into a zuul checks-ui plugin is not entirely outside of my wheelhouse. | 20:25 |
| @jim:acmegating.com | that'd be great :) | 20:31 |
| @jim:acmegating.com | Michael Kelly: Clark and ianw also have interest/expertise in this (in addition to myself). we'd probably want the repo hosted on gerrit's gerrit; i can help with getting ci set up for it. (i realize you haven't volunteered to start immediately and probably want to get a little more familiar with zuul first (including the rest api) -- just throwing out that info for if/when you decide to get started or feel like brainstorming further) | 20:34 |
| @michael_kelly_anet:matrix.org | Sounds good. | 20:35 |
| @jim:acmegating.com | also, i have a matrix bridge set up for the gerrit slack, let me know if you ever want an invite (this goes for anyone else too) | 20:36 |
| @michael_kelly_anet:matrix.org | Will do. | 20:37 |
| -@gerrit:opendev.org- Michael Kelly proposed: | 20:42 | |
| - [zuul/zuul-operator] 853585: kopf: Upgrade requirement to 1.35.6 https://review.opendev.org/c/zuul/zuul-operator/+/853585 | ||
| - [zuul/zuul-operator] 853586: Only listen for updates to known secrets https://review.opendev.org/c/zuul/zuul-operator/+/853586 | ||
| - [zuul/zuul-operator] 853592: Allow the specification of storageClassName in PVCs https://review.opendev.org/c/zuul/zuul-operator/+/853592 | ||
| - [zuul/zuul-operator] 853695: Prefix zuul-specific resources with instance name https://review.opendev.org/c/zuul/zuul-operator/+/853695 | ||
| - [zuul/zuul-operator] 853696: Prefix nodepool specific resources with instance name https://review.opendev.org/c/zuul/zuul-operator/+/853696 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 849887: Add Nodepool in Zuul spec https://review.opendev.org/c/zuul/zuul/+/849887 | 20:49 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 849887: Add Nodepool in Zuul spec https://review.opendev.org/c/zuul/zuul/+/849887 | 20:50 | |
| @clarkb:matrix.org | Yes a checks UI plugin for zuul would be wonderful | 21:15 |
| -@gerrit:opendev.org- Michael Kelly proposed: | 21:19 | |
| - [zuul/zuul-operator] 853592: Allow the specification of storageClassName in PVCs https://review.opendev.org/c/zuul/zuul-operator/+/853592 | ||
| - [zuul/zuul-operator] 853695: Prefix zuul-specific resources with instance name https://review.opendev.org/c/zuul/zuul-operator/+/853695 | ||
| - [zuul/zuul-operator] 853696: Prefix nodepool specific resources with instance name https://review.opendev.org/c/zuul/zuul-operator/+/853696 | ||
| @clarkb:matrix.org | corvus: I left comments on https://review.opendev.org/c/zuul/zuul/+/848516 I don't really think anything needs to be changed though so I +2'd if you want to approve it | 21:48 |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853603: zuul_stream: handle non-string msg value https://review.opendev.org/c/zuul/zuul/+/853603 | 21:52 | |
| -@gerrit:opendev.org- Michael Kelly proposed: | 21:55 | |
| - [zuul/zuul-operator] 853592: Allow the specification of storageClassName in PVCs https://review.opendev.org/c/zuul/zuul-operator/+/853592 | ||
| - [zuul/zuul-operator] 853695: Prefix zuul-specific resources with instance name https://review.opendev.org/c/zuul/zuul-operator/+/853695 | ||
| - [zuul/zuul-operator] 853696: Prefix nodepool specific resources with instance name https://review.opendev.org/c/zuul/zuul-operator/+/853696 | ||
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853603: zuul_stream: handle non-string msg value https://review.opendev.org/c/zuul/zuul/+/853603 | 22:01 | |
| @clarkb:matrix.org | ianw: ^ question on that one | 22:05 |
| @iwienand:matrix.org | Clark: thanks, yeah good point we can move that in where we inspect this as a string (do the rstrip()) | 22:26 |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853603: zuul_stream: handle non-string msg value https://review.opendev.org/c/zuul/zuul/+/853603 | 22:31 | |
| -@gerrit:opendev.org- Michael Kelly proposed: | 22:32 | |
| - [zuul/zuul-operator] 853695: Prefix zuul-specific resources with instance name https://review.opendev.org/c/zuul/zuul-operator/+/853695 | ||
| - [zuul/zuul-operator] 853696: Prefix nodepool specific resources with instance name https://review.opendev.org/c/zuul/zuul-operator/+/853696 | ||
| -@gerrit:opendev.org- Michael Kelly proposed: | 22:58 | |
| - [zuul/zuul-operator] 853695: Prefix zuul-specific resources with instance name https://review.opendev.org/c/zuul/zuul-operator/+/853695 | ||
| - [zuul/zuul-operator] 853696: Prefix nodepool specific resources with instance name https://review.opendev.org/c/zuul/zuul-operator/+/853696 | ||
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853208: zuul-stream : Test against a Python 2.7 container https://review.opendev.org/c/zuul/zuul/+/853208 | 23:54 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!