*** osmanlicilegi is now known as Guest7485 | 00:18 | |
*** rlandy_ is now known as rlandy|out | 00:43 | |
ianw | happily there's a pull request on our grafana issues with the beta -> https://github.com/grafana/grafana/issues/53275 | 00:53 |
---|---|---|
ianw | unhappily i can not build a container with it to test, it seems like "docker build" is broken from some npm/javascript reason i don't want to think about | 00:53 |
ianw | if only there was some way that we could develop software and commit changes to ensure that developers wanting to help test your project always have a main branch that builds ... | 00:54 |
ianw | frickler: thanks for looking at the openeuler stuff. i will go in and manually delete the old .wsp files to cleanup https://grafana.opendev.org/d/f3089338b3/nodepool-dib-status?orgId=1 | 01:29 |
ianw | otherwise they'll sit for a month or two until they get cleaned up for non-updating | 01:29 |
ianw | i did apply the upstream change on the 9.0.1-beta1 tag and it fixed our graphs, so that's good. opendev testing wins again :) | 01:38 |
opendevreview | Merged opendev/system-config master: paste: move certificate to group variable https://review.opendev.org/c/opendev/system-config/+/848917 | 02:10 |
*** pojadhav is now known as pojadhav|ruck | 02:36 | |
opendevreview | Merged opendev/system-config master: mirror: retwork testinfra testing https://review.opendev.org/c/opendev/system-config/+/848912 | 03:50 |
opendevreview | Merged opendev/system-config master: mirror-update: move testing to mirror-update99 https://review.opendev.org/c/opendev/system-config/+/848919 | 03:50 |
*** lbragstad8 is now known as lbragstad | 04:48 | |
*** raukadah is now known as chandankumar | 04:51 | |
*** marios is now known as marios|ruck | 05:12 | |
*** pojadhav|ruck is now known as pojadhav|rover | 05:26 | |
*** ysandeep|PTO is now known as ysandeep | 05:38 | |
*** tobias-urdin5 is now known as tobias-urdin | 06:03 | |
*** ysandeep is now known as ysandeep|afk | 06:52 | |
*** gibi_pto is now known as gibi | 07:27 | |
*** jpena|off is now known as jpena | 07:35 | |
*** undefined is now known as Guest7674 | 07:59 | |
opendevreview | Simon Westphahl proposed zuul/zuul-jobs master: Supply missing error message on S3 upload failure https://review.opendev.org/c/zuul/zuul-jobs/+/852383 | 08:19 |
opendevreview | Simon Westphahl proposed zuul/zuul-jobs master: Supply missing error message on S3 upload failure https://review.opendev.org/c/zuul/zuul-jobs/+/852383 | 08:28 |
opendevreview | Simon Westphahl proposed zuul/zuul-jobs master: Supply missing error message on S3 upload failure https://review.opendev.org/c/zuul/zuul-jobs/+/852383 | 08:58 |
opendevreview | Simon Westphahl proposed zuul/zuul-jobs master: Supply missing error message on S3 upload failure https://review.opendev.org/c/zuul/zuul-jobs/+/852383 | 09:00 |
*** ysandeep|afk is now known as ysandeep | 09:33 | |
*** rlandy|out is now known as rlandy | 10:29 | |
*** dviroel_ is now known as dviroel | 11:38 | |
*** dhill is now known as Guest7696 | 11:55 | |
opendevreview | Merged openstack/project-config master: Add new project "whitebox-neutron-tempest-plugin" in the x/ namespace https://review.opendev.org/c/openstack/project-config/+/851031 | 11:57 |
*** elodille1 is now known as elodilles | 12:11 | |
*** Guest7674 is now known as rcastillo | 12:35 | |
*** pojadhav is now known as pojadhav|rover | 13:30 | |
*** dasm|off is now known as dasm | 13:31 | |
*** yoctozepto is now known as yoctozepto[pto] | 13:46 | |
opendevreview | Vanou Ishii proposed openstack/diskimage-builder master: Fix wrong yum.conf name of CentOS 9 Stream https://review.opendev.org/c/openstack/diskimage-builder/+/852408 | 13:53 |
*** hjensas is now known as hjensas|afk | 14:07 | |
NeilHanlon | good morning folks :) i think https://review.opendev.org/c/openstack/project-config/+/852167 is ready to go, if/when someone has a moment to move it along | 14:09 |
fungi | looks like i'm already +2 on that one, but ideally we'll have a second core reviewer approve it | 14:10 |
NeilHanlon | ack | 14:12 |
*** marios|ruck is now known as marios|ruck|call | 14:49 | |
opendevreview | Clark Boylan proposed opendev/system-config master: WIP Add a mailman3 list server https://review.opendev.org/c/opendev/system-config/+/851248 | 15:13 |
clarkb | I'm hoping that shows vhosts are properly work via the screenshots and that we don't try to create lists that already exist. Next up I think I'm going to try and directly inspect the DB to determien if our super user exists and toggle the email verification for that user from there | 15:14 |
*** marios|ruck|call is now known as marios|ruck | 15:15 | |
*** dviroel is now known as dviroel|lunch | 15:29 | |
*** ysandeep is now known as ysandeep|out | 15:30 | |
*** marios|ruck is now known as marios|out | 15:55 | |
opendevreview | Clark Boylan proposed opendev/system-config master: WIP Add a mailman3 list server https://review.opendev.org/c/opendev/system-config/+/851248 | 15:55 |
*** jpena is now known as jpena|off | 16:34 | |
*** dviroel|lunch is now known as dviroel | 16:38 | |
clarkb | this is weird the admin user I'm creating has its email address verified now | 16:41 |
clarkb | maybe because I called the user "admin" there is special handling? | 16:41 |
clarkb | I guess we can move on from that problem | 16:41 |
clarkb | hrm I think I found another bug in the upstream docker images. DJANGO_ALLOWED_HOSTS is meant to be a list of allowed hosts that we will serve content for, but the upstream settings.py simple os.getenviron()s that var | 16:49 |
clarkb | and thatdoesn't deserialize the textual list to a pyhton list of strings which is what django expects | 16:50 |
opendevreview | Clark Boylan proposed opendev/system-config master: WIP Add a mailman3 list server https://review.opendev.org/c/opendev/system-config/+/851248 | 16:59 |
clarkb | I've now made a second modification to the upstream settings.py file that we're bind mounting over the container's version :/ | 17:01 |
clarkb | I think once we've got something much closer to functional I'll need to write some bug reports for upstream | 17:03 |
fungi | yeah, i have a feeling we're working outside the bounds of how they expect people deploy things | 17:25 |
fungi | python 3.11.0rc1 was just tagged! | 17:28 |
opendevreview | Clark Boylan proposed opendev/system-config master: WIP Add a mailman3 list server https://review.opendev.org/c/opendev/system-config/+/851248 | 17:30 |
clarkb | fungi: I'm hoping that maybe by tomorrow the test created deployment will be in a place that you can poke at and think about mailing list configuration items and maybe also test the exim and mail transportation functionality | 17:34 |
clarkb | Once I get these last couple of details done its mostly just backups, log rotation, and then actual mailing list (and maybe user?) configuration | 17:34 |
clarkb | re user management/configuration It looks like we can assign email addresses as list owners similarly to what we've done in the past with mm2. However, I have no idea if that will work if the user then shows up and tries to create an account for that email address | 17:36 |
clarkb | that might complicate our migration. However, once migrated we can probably just tell users they must have registered and accoubt before adding their new lists | 17:37 |
fungi | yes, that's worth testing | 17:37 |
fungi | longer term, i think we also want to integrate with keycloak | 17:37 |
clarkb | yes, one of the things I've done there is disabled "social" auth but in theory we can turn that back on and tie it to the central auth | 17:37 |
clarkb | mostly I disabled it because I don't want people to have 5 accounts for 5 different identity sources | 17:38 |
clarkb | better to make a local account | 17:38 |
clarkb | though I guess I'm assuming my ALLOWED_HOSTS fix will fix the vhosting | 17:46 |
clarkb | if it doesn't fix the vhosting there may still be a bit of work to do | 17:46 |
clarkb | I think I just found a bug in mm3 | 18:09 |
clarkb | I'm trying to set advertized: False or True depending on whether or not hte list is private. The mm3 api is accepting my boolean json value for true and false and trying to .lower() it | 18:10 |
fungi | hah | 18:10 |
fungi | and lower-case is invalid? | 18:10 |
clarkb | well thats the next thing to try, sending it a str valud of 'true' or 'false' | 18:11 |
clarkb | its just weird that their docs show actual boolean values and then they don't work | 18:11 |
opendevreview | Clark Boylan proposed opendev/system-config master: WIP Add a mailman3 list server https://review.opendev.org/c/opendev/system-config/+/851248 | 18:11 |
clarkb | I also found an issue with the way I collect mm3 service logs in the CI job. It only works when ansible is successful | 18:15 |
clarkb | The reason it is that way is the di is logs/ and for both mailman core and mailman web and I want to copy the entire dir but differentiate between the two so I do a symlink in the post run playbook | 18:15 |
clarkb | oh except it isn't a true post run playbook it is the "test run" playbook which we must only run if the other ansible succeeds. I think I can fix this by having another playbook do the thing and treat it as a regular run playbook | 18:16 |
opendevreview | Clark Boylan proposed opendev/system-config master: WIP Add a mailman3 list server https://review.opendev.org/c/opendev/system-config/+/851248 | 18:48 |
clarkb | getting idempotency correct when you're inspecting the db is fun | 18:49 |
opendevreview | Clark Boylan proposed opendev/system-config master: WIP Add a mailman3 list server https://review.opendev.org/c/opendev/system-config/+/851248 | 19:11 |
*** tosky_ is now known as tosky | 19:25 | |
*** undefined is now known as Guest7733 | 19:30 | |
*** Guest7733 is now known as rcastillo | 19:31 | |
clarkb | ok that latest patchset is diempotent and the vhosting works | 19:36 |
*** rlandy is now known as rlandy|brb | 20:09 | |
*** rlandy|brb is now known as rlandy | 20:55 | |
opendevreview | Clark Boylan proposed opendev/system-config master: WIP Add a mailman3 list server https://review.opendev.org/c/opendev/system-config/+/851248 | 21:08 |
clarkb | fungi: ^ assuming I didn't break anything with these latest updates I think the resulting held node for that will be ready for testing of mailman more interactively to sort out things like list ownership and exim configs and mm2 migrations | 21:08 |
clarkb | that is really really close I think | 21:09 |
fungi | and then we'll push a dnm change to fail the deploy test job with an autohold in place, i guess? | 21:13 |
*** dviroel is now known as dviroel|out | 21:22 | |
clarkb | fungi: I've already been doing that all day so ya we should have one asap | 21:44 |
fungi | oh, hah | 21:44 |
fungi | and thanks! i've been quite distracted with other tasks | 21:45 |
clarkb | fungi: 104.130.26.212 that last run didn't explode on the ansible I added. I think thats about where I need to hand it off for help as its the mailman and exim details now | 21:46 |
clarkb | as far as the service goes with its deployment, base config, log rotation, backups etc I think we're good. Oh I do need to exclude the local db backup from borg though | 21:46 |
clarkb | The skeleton for managing lists via the rest api in ansible is all there too. Just need to figure out what we want to set setttinsg to and update the bits already in ansible | 21:47 |
fungi | hopefully my tomorrow will be caught up on other work and i can take a closer look | 21:47 |
opendevreview | Clark Boylan proposed opendev/system-config master: WIP Add a mailman3 list server https://review.opendev.org/c/opendev/system-config/+/851248 | 21:53 |
clarkb | That is mostly a bookkeeping update. I won't bother holding a new node. The old one should be fine | 21:53 |
clarkb | infra-root ^ feel free to poke around that change at this point. It is still WIP but the held node at 104.130.26.212 as well as the bones of the change itself are worth looking at I think | 21:55 |
clarkb | infra-root I'm going to work on getting our meeting agenda sent out. Please add anything you'd like on the agenda in the next hour or so | 21:58 |
*** dasm is now known as dasm|off | 22:03 | |
*** rlandy is now known as rlandy|bbl | 22:24 | |
opendevreview | Merged openstack/project-config master: Add rockylinux 9 to nodepool https://review.opendev.org/c/openstack/project-config/+/852167 | 22:31 |
ianw | clarkb: excellent stuff; dropped some minor comments | 22:34 |
clarkb | ianw: thanks responded to your questions | 22:41 |
clarkb | I'll work on addressing feedback tomorrow | 22:41 |
clarkb | Ok I've made edits to the agenda. What am I forgetting? | 22:56 |
ianw | clarkb: yeah, sometimes you just can not convince jinja to do things that seem sane :) | 23:19 |
ianw | e.g. if anyone can get | 23:19 |
ianw | sed -i -e '$a\' <file> into a shell: statement :) | 23:19 |
ianw | i've had a heck of a time getting https://review.opendev.org/q/topic:selfsigned-shared-ca merged with job timesouts; all basically random | 23:27 |
ianw | https://zuul.opendev.org/t/openstack/builds?project=opendev%2Fsystem-config&result=TIMED_OUT&skip=0 | 23:28 |
clarkb | is haveged installed? | 23:28 |
clarkb | maybe we don't have enough randomness? | 23:29 |
ianw | excluding two arm64 timeouts in that list above, the first 10 entries i've looked at area ll running in OVH GRA1 | 23:33 |
clarkb | I think the ovh servers have more limited disk iops | 23:40 |
ianw | https://paste.opendev.org/show/bm7jj1xw2U7SBaaNwnES/ | 23:56 |
ianw | via little script https://paste.opendev.org/show/b6TehZzJ7SxKgJOUBCKX/ | 23:57 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!