ianw | i feel like i've held nodes and inspected the mariadb to ensure it's connecting, but not before recent changes. will look | 00:00 |
---|---|---|
ianw | i'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 that | 00:02 |
clarkb | ianw: but also the exclude isn't on both hosts (its on the host I don't expect will run a mariadb?) | 00:04 |
clarkb | basically we want 01 to be pretty steady state I think then 02 to run mariadb? but maybe I'm wrong about your plan there | 00:05 |
ianw | i think at one point i thought we should switch the production db on 01 to mariadb first to validate the whole thing | 00:05 |
ianw | but at this point i think we don't bother with that | 00:06 |
clarkb | ya 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 change | 00:06 |
clarkb | it seems to try to leave 01 alone as much as possible, except for that exclude being on 01 and not 02 | 00:06 |
ianw | yep, that's the current plan. thanks, good catch on the inconsistencies there | 00:08 |
ianw | i'll add a follow-on to switch review02 to mariadb and we can evaluate that as a separate step. | 00:11 |
ianw | i'm pretty sure i already made a script to import the data from the old trove instance | 00:11 |
opendevreview | Ian Wienand proposed opendev/system-config master: gerrit: add mariadb_container option https://review.opendev.org/c/opendev/system-config/+/775961 | 00:15 |
opendevreview | Ian Wienand proposed opendev/system-config master: review02 : switch reviewdb to mariadb_container type https://review.opendev.org/c/opendev/system-config/+/795192 | 00:18 |
ianw | i 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 |
ianw | i'll try to jump on the host under test now and confirm something in the db | 00:29 |
opendevreview | Merged opendev/system-config master: Disable eavesdrop direct publishing job https://review.opendev.org/c/opendev/system-config/+/795015 | 00:34 |
opendevreview | Merged opendev/system-config master: Disable puppet runs on eavesdrop https://review.opendev.org/c/opendev/system-config/+/794051 | 00:34 |
opendevreview | Merged opendev/system-config master: Add eavesdrop01.opendev.org server https://review.opendev.org/c/opendev/system-config/+/795006 | 00:40 |
*** bhagyashris_ has joined #opendev | 01:01 | |
*** bhagyashris has quit IRC | 01:09 | |
*** ykarel|away has joined #opendev | 03:48 | |
opendevreview | Merged opendev/meetbot master: Switch to setuptools https://review.opendev.org/c/opendev/meetbot/+/793716 | 03:55 |
opendevreview | Merged opendev/meetbot master: Update to Python 3 / Limnoria plugin https://review.opendev.org/c/opendev/meetbot/+/793869 | 03:55 |
opendevreview | Merged opendev/meetbot master: Remove commented out getattr failure https://review.opendev.org/c/opendev/meetbot/+/793873 | 03:55 |
opendevreview | Merged opendev/meetbot master: Update tests https://review.opendev.org/c/opendev/meetbot/+/793874 | 03:55 |
opendevreview | Merged opendev/meetbot master: Fix old file() alias for open() https://review.opendev.org/c/opendev/meetbot/+/793875 | 03:56 |
opendevreview | Merged opendev/meetbot master: Remove encode/decode wrappers https://review.opendev.org/c/opendev/meetbot/+/793876 | 03:56 |
*** ykarel_ has joined #opendev | 04:15 | |
*** ykarel|away has quit IRC | 04:21 | |
*** hashar has joined #opendev | 04:23 | |
opendevreview | Merged opendev/system-config master: Move gerritbot/accessbot to new eavesdrop server https://review.opendev.org/c/opendev/system-config/+/795013 | 04:23 |
*** ykarel_ is now known as ykarel | 04:25 | |
*** opendevreview has quit IRC | 04:26 | |
*** opendevreview has joined #opendev | 04:37 | |
opendevreview | Ian Wienand proposed opendev/system-config master: Cleanup ask.openstack.org https://review.opendev.org/c/opendev/system-config/+/795207 | 04:37 |
ianw | ^ that came from eavesdrop01.opendev.org; so the gerritbot migration is complete | 04:37 |
ianw | #status log gerritbot now running from eavesdrop01.opendev.org | 04:37 |
opendevstatus | ianw: finished logging | 04:37 |
*** marios has joined #opendev | 05:06 | |
*** ysandeep has joined #opendev | 05:11 | |
*** ysandeep is now known as ysandeep|ruck | 05:11 | |
*** hashar has quit IRC | 05:40 | |
opendevreview | Ian Wienand proposed opendev/system-config master: Run statusbot from eavesdrop01.opendev.org https://review.opendev.org/c/opendev/system-config/+/795213 | 05:58 |
* yoctozepto getting 503 on https://meetings.opendev.org/meetings/ | 06:02 | |
ianw | yoctozepto: hrm, looking | 06:04 |
ianw | oh, i think i've dropped it from the webservers group | 06:06 |
yoctozepto | nice | 06:06 |
*** bhagyashris_ is now known as bhagyashris | 06:08 | |
opendevreview | Ian Wienand proposed opendev/system-config master: Restore eavesdrop01.openstack.org to webservers group https://review.opendev.org/c/opendev/system-config/+/795215 | 06:09 |
ianw | yoctozepto: should be able to access it now, i'll merge ^ to restore it for good | 06:17 |
yoctozepto | ianw: thanks | 06:18 |
*** david-lyle has joined #opendev | 06:20 | |
*** dklyle has quit IRC | 06:20 | |
*** ralonsoh has joined #opendev | 06:34 | |
*** Adri2000 has joined #opendev | 06:48 | |
*** david-lyle has quit IRC | 06:50 | |
*** gibi has quit IRC | 06:55 | |
*** gibi has joined #opendev | 06:55 | |
opendevreview | Merged opendev/system-config master: Restore eavesdrop01.openstack.org to webservers group https://review.opendev.org/c/opendev/system-config/+/795215 | 06:56 |
*** odyssey4me has joined #opendev | 06:59 | |
*** odyssey4me has quit IRC | 07:00 | |
*** odyssey4me has joined #opendev | 07:01 | |
*** ykarel_ has joined #opendev | 07:01 | |
*** ykarel has quit IRC | 07:03 | |
*** amoralej|off is now known as amoralej | 07:20 | |
*** andrewbonney has joined #opendev | 07:29 | |
*** rpittau|afk is now known as rpittau | 07:30 | |
*** jpena|off is now known as jpena | 07:31 | |
*** hashar has joined #opendev | 07:49 | |
*** elodilles is now known as elodilles_pto | 07:55 | |
*** ykarel_ is now known as ykarel | 08:00 | |
*** ysandeep|ruck is now known as ysandeep|lunch | 08:03 | |
*** lucasagomes has joined #opendev | 08:25 | |
*** tosky has joined #opendev | 08:39 | |
*** lucasagomes has quit IRC | 08:41 | |
*** ysandeep|lunch has quit IRC | 08:52 | |
*** lucasagomes has joined #opendev | 08:53 | |
*** ysandeep has joined #opendev | 08:55 | |
opendevreview | Ian Wienand proposed opendev/system-config master: Run statusbot from eavesdrop01.opendev.org https://review.opendev.org/c/opendev/system-config/+/795213 | 08:58 |
*** ysandeep is now known as ysandeep|ruck | 09:02 | |
*** lucasagomes has quit IRC | 09:03 | |
*** lucasagomes has joined #opendev | 09:25 | |
opendevreview | Ian Wienand proposed opendev/system-config master: Run statusbot from eavesdrop01.opendev.org https://review.opendev.org/c/opendev/system-config/+/795213 | 09:34 |
*** lucasagomes has quit IRC | 09:38 | |
*** lucasagomes has joined #opendev | 09:39 | |
*** lucasagomes has quit IRC | 10:01 | |
*** lucasagomes has joined #opendev | 10:18 | |
*** lucasagomes has quit IRC | 10:29 | |
ysandeep|ruck | #opendev we are noticing many job failing while trying to install pip packages.. | 10:30 |
ysandeep|ruck | for 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.txt | 10:30 |
ysandeep|ruck | ~~~ | 10:30 |
ysandeep|ruck | 2021-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_64 | 10:30 |
ysandeep|ruck | 2021-06-08 10:24:04.247476 | primary | ERROR: Could not find a version that satisfies the requirement selinux (from versions: none) | 10:30 |
ysandeep|ruck | 2021-06-08 10:24:04.247565 | primary | ERROR: No matching distribution found for selinux | 10:30 |
ysandeep|ruck | ~~~ | 10:30 |
ysandeep|ruck | another example: https://d28efda77c4dcd7255de-ad135f72b22a132e11a904324fbc4e60.ssl.cf2.rackcdn.com/793383/9/gate/tripleo-ci-centos-8-containers-multinode/520ae4d/job-output.txt | 10:30 |
jrosser | ysandeep|ruck: fastly cdn is having a gigantic outage at the moment which is certainly affecting pypi.org | 10:32 |
ysandeep|ruck | jrosser: ack thanks! for info | 10:33 |
*** lucasagomes has joined #opendev | 11:00 | |
*** ysandeep|ruck is now known as ysandeep|afk | 11:02 | |
*** lucasagomes has quit IRC | 11:08 | |
*** odyssey4me has quit IRC | 11:16 | |
*** ysandeep|afk is now known as ysandeep | 11:29 | |
*** ysandeep is now known as ysandeep|ruck | 11:29 | |
*** jpena is now known as jpena|lunch | 11:31 | |
*** odyssey4me has joined #opendev | 12:01 | |
opendevreview | Martin Chacon Piza proposed opendev/irc-meetings master: Update chair for Monasca Team Meeting https://review.opendev.org/c/opendev/irc-meetings/+/795306 | 12:06 |
*** dmitriis has joined #opendev | 12:20 | |
*** jpena|lunch is now known as jpena | 12:31 | |
dmitriis | o/ | 12:35 |
dmitriis | I 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 to | 12:35 |
dmitriis | https://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 guide | 12:35 |
dmitriis | https://docs.opendev.org/opendev/infra-manual/latest/creators.html#voting-third-party-ci. Thanks in advance! | 12:35 |
*** dmitriis has quit IRC | 12:43 | |
*** dmitriis has joined #opendev | 12:43 | |
opendevreview | Merged opendev/irc-meetings master: Update chair for Monasca Team Meeting https://review.opendev.org/c/opendev/irc-meetings/+/795306 | 12:44 |
*** amoralej is now known as amoralej|lunch | 13:00 | |
*** lucasagomes has joined #opendev | 13:23 | |
*** amoralej|lunch is now known as amoralej | 13:39 | |
opendevreview | Mohammed Naser proposed zuul/zuul-jobs master: Bump default Helm version to 2.17.0 https://review.opendev.org/c/zuul/zuul-jobs/+/795329 | 13:40 |
*** elodilles_pto has quit IRC | 13:41 | |
*** elodilles has joined #opendev | 13:43 | |
mnaser | fwiw 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 |
opendevreview | Benjamin Schanzel proposed zuul/zuul-jobs master: Add a meta log upload role with a failover mechanism https://review.opendev.org/c/zuul/zuul-jobs/+/795336 | 13:56 |
*** lucasagomes has quit IRC | 13:59 | |
opendevreview | Benjamin Schanzel proposed zuul/zuul-jobs master: Add a meta log upload role with a failover mechanism https://review.opendev.org/c/zuul/zuul-jobs/+/795336 | 13:59 |
mordred | dmitriis: 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 needed | 14:03 |
*** lucasagomes has joined #opendev | 14:03 | |
dmitriis | mordred: will do, tyvm | 14:03 |
opendevreview | Dmitrii Shcherbakov proposed openstack/project-config master: Add label-Verified to snaps.config https://review.opendev.org/c/openstack/project-config/+/795341 | 14:05 |
*** lucasagomes has quit IRC | 14:18 | |
*** lucasagomes has joined #opendev | 14:21 | |
*** ykarel has quit IRC | 14:41 | |
*** ykarel has joined #opendev | 14:42 | |
*** ykarel has quit IRC | 14:46 | |
*** ykarel has joined #opendev | 14:47 | |
opendevreview | Dmitrii Shcherbakov proposed openstack/project-config master: Add label-Verified to snaps.config https://review.opendev.org/c/openstack/project-config/+/795341 | 14:48 |
*** dklyle has joined #opendev | 14:50 | |
*** ysandeep|ruck is now known as ysandeep|away | 15:12 | |
clarkb | mnaser: is there any more context on that? Is this zuul trying to talk to a test node? | 15:18 |
mnaser | clarkb: yep, just caught that in the console log before it retried | 15:19 |
opendevreview | Merged openstack/project-config master: Add label-Verified to snaps.config https://review.opendev.org/c/openstack/project-config/+/795341 | 15:21 |
fungi | dmitriis: that ^ should deploy shortly, and then i'll try to remember to set the ownership for it | 15:22 |
clarkb | fungi: you need to set group ownership on that one not the initial seed member iirc | 15:23 |
dmitriis | fungi: ack | 15:23 |
clarkb | (we set -core to own -ci) | 15:23 |
fungi | yep | 15:23 |
fungi | that's the plan | 15:23 |
clarkb | cool I was trying to remember how we did that when I reviewed the change | 15:23 |
clarkb | mnaser: cool, system is working as expected then :) | 15:23 |
clarkb | The internet isn't, but zuul is | 15:24 |
fungi | pretty sure that's also the inetrnet working as much as expected, which is to say not entirely | 15:24 |
mordred | I mean - the internet is working fine | 15:26 |
mordred | there is a company out there with some issues | 15:27 |
clarkb | I hear fastly broke large portions of it earlier today | 15:27 |
mordred | yeah. well - that's what I'm saying | 15:27 |
clarkb | I don't think the IP above would've been impacted by that but who knows could've been some flooded pipes in the aftermatch | 15:27 |
clarkb | *aftermath | 15:27 |
* fungi retreats to the far corners of arpanet | 15:27 | |
opendevreview | Merged openstack/project-config master: New Project Request: airship/image-builder https://review.opendev.org/c/openstack/project-config/+/795118 | 15:31 |
fungi | clarkb: did you see aj's question on https://review.opendev.org/792710 ? | 15:31 |
clarkb | fungi: I had not. Looking now | 15:31 |
clarkb | fungi: responded | 15:32 |
*** ysandeep|away has quit IRC | 15:33 | |
*** ysandeep|away has joined #opendev | 15:34 | |
fungi | dmitriis: done, let us know if you need anything else | 15:37 |
fungi | clarkb: 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 there | 15:39 |
fungi | and have demoted my account again now that i'm done | 15:39 |
fungi | in 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 it | 15:41 |
* fungi shakes fist at heisenbug | 15:41 | |
clarkb | fungi: 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 deal | 15:42 |
*** whoami-rajat has quit IRC | 15:46 | |
dmitriis | fungi: 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|out | 16:02 | |
*** ykarel is now known as ykarel|away | 16:02 | |
*** ykarel|away has quit IRC | 16:13 | |
*** lucasagomes has quit IRC | 16:15 | |
*** rpittau is now known as rpittau|afk | 16:31 | |
*** jpena is now known as jpena|off | 16:33 | |
opendevreview | Merged openstack/project-config master: Drop logstash filter jobs https://review.opendev.org/c/openstack/project-config/+/792710 | 16:37 |
*** ysandeep|away has quit IRC | 16:38 | |
*** marios|out has quit IRC | 16:39 | |
opendevreview | Balazs Gibizer proposed opendev/irc-meetings master: Move nova meeting to #openstack-nova https://review.opendev.org/c/opendev/irc-meetings/+/795377 | 16:41 |
opendevreview | Ghanshyam proposed openstack/project-config master: Move devstack-gate jobs list to in-tree https://review.opendev.org/c/openstack/project-config/+/795379 | 16:51 |
*** ralonsoh has quit IRC | 17:01 | |
*** ykarel|away has joined #opendev | 17:06 | |
*** amoralej is now known as amoralej|off | 17:30 | |
*** andrewbonney has quit IRC | 17:32 | |
*** ykarel|away has quit IRC | 17:43 | |
opendevreview | Merged opendev/system-config master: Forward openstack-security ML to openstack-discuss https://review.opendev.org/c/opendev/system-config/+/794659 | 17: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 |
opendevstatus | fungi: finished logging | 17:56 |
opendevreview | Rajath 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/+/795396 | 18:29 |
opendevreview | Rajath 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/+/795396 | 18:30 |
opendevreview | Rajath Av proposed openstack/project-config master: Changes to be committed: modified: zuul.d/projects.yaml https://review.opendev.org/c/openstack/project-config/+/795397 | 18:42 |
opendevreview | Rajath Av proposed openstack/project-config master: Changes to be committed: modified: zuul.d/projects.yaml https://review.opendev.org/c/openstack/project-config/+/795396 | 18:43 |
opendevreview | Rajath Av proposed openstack/project-config master: Changes to be committed: modified: zuul.d/projects.yaml https://review.opendev.org/c/openstack/project-config/+/795397 | 18:45 |
*** rav_ has joined #opendev | 18:47 | |
rav_ | https://review.opendev.org/c/openstack/project-config/+/795397 | 19:09 |
clarkb | rav_: 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' line | 19:10 |
rav_ | Oh i see. I thought of adding both in .zuul.yaml thanks for pointing out, will make the necessary changes | 19:11 |
opendevreview | Rajath Av proposed openstack/project-config master: Changes to be committed: modified: zuul.d/projects.yaml https://review.opendev.org/c/openstack/project-config/+/795397 | 19:14 |
*** stevebaker has quit IRC | 19: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 |
clarkb | rav_: 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 merged | 19:42 |
fungi | rav_: 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 moment | 19:42 |
rav_ | alrighty.. Got it | 19:42 |
rav_ | thanks for your time | 19:42 |
*** rav_ has quit IRC | 19:51 | |
opendevreview | Merged openstack/project-config master: Changes to be committed: modified: zuul.d/projects.yaml https://review.opendev.org/c/openstack/project-config/+/795397 | 19:55 |
clarkb | infra-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 mergeable | 19:58 |
ianw | clarkb: 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 |
ianw | i 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 strongly | 20:03 |
ianw | infra-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 |
ianw | that should be fine now, as we've updated ARA so it is compatible | 20:06 |
opendevreview | Clark Boylan proposed opendev/system-config master: Remove special x/ handling patch in gerrit https://review.opendev.org/c/opendev/system-config/+/791995 | 20:07 |
opendevreview | Clark 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/+/795409 | 20:07 |
clarkb | ianw: 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 critical | 20:08 |
clarkb | infra-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 restart | 20:08 |
clarkb | this will pull in the newer zuul status dashboard plugin too | 20:08 |
clarkb | ianw: if you'd prefer to not have local copies then I can switch to +2 at this point I think | 20:09 |
ianw | clarkb: 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 |
clarkb | ianw: ++ | 20:11 |
opendevreview | Tristan Cacqueray proposed zuul/zuul-jobs master: fetch-translation-output: introduce zuul_use_fetch_output https://review.opendev.org/c/zuul/zuul-jobs/+/681887 | 20:12 |
opendevreview | Tristan Cacqueray proposed zuul/zuul-jobs master: fetch-javascript-content-tarball: introduce zuul_use_fetch_output https://review.opendev.org/c/zuul/zuul-jobs/+/681903 | 20:12 |
*** stevebaker has joined #opendev | 20:19 | |
opendevreview | Ian Wienand proposed opendev/system-config master: review02 : switch reviewdb to mariadb_container type https://review.opendev.org/c/opendev/system-config/+/795192 | 20:21 |
ianw | argh put it in the wrong change | 20:21 |
opendevreview | Ian Wienand proposed opendev/system-config master: gerrit: add mariadb_container option https://review.opendev.org/c/opendev/system-config/+/775961 | 20:25 |
opendevreview | Ian Wienand proposed opendev/system-config master: review02 : switch reviewdb to mariadb_container type https://review.opendev.org/c/opendev/system-config/+/795192 | 20:25 |
ianw | clarkb: ^ that is what we discussed now | 20:25 |
clarkb | cool I'll try to dive into reviews for all those called out changes when I get back from a bike ride and some food | 20:25 |
ianw | thanks; the meetbot ones are probably the most interseting | 20:26 |
opendevreview | Tristan Cacqueray proposed zuul/zuul-jobs master: fetch-translation-output: introduce zuul_use_fetch_output https://review.opendev.org/c/zuul/zuul-jobs/+/681887 | 20:26 |
opendevreview | Tristan Cacqueray proposed zuul/zuul-jobs master: fetch-javascript-content-tarball: introduce zuul_use_fetch_output https://review.opendev.org/c/zuul/zuul-jobs/+/681903 | 20:26 |
opendevreview | Tristan Cacqueray proposed zuul/zuul-jobs master: fetch-translation-output: introduce zuul_use_fetch_output https://review.opendev.org/c/zuul/zuul-jobs/+/681887 | 20:43 |
opendevreview | Tristan Cacqueray proposed zuul/zuul-jobs master: fetch-javascript-output: introduce zuul_use_fetch_output https://review.opendev.org/c/zuul/zuul-jobs/+/681748 | 20:48 |
*** hashar has quit IRC | 21:06 | |
opendevreview | Tim 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/+/795419 | 21:25 |
opendevreview | Merged zuul/zuul-jobs master: fetch-translation-output: introduce zuul_use_fetch_output https://review.opendev.org/c/zuul/zuul-jobs/+/681887 | 21:44 |
*** cloudnull has quit IRC | 21:50 | |
*** cloudnull has joined #opendev | 21:51 | |
*** dmitriis has quit IRC | 22:02 | |
opendevreview | Ian Wienand proposed opendev/system-config master: Run statusbot from eavesdrop01.opendev.org https://review.opendev.org/c/opendev/system-config/+/795213 | 22:13 |
*** dmitriis has joined #opendev | 22:13 | |
ianw | clarkb: 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 quiet | 22:43 |
clarkb | fungi: ^ do you want to review those changes maybe? | 22:44 |
*** dmitriis has quit IRC | 22:44 | |
clarkb | fungi: specifically https://review.opendev.org/c/opendev/system-config/+/795409 and its child | 22:47 |
fungi | sure what's the topic string for them? | 22:48 |
clarkb | topic:check-gerrit-url-handling | 22:48 |
clarkb | but only the most recent two changes. The 3.4 change needs a reabse | 22:48 |
fungi | thanks lookin | 22:49 |
clarkb | ianw: re the eavesdrop/meetings stuff do we need to make an announcement of the fqdn change from eavesdrop to meetings? | 22:50 |
ianw | i haven't as intended that everything redirected to exactly the same data | 22:52 |
fungi | yeah, we can always post a "didja notice?" but really it should be transparent for everyone | 22:52 |
ianw | but we could; maybe let's get everything 100% migrated first? | 22:52 |
clarkb | meetings.opendev.org is a new canonical name though isn't it? | 22:52 |
clarkb | I 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 |
ianw | eavesdrop.openstack.org is redirecting | 22:54 |
ianw | meetings.opendev.org proxies /irclogs and /meetings to eavesdrop01 for now | 22:54 |
clarkb | ah 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 transparent | 22:54 |
ianw | i 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 statically | 22:55 |
clarkb | I was looking for that when reviewing the chagnes but I guess its already there which is why I wasnt finding it | 22:55 |
*** dmitriis has joined #opendev | 22:55 | |
ianw | 'eavesdrop' always seemed a bit weird for the meeting ICS files :) | 22:55 |
ianw | i 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 |
clarkb | I think we'd only have that for reads when serving large files out of the mirrors | 22:56 |
ianw | most other things like mirror-update, executors, etc. just pull in a kerberos token when they need to write something on afs ad-hoc | 22:57 |
clarkb | ianw: fwiw I've gotten through all but the statusbot change so far. And now I need to eat some very late lunch | 22:57 |
clarkb | will try to review statusbot after too | 22:57 |
ianw | thanks; yeah statusbot is still WIP and probably not working yet | 22:58 |
*** tosky has quit IRC | 23:03 | |
opendevreview | Merged opendev/system-config master: Update Gerrit plugin versions for Gerrit 3.2 and 3.3 https://review.opendev.org/c/opendev/system-config/+/795409 | 23:20 |
ianw | can 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 |
ianw | oh i see it, some {% accidentally ending with }} | 23:32 |
clarkb | ianw: {% if statusbot_wiki_thankspageid }} | 23:32 |
clarkb | total width is balanced because the left } should be a % | 23:32 |
ianw | yeah; a line number would have been great thanks ansible | 23:33 |
opendevreview | Ian Wienand proposed opendev/system-config master: limnoria/meetbot setup on eavesdrop01.opendev.org https://review.opendev.org/c/opendev/system-config/+/793719 | 23:35 |
opendevreview | Ian Wienand proposed opendev/system-config master: Move meetbot config to eavesdrop01.opendev.org https://review.opendev.org/c/opendev/system-config/+/795007 | 23:35 |
opendevreview | Ian Wienand proposed opendev/system-config master: Cleanup eavesdrop puppet references https://review.opendev.org/c/opendev/system-config/+/795014 | 23:35 |
opendevreview | Ian Wienand proposed opendev/system-config master: Run statusbot from eavesdrop01.opendev.org https://review.opendev.org/c/opendev/system-config/+/795213 | 23:35 |
clarkb | statusbot 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 statusbot | 23:35 |
clarkb | if it was a daemon I'd suggest we not collocate them to avoid confusion over what the container is running | 23:36 |
ianw | clarkb: thanks for review, that has addressed comments on 793719 | 23:36 |
clarkb | looking | 23:36 |
ianw | it 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.yaml | 23:37 |
clarkb | ah it is accesbot that I'm confusing with statusbot. Accessbot runs in the way I describe | 23:37 |
clarkb | in that case I'll need to think about this and better understand the implications of running it that way | 23:38 |
clarkb | ianw: the service name on that docker-compose entry is gerritbot but should be statusbot | 23:38 |
ianw | ahh, yeah copy-paste. | 23:38 |
ianw | i know that it should be a separate container | 23:39 |
clarkb | ya it will run as a separate container using the same image | 23:39 |
clarkb | that is probably ok as the concern is running multiple processes in a single container | 23:39 |
ianw | i 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 container | 23:40 |
clarkb | mordred: ^ you probably have teh strongest opinions on this stuff. What do you think | 23:40 |
opendevreview | Ian Wienand proposed opendev/system-config master: Run statusbot from eavesdrop01.opendev.org https://review.opendev.org/c/opendev/system-config/+/795213 | 23:41 |
mordred | clarkb: reading | 23:59 |
mordred | I don't think it needs to be its own container image | 23:59 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!