clarkb | fungi: looks like mm3 upstream says we should test a dummy site Iguess. | 00:26 |
---|---|---|
clarkb | *dummy default site | 00:26 |
clarkb | fungi: stephenfin: ok sounds like they are open to a PR to make forking a subprocess configurable or detectable or something. I asked for more clarification on that. But I have zero desire to write the patch myself | 00:34 |
clarkb | feel free to use any and all of the hacky thing I posted there if you end up doing it | 00:34 |
clarkb | also there is a bug in that setuptools raising an exception shouldn't crash the whole thing either | 00:35 |
opendevreview | Merged openstack/diskimage-builder master: tox jobs: pin to correct nodesets; use host networking for containerfile https://review.opendev.org/c/openstack/diskimage-builder/+/867579 | 00:52 |
*** pojadhav is now known as pojadhav|ruck | 02:21 | |
opendevreview | Merged openstack/diskimage-builder master: Fix issue in extract image https://review.opendev.org/c/openstack/diskimage-builder/+/850882 | 03:43 |
opendevreview | Ian Wienand proposed openstack/project-config master: gerrit/acl : remove deprecated copy conditions https://review.opendev.org/c/openstack/project-config/+/867931 | 03:46 |
opendevreview | Frikin Evgenii proposed openstack/diskimage-builder master: Add variable for check installing python3 in yum element https://review.opendev.org/c/openstack/diskimage-builder/+/856577 | 03:51 |
*** akahat is now known as akahat|ruck | 05:21 | |
*** pojadhav is now known as pojadhav|rover | 05:21 | |
pojadhav|rover | akahat|ruck, good morning !! | 05:22 |
opendevreview | Ian Wienand proposed openstack/project-config master: gerrit/acl : remove deprecated copy conditions https://review.opendev.org/c/openstack/project-config/+/867931 | 05:37 |
*** ralonsoh__ is now known as ralonsoh | 07:35 | |
*** ysandeep is now known as ysandeep|afk | 08:05 | |
*** ysandeep|afk is now known as ysandeep | 08:28 | |
*** jpena|off is now known as jpena | 08:40 | |
*** akahat|ruck is now known as akahat|ruck|lunch | 09:08 | |
*** anbanerj is now known as frenzy_friday | 09:13 | |
*** bhagyashris is now known as bhagyashris|sick | 09:26 | |
*** akahat|ruck|lunch is now known as akahat|ruck | 10:17 | |
*** ysandeep is now known as ysandeep|afk | 10:24 | |
frickler | does anyone else see formatting issues in the gerrit reply popup? the +2 vote boxes overlap with the vote descriptions for me | 10:43 |
*** kopecmartin is now known as kopecmartin|sick | 10:45 | |
*** rlandy|out is now known as rlandy | 11:11 | |
*** dviroel|rover|afk is now known as dviroel|rover | 11:16 | |
*** ysandeep|afk is now known as ysandeep | 12:13 | |
*** dviroel|rover is now known as dviroel | 12:20 | |
*** jpena is now known as jpena|off | 12:44 | |
*** ysandeep__ is now known as ysandeep | 13:02 | |
*** dasm|off is now known as dasm | 13:36 | |
*** dhill is now known as Guest2471 | 13:40 | |
*** jpena|off is now known as jpena | 14:06 | |
*** chkumar|ruck is now known as chandankumar | 14:31 | |
*** ysandeep is now known as ysandeep|out | 15:02 | |
*** dviroel is now known as dviroel|lunch | 15:05 | |
*** jpena is now known as jpena|off | 15:41 | |
fungi | comparing https://opendev.org/opendev/system-config/src/branch/master/playbooks/roles/mailman3/tasks/create_lists.yaml#L1-L25 to https://docs.mailman3.org/projects/mailman/en/latest/src/mailman/rest/docs/domains.html i can't see anything obvious we're missing in our implementation, and can't help but wonder if the rest api docs are incomplete or whether there's some bug with domain creation | 15:43 |
fungi | that they aren't getting added completely. the docs would seem to imply that a corresponding "web host" (django "site") gets created when we post to domains, but that doesn't appear to actually be happening from what i can see | 15:43 |
opendevreview | Jeremy Stanley proposed opendev/zone-opendev.org master: Dummy mailman hostname to house the list of lists https://review.opendev.org/c/opendev/zone-opendev.org/+/867981 | 15:52 |
Clark[m] | fungi: I suspect the issue is mailman-core is what does the rest API and Django isn't aware of it directly? I would need to read more to be sure though | 15:54 |
Clark[m] | fungi: do we need to actually host DNS records for the dummy default domain? We should be able to create it and ignore it? | 15:55 |
Clark[m] | Though maybe this will make list administration easier so could see doing it for that reason. | 15:55 |
*** dviroel|lunch is now known as dviroel | 15:58 | |
fungi | my thought is that if it also wants to use it to source notifications from or something, then we need it to resolve | 16:15 |
fungi | i was typing up the changes from a parking lot, so need to finish and push them up now that i'm home again | 16:17 |
*** marios is now known as marios|out | 16:20 | |
opendevreview | Jeremy Stanley proposed opendev/system-config master: Separate mailman default hostname https://review.opendev.org/c/opendev/system-config/+/867986 | 16:56 |
opendevreview | Jeremy Stanley proposed opendev/system-config master: Use magic domain guessing in Mailman 3 https://review.opendev.org/c/opendev/system-config/+/867987 | 16:56 |
clarkb | fungi: were we going to test it on the test node first? or maybe you already did that | 16:57 |
clarkb | mostly thinking we may want to avoid adding another unnecessary domain if it doesn't help | 16:58 |
fungi | oh, no the one necessitates the other | 16:58 |
fungi | site_id=0 doesn't work unless you add mail hosts. then the filtering goes sideways unless you also add web hosts to associate with each of them, but then the first web host still lists all the mailing lists in postorius | 16:58 |
fungi | that's what i was trying to get at with my post to the mm3 users ml | 16:59 |
clarkb | thats our assumption though right? it could be something specific to lists.opendev.org maybe? | 17:00 |
clarkb | I mean I think it is a good assumption, just wondering if we should test it before we deploy things to production | 17:00 |
fungi | well, it was specific to the held lists99 node too | 17:00 |
fungi | which is where i was testing this stuff out | 17:00 |
fungi | i didn't test it on the production server at all | 17:00 |
clarkb | right not the host, but the config in mm3 for that domain | 17:00 |
clarkb | put another way, I'm concerned that if we add a new domain we won't be able to make it the default domain and change the behavior for lists.opendev.org | 17:01 |
fungi | oh, i see what you're saying. i wanted to get an actual test result though to see what this does in ci | 17:01 |
clarkb | I think thefirst domain created is the default default. But I'm not sure if we can change that | 17:01 |
clarkb | ah I see | 17:01 |
fungi | since doing it to a held node after the fact doesn't really replicate deployment from scratch | 17:01 |
opendevreview | Jeremy Stanley proposed opendev/system-config master: DNM force mm3 failure to hold the node https://review.opendev.org/c/opendev/system-config/+/855292 | 17:02 |
clarkb | I'm going to work on finishing the iweb removal this morning | 17:04 |
fungi | i've got an autohold set for that now | 17:04 |
clarkb | At this point I think I need to check nodepool for any remaining content in the provider, but there shouldn't be at this point | 17:04 |
clarkb | then I can land the last couple of changes and then manually delete the mirror node | 17:04 |
fungi | sounds good, thanks! | 17:07 |
clarkb | `nodepool list | grep iweb` and `nodepool image-list | grep iweb` are both empty. | 17:07 |
clarkb | I'm going to approve the change to remove the provider from nodepool now | 17:07 |
opendevreview | Clark Boylan proposed opendev/zone-opendev.org master: Remove iweb mirror records https://review.opendev.org/c/opendev/zone-opendev.org/+/867988 | 17:11 |
clarkb | fungi: ^ that will conflict with your dns record updates. I'm not in a hurry to land mine and will happily rebase if necessary. I just didn't want ot forget to do that part of the cleanup | 17:11 |
opendevreview | Merged openstack/project-config master: Remove iweb nodepool configs https://review.opendev.org/c/openstack/project-config/+/867264 | 17:27 |
clarkb | ok thatappears to have applied. I'm approving the system-config chnge now | 17:49 |
clarkb | infra-root ^ that updates inventory files so will trigger a bunch of jobs. There isn't anything else that we should defer to with priority right? | 17:50 |
fungi | nothing i'm aware of, no | 17:51 |
opendevreview | Joshua Watt proposed zuul/zuul-jobs master: push-to-intermediate-registry: Handle container_images https://review.opendev.org/c/zuul/zuul-jobs/+/867991 | 17:55 |
clarkb | ok I'm sending it in | 18:00 |
*** rlandy is now known as rlandy|brb | 18:13 | |
*** rlandy|brb is now known as rlandy | 18:41 | |
opendevreview | Merged opendev/system-config master: Cleanup iweb/inap provider content https://review.opendev.org/c/opendev/system-config/+/867267 | 18:54 |
*** dviroel is now known as dviroel|out | 20:30 | |
opendevreview | Clark Boylan proposed zuul/zuul-jobs master: Add nox role and some simple jobs https://review.opendev.org/c/zuul/zuul-jobs/+/868002 | 20:32 |
opendevreview | Clark Boylan proposed opendev/bindep master: Fix bindep on Arch with packaging>=22.0 https://review.opendev.org/c/opendev/bindep/+/868003 | 20:35 |
opendevreview | Clark Boylan proposed opendev/bindep master: Use nox https://review.opendev.org/c/opendev/bindep/+/868004 | 20:35 |
clarkb | corvus: ^ fyi | 20:36 |
clarkb | corvus: I realized that nox may not work with old python3.5 as the hosting python3. But it seems to be able to drive old python just fine | 20:36 |
clarkb | corvus: honestly tools like bindep are simple neough that if we stop explicitly testing that compat I won't be sad | 20:37 |
clarkb | but it makes bindep a good test case | 20:37 |
opendevreview | Clark Boylan proposed zuul/zuul-jobs master: Add nox role and some simple jobs https://review.opendev.org/c/zuul/zuul-jobs/+/868002 | 20:40 |
clarkb | corvus: 868004 just ran a few nox jobs. python3.5 looks broken as expected. And I've got a bug in the docs target. There is still a bit to do as far as writing readmes and descriptions and docs and stuff. I haven't had lunch yet but will poke at refining this stuff after that | 20:46 |
opendevreview | Clark Boylan proposed zuul/zuul-jobs master: Add nox role and some simple jobs https://review.opendev.org/c/zuul/zuul-jobs/+/868002 | 21:34 |
opendevreview | Clark Boylan proposed zuul/zuul-jobs master: Add nox role and some simple jobs https://review.opendev.org/c/zuul/zuul-jobs/+/868002 | 21:39 |
opendevreview | Clark Boylan proposed opendev/bindep master: Use nox https://review.opendev.org/c/opendev/bindep/+/868004 | 21:39 |
opendevreview | Clark Boylan proposed opendev/bindep master: Use nox https://review.opendev.org/c/opendev/bindep/+/868004 | 21:51 |
*** dasm is now known as dasm|off | 21:54 | |
opendevreview | Clark Boylan proposed opendev/bindep master: Use nox https://review.opendev.org/c/opendev/bindep/+/868004 | 21:54 |
*** osmanlicilegi is now known as Guest0 | 22:33 | |
*** ChanServ changes topic to "OpenDev is a space for collaborative Open Source software development | https://opendev.org/ | channel logs https://meetings.opendev.org/irclogs/%23opendev/" | 22:43 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!