| opendevreview | Merged zuul/zuul-jobs master: enable-kubernetes: Fix jammy install, improve pod test https://review.opendev.org/c/zuul/zuul-jobs/+/863810 | 00:01 |
|---|---|---|
| opendevreview | Ian Wienand proposed opendev/system-config master: openafs: copy dkms log directory https://review.opendev.org/c/opendev/system-config/+/864148 | 00:20 |
| *** dhill is now known as Guest1004 | 00:22 | |
| opendevreview | Ian Wienand proposed opendev/system-config master: openafs: copy dkms log directory https://review.opendev.org/c/opendev/system-config/+/864148 | 00:39 |
| ianw | clarkb: are you ok with the global bump to the pre version? sorry took a bunch of revisions to get the file name right :/ | 01:04 |
| Clark[m] | ianw: it would just be for CentOS right? | 02:03 |
| Clark[m] | I don't think we should update the zuul executors or mirror nodes etc | 02:03 |
| ianw | yeah only centos, the rpm distros | 02:20 |
| Clark[m] | Ya I think that should be fine. I left a newer review on one of the older patchsets indicating I was ok with that after my suggestion of keeping CentOS 8 back | 02:38 |
| opendevreview | Michael Kelly proposed zuul/zuul-jobs master: helm: Add job for linting helm charts https://review.opendev.org/c/zuul/zuul-jobs/+/861799 | 04:16 |
| *** yadnesh|away is now known as yadnesh | 04:33 | |
| *** marios is now known as marios|ruck | 06:04 | |
| opendevreview | wangxiyuan proposed opendev/system-config master: Drop openEuler 20.03 LTS SP2 repo mirror https://review.opendev.org/c/opendev/system-config/+/848796 | 06:29 |
| opendevreview | wangxiyuan proposed opendev/system-config master: Drop openEuler 20.03 LTS SP2 repo mirror https://review.opendev.org/c/opendev/system-config/+/848796 | 06:29 |
| *** jpena|off is now known as jpena | 08:23 | |
| *** ysandeep|out is now known as ysandeep | 08:54 | |
| *** dviroel|biab is now known as dviroel | 09:36 | |
| *** bbezak_ is now known as bbezak | 09:51 | |
| *** bbezak is now known as Guest1060 | 09:52 | |
| *** dviroel is now known as dviroel|doc-appt | 09:52 | |
| *** Guest1060 is now known as bbezak | 09:56 | |
| opendevreview | Nikolay Vinogradov proposed openstack/project-config master: Add manila-infinidat backend charm to OpenStack charms https://review.opendev.org/c/openstack/project-config/+/863957 | 10:03 |
| opendevreview | Nikolay Vinogradov proposed openstack/project-config master: Add the cinder-infinidat charm to Openstack charms https://review.opendev.org/c/openstack/project-config/+/863954 | 10:05 |
| opendevreview | Nikolay Vinogradov proposed openstack/project-config master: Add the infinidat-tools charm to Openstack charms https://review.opendev.org/c/openstack/project-config/+/863955 | 10:06 |
| opendevreview | Tama McGlinn proposed opendev/git-review master: Add message option https://review.opendev.org/c/opendev/git-review/+/864098 | 10:08 |
| opendevreview | Nikolay Vinogradov proposed openstack/project-config master: Add manila-infinidat charm to OpenStack charms https://review.opendev.org/c/openstack/project-config/+/863957 | 10:08 |
| opendevreview | Nikolay Vinogradov proposed openstack/project-config master: Add manila-infinidat charm to OpenStack charms https://review.opendev.org/c/openstack/project-config/+/863957 | 10:09 |
| *** ysandeep is now known as ysandeep|out | 10:43 | |
| frickler | clarkb: if you fancy creating more gerrit bug reports: the patchset description added in https://review.opendev.org/c/opendev/git-review/+/864098 doesn't make it into the notification mails | 11:07 |
| frickler | might also be interesting for gertty users to check how it is handled there | 11:07 |
| *** dviroel|doc-appt is now known as dviroel | 11:24 | |
| *** yadnesh is now known as yadnesh|away | 12:47 | |
| fungi | gertty shows it like a review comment but associated with the patchset upload event in the review timeline, much like how the gerrit webui displays it | 13:13 |
| frickler | o.k., so working as expected it seems, nice | 13:40 |
| *** dasm|off is now known as dasm | 14:09 | |
| opendevreview | Merged opendev/system-config master: Add ssl support to zookeeper-statsd and fix latency handling https://review.opendev.org/c/opendev/system-config/+/864146 | 15:37 |
| *** dviroel is now known as dviroel|lunch | 15:54 | |
| *** knikolla[m] is now known as knikolla | 16:06 | |
| *** marios|ruck is now known as marios | 16:16 | |
| *** marios is now known as marios|out | 16:16 | |
| *** marios|out is now known as marios | 16:16 | |
| *** marios is now known as marios|out | 16:23 | |
| clarkb | frickler: fungi: https://review.opendev.org/c/opendev/system-config/+/861925 is an easy review. THen I'll try to land the child when I get back next week. (I'd do it today but I've already got a todo list long enough to keep me busy today :) ) | 16:31 |
| clarkb | that cleans up our uwsgi docker image | 16:31 |
| *** iurygregory__ is now known as iurygregory | 16:44 | |
| opendevreview | Clark Boylan proposed opendev/system-config master: Update Gerrit images to 3.5.4 and 3.6.3 https://review.opendev.org/c/opendev/system-config/+/864217 | 16:48 |
| clarkb | that is one less thing on the todo list :) they actually improved the copy approvals process that we need to run before upgrading to 3.6 which is nice | 16:49 |
| *** dviroel|lunch is now known as dviroel | 16:59 | |
| *** mnaser__ is now known as mnaser | 17:05 | |
| *** jpena is now known as jpena|off | 17:21 | |
| opendevreview | Merged opendev/system-config master: Cleanup uwsgi build hacks https://review.opendev.org/c/opendev/system-config/+/861925 | 17:32 |
| clarkb | https://bugs.chromium.org/p/gerrit/issues/detail?id=16418 gerrit web ui rebase bug | 17:36 |
| opendevreview | Clark Boylan proposed opendev/lodgeit master: Update uwsgi image to python 3.10 https://review.opendev.org/c/opendev/lodgeit/+/864220 | 17:42 |
| opendevreview | Clark Boylan proposed opendev/lodgeit master: Drop python 3.6 testing https://review.opendev.org/c/opendev/lodgeit/+/864221 | 17:42 |
| clarkb | and these two changes are followups to the uwsgi update | 17:43 |
| *** dviroel_ is now known as dviroel | 18:02 | |
| opendevreview | Clark Boylan proposed opendev/lodgeit master: Update uwsgi image to python 3.10 https://review.opendev.org/c/opendev/lodgeit/+/864220 | 18:31 |
| opendevreview | Clark Boylan proposed opendev/lodgeit master: Drop python 3.6 testing https://review.opendev.org/c/opendev/lodgeit/+/864221 | 18:31 |
| opendevreview | Clark Boylan proposed opendev/lodgeit master: Modernize test suite https://review.opendev.org/c/opendev/lodgeit/+/864222 | 18:31 |
| clarkb | that ended up being far more involved than I anticipated | 18:32 |
| clarkb | timburke: ^ I know this is far far simpler than the swift test suite and yall are looking at pytest but 864222 is a conversion from nose to python unittest with stestr as the runner if that is useful | 18:34 |
| *** dviroel is now known as dviroel|afk | 20:12 | |
| ianw | clarkb: we should probably run a system-config change to try deploying the lodgeit continaer with 3.10 -- i don't think the gate does that? | 20:36 |
| ianw | my concern there is the super-ancient version of Werkzeug | 20:36 |
| ianw | istr getting into some sort of recursive loop trying to update that ... | 20:37 |
| clarkb | ianw: werkzeug is exercised in the test suite at least. But ya that might be worthwhile | 20:38 |
| ianw | https://review.opendev.org/c/opendev/lodgeit/+/693378 ... that was where i ended up (giving up) | 20:38 |
| ianw | looks like that particular issue may have been fixed in sept 2021, so possibly it can be upgraded | 20:40 |
| *** dasm is now known as dasm|off | 22:28 | |
| ianw | hrm, those gerrit updates failed getting nodejs for jammy | 23:36 |
| ianw | it's trying to get node.js 10, which might be the problem | 23:41 |
| ianw | "The minimum nodejs version supported is 10.x+. We recommend at least the latest LTS (v16 as of October 2022)." ... so we're due an update anyway | 23:41 |
| ianw | (https://gerrit.googlesource.com/gerrit/+/master/polygerrit-ui/README.md#installing-node_js-and-npm-packages) | 23:41 |
| clarkb | thats interesting they would bump the requirement in the middle of a stable gerrit release | 23:44 |
| opendevreview | Ian Wienand proposed opendev/system-config master: gerrit-build: update to nodejs 19.x https://review.opendev.org/c/opendev/system-config/+/864234 | 23:44 |
| ianw | clarkb: oh, well that's master | 23:45 |
| ianw | but i think it's us that has bumped where we're building it to jammy, and there's no 10.x packages | 23:45 |
| clarkb | right, but we would've built with focal before? | 23:45 |
| clarkb | which should be even older? | 23:45 |
| clarkb | oh wait I see | 23:46 |
| ianw | which is probably a good thing, because ensure-nodejs pulles form the nodesource repos, which were built for focal | 23:46 |
| clarkb | ianw: I think nodejs does a stable even release and unstable odd release. I think we may want 18 and not 19 | 23:47 |
| clarkb | yup that appears to be the case. I'll leave a comment on the change suggesting 18 instead | 23:47 |
| ianw | good point, 18 is marked as lts | 23:47 |
| opendevreview | Ian Wienand proposed opendev/system-config master: gerrit-build: update to nodejs 19.x https://review.opendev.org/c/opendev/system-config/+/864234 | 23:48 |
| opendevreview | Ian Wienand proposed opendev/system-config master: gerrit-build: update to nodejs 18.x https://review.opendev.org/c/opendev/system-config/+/864234 | 23:48 |
| clarkb | +2 thanks | 23:49 |
| ianw | that will either work or explode in a 400-line backtrace of incomprehensible javascript reflections/abstraction/encapsulation nonsense :) | 23:49 |
| clarkb | if it explodes we can try 16, then 14, then 12, etc :) | 23:49 |
| clarkb | 16 is a very popular version | 23:49 |
| ianw | while (explode); do version -= 2; done :) | 23:51 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!