Tuesday, 2021-06-08

ianwi feel like i've held nodes and inspected the mariadb to ensure it's connecting, but not before recent changes.  will look00:00
ianwi'm not sure what that backup comment i made was.  we don't need the on-disk db as you say, only the dump data.  i will clarify that00:02
clarkbianw: but also the exclude isn't on both hosts (its on the host I don't expect will run a mariadb?)00:04
clarkbbasically we want 01 to be pretty steady state I think then 02 to run mariadb? but maybe I'm wrong about your plan there00:05
ianwi think at one point i thought we should switch the production db on 01 to mariadb first to validate the whole thing00:05
ianwbut at this point i think we don't bother with that00:06
clarkbya and I think memory is tighter than ever. But also I thought that was your intention as 01 doesn't switch the db type in that change00:06
clarkbit seems to try to leave 01 alone as much as possible, except for that exclude being on 01 and not 0200:06
ianwyep, that's the current plan.  thanks, good catch on the inconsistencies there00:08
ianwi'll add a follow-on to switch review02 to mariadb and we can evaluate that as a separate step.00:11
ianwi'm pretty sure i already made a script to import the data from the old trove instance00:11
opendevreviewIan Wienand proposed opendev/system-config master: gerrit: add mariadb_container option  https://review.opendev.org/c/opendev/system-config/+/77596100:15
opendevreviewIan Wienand proposed opendev/system-config master: review02 : switch reviewdb to mariadb_container type  https://review.opendev.org/c/opendev/system-config/+/79519200:18
ianwi agree it's hard to see obvious signs that gerrit connected to the database during testing.  i wonder if it only does after files are reviewed?00:29
ianwi'll try to jump on the host under test now and confirm something in the db00:29
opendevreviewMerged opendev/system-config master: Disable eavesdrop direct publishing job  https://review.opendev.org/c/opendev/system-config/+/79501500:34
opendevreviewMerged opendev/system-config master: Disable puppet runs on eavesdrop  https://review.opendev.org/c/opendev/system-config/+/79405100:34
opendevreviewMerged opendev/system-config master: Add eavesdrop01.opendev.org server  https://review.opendev.org/c/opendev/system-config/+/79500600:40
*** bhagyashris_ has joined #opendev01:01
*** bhagyashris has quit IRC01:09
*** ykarel|away has joined #opendev03:48
opendevreviewMerged opendev/meetbot master: Switch to setuptools  https://review.opendev.org/c/opendev/meetbot/+/79371603:55
opendevreviewMerged opendev/meetbot master: Update to Python 3 / Limnoria plugin  https://review.opendev.org/c/opendev/meetbot/+/79386903:55
opendevreviewMerged opendev/meetbot master: Remove commented out getattr failure  https://review.opendev.org/c/opendev/meetbot/+/79387303:55
opendevreviewMerged opendev/meetbot master: Update tests  https://review.opendev.org/c/opendev/meetbot/+/79387403:55
opendevreviewMerged opendev/meetbot master: Fix old file() alias for open()  https://review.opendev.org/c/opendev/meetbot/+/79387503:56
opendevreviewMerged opendev/meetbot master: Remove encode/decode wrappers  https://review.opendev.org/c/opendev/meetbot/+/79387603:56
*** ykarel_ has joined #opendev04:15
*** ykarel|away has quit IRC04:21
*** hashar has joined #opendev04:23
opendevreviewMerged opendev/system-config master: Move gerritbot/accessbot to new eavesdrop server  https://review.opendev.org/c/opendev/system-config/+/79501304:23
*** ykarel_ is now known as ykarel04:25
*** opendevreview has quit IRC04:26
*** opendevreview has joined #opendev04:37
opendevreviewIan Wienand proposed opendev/system-config master: Cleanup ask.openstack.org  https://review.opendev.org/c/opendev/system-config/+/79520704:37
ianw^ that came from eavesdrop01.opendev.org; so the gerritbot migration is complete04:37
ianw#status log gerritbot now running from eavesdrop01.opendev.org04:37
opendevstatusianw: finished logging04:37
*** marios has joined #opendev05:06
*** ysandeep has joined #opendev05:11
*** ysandeep is now known as ysandeep|ruck05:11
*** hashar has quit IRC05:40
opendevreviewIan Wienand proposed opendev/system-config master: Run statusbot from eavesdrop01.opendev.org  https://review.opendev.org/c/opendev/system-config/+/79521305:58
* yoctozepto getting 503 on https://meetings.opendev.org/meetings/06:02
ianwyoctozepto: hrm, looking06:04
ianwoh, i think i've dropped it from the webservers group06:06
yoctozeptonice06:06
*** bhagyashris_ is now known as bhagyashris06:08
opendevreviewIan Wienand proposed opendev/system-config master: Restore eavesdrop01.openstack.org to webservers group  https://review.opendev.org/c/opendev/system-config/+/79521506:09
ianwyoctozepto: should be able to access it now, i'll merge ^ to restore it for good06:17
yoctozeptoianw: thanks06:18
*** david-lyle has joined #opendev06:20
*** dklyle has quit IRC06:20
*** ralonsoh has joined #opendev06:34
*** Adri2000 has joined #opendev06:48
*** david-lyle has quit IRC06:50
*** gibi has quit IRC06:55
*** gibi has joined #opendev06:55
opendevreviewMerged opendev/system-config master: Restore eavesdrop01.openstack.org to webservers group  https://review.opendev.org/c/opendev/system-config/+/79521506:56
*** odyssey4me has joined #opendev06:59
*** odyssey4me has quit IRC07:00
*** odyssey4me has joined #opendev07:01
*** ykarel_ has joined #opendev07:01
*** ykarel has quit IRC07:03
*** amoralej|off is now known as amoralej07:20
*** andrewbonney has joined #opendev07:29
*** rpittau|afk is now known as rpittau07:30
*** jpena|off is now known as jpena07:31
*** hashar has joined #opendev07:49
*** elodilles is now known as elodilles_pto07:55
*** ykarel_ is now known as ykarel08:00
*** ysandeep|ruck is now known as ysandeep|lunch08:03
*** lucasagomes has joined #opendev08:25
*** tosky has joined #opendev08:39
*** lucasagomes has quit IRC08:41
*** ysandeep|lunch has quit IRC08:52
*** lucasagomes has joined #opendev08:53
*** ysandeep has joined #opendev08:55
opendevreviewIan Wienand proposed opendev/system-config master: Run statusbot from eavesdrop01.opendev.org  https://review.opendev.org/c/opendev/system-config/+/79521308:58
*** ysandeep is now known as ysandeep|ruck09:02
*** lucasagomes has quit IRC09:03
*** lucasagomes has joined #opendev09:25
opendevreviewIan Wienand proposed opendev/system-config master: Run statusbot from eavesdrop01.opendev.org  https://review.opendev.org/c/opendev/system-config/+/79521309:34
*** lucasagomes has quit IRC09:38
*** lucasagomes has joined #opendev09:39
*** lucasagomes has quit IRC10:01
*** lucasagomes has joined #opendev10:18
*** lucasagomes has quit IRC10:29
ysandeep|ruck#opendev we are noticing many job failing while trying to install pip packages..10:30
ysandeep|ruckfor example: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_7ce/795155/6/check/tripleo-ci-centos-8-undercloud-containers/7cee319/job-output.txt10:30
ysandeep|ruck~~~10:30
ysandeep|ruck2021-06-08 10:24:01.508302 | primary | Looking in indexes: https://mirror.gra1.ovh.opendev.org/pypi/simple, https://mirror.gra1.ovh.opendev.org/wheel/centos-8-x86_6410:30
ysandeep|ruck2021-06-08 10:24:04.247476 | primary | ERROR: Could not find a version that satisfies the requirement selinux (from versions: none)10:30
ysandeep|ruck2021-06-08 10:24:04.247565 | primary | ERROR: No matching distribution found for selinux10:30
ysandeep|ruck~~~10:30
ysandeep|ruckanother example: https://d28efda77c4dcd7255de-ad135f72b22a132e11a904324fbc4e60.ssl.cf2.rackcdn.com/793383/9/gate/tripleo-ci-centos-8-containers-multinode/520ae4d/job-output.txt10:30
jrosserysandeep|ruck: fastly cdn is having a gigantic outage at the moment which is certainly affecting pypi.org10:32
ysandeep|ruckjrosser: ack thanks! for info10:33
*** lucasagomes has joined #opendev11:00
*** ysandeep|ruck is now known as ysandeep|afk11:02
*** lucasagomes has quit IRC11:08
*** odyssey4me has quit IRC11:16
*** ysandeep|afk is now known as ysandeep11:29
*** ysandeep is now known as ysandeep|ruck11:29
*** jpena is now known as jpena|lunch11:31
*** odyssey4me has joined #opendev12:01
opendevreviewMartin Chacon Piza proposed opendev/irc-meetings master: Update chair for Monasca Team Meeting  https://review.opendev.org/c/opendev/irc-meetings/+/79530612:06
*** dmitriis has joined #opendev12:20
*** jpena|lunch is now known as jpena12:31
dmitriiso/12:35
dmitriisI wonder if somebody with group creation rights could help me out with creation of a new group called "snaps-ci". I am a member of snaps-core https://review.opendev.org/admin/groups/49e83018fb79bbfa1116d89a8a60804a5dc64dca,members and I need this for adding label-Verified to12:35
dmitriishttps://opendev.org/openstack/project-config/src/branch/master/gerrit/acls/x/snaps.config. However, I do not have rights to create new groups in the opendev gerrit https://gerrit-review.googlesource.com/Documentation/cmd-create-group.html#_access. The group name is based on the one suggested in this guide12:35
dmitriishttps://docs.opendev.org/opendev/infra-manual/latest/creators.html#voting-third-party-ci. Thanks in advance!12:35
*** dmitriis has quit IRC12:43
*** dmitriis has joined #opendev12:43
opendevreviewMerged opendev/irc-meetings master: Update chair for Monasca Team Meeting  https://review.opendev.org/c/opendev/irc-meetings/+/79530612:44
*** amoralej is now known as amoralej|lunch13:00
*** lucasagomes has joined #opendev13:23
*** amoralej|lunch is now known as amoralej13:39
opendevreviewMohammed Naser proposed zuul/zuul-jobs master: Bump default Helm version to 2.17.0  https://review.opendev.org/c/zuul/zuul-jobs/+/79532913:40
*** elodilles_pto has quit IRC13:41
*** elodilles has joined #opendev13:43
mnaserfwiw just caught this --  "msg": "Data could not be sent to remote host \"188.212.109.213\". Make sure this host can be reached over ssh: ssh: connect to host 188.212.109.213 port 22: No route to host\r\n",13:47
opendevreviewBenjamin Schanzel proposed zuul/zuul-jobs master: Add a meta log upload role with a failover mechanism  https://review.opendev.org/c/zuul/zuul-jobs/+/79533613:56
*** lucasagomes has quit IRC13:59
opendevreviewBenjamin Schanzel proposed zuul/zuul-jobs master: Add a meta log upload role with a failover mechanism  https://review.opendev.org/c/zuul/zuul-jobs/+/79533613:59
mordreddmitriis: just push up a patch to that file in the project-config repo and pretend the group exists. the automation will create it as needed, and then someone here can add accounts to it as needed14:03
*** lucasagomes has joined #opendev14:03
dmitriismordred: will do, tyvm14:03
opendevreviewDmitrii Shcherbakov proposed openstack/project-config master: Add label-Verified to snaps.config  https://review.opendev.org/c/openstack/project-config/+/79534114:05
*** lucasagomes has quit IRC14:18
*** lucasagomes has joined #opendev14:21
*** ykarel has quit IRC14:41
*** ykarel has joined #opendev14:42
*** ykarel has quit IRC14:46
*** ykarel has joined #opendev14:47
opendevreviewDmitrii Shcherbakov proposed openstack/project-config master: Add label-Verified to snaps.config  https://review.opendev.org/c/openstack/project-config/+/79534114:48
*** dklyle has joined #opendev14:50
*** ysandeep|ruck is now known as ysandeep|away15:12
clarkbmnaser: is there any more context on that? Is this zuul trying to talk to a test node?15:18
mnaserclarkb: yep, just caught that in the console log before it retried15:19
opendevreviewMerged openstack/project-config master: Add label-Verified to snaps.config  https://review.opendev.org/c/openstack/project-config/+/79534115:21
fungidmitriis: that ^ should deploy shortly, and then i'll try to remember to set the ownership for it15:22
clarkbfungi: you need to set group ownership on that one not the initial seed member iirc15:23
dmitriisfungi: ack15:23
clarkb(we set -core to own -ci)15:23
fungiyep15:23
fungithat's the plan15:23
clarkbcool I was trying to remember how we did that when I reviewed the change15:23
clarkbmnaser: cool, system is working as expected then :)15:23
clarkbThe internet isn't, but zuul is15:24
fungipretty sure that's also the inetrnet working as much as expected, which is to say not entirely15:24
mordredI mean - the internet is working fine15:26
mordredthere is a company out there with some issues15:27
clarkbI hear fastly broke large portions of it earlier today15:27
mordredyeah. well - that's what I'm saying15:27
clarkbI don't think the IP above would've been impacted by that but who knows could've been some flooded pipes in the aftermatch15:27
clarkb*aftermath15:27
* fungi retreats to the far corners of arpanet15:27
opendevreviewMerged openstack/project-config master: New Project Request: airship/image-builder  https://review.opendev.org/c/openstack/project-config/+/79511815:31
fungiclarkb: did you see aj's question on https://review.opendev.org/792710 ?15:31
clarkbfungi: I had not. Looking now15:31
clarkbfungi: responded15:32
*** ysandeep|away has quit IRC15:33
*** ysandeep|away has joined #opendev15:34
fungidmitriis: done, let us know if you need anything else15:37
fungiclarkb: worth noting, there is no gerrit ssh cli syntax for changing group ownership, i ended up temporarily elevating my webui account to admin so i could do it there15:39
fungiand have demoted my account again now that i'm done15:39
fungiin other news, statusbot is still running crash-free foregrounded in a screen session... i suppose we can just leave it there until ianw has it going on the new server and then kill it15:41
* fungi shakes fist at heisenbug15:41
clarkbfungi: interesting, I think you can set it on group creation. Maybe we should consider modifying jeepyb, though we so rarely add -ci groups it shouldn't be a big deal15:42
*** whoami-rajat has quit IRC15:46
dmitriisfungi: tyvm, will do a check soon but the group page looks good (https://review.opendev.org/admin/groups/1565234147bc057b7ebe2f889162467d9d55447c)15:46
*** marios is now known as marios|out16:02
*** ykarel is now known as ykarel|away16:02
*** ykarel|away has quit IRC16:13
*** lucasagomes has quit IRC16:15
*** rpittau is now known as rpittau|afk16:31
*** jpena is now known as jpena|off16:33
opendevreviewMerged openstack/project-config master: Drop logstash filter jobs  https://review.opendev.org/c/openstack/project-config/+/79271016:37
*** ysandeep|away has quit IRC16:38
*** marios|out has quit IRC16:39
opendevreviewBalazs Gibizer proposed opendev/irc-meetings master: Move nova meeting to #openstack-nova  https://review.opendev.org/c/opendev/irc-meetings/+/79537716:41
opendevreviewGhanshyam proposed openstack/project-config master: Move devstack-gate jobs list to in-tree  https://review.opendev.org/c/openstack/project-config/+/79537916:51
*** ralonsoh has quit IRC17:01
*** ykarel|away has joined #opendev17:06
*** amoralej is now known as amoralej|off17:30
*** andrewbonney has quit IRC17:32
*** ykarel|away has quit IRC17:43
opendevreviewMerged opendev/system-config master: Forward openstack-security ML to openstack-discuss  https://review.opendev.org/c/opendev/system-config/+/79465917:50
fungi#status log Dumped archival copies of openstack-security ML configuration and subscriber list in /home/fungi on lists.o.o, then removed the ML from mailman (leaving archives intact)17:56
opendevstatusfungi: finished logging17:56
opendevreviewRajath Av proposed openstack/project-config master: Changes to be committed: modified: zuul.d/projects.yaml Change-Id: Ib712abce33b5f552965be874d9965a24eb7ec105  https://review.opendev.org/c/openstack/project-config/+/79539618:29
opendevreviewRajath Av proposed openstack/project-config master: Changes to be committed: modified: zuul.d/projects.yaml Change-Id: Ib712abce33b5f552965be874d9965a24eb7ec105 Adding this change as networking-infoblox plugin has been migrated to support python3.6  https://review.opendev.org/c/openstack/project-config/+/79539618:30
opendevreviewRajath Av proposed openstack/project-config master: Changes to be committed: modified: zuul.d/projects.yaml  https://review.opendev.org/c/openstack/project-config/+/79539718:42
opendevreviewRajath Av proposed openstack/project-config master: Changes to be committed: modified: zuul.d/projects.yaml  https://review.opendev.org/c/openstack/project-config/+/79539618:43
opendevreviewRajath Av proposed openstack/project-config master: Changes to be committed: modified: zuul.d/projects.yaml  https://review.opendev.org/c/openstack/project-config/+/79539718:45
*** rav_ has joined #opendev18:47
rav_https://review.opendev.org/c/openstack/project-config/+/79539719:09
clarkbrav_: sorry I had to switch to a meeting, but that latest version removes your pypi publishing jobs too which is fine if you don't want to publsih to pypi. If you do want to publish to pypi then you should only remove the '- openstack-python-jobs-neutron' line19:10
rav_Oh i see. I thought of adding both in .zuul.yaml thanks for pointing out, will make the necessary changes19:11
opendevreviewRajath Av proposed openstack/project-config master: Changes to be committed: modified: zuul.d/projects.yaml  https://review.opendev.org/c/openstack/project-config/+/79539719:14
*** stevebaker has quit IRC19:33
rav_Do someone need to verify workflow too? its my first commit on opendev so i dont know how this review system works.19:40
clarkbrav_: Verified is set by our CI system Zuul. Once that comes back +1 and two reviewrs +2 code review the change the second reviewer will +1 Workflow  to approve the change. At that point zuul tests it again against the current state of things and if that succeeds it is merged19:42
fungirav_: yes, one of our core reviewers will need to add a workflow +1 vote after zuul is happy with the change, but we're in our weekly meeting at the moment19:42
rav_alrighty.. Got it19:42
rav_thanks for your time19:42
*** rav_ has quit IRC19:51
opendevreviewMerged openstack/project-config master: Changes to be committed: modified: zuul.d/projects.yaml  https://review.opendev.org/c/openstack/project-config/+/79539719:55
clarkbinfra-root doing a quick check we do indeed need to update our plugin versions on our gerrit builds. I'll push a change up to do that then rebase the x/ path handling cleanup on that then we should have something mergeable19:58
ianwclarkb: on 775961 i think i misunderstood, you're saying you think we should also backup the mariadb to a dump file on disk, as well as to the remote servers?20:03
ianwi personally probably don't think it's necessary because it's quite easy to retrieve from borg, but i can add it if you feel strongly20:03
ianwinfra-root: oh i meant to also call out https://review.opendev.org/c/opendev/system-config/+/792866 to update bridge to the latest ansible (4.0.0, which is really 2.11 "ansible" as we know it, with the other bits added on)20:05
ianwthat should be fine now, as we've updated ARA so it is compatible20:06
opendevreviewClark Boylan proposed opendev/system-config master: Remove special x/ handling patch in gerrit  https://review.opendev.org/c/opendev/system-config/+/79199520:07
opendevreviewClark Boylan proposed opendev/system-config master: Update Gerrit plugin versions for Gerrit 3.2 and 3.3  https://review.opendev.org/c/opendev/system-config/+/79540920:07
clarkbianw: yes, becauase it can be helpful to not need to go remote to grab that in some situations. But I don't think it is critical20:08
clarkbinfra-root ok https://review.opendev.org/c/opendev/system-config/+/795409 updates gerrit plugin versiosn then https://review.opendev.org/c/opendev/system-config/+/791995 unforks us. If those look good to you I think we can land those and plan for a gerrit restart20:08
clarkbthis will pull in the newer zuul status dashboard plugin too20:08
clarkbianw: if you'd prefer to not have local copies then I can switch to +2 at this point I think20:09
ianwclarkb: i'm not sure it's worth managing; how about i update with a pre-canned example of how to pull the dump from borg?20:10
clarkbianw: ++20:11
opendevreviewTristan Cacqueray proposed zuul/zuul-jobs master: fetch-translation-output: introduce zuul_use_fetch_output  https://review.opendev.org/c/zuul/zuul-jobs/+/68188720:12
opendevreviewTristan Cacqueray proposed zuul/zuul-jobs master: fetch-javascript-content-tarball: introduce zuul_use_fetch_output  https://review.opendev.org/c/zuul/zuul-jobs/+/68190320:12
*** stevebaker has joined #opendev20:19
opendevreviewIan Wienand proposed opendev/system-config master: review02 : switch reviewdb to mariadb_container type  https://review.opendev.org/c/opendev/system-config/+/79519220:21
ianwargh put it in the wrong change20:21
opendevreviewIan Wienand proposed opendev/system-config master: gerrit: add mariadb_container option  https://review.opendev.org/c/opendev/system-config/+/77596120:25
opendevreviewIan Wienand proposed opendev/system-config master: review02 : switch reviewdb to mariadb_container type  https://review.opendev.org/c/opendev/system-config/+/79519220:25
ianwclarkb: ^ that is what we discussed now20:25
clarkbcool I'll try to dive into reviews for all those called out changes when I get back from a bike ride and some food20:25
ianwthanks; the meetbot ones are probably the most interseting20:26
opendevreviewTristan Cacqueray proposed zuul/zuul-jobs master: fetch-translation-output: introduce zuul_use_fetch_output  https://review.opendev.org/c/zuul/zuul-jobs/+/68188720:26
opendevreviewTristan Cacqueray proposed zuul/zuul-jobs master: fetch-javascript-content-tarball: introduce zuul_use_fetch_output  https://review.opendev.org/c/zuul/zuul-jobs/+/68190320:26
opendevreviewTristan Cacqueray proposed zuul/zuul-jobs master: fetch-translation-output: introduce zuul_use_fetch_output  https://review.opendev.org/c/zuul/zuul-jobs/+/68188720:43
opendevreviewTristan Cacqueray proposed zuul/zuul-jobs master: fetch-javascript-output: introduce zuul_use_fetch_output  https://review.opendev.org/c/zuul/zuul-jobs/+/68174820:48
*** hashar has quit IRC21:06
opendevreviewTim Burke proposed zuul/zuul-jobs master: build-python-release: Add flag for whether to build a wheel or not  https://review.opendev.org/c/zuul/zuul-jobs/+/79541921:25
opendevreviewMerged zuul/zuul-jobs master: fetch-translation-output: introduce zuul_use_fetch_output  https://review.opendev.org/c/zuul/zuul-jobs/+/68188721:44
*** cloudnull has quit IRC21:50
*** cloudnull has joined #opendev21:51
*** dmitriis has quit IRC22:02
opendevreviewIan Wienand proposed opendev/system-config master: Run statusbot from eavesdrop01.opendev.org  https://review.opendev.org/c/opendev/system-config/+/79521322:13
*** dmitriis has joined #opendev22:13
ianwclarkb: if you want to go with your gerrit updates and get the images promoted, i can look at a quick restart in a few hours when it's usually very quiet22:43
clarkbfungi: ^ do you want to review those changes maybe?22:44
*** dmitriis has quit IRC22:44
clarkbfungi: specifically https://review.opendev.org/c/opendev/system-config/+/795409 and its child22:47
fungisure what's the topic string for them?22:48
clarkbtopic:check-gerrit-url-handling22:48
clarkbbut only the most recent two changes. The 3.4 change needs a reabse22:48
fungithanks lookin22:49
clarkbianw: re the eavesdrop/meetings stuff do we need to make an announcement of the fqdn change from eavesdrop to meetings?22:50
ianwi haven't as intended that everything redirected to exactly the same data22:52
fungiyeah, we can always post a "didja notice?" but really it should be transparent for everyone22:52
ianwbut we could; maybe let's get everything 100% migrated first?22:52
clarkbmeetings.opendev.org is a new canonical name though isn't it?22:52
clarkbI think once we cut over from eavesdrop.openstack.org to eavesdrop01.opendev.org people will notice since you'll only be able to access that info via meetings.opendev.org? (or maybe there is some plan to make eavesdrop.openstack.org go to the same location?)22:53
ianweavesdrop.openstack.org is redirecting22:54
ianwmeetings.opendev.org proxies /irclogs and /meetings to eavesdrop01 for now22:54
clarkbah I see in DNS we point eavesdrop.o.o at meetings.opendev.o and then we must haev a redirect. Ok ya I agree that should be transparent22:54
ianwi was half-thinking that we make something like "irclogs.opendev.org" to house them in the future, and possibly export the logs from eavesdrop01.opendev.org to AFS and just serve them statically22:55
clarkbI was looking for that when reviewing the chagnes but I guess its already there which is why I wasnt finding it22:55
*** dmitriis has joined #opendev22:55
ianw'eavesdrop' always seemed a bit weird for the meeting ICS files :)22:55
ianwi don't know if we have anything else that has long-opened AFS connections like limnoria would need to write directly to afs though?22:56
clarkbI think we'd only have that for reads when serving large files out of the mirrors22:56
ianwmost other things like mirror-update, executors, etc. just pull in a kerberos token when they need to write something on afs ad-hoc22:57
clarkbianw: fwiw I've gotten through all but the statusbot change so far. And now I need to eat some very late lunch22:57
clarkbwill try to review statusbot after too22:57
ianwthanks; yeah statusbot is still WIP and probably not working yet22:58
*** tosky has quit IRC23:03
opendevreviewMerged opendev/system-config master: Update Gerrit plugin versions for Gerrit 3.2 and 3.3  https://review.opendev.org/c/opendev/system-config/+/79540923:20
ianwcan anyone spot the unintended } in https://5b55da37df53ecd854ad-c393df920c3ba3d74686e2bffd0842f4.ssl.cf1.rackcdn.com/795213/4/check/system-config-run-eavesdrop/ab371c2/bridge.openstack.org/ara-report/results/322.html#msg?23:31
ianwoh i see it, some {% accidentally ending with }}23:32
clarkbianw: {% if statusbot_wiki_thankspageid }}23:32
clarkbtotal width is balanced because the left } should be a %23:32
ianwyeah; a line number would have been great thanks ansible23:33
opendevreviewIan Wienand proposed opendev/system-config master: limnoria/meetbot setup on eavesdrop01.opendev.org  https://review.opendev.org/c/opendev/system-config/+/79371923:35
opendevreviewIan Wienand proposed opendev/system-config master: Move meetbot config to eavesdrop01.opendev.org  https://review.opendev.org/c/opendev/system-config/+/79500723:35
opendevreviewIan Wienand proposed opendev/system-config master: Cleanup eavesdrop puppet references  https://review.opendev.org/c/opendev/system-config/+/79501423:35
opendevreviewIan Wienand proposed opendev/system-config master: Run statusbot from eavesdrop01.opendev.org  https://review.opendev.org/c/opendev/system-config/+/79521323:35
clarkbstatusbot is not a persistently running daemon right? we only run it when we have changes to make? Considering that i think having it in the ircbot container is probably fine then we can exec in and run statusbot23:35
clarkbif it was a daemon I'd suggest we not collocate them to avoid confusion over what the container is running23:36
ianwclarkb: thanks for review, that has addressed comments on 79371923:36
clarkblooking23:36
ianwit is a daemon; i'm proposing running it via entrypoint @ https://review.opendev.org/c/opendev/system-config/+/795213/5/playbooks/roles/statusbot/files/docker-compose.yaml23:37
clarkbah it is accesbot that I'm confusing with statusbot. Accessbot runs in the way I describe23:37
clarkbin that case I'll need to think about this and better understand the implications of running it that way23:38
clarkbianw: the service name on that docker-compose entry is gerritbot but should be statusbot23:38
ianwahh, yeah copy-paste.23:38
ianwi know that it should be a separate container23:39
clarkbya it will run as a separate container using the same image23:39
clarkbthat is probably ok as the concern is running multiple processes in a single container23:39
ianwi just don't know if we need another whole container in our life to handle this bot.  if people feel strongly i can pull it out from being a sibling into it's own container23:40
clarkbmordred: ^ you probably have teh strongest opinions on this stuff. What do you think23:40
opendevreviewIan Wienand proposed opendev/system-config master: Run statusbot from eavesdrop01.opendev.org  https://review.opendev.org/c/opendev/system-config/+/79521323:41
mordredclarkb: reading23:59
mordredI don't think it needs to be its own container image23:59

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