| clarkb | corvus: or maybe mailman looks at both files? I'm honestly surprised that the docker images didn't expose that setting as an env var | 00:00 |
|---|---|---|
| corvus | (i say weird because it seems strange to do that for an auto-generated file) | 00:00 |
| clarkb | because thats how literally everything else is done... | 00:00 |
| clarkb | corvus: fungi https://github.com/maxking/docker-mailman/blob/main/core/docker-entrypoint.sh#L185-L189 | 00:03 |
| clarkb | so ya that should do it I expect. I really wonder why this isn't exposed as an env var like everything else... | 00:04 |
| corvus | clarkb: hrm, the autogen file says it was created by mailman on 2022-12-01 04:54:40 UTC | 00:05 |
| corvus | since that was written out to our bind-mounted volume... | 00:05 |
| corvus | won't that entrypoint cat/append it again each time it starts? | 00:05 |
| corvus | ^ it=the extra file | 00:06 |
| clarkb | https://github.com/maxking/docker-mailman/blob/main/core/docker-entrypoint.sh#L149 it writes to it without >> to start | 00:06 |
| corvus | hrm, none of those mta things are in the file i'm looking at | 00:07 |
| clarkb | /etc/mailman.cfg in the container has those settings. I wonder if they aren't looking in the asme place :/ | 00:08 |
| corvus | so what's the /var/etc/mailman.cfg file then? | 00:08 |
| clarkb | it says that yo ucan edit that file nad mailman will never override it. Maybe it is applied over the /etc/mailman.cfg? | 00:09 |
| clarkb | so ya I think we can write to that as well. Either way should work I expect | 00:09 |
| corvus | oh maybe. anyway, i now agree that just writing the -extra file seems like the best/first thing to try. | 00:09 |
| clarkb | and what do we set site owner to if we have multiple vhosts. I guess lists.opendev.org is canonical? | 00:10 |
| clarkb | something like mailman@lists.opendev.org ? | 00:10 |
| corvus | yeah, i think it should be something like that | 00:10 |
| clarkb | I can work on that change | 00:10 |
| clarkb | https://github.com/maxking/docker-mailman/blob/main/core/docker-entrypoint.sh#L27 is where it actually resets the values | 00:13 |
| clarkb | corvus: oh the readme says it should point to a human | 00:15 |
| clarkb | should that be root@lists01.opendev.org then our normal root email will be sent? | 00:15 |
| fungi | corvus: oh, i got something similar for a (supposedly defunct) subscriber to one of the mailing lists i'm an owner on | 00:15 |
| clarkb | I'll push up with root@lists01.opendev.org as the setting and yall that understand email better can edit :) | 00:16 |
| fungi | i agree if we have to pick a specific domain for the server, it's opendev.org | 00:17 |
| fungi | i think it'll probably be a matter of merging and then seeing if we disagree with what the server is doing afterwards | 00:18 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Set mailman3 site-owner https://review.opendev.org/c/opendev/system-config/+/866632 | 00:21 |
| clarkb | something like that maybe | 00:21 |
| clarkb | yall should feel free to update that change | 00:22 |
| clarkb | I think I got the paths correct to map between host side and bind mount location though | 00:23 |
| clarkb | the team meeting agenda email is out and I received it back from the list server. It also appears to be in the archive | 00:27 |
| *** dviroel is now known as dviroel|out | 00:28 | |
| ianw | clarkb: i wonder if we timestamped in the requirements file as a comment the current day (2022-12-06) only ... i wonder if that would then report "changed" once a day, and so we'd effecitvely run an update on the virtualenv once a day, but for the most part skip it | 00:29 |
| ianw | it would be a question of if the act of making the timestamp caused the template module to think it changed, or the actual file content | 00:30 |
| clarkb | I believe it is based on hashes and file content | 00:30 |
| ianw | seems to have a chance of working. just this should run in every hourly jobset, that's why i'm trying to think of ways we can skip external access as much as possible | 00:31 |
| ianw | yeah, a local test confirmed that putting the same date in doesn't result in changed | 00:39 |
| opendevreview | Ian Wienand proposed opendev/system-config master: install-ansible: overhaul install ansible requirements https://review.opendev.org/c/opendev/system-config/+/866542 | 00:56 |
| opendevreview | Ian Wienand proposed opendev/system-config master: install-ansible: update venv once a day https://review.opendev.org/c/opendev/system-config/+/866633 | 00:56 |
| opendevreview | Ian Wienand proposed opendev/system-config master: install-ansible: overhaul install ansible requirements https://review.opendev.org/c/opendev/system-config/+/866542 | 01:20 |
| opendevreview | Ian Wienand proposed opendev/system-config master: install-ansible: update venv once a day https://review.opendev.org/c/opendev/system-config/+/866633 | 01:20 |
| opendevreview | Ian Wienand proposed opendev/system-config master: install-ansible: overhaul install ansible requirements https://review.opendev.org/c/opendev/system-config/+/866542 | 02:10 |
| opendevreview | Ian Wienand proposed opendev/system-config master: install-ansible: update venv once a day https://review.opendev.org/c/opendev/system-config/+/866633 | 02:10 |
| opendevreview | Ian Wienand proposed opendev/system-config master: install-ansible: overhaul install ansible requirements https://review.opendev.org/c/opendev/system-config/+/866542 | 02:28 |
| opendevreview | Ian Wienand proposed opendev/system-config master: install-ansible: update venv once a day https://review.opendev.org/c/opendev/system-config/+/866633 | 02:28 |
| opendevreview | Ian Wienand proposed opendev/system-config master: install-ansible: update venv once a day https://review.opendev.org/c/opendev/system-config/+/866633 | 03:50 |
| *** yadnesh|away is now known as yadnesh | 04:45 | |
| opendevreview | Ian Wienand proposed opendev/system-config master: create-venv: make update venv once per day https://review.opendev.org/c/opendev/system-config/+/866644 | 05:27 |
| opendevreview | Ian Wienand proposed opendev/system-config master: pip: use latest instead of upgrade https://review.opendev.org/c/opendev/system-config/+/866645 | 05:27 |
| opendevreview | Ian Wienand proposed opendev/system-config master: create-venv: make update venv once per day https://review.opendev.org/c/opendev/system-config/+/866644 | 05:44 |
| opendevreview | Ian Wienand proposed opendev/system-config master: pip: use latest instead of upgrade https://review.opendev.org/c/opendev/system-config/+/866645 | 05:44 |
| *** ysandeep is now known as ysandeep|afk | 05:47 | |
| opendevreview | Ian Wienand proposed opendev/system-config master: create-venv: make update venv once per day https://review.opendev.org/c/opendev/system-config/+/866644 | 05:59 |
| opendevreview | Ian Wienand proposed opendev/system-config master: pip: use latest instead of upgrade https://review.opendev.org/c/opendev/system-config/+/866645 | 05:59 |
| opendevreview | Ian Wienand proposed opendev/system-config master: create-venv: make update venv once per day https://review.opendev.org/c/opendev/system-config/+/866644 | 06:07 |
| opendevreview | Ian Wienand proposed opendev/system-config master: pip: use latest instead of upgrade https://review.opendev.org/c/opendev/system-config/+/866645 | 06:07 |
| *** marios is now known as marios|ruck | 06:08 | |
| opendevreview | Ian Wienand proposed opendev/system-config master: create-venv: make upgrade venv once per day https://review.opendev.org/c/opendev/system-config/+/866644 | 06:20 |
| opendevreview | Ian Wienand proposed opendev/system-config master: pip: use latest instead of upgrade https://review.opendev.org/c/opendev/system-config/+/866645 | 06:20 |
| opendevreview | Ian Wienand proposed opendev/system-config master: install-ansible: update venv once a day https://review.opendev.org/c/opendev/system-config/+/866633 | 06:28 |
| opendevreview | Ian Wienand proposed opendev/system-config master: create-venv: make upgrade venv once per day https://review.opendev.org/c/opendev/system-config/+/866644 | 06:28 |
| opendevreview | Ian Wienand proposed opendev/system-config master: pip: use latest instead of upgrade https://review.opendev.org/c/opendev/system-config/+/866645 | 06:28 |
| *** ysandeep|afk is now known as ysandeep | 07:00 | |
| *** yadnesh is now known as yadnesh|afk | 08:13 | |
| *** yadnesh|afk is now known as yadnesh | 09:20 | |
| *** ysandeep is now known as ysandeep|lunch | 09:21 | |
| opendevreview | Tim Beermann proposed zuul/zuul-jobs master: Add yamllint job. https://review.opendev.org/c/zuul/zuul-jobs/+/866679 | 10:13 |
| *** jpodivin_ is now known as jpodivin | 10:14 | |
| *** ysandeep|lunch is now known as ysandeep | 10:18 | |
| opendevreview | Tim Beermann proposed zuul/zuul-jobs master: Add yamllint job. https://review.opendev.org/c/zuul/zuul-jobs/+/866679 | 10:25 |
| opendevreview | Tim Beermann proposed zuul/zuul-jobs master: Add yamllint job. https://review.opendev.org/c/zuul/zuul-jobs/+/866679 | 10:41 |
| *** dviroel|out is now known as dviroel | 11:12 | |
| *** rlandy|out is now known as rlandy|rover | 11:18 | |
| *** yadnesh is now known as yadnesh|afk | 12:00 | |
| *** frenzy_friday is now known as frenzy_friday|food | 12:21 | |
| *** yadnesh|afk is now known as yadnesh | 13:10 | |
| *** frenzy_friday|food is now known as frenzy_friday | 13:30 | |
| *** ysandeep is now known as ysandeep|dinner | 13:34 | |
| *** dhill is now known as Guest1131 | 13:38 | |
| *** dasm|off is now known as dasm | 14:04 | |
| *** jpodivin_ is now known as jpodivin | 14:17 | |
| Tengu | clarkb, fungi heya! care to nudge https://review.opendev.org/c/opendev/system-config/+/866175 ? that one would really help tripleo :) | 14:43 |
| opendevreview | Merged openstack/project-config master: Allow Ironic/Sushy cores to toggle WIP state https://review.opendev.org/c/openstack/project-config/+/863931 | 14:45 |
| opendevreview | Merged openstack/project-config master: Add os_skyline repo to CI https://review.opendev.org/c/openstack/project-config/+/863167 | 14:46 |
| *** ysandeep|dinner is now known as ysandeep__ | 14:46 | |
| Tengu | thanks fungi :) | 15:20 |
| fungi | yw | 15:27 |
| *** dviroel is now known as dviroel|lunch | 15:31 | |
| opendevreview | Merged opendev/system-config master: Correct how ansible-galaxy is proxified https://review.opendev.org/c/opendev/system-config/+/866175 | 15:32 |
| *** pojadhav is now known as pojadhav|afk | 15:32 | |
| Tengu | \o/ time to go configure our projects to use this proxy endpoint! | 15:37 |
| fungi | Tengu: after you see the deploy pipeline jobs complete | 15:44 |
| Tengu | I have a WIP change that should also take advantage of the capability to pass a list of servers for ansible-galaxy. | 15:45 |
| Tengu | so it's also a good test | 15:45 |
| *** ysandeep__ is now known as ysandeep|out | 15:50 | |
| *** yadnesh is now known as yadnesh|away | 15:51 | |
| clarkb | fungi: an hour ago a thread on the mm3 list was started and people on it suggest not upgrading to the latest version of mailman (3.3.7) The bug filed around the problem seems related to postfix and affects mysql/mariadb with 10 or more lists https://gitlab.com/mailman/mailman/-/issues/1044 | 15:54 |
| clarkb | fungi: sorry more than 10 lists. lists.opendev.org has exactly 10 lists and zuul-ci has fewer. I think that means even if the issue affects exim we're ok for now | 15:55 |
| clarkb | fungi: but we should keep an eye on this prior to migrating other list domains | 15:55 |
| clarkb | or adding more lists | 15:56 |
| fungi | yes, and be prepared to roll out an update once there's a fixed release | 15:56 |
| fungi | thanks for the heads up! | 15:56 |
| clarkb | I'm glad we're didn't migrate everything all at once :) | 15:57 |
| clarkb | oh sqlalchemy version may also play a part | 15:58 |
| clarkb | we don't restrict the sqlalchemy version so were almost certainly running the current affected version | 15:59 |
| clarkb | fungi: maybe we should also check our core container logs for those log messages to double check? | 16:00 |
| clarkb | I've grepped 'unbuffered result' in /var/lib/mailman/core/var/logs/mailman.log which should be our core service log and get no results | 16:06 |
| clarkb | so ya lets watch this but I think we are ok for now | 16:06 |
| clarkb | cc corvus you may want to find an excuse to send emai lto the zuul list just to be sure that looks good | 16:06 |
| clarkb | it is possible that exim is unaffected | 16:07 |
| opendevreview | Monty Taylor proposed opendev/system-config master: Retire mordred as infra-root https://review.opendev.org/c/opendev/system-config/+/866781 | 16:21 |
| frickler | fungi: looks like https://review.opendev.org/c/opendev/system-config/+/489711 is still relevant and it might be a good opportunity to rebase it on top of ^^ | 16:28 |
| * frickler just noticed that patch in the list of conflicts for the above | 16:28 | |
| *** dviroel|lunch is now known as dviroel | 16:34 | |
| fungi | may need to knock some cobwebs off that one, yeah | 16:38 |
| opendevreview | Jeremy Stanley proposed opendev/system-config master: Clean up SB accounts for former admins https://review.opendev.org/c/opendev/system-config/+/489711 | 16:52 |
| fungi | frickler: the change was almost entirely obsolete, once i factored out things which have been cleaned up in years since that's ^ all that was left | 16:52 |
| *** marios|ruck is now known as marios|out | 16:55 | |
| opendevreview | Monty Taylor proposed opendev/system-config master: Retire mordred as infra-root https://review.opendev.org/c/opendev/system-config/+/866781 | 17:04 |
| *** pojadhav|afk is now known as pojadhav | 18:37 | |
| *** dviroel is now known as dviroel|biab | 19:48 | |
| clarkb | fwiw I've skimmed nodepool grafana stuff ofr any launcher issues and not seeing anything other than linaro being sad but it has been for a bit and we seem to no longer have graphite data for api response times | 19:50 |
| clarkb | I'll try to look into that tomorrow. I don't think it is critical but is a very nice to have so worth sorting out | 19:50 |
| opendevreview | Ian Wienand proposed opendev/system-config master: [dnm] hold gerrit nodes https://review.opendev.org/c/opendev/system-config/+/866798 | 20:00 |
| *** dasm is now known as dasm|off | 22:51 | |
| ianw | https://173.231.255.71/ -- the only visual thing i can see is the "status" column. it's not immediately clear what that means | 22:57 |
| *** rlandy|rover is now known as rlandy|out | 23:01 | |
| ianw | ok, so it looks like a visual representation of the submit-requirement rules; i.e. https://opendev.org/zuul/zuul/src/commit/532c30469ffc1a1b35d812455fa8271b392915cd/doc/source/examples/playbooks/project.config#L83 | 23:01 |
| fungi | oof, new python releases just now: 3.12.0a3, 3.11.1, 3.10.9, 3.9.16, 3.8.16, 3.7.16 | 23:17 |
| fungi | guess i know what my workstation is going to be compiling all night | 23:17 |
| JayF | you need some cpu juice to help? | 23:23 |
| JayF | serious question, if you're doing local compiles I do have a beefy box I can run a script on if you wanna split up the work | 23:23 |
| ianw | i guess our CI doesn't set the Workflow label in the test gerrit | 23:28 |
| fungi | nah, they go pretty quickly all things considered. it's just 6 compilations of the interpreter and stdlib from source is a lot to ask of any computer when it should be off enjoying a few evening pints | 23:29 |
| fungi | thanks for the offer though JayF! | 23:29 |
| ianw | i don't think we have submit-requirement rules, so i wonder if the column disappears | 23:30 |
| JayF | If you have a powerful PC, and it's not doubling as a space heater every now and then, it's like never walking your dog ;) | 23:30 |
| clarkb | ianw: updating to the new rules spec is somethign we should do after the 3.6 upgrade to be prepared for 3.7 | 23:33 |
| clarkb | but the old stuff should continue to work on 3.6 | 23:34 |
| opendevreview | Merged opendev/system-config master: Set mailman3 site-owner https://review.opendev.org/c/opendev/system-config/+/866632 | 23:39 |
| ianw | clarkb: yeah -- i don't exactly know where it's getting it's submit requirements from (the ci version). maybe it's the defaults? | 23:43 |
| ianw | project.config doesn't have any submit-requirement rules i can see | 23:43 |
| ianw | so, it might be slightly questionable if this "Status" column will show the right thing in our production | 23:44 |
| ianw | it might be something we just go with, and sort out as we go along | 23:44 |
| ianw | i can't see any other user visible changes, just by flipping between tabs | 23:45 |
| ianw | we could probably turn it off ... SUBMIT_REQUIREMENTS_UI = 'UiFeature__submit_requirements_ui' | 23:49 |
| ianw | that works, it's turned off on that host now | 23:53 |
| ianw | i think maybe leave it on, since we want to think about the submit requirements updates. if it becomes a problem, we know we can disable it easily with a config switch? | 23:54 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!