| -@gerrit:opendev.org- Michael Kelly proposed: | 05:33 | |
| - [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 | ||
| - [zuul/zuul-operator] 861488: helm: Add a basic helm chart for zuul-operator https://review.opendev.org/c/zuul/zuul-operator/+/861488 | ||
| - [zuul/zuul-operator] 862390: helm: Add cert-manager as optional dependency https://review.opendev.org/c/zuul/zuul-operator/+/862390 | ||
| - [zuul/zuul-operator] 863191: helm: Add pxc-operator as optional dependency https://review.opendev.org/c/zuul/zuul-operator/+/863191 | ||
| - [zuul/zuul-operator] 863439: doc: Re-write install doc to use helm chart https://review.opendev.org/c/zuul/zuul-operator/+/863439 | ||
| - [zuul/zuul-operator] 861279: bug: Select scheduler pod based on instance name on update https://review.opendev.org/c/zuul/zuul-operator/+/861279 | ||
| - [zuul/zuul-operator] 863474: pxc: Don't install PXC operator from Zuul operator https://review.opendev.org/c/zuul/zuul-operator/+/863474 | ||
| - [zuul/zuul-operator] 863475: cert-manager: Don't install cert-manager via zuul-operator https://review.opendev.org/c/zuul/zuul-operator/+/863475 | ||
| - [zuul/zuul-operator] 863476: k8s: Enable administrator to limit the watched namespace scope https://review.opendev.org/c/zuul/zuul-operator/+/863476 | ||
| - [zuul/zuul-operator] 863477: k8s: Allow use of a default image version besides latest https://review.opendev.org/c/zuul/zuul-operator/+/863477 | ||
| -@gerrit:opendev.org- Joshua Watt proposed: [zuul/zuul] 863068: executor: Skip line mapping for special Gerrit files https://review.opendev.org/c/zuul/zuul/+/863068 | 12:49 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Clark Boylan: [zuul/zuul] 863326: Fix config-errors dedicated page https://review.opendev.org/c/zuul/zuul/+/863326 | 17:34 | |
| @clarkb:matrix.org | woot | 18:13 |
|---|---|---|
| @clarkb:matrix.org | corvus: I think if you want to restart the web services in opendev to triple check the two web fixes that would be reaosnable before making 8.0.1 but also that is probabl overkill since we've checked it otherwise | 18:18 |
| @jim:acmegating.com | Clark: i will do that and then propose a release tag after lunch | 18:30 |
| @jim:acmegating.com | they are easy to restart | 18:30 |
| @jim:acmegating.com | zuul-maint: how does this look for a zuul release? | 20:12 |
| commit 4a5089338848b67dae2238f166413a8d9af60d0f (HEAD -> master, tag: 8.0.1, origin/master, refs/changes/26/863326/1) | ||
| @clarkb:matrix.org | corvus: when the releaes is done do you think you have time to review https://review.opendev.org/c/zuul/zuul-jobs/+/863098 ? that should all be working now for dns zone checks on debuntu distros | 20:31 |
| @jpew:matrix.org | I'm using DIB created images with Zuul with very liitle modifications from the stock images, and I get a lot of these types of warnings when I run some commands: `bash: warning: setlocale: LC_ALL: cannot change locale (C.UTF-8)` I can't quite figure out what's wrong in the image that's causing this | 20:32 |
| @jpew:matrix.org | It seems like DIB is generating locale data for en_US.UTF-8, but setting the default to C.UTF-8 (which it doesn't generate locale data for) | 20:33 |
| @clarkb:matrix.org | jpew: are you building using a -minimal element for the distro? I suspect that the distro's bash install defaults to c.utf-8 but some package is missing to include that information | 20:34 |
| @jpew:matrix.org | Clark: Ya, `ubuntu-minimal` | 20:35 |
| @clarkb:matrix.org | locale-c.utf-8 appears to be the pacakge to install that | 20:36 |
| @clarkb:matrix.org | I don't think we install that directly on our ubuntu images and we don't have that problem so we must pull it in via some other mechansim | 20:36 |
| @clarkb:matrix.org | hrm that doesn't exist on ubuntu maybe it is a debianism | 20:36 |
| @jpew:matrix.org | I think so | 20:37 |
| @jpew:matrix.org | On Ubuntu, (I think) you need libc-bin to pull in the definitions, then you have to run `locale-gen` to generate the actual files | 20:38 |
| @jpew:matrix.org | * On Ubuntu, (I think) you need libc-bin to pull in the definitions, then you have to run `locale-gen` to generate the actual compiled locales | 20:38 |
| @jpew:matrix.org | (hopefully I understand that correctly; I'm not very familar with locales) | 20:38 |
| @clarkb:matrix.org | ok quickly checking our ubuntu jammy image also built from ubuntu-minimal `locale -a` reports C and C.utf8` are valid and `locale` says we are using `C.UTF-8` | 20:41 |
| @clarkb:matrix.org | * ok quickly checking our ubuntu jammy image also built from ubuntu-minimal `locale -a` reports `C` and `C.utf8`are valid and`locale`says we are using`C.UTF-8` | 20:41 |
| @clarkb:matrix.org | something we are doing must build the locale | 20:41 |
| @clarkb:matrix.org | Looks like the debootstrap element installs the locales package which I think generates this stuff. | 20:43 |
| @clarkb:matrix.org | https://opendev.org/openstack/diskimage-builder/src/branch/master/diskimage_builder/elements/debootstrap/install.d/12-debian-locale-gen that file does something related to this | 20:46 |
| @clarkb:matrix.org | reading that script and some of the git history and comparing to our prod builds it appears the C.utf8 must be part of the default via the locales package? Then separately that script also generats en_US.utf9 ? | 20:50 |
| @clarkb:matrix.org | * reading that script and some of the git history and comparing to our prod builds it appears the C.utf8 must be part of the default via the locales package? Then separately that script also generats en\_US.utf8 ? | 20:50 |
| @clarkb:matrix.org | jpew: maybe double check `locale -a` to see what is available? And work backward from there. But it appears C.utf8 is the default and we add in en_US.utf8 by default (I think because for a long time there wasn't a C.utf8 so this got us utf8 way back when. We may not need to generate en_US.utf8 anymore) | 20:53 |
| @clarkb:matrix.org | I guess the other thing to check is your libc too old for C.utf8? | 20:57 |
| @clarkb:matrix.org | https://sourceware.org/glibc/wiki/Proposals/C.UTF-8 says glibc 2.35 or newer | 20:58 |
| @jim:acmegating.com | > <@clarkb:matrix.org> corvus: when the releaes is done do you think you have time to review https://review.opendev.org/c/zuul/zuul-jobs/+/863098 ? that should all be working now for dns zone checks on debuntu distros | 20:58 |
| reviewed! i did the testing catch that path thing? | ||
| @jim:acmegating.com | pushed zuul 8.0.1 | 20:58 |
| @clarkb:matrix.org | > <@jim:acmegating.com> reviewed! i did the testing catch that path thing? | 20:59 |
| For debian specifically since /usr/sbin isn't in the path there by default. It worked on ubuntu as is. But since the role may have been used on debian previously this caught a potential issue | ||
| @clarkb:matrix.org | hrm focal has 2.31 let me check a focal node | 21:00 |
| @jpew:matrix.org | It's ubuntu focal, so I don't think it should be too old | 21:00 |
| @jpew:matrix.org | Oh, wow, that is really new | 21:00 |
| @clarkb:matrix.org | I thought this got handled earlier fwiw. | 21:01 |
| @jpew:matrix.org | `locale -a` on my focal node shows: | 21:03 |
| ``` | ||
| C | ||
| C.UTF-8 | ||
| POSIX | ||
| en_US.UTF-8 | ||
| ``` | ||
| @clarkb:matrix.org | ya our focal looks similar basically looks the same as jammy | 21:04 |
| @clarkb:matrix.org | jpew: `validlocale` is a command that takes a locale and hcecks if it is valid | 21:05 |
| @clarkb:matrix.org | `validlocale C.UTF-8` says `locale 'C.UTF-8' valid and available` on our focal node | 21:06 |
| @jpew:matrix.org | Ours too... but we still get the error :/ | 21:10 |
| @jpew:matrix.org | Well, warning | 21:10 |
| @clarkb:matrix.org | that implies to me the issue isnt with locale generation. Since the locale is there and validates | 21:13 |
| @jpew:matrix.org | Clark: That seems correct | 21:14 |
| @jpew:matrix.org | I wonder if we see this on Jammy nodes.... cant check ATM though | 21:14 |
| @jpew:matrix.org | Ya, even if I manually (re-)generate it with `locale-gen` it still spews the warning. I wonder if it's the C-library | 21:15 |
| @clarkb:matrix.org | jpew: you might try setting it to en_US.utf8 just to see if that generated locale works. Another option would be to install locales-all and see if that fixes it | 21:15 |
| @clarkb:matrix.org | basically just trying to brute force something that works then you have something to compare against | 21:15 |
| @clarkb:matrix.org | corvus: I see docker images now and the package on pypi | 21:20 |
| @jim:acmegating.com | Clark: \o/ also -- a successful entry in the buildsets list ;) | 21:21 |
| @jim:acmegating.com | i will send out an email in a little bit | 21:21 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 863569: Document pct_used_hdd stat https://review.opendev.org/c/zuul/zuul/+/863569 | 22:36 | |
| @jim:acmegating.com | email sent | 23:15 |
| -@gerrit:opendev.org- Zuul merged on behalf of Clark Boylan: [zuul/zuul-jobs] 863098: Fix check zone role for Jammy https://review.opendev.org/c/zuul/zuul-jobs/+/863098 | 23:51 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!