Friday, 2022-09-09

fungias an aside, i notice by default hyperkitty is using gravatar for user icons00:01
clarkblooks like the colibri fix may have worked00:01
clarkbthat is probably good for revie wnow if we want to assume that we just need a random keystore and try that00:01
clarkbotherwise maybe a hold and try to test?00:02
fungii won't really be able to test tomorrow, except possibly in the evening, but sounds worth rolling out00:03
fungiapparently we should be able to set HYPERKITTY_ENABLE_GRAVATAR to false00:05
*** ysandeep|out is now known as ysandeep00:08
fungii've got a revision prepped for that, just want to wait for the imports to complete before i test adding it on the held server00:17
fungiopenstack-stable-maint import is taking a long time at the "Computing thread structure" step00:54
fungireal    68m42.988s00:56
fungifor just that one ml00:56
fungijust the import step00:57
fungiIndexing 99549 emails00:57
fungiit will probably hit 100k messages next week at the current rate00:57
fungiby comparison, openstack-discuss less than a third of that, at 30358 messages00:59
fungistill indexing01:09
fungi18m23.747s to index it01:14
fungiso roughly 90 minutes to import and index that ml01:15
fungirelease-announce is not exactly small either01:25
fungi13321 messages in it01:31
fungifull site import required 2h24m36s wall clock time01:39
opendevreviewJeremy Stanley proposed opendev/system-config master: Add a mailman3 list server  https://review.opendev.org/c/opendev/system-config/+/85124801:40
fungiclarkb: ^ that worked to turn off gravatar integration01:40
*** rlandy|bbl is now known as rlandy02:03
*** rlandy is now known as rlandy|out02:06
opendevreviewIan Wienand proposed opendev/system-config master: bootstrap-bridge: drop pip3 role  https://review.opendev.org/c/opendev/system-config/+/85659302:47
opendevreviewIan Wienand proposed opendev/system-config master: bootstrap-bridge: drop pip3 role  https://review.opendev.org/c/opendev/system-config/+/85659303:40
*** ysandeep is now known as ysandeep|out03:42
opendevreviewIan Wienand proposed opendev/system-config master: [wip] bootstrap-bridge: drop pip3 role/add venv  https://review.opendev.org/c/opendev/system-config/+/85659304:51
*** ykarel|away is now known as ykarel05:01
opendevreviewIan Wienand proposed opendev/system-config master: [wip] bootstrap-bridge: drop pip3 role/add venv  https://review.opendev.org/c/opendev/system-config/+/85659305:06
opendevreviewIan Wienand proposed opendev/system-config master: install-ansible: remove testinfra version install workaround  https://review.opendev.org/c/opendev/system-config/+/85247505:16
opendevreviewIan Wienand proposed opendev/system-config master: testinfra: install with ansible extras  https://review.opendev.org/c/opendev/system-config/+/85247605:16
opendevreviewIan Wienand proposed opendev/system-config master: install-ansible: remove stevedore workaround  https://review.opendev.org/c/opendev/system-config/+/85247705:16
opendevreviewIan Wienand proposed opendev/system-config master: install-ansible: remove stub install for ARA  https://review.opendev.org/c/opendev/system-config/+/85247805:16
opendevreviewIan Wienand proposed opendev/system-config master: [wip] bootstrap-bridge: drop pip3 role/add venv  https://review.opendev.org/c/opendev/system-config/+/85659305:16
fricklerclarkb: thx for the reminder, cleaned those up05:48
fricklerhttps://zuul.opendev.org/t/openstack/nodes still shows node 0027999262 being held since 8 months now without a matching autohold, seems it got leaked somehow?05:49
*** pojadhav|out is now known as pojadhav06:00
*** chkumar|ruck is now known as chandankumar06:29
*** ysandeep|out is now known as ysandeep06:49
opendevreviewIan Wienand proposed opendev/system-config master: [wip] bootstrap-bridge: drop pip3 role/add venv  https://review.opendev.org/c/opendev/system-config/+/85659306:56
opendevreviewBrin Zhang proposed openstack/project-config master: Add api-ref-job for Venus  https://review.opendev.org/c/openstack/project-config/+/85638207:19
*** jpena|off is now known as jpena07:35
elodillesfungi clarkb : yes, unfortunately, anytime something is fixed and periodic-stable fail count goes down comes another issue :/ for now we have ~44 new failing job on ussuri+train because of new virtualenv release (with new setuptools version bundled in for py37 as well)... I've pushed fix for them already, but needs stable maintainers to review ( https://review.opendev.org/q/topic:cap-virtualenv-py37 07:37
elodilles)...07:37
elodilles(and of course those cover up another gate issues with test jobs that are not covered with periodic-stable...)07:39
opendevreviewBrin Zhang proposed openstack/project-config master: Add api-ref-job for Venus  https://review.opendev.org/c/openstack/project-config/+/85638207:40
*** bhagyashris is now known as bhagyashris|ruck07:57
opendevreviewMerged openstack/project-config master: Add api-ref-job for Venus  https://review.opendev.org/c/openstack/project-config/+/85638207:58
fricklermaybe we should rename the pipeline to "periodic-unstable" ;)08:02
opendevreviewIan Wienand proposed opendev/system-config master: [wip] bootstrap-bridge: drop pip3 role/add venv  https://review.opendev.org/c/opendev/system-config/+/85659308:03
*** arxcruz|rover is now known as arxcruz08:10
opendevreviewFrikin Evgenii proposed openstack/diskimage-builder master: Add variable for check installing python3 in yum element  https://review.opendev.org/c/openstack/diskimage-builder/+/85657708:16
opendevreviewFrikin Evgenii proposed openstack/diskimage-builder master: Add variable for check installing python3 in yum element  https://review.opendev.org/c/openstack/diskimage-builder/+/85657708:20
elodillesfrickler: well, it is stable, as it breaks on a stable way :)08:27
*** ysandeep is now known as ysandeep|lunch08:29
*** ysandeep|lunch is now known as ysandeep09:48
gthiemongeHi, we have a failure in the octavia tips jobs on ussuri, I don't understand what happens: https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_2a7/856399/1/check/octavia-tox-py37-tips/2a7226f/tox/py37-siblings.txt09:53
gthiemongeany ideas?09:53
fricklergthiemonge: I think that is what elodilles mentioned earlier?09:55
gthiemongeoops i should have read the backlog09:56
gthiemongefrickler: thanks09:57
gthiemongeI don't see a patch for octavia, I'll propose one, thanks09:57
fungigthiemonge: did you notice that because you've been paying attention to the periodic stable job failures, or did it not come to your attention until someone tried to merge a backport? i'm curious how much the various project teams pay attention to periodic job runs, if at all10:22
fricklerfungi: that log at least came from a dedicated test patch, so kind of a mixture of both I guess https://review.opendev.org/c/openstack/octavia/+/856399/10:23
fungii guess triggered by a failure someone observed in check and then pushed in order to figure out if it was caused by the patch or not10:24
fungiperhaps without knowing they could just check the periodic job results which were also failing10:24
fricklerthey didn't fail, because only the -tips variant seems to be affected https://zuul.opendev.org/t/openstack/builds?project=openstack%2Foctavia&pipeline=periodic-stable&skip=010:25
fungiinteresting10:30
fungiand yeah, they're not running any -tips jobs in periodic-stable10:32
fungiso i guess my question is a bit more academic. would they have noticed if it were being run periodically? maybe elodilles would have noticed and pushed a patch for them at least10:33
fungibut if nothing else, that probably explains why they didn't get one10:34
*** rlandy|out is now known as rlandy10:34
gthiemongefungi: I noticed it in some backports to stable/ussuri10:57
fungigot it. thanks for confirming!10:59
*** pojadhav is now known as pojadhav|sick11:05
*** ysandeep is now known as ysandeep|brb13:19
*** dasm|off is now known as dasm13:20
*** ysandeep|brb is now known as ysandeep13:30
*** ysandeep is now known as ysandeep|out14:37
*** dviroel is now known as dviroel|lunch14:52
opendevreviewJeremy Stanley proposed opendev/system-config master: Redirect openstack-infra specs to opendev docs  https://review.opendev.org/c/opendev/system-config/+/85682815:21
opendevreviewMerged openstack/project-config master: Add STS-Silicom app to StarlingX  https://review.opendev.org/c/openstack/project-config/+/85653315:35
clarkbfungi: looking at mailman list configuration options at https://docs.mailman3.org/projects/mailman/en/latest/src/mailman/rest/docs/listconf.html one thing we could possibly do is set all lists to not be advertised and not accept posts15:35
clarkbthen assuming the import will override those settings we'll be good and new lists can avoid setting the "disable" flag in the first place15:35
clarkbThere doesn't seem to be a "this list exists but does not operate" flag though15:36
clarkbI think the set advertised to false and allow_list_posts to false option ins't perfect because people could still potentially subcsribe before migration and maybe the migration would undo their subscription if not subscribed on the mm2 side? also the list admin could potential start it up15:37
clarkbAnother option is that we script up the list creation and add it to the import process. I kinda like this option because it helps keep control on the migration side of things15:38
clarkbI know you're AFK for much of today so no rush. But if you can give that some thought and let me know what you think I'm happy to try and update changes or import scripts etc to match15:38
fungiclarkb: alternative... we could just make separate child changes which add the lists for those sites and then if we want to test with them we hold a node from the one at the end of the series, but keep them wip and merge when we're ready for each migration?15:43
fungiless worried about the test nodes because we block outbound e-mail anyway15:44
clarkbya that should work too. Then merge the changes near when we plan to migrate each site15:45
clarkbseparately my java keystore is apparently not a valid keystore in the jvb change :?15:45
clarkbhttps://zuul.opendev.org/t/openstack/build/e58057733ac94114aae84c31c2e8f26c/log/meetpad01.opendev.org/docker/jitsi-meet-docker_jvb_1.txt#4115:46
*** dviroel|lunch is now known as dviroel15:53
opendevreviewClark Boylan proposed opendev/system-config master: Update colibri for all the JVBs  https://review.opendev.org/c/opendev/system-config/+/85655315:55
clarkbI'm hoping it was just a case of container vs host paths being different for the same bind mounted file15:55
*** marios is now known as marios|out15:59
opendevreviewBrian Rosmaita proposed opendev/system-config master: Redirect openstack-specs to git repo on opendev  https://review.opendev.org/c/opendev/system-config/+/85683416:19
*** jpena is now known as jpena|off16:35
clarkbok the colibri websocket jvb change looks happy now16:50
clarkbI think that is ready for review and cautious testing/approval16:50
funginice!16:55
fungii can try to look at it once i'm home (definitely not from this rest area parking lot)16:56
clarkbits still an open quetion if the keystore needs coordinated content for authentication/trust, but otherwise I don't see errors in startup16:57
clarkbfungi: I've just double checked on the lists stuff and we do actually define them all on the test node. It was only for the production server where I removed them17:29
clarkbfungi: so they should all be there. When you get a chance can you point at the error you saw?17:29
fungiclarkb: this was the import log... https://paste.opendev.org/show/81666818:48
fungiunfortunately i seem to have missed redirecting stderr18:48
fungiso i guess i didn't capture it18:48
fungibut i can do another run for another site real quick and do so18:49
fungii think it's not happening for every list18:50
fungifor example, i saw it when importing legal-discuss@lists.openstack.org18:51
clarkboh its happening for the lists we don't create? I took the list from the existing mm2 ansible18:55
fungiyeah, that was never complete18:56
clarkbfungi: is our list in ansible incomplete then? That may be the issue we need to reconcile it against relaity. Probably best to start iwth mm2 and then port that over to the mm3 change18:56
fungiwe used to just treat the puppet manifest as a convenient way to ask the automation to create a new list, but we never added older lists to it18:57
fungii can update the ansible to add all the missing ones if that's desirable (seems like it probably is for creating the new server, at least)18:58
clarkbya I think at the very least we seem to want it for the mm3 change18:59
fungithough that's also an even better reason to split the list creation up into different changes per-site. i'm hoping to do one more pass of retiring unused mailing lists for openstack before we end up migrating lists nobody is using, so that would give us time to do final cleanup between when opendev/zuul have earlier migrations and when the other sites move19:02
fungii'm happy to do the splitting too while i'm cross-checking all the lists19:03
clarkbfungi: I think we should continue to have all the lists in hte test node. Then have split up changes for the prod node host vars adding each site19:03
fungioh, right that makes sense19:04
fungiyes i agree with that19:04
fungii didn't dawn on me those were separated19:04
clarkbI think the mm3 change as is only has opendev lists in the prod hots vars but attempted to have all lists on the test node (and failed at that because the mm2 listing is incomplete)19:04
fungisure, i can add them19:06
*** dviroel is now known as dviroel|out20:02
fungiclarkb: do we want to migrate https://lists.openstack.org/cgi-bin/mailman/listinfo/test-list do you think? or create a new one for lists.opendev.org?21:40
fungiit has no posts, apparently21:41
fungii'm leaning toward just deleting it21:41
opendevreviewJeremy Stanley proposed opendev/system-config master: Add a mailman3 list server  https://review.opendev.org/c/opendev/system-config/+/85124821:50
clarkbfungi: ++ to cleaning it up21:50
fungiclarkb: ^ that's adding all the missing lists and alpha-ordering everything (sites and lists) except for the opendev site is still first since that becomes the default domain21:51
fungia bit of cleanup we could consider is moving the listdomain parameter into the site name since the name is no longer used21:53
fungiso just name the sites by their fqdns in the future21:53
funginot urgent we do it before migration though21:53
fungi#status log Deleted unused OpenStack "test-list" mailing list21:55
opendevstatusfungi: finished logging21:55
fungii can't find any reference to it in our configs, so it may have been created manually21:55
clarkbfungi: we do use it for the apache vhost aliases22:00
clarkbit being listdomain. We might be able to refactor that though22:01
clarkbfungi: for those new lists you can set 'private: true' in the list def for private lists. I thought I had done that before for osme lists but apparently not22:14
clarkbfungi: I know it isn't critical for these lists due to the import process but still probably a good idea to serve as an exampl eto new lists and to help us keep that metadta straight22:14
fungiclarkb: yeah, that's what i mean, refactor to use name instead, and move the listdomain values into name, since the current name is useless, but yes it's not super important22:27
clarkbfungi: right its useless for creating lists but those values inform the apache vhost config as to what aliases to accept22:27
clarkbotherwise the vhosting wouldn't work22:27
fungiand yeah, we can add the private flag, i don't really have an opinion, stuff like description and owner already gets out of sync quickly22:27
fungiclarkb: the listdomain values are important yes, i mean the name values aren't22:28
fungithey used to be the names of the separate directory trees we used for the old multi-domain setup we had22:28
clarkboh I see the name on the site22:29
clarkbya I'm not sure if we use that anywhere anymore. Probably not22:29
fungithough maybe we could just drop the "name" parameters entirely, i don't suppose we need to play musical variable names22:29
opendevreviewJeremy Stanley proposed opendev/system-config master: Add a mailman3 list server  https://review.opendev.org/c/opendev/system-config/+/85124823:04
fungiclarkb: ^ flagged the private lists and also tried removing the name params to see if it still works23:04
clarkbthanks23:05
*** dasm is now known as dasm|off23:13

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