Monday, 2023-01-30

*** rlandy is now known as rlandy|out00:53
*** yadnesh|away is now known as yadnesh05:12
*** pojadhav is now known as pojadhav|ruck05:14
ianwi'm running a prune on the backup0206:11
ianwroot screen06:11
*** akahat|rover is now known as akahat07:05
*** pojadhav is now known as pojadhav|ruck07:24
*** jpena|off is now known as jpena08:43
*** elodilles is now known as elodilles_afk09:39
opendevreviewTobias Henkel proposed zuul/zuul-jobs master: Fix creating bucket in s3 log upload  https://review.opendev.org/c/zuul/zuul-jobs/+/87210409:52
*** rlandy|out is now known as rlandy11:09
*** elodilles_afk is now known as elodilles11:12
*** dviroel_ is now known as dviroel|rover11:24
*** ysandeep is now known as ysandeep|afk11:55
*** ysandeep|afk is now known as ysandeep12:56
*** tkajinam is now known as Guest288213:11
*** dasm|off is now known as dasm13:57
*** yadnesh is now known as yadnesh|away15:01
fungilucasagomes: our root alias got an invite from rtd to join ovn-bgp-agent, supposedly initiated by your account there. did you need something done?15:15
*** ysandeep is now known as ysandeep|out15:23
fungiokay, so getting closer on the mm3 fixups. on the new held node (104.130.219.31) it still had the initial mailman site as lists.opendev.org rather than mailman.opendev.org so i'm clearly missing something there. after creating corresponding sites and mailman domains through the django admin interface i can see both postorius and hyperkitty correctly filtering the lists for each site.15:57
fungiunfortunately hyperkitty still says lists.openstack.org in the top-left corner of all the sites, so i need to figure out what's missing for that part (i thought we had already addressed it in the proposed changes, but this would indicate otherwise)15:57
fungii'm restarting the containers on the held node to see if anything changes15:58
lucasagomesfungi, hi there, ah... I created the account and added the openstackci as an admin16:12
lucasagomesfungi, lemme get the patch16:12
lucasagomesfungi, https://review.opendev.org/c/x/ovn-bgp-agent/+/87203216:13
lucasagomesfungi, does it need some approval ?16:13
Clark[m]As a heads up slow start for me today due to a doctor's checkup16:13
fungilucasagomes: i'm not super familiar with rtd, so unsure whether the refresh trigger in the rtd api requires the zuul account to be a member of the project. if so, we'll need to accept the invitation from rtd, i think?16:14
fungiokay, good news on the mm3 front... after a full container restart and page refresh, hyperkitty is now showing the lists' individual display names as intended16:15
*** dviroel|rover is now known as dviroel|rover|lunch16:15
fungiso it seems like all we really need at this point is a mechanism for adding the sites/domains in django and then restarting the containers maybe? (possible the container restart is only needed after initial installation and not on addition of new domains)16:16
lucasagomesfungi, I see... I will keep an eye on it on future updates in the doc. If needed I will ask you to accept the invitation16:16
lucasagomesfungi, thanks for the heads up!16:16
fungilucasagomes: note that the invite has a two-week expiration on it, so try to let us know before then16:17
lucasagomesfungi, will do16:17
lucasagomeswe have a patch changing the docs right now16:17
lucasagomesshould be merged this week16:17
fungilucasagomes: basically, rtd rebuilds the content periodically if it notices the repository has changed, the zuul job hits a webhook to tell it to check immediately, so in theory you should see the content update within a few minutes of the job running16:18
lucasagomesfungi, I see... if that's the case there's no need to have the openstack user as admin. I just added because I had another project on rtd where that openstack user was the admin16:21
lucasagomesso I mimic'd it16:21
lucasagomesbut should be fine then16:21
lucasagomeslemme revoke the invitation and see if it will work without it16:22
*** marios is now known as marios|out16:54
*** dviroel|rover|lunch is now known as dviroel|rover17:20
*** jpena is now known as jpena|off17:39
clarkbfungi: so if I'm parsing the mm3 stuff correctly these are the steps we need to take: deploy mm3 with site id = 1 and create vhosts, switch site id to 0, add django web host entries for each domain, restart containers?17:47
funginah, the only thing that needed SITE_ID=1 is the initial database setup17:50
fungiso the current change temporarily switches to SITE_ID=1, does the db setup, then switches back to SITE_ID=0 and proceeds with the deployment17:51
fungithere's a flagfile created at ~mailman/bootstrapped which indicates that initial db setup has been performed so that ansible knows not to toggle the config again after the initial deployment17:52
clarkbgotcha so is the process more like set site_id = 1, create db, set site_id = 0, deploy the rest of mm3, update web hosts in django, restart containers?17:53
fungii think so, yes17:57
fungiat least i did those later steps on the held node and they seemed to work. i didn't touch settings.py at all17:57
fungihttps://review.opendev.org/867987 is the current state of that17:57
fungilooks like i added the config revert after the list creation step, but can try moving it earlier17:58
fungithe biggest unknown is what is involved in creating sites and domains through django's api instead of the admin webui18:00
clarkbfungi: there aer a number of django commands through the management script iirc. Its possible there is one that might do this?18:03
clarkbif not we may need to insert into the db directly (ugh). I think I looked at the mm3 api and it doesn't have it exposed that way18:04
opendevreviewJeremy Stanley proposed opendev/infra-manual master: General updates for the Test Environment chapter  https://review.opendev.org/c/opendev/infra-manual/+/87220118:17
clarkbfungi: looks like the django-mailman3 repo's MailDomain has a foreign key on django.contrib.sites.models.Site and it is this that we need to update (both the mailmaindomain table and the site table)18:17
fungiyeah, adding them through the admin webui you can clearly see the dependency (once you create sites, you can choose them in the drop-down while creating domains)18:19
clarkbI'm not seeing any obvious command line tools for this :/18:20
fungiargh18:20
fungii can ask on the ml18:20
clarkbhttps://pypi.org/project/django-api-admin/ would seem to indicate that django doesn't generically expose this if there is a special library made to do some of this18:21
fungii asked on mailman-users, maybe someone will come back with a less fragile solution18:32
fungihttps://lists.mailman3.org/archives/list/mailman-users@mailman3.org/message/OHAKSEYYL6REAHWEEZZRA7YZM4O7DPLH/18:33
fungiclarkb: revisiting https://lists.mailman3.org/archives/list/mailman-users@mailman3.org/thread/JCRYKMJKCRAWNTZPGH2MKGJPGPOMCHJZ/ the last message seems to indicate that poking things directly into the db is indeed the alternative to using the admin webui, but hopefully a response to my post will confirm or refute that18:40
clarkback18:50
fungialso i think my addition of a mailman.opendev.org site didn't do anything because we don't create it if we don't have any lists to add to it, so i'll update the change to make a test list and see if that works18:54
opendevreviewClark Boylan proposed opendev/system-config master: Revert "Update git in gitea images"  https://review.opendev.org/c/opendev/system-config/+/87220919:11
opendevreviewClark Boylan proposed opendev/system-config master: Revert "hound: use updated git packages"  https://review.opendev.org/c/opendev/system-config/+/87221119:13
clarkbif you accidentall use -d instead of -t git-review fails, but without a clear indication of why (potential improvement here if someone has time)19:14
fungioh -d without a change number?19:15
opendevreviewClark Boylan proposed opendev/system-config master: Flip Gerrit image to install distro git  https://review.opendev.org/c/opendev/system-config/+/87221219:17
clarkbfungi: ya -d foo_topic19:17
clarkbmy typing driver failed me there :)19:18
clarkbits a super minor thing, just thought I'd mention it19:19
ianw#status log ran backup prune on backup02.ca-ymq-120:43
opendevstatusianw: finished logging20:43
opendevreviewAde Lee proposed zuul/zuul-jobs master: Add ubuntu to enable-fips role  https://review.opendev.org/c/zuul/zuul-jobs/+/86688121:35
opendevreviewAde Lee proposed openstack/project-config master: Add base openstack FIPS job  https://review.opendev.org/c/openstack/project-config/+/87222221:36
*** dviroel|rover is now known as dviroel|out21:54
opendevreviewIan Wienand proposed zuul/zuul-jobs master: promote-docker-image: also accept OCI manifest formats  https://review.opendev.org/c/zuul/zuul-jobs/+/87223022:40
opendevreviewMerged zuul/zuul-jobs master: promote-docker-image: also accept OCI manifest formats  https://review.opendev.org/c/zuul/zuul-jobs/+/87223023:03
*** rlandy is now known as rlandy|bbl23:14
opendevreviewMerged opendev/system-config master: Revert "hound: use updated git packages"  https://review.opendev.org/c/opendev/system-config/+/87221123:56

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!