Thursday, 2020-09-10

*** DSpider has quit IRC00:09
ianwyeah, 04 was just brought up to not conflict with the openstack.org versions00:16
ianwback in ancient history; when we all lived care-free outside and with clean air; i.e. a few months ago00:17
fungiin the beforetime, the long-longago00:19
ianwfungi: speaking of, i'm not letting my ipv6 issues go without a fight.  NAT works for me, but libvirt still has issues https://www.redhat.com/archives/libvir-list/2020-September/msg00528.html00:21
fungiwow, yeah00:24
fungiipv6 nat... /me shudders00:24
fungibut i guess if it saves you from doing dhcp6-pd from your router through your workstation/laptop to the vm...00:25
ianwfungi: if you're in a reviewing mode, i'd like to get rid of https://review.opendev.org/#/q/status:open+topic:s-c-ansible-devel-testing00:32
*** mlavalle has quit IRC00:47
openstackgerritArun S A G proposed opendev/gerritlib master: Add support for TCP keep alive in gerritlib  https://review.opendev.org/75084900:56
openstackgerritMerged opendev/system-config master: tox: drop test-requirements.txt  https://review.opendev.org/74732801:18
openstackgerritMerged opendev/system-config master: Collect tox logs  https://review.opendev.org/74732901:18
ianwthanks!01:21
ianwgentoo trying to build.  there's also a request up for f32 which i can work on soon01:21
ianw2020-09-10 01:47:37.345 | + locale-gen -q01:50
ianw2020-09-10 01:47:38.924 |  * C.UTF-8: [error] cannot write output files to `C.UTF-8': File exists01:50
ianw2020-09-10 01:47:40.158 |  * Adding locales to archive ...                                          [ !! ]01:50
ianwgentoo fails here now01:51
ianwprometheanfire: ^01:51
fungiianw: should 747597 still be getting a POST_FAILURE on system-config-run-base-ansible-devel or is that stale?01:52
ianwfungi: umm, i think that's probably stale from before we had the projects in project-config01:54
ianwalthough we'll need those ara changes in for it to work01:54
fungiyeah, i figured we were still waiting on that01:54
ianwdmsimard: maybe when you get a sec you can look over the 2.10 fixes for stable @ https://review.opendev.org/#/c/747337/01:55
fungibut once dmsimard has time to decide there, hopefully we'll have that job back01:55
ianwyeah, and we can speculatively test a lot of the community module changes too with it, which is pretty cool01:55
fungiagreed, that'll be slick01:56
prometheanfireianw: hi02:07
prometheanfirehuh, that's new02:07
prometheanfiremaybe we should be running with `-q -k -u`02:08
prometheanfireianw: that's running in an infra element?  If so, it can probably just be removed, since the base dib gentoo stuff runs it for us02:36
ianwthat's in ... /tmp/in_target.d/pre-install.d/02-gentoo-00-set-profile02:37
prometheanfireor if it's the gentoo element that's failed, 02-gentoo-00-set-profile, then maybe that can be removed02:37
prometheanfireit's odd02:38
prometheanfirelet me build a basic image and see if I can reproduce02:38
ianwhttps://nb01.opendev.org/gentoo-17-0-systemd-0000143982.log02:38
ianwnb01.openstack.org now points to some other host, we should remove that02:39
prometheanfiresystemd profile running with these elements 'gentoo simple-init growroot vm openssh-server block-device-efi enable-serial-console'02:40
prometheanfirehmm02:42
prometheanfire2020-09-10 02:40:47.395 | dib-run-parts 02-gentoo-00-set-profile completed02:42
ianw#status log removed old nb01/2.openstack.org dns entries02:46
openstackstatusianw: finished logging02:46
prometheanfireianw: I'm not sure why that happened, only thing I can think of is if locale-gen is running earlier03:23
prometheanfirebut I don't see it in the log03:23
openstackgerritIan Wienand proposed openstack/project-config master: Add Fedora 32 builds  https://review.opendev.org/75064203:45
openstackgerritMerged openstack/project-config master: Add Fedora 32 builds  https://review.opendev.org/75064204:06
*** ykarel|away has joined #opendev04:22
*** ykarel|away is now known as ykarel04:23
*** ykarel has quit IRC04:30
*** bhagyashris|rove is now known as bhagyashri|rover04:32
*** ykarel has joined #opendev04:37
*** ykarel_ has joined #opendev04:51
*** ykarel has quit IRC04:54
*** user_19173783170 has joined #opendev04:55
user_19173783170hello04:56
*** fressi has joined #opendev04:56
*** cloudnull has quit IRC05:08
*** ykarel__ has joined #opendev05:12
*** ykarel_ has quit IRC05:14
*** ysandeep|away is now known as ysandeep05:18
*** ykarel__ is now known as ykarel05:22
*** user_19173783170 has quit IRC05:31
*** ykarel_ has joined #opendev06:01
*** fressi has quit IRC06:01
*** ykarel has quit IRC06:03
*** cloudnull has joined #opendev06:05
*** ykarel_ is now known as ykarel06:15
*** qchris has quit IRC06:21
*** qchris has joined #opendev06:33
AJaegerinfra-root, please review https://review.opendev.org/#/c/597402/ to retire pynotedb.06:38
AJaegergmann, clarkb , we're down to 52 errors - the stable branches for midonet need the x/networking-l2gw changes as well...06:39
AJaegerianw: thanks! Could you review https://review.opendev.org/#/c/750635/3 as well? Then it's removed completely...06:45
ianwlgtm, thanks06:46
AJaegerthanks06:50
*** hashar has joined #opendev06:50
openstackgerritJan Kubovy proposed zuul/zuul-jobs master: WIP: Temporary disable sudo revocation  https://review.opendev.org/75088106:53
openstackgerritMerged openstack/project-config master: pynotedb: remove project from infrastructure systems  https://review.opendev.org/75063506:58
openstackgerritJan Kubovy proposed zuul/zuul-jobs master: WIP: Temporary disable sudo revocation for tox  https://review.opendev.org/75088107:12
AJaegergmann, clarkb: There were other repo retirements besides networking-l2gw as well - but that repo has the most fallout. I've send patches for some other repos already.07:13
*** priteau has joined #opendev07:16
*** fressi has joined #opendev07:24
*** tosky has joined #opendev07:32
*** priteau has quit IRC07:47
*** andrewbonney has joined #opendev07:47
*** priteau has joined #opendev07:53
*** moppy has quit IRC08:01
*** moppy has joined #opendev08:02
openstackgerritSorin Sbarnea (zbr) proposed opendev/elastic-recheck master: Enable configuration via environment variables  https://review.opendev.org/74689008:05
openstackgerritSorin Sbarnea (zbr) proposed opendev/elastic-recheck master: Refreshed bindep  https://review.opendev.org/75090408:27
openstackgerritSorin Sbarnea (zbr) proposed opendev/elastic-recheck master: Use pytest for queries  https://review.opendev.org/75044508:29
*** jaicaa has quit IRC08:29
*** jaicaa has joined #opendev08:31
*** DSpider has joined #opendev08:31
*** ysandeep is now known as ysandeep|lunch08:34
*** ykarel_ has joined #opendev08:35
*** ykarel has quit IRC08:36
*** ykarel_ is now known as ykarel|lunch08:40
*** ykarel|lunch has quit IRC09:12
*** dtantsur|afk is now known as dtantsur09:14
*** ykarel has joined #opendev09:19
*** fressi has quit IRC09:21
*** ysandeep|lunch is now known as ysandeep09:35
*** hashar has quit IRC09:58
*** hashar has joined #opendev09:58
*** fressi has joined #opendev10:01
openstackgerritMerged opendev/elastic-recheck master: Refreshed bindep  https://review.opendev.org/75090410:04
*** fressi has quit IRC10:13
openstackgerritSorin Sbarnea (zbr) proposed opendev/elastic-recheck master: Use pytest for queries  https://review.opendev.org/75044510:24
openstackgerritSorin Sbarnea (zbr) proposed opendev/elastic-recheck master: Enable configuration via environment variables  https://review.opendev.org/74689010:24
*** fressi has joined #opendev10:25
*** fressi has quit IRC10:44
openstackgerritSorin Sbarnea (zbr) proposed opendev/elastic-recheck master: Create elastic-recheck container image  https://review.opendev.org/75095810:54
*** dmellado has quit IRC11:18
openstackgerritSorin Sbarnea (zbr) proposed opendev/elastic-recheck master: Create elastic-recheck container image  https://review.opendev.org/75095811:19
*** dmellado has joined #opendev11:19
openstackgerritSorin Sbarnea (zbr) proposed opendev/elastic-recheck master: Create elastic-recheck container image  https://review.opendev.org/75095811:32
*** dmellado has quit IRC11:33
openstackgerritSorin Sbarnea (zbr) proposed opendev/elastic-recheck master: Create elastic-recheck container image  https://review.opendev.org/75095811:34
*** dmellado has joined #opendev11:34
openstackgerritMerged opendev/elastic-recheck master: Use pytest for queries  https://review.opendev.org/75044511:38
openstackgerritSorin Sbarnea (zbr) proposed opendev/elastic-recheck master: pylint fixes  https://review.opendev.org/75096311:42
openstackgerritSorin Sbarnea (zbr) proposed opendev/elastic-recheck master: pylint: fixed imports  https://review.opendev.org/75096812:04
*** dmellado has quit IRC12:10
*** dmellado has joined #opendev12:11
openstackgerritMerged opendev/elastic-recheck master: pylint fixes  https://review.opendev.org/75096312:14
*** hashar has quit IRC12:31
*** hashar has joined #opendev12:37
*** stephenfin has joined #opendev12:37
*** hashar has quit IRC12:40
openstackgerritSorin Sbarnea (zbr) proposed opendev/elastic-recheck master: pylint: fixed imports  https://review.opendev.org/75096812:47
mnaserinfra-root: https://review.opendev.org/#/c/750519/ does this failure happen because someone needs to create an afs vol for the ceph debs?12:48
*** ykarel_ has joined #opendev12:48
*** ykarel has quit IRC12:51
fungimnaser: judging from vos listvol output, i expect so: http://paste.openstack.org/show/797716/13:05
fungihttps://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_b8d/750519/1/check/system-config-run-mirror-x86/b8d8998/bridge.openstack.org/ara-report/result/f9f3445c-f5c0-41e7-a30f-0012aaa9f081/13:10
fungi"src file does not exist, use \"force=yes\" if you really want to create the link: /afs/openstack.org/mirror/ceph-deb-octopus"13:10
mnaserfungi: yeah, that's my thought too.  could i get that volume? :-)13:11
fungiit'll probably be 30 minutes, i need to finish up a couple things and switch computers13:11
fungibut syre13:11
fungisure13:11
*** priteau has quit IRC13:11
mnaserfungi: unless i can push up a system-config patch, im not sure if that's managed manually or not13:12
mnaseri can gladly help with that too13:12
dmsimardfungi, ianw: thanks for the ping, I +W'd the ara fix13:12
fungimnaser: it's not orchestrated unfortunately, i need to auth to kerberos and then run the commands documented here: afs.html13:13
fungier, here: https://docs.opendev.org/opendev/system-config/latest/afs.html#mirrors13:14
mnaserfungi: ah gotcha, okay, i'll hold off.  thank you :)13:14
*** priteau has joined #opendev13:15
openstackgerritSean McGinnis proposed openstack/project-config master: Remove noop jobs for python-adjutantclient  https://review.opendev.org/75099013:25
*** ykarel_ is now known as ykarel13:25
*** Goneri has joined #opendev13:26
*** hashar has joined #opendev13:48
openstackgerritSorin Sbarnea (zbr) proposed opendev/elastic-recheck master: pylint: fixed logging-not-lazy  https://review.opendev.org/75099713:57
openstackgerritMerged opendev/elastic-recheck master: pylint: fixed imports  https://review.opendev.org/75096814:11
*** ykarel is now known as ykarel|away14:18
openstackgerritSorin Sbarnea (zbr) proposed opendev/elastic-recheck master: pylint: 6 more  https://review.opendev.org/75100114:29
openstackgerritAndreas Jaeger proposed openstack/project-config master: Retire devstack-plugin-pika project  https://review.opendev.org/74871214:40
*** ykarel|away has quit IRC14:45
openstackgerritSorin Sbarnea (zbr) proposed opendev/elastic-recheck master: Create elastic-recheck container image  https://review.opendev.org/75095815:01
*** avass has joined #opendev15:09
openstackgerritSorin Sbarnea (zbr) proposed opendev/elastic-recheck master: pytlint: 4 more  https://review.opendev.org/75102115:12
openstackgerritSorin Sbarnea (zbr) proposed opendev/elastic-recheck master: pylint: 4 more  https://review.opendev.org/75102115:16
*** hashar has quit IRC15:17
openstackgerritMerged opendev/elastic-recheck master: pylint: fixed logging-not-lazy  https://review.opendev.org/75099715:20
openstackgerritMerged opendev/elastic-recheck master: pylint: 6 more  https://review.opendev.org/75100115:25
*** mlavalle has joined #opendev16:02
*** dtantsur is now known as dtantsur|afk16:22
*** ysandeep is now known as ysandeep|away16:25
openstackgerritOleksandr Kozachenko proposed openstack/project-config master: Add some more monasca projects in vexxhost tenant  https://review.opendev.org/75104616:31
openstackgerritOleksandr Kozachenko proposed openstack/project-config master: Add some more monasca projects in vexxhost tenant  https://review.opendev.org/75104616:32
*** dmellado_ has joined #opendev16:33
*** dmellado has quit IRC16:34
*** dmellado_ is now known as dmellado16:34
fungiokay... what am i doing wrong? from a shell on mirror.ca-ymq-1.vexxhost.opendev.org i can `ls -l /afs/openstack.org/mirror/ceph-deb-octopus/` and see a dummy file i created in there, but https://mirror.ca-ymq-1.vexxhost.opendev.org/ceph-deb-octopus/ returns 404 even after fully restarting apache16:36
fungimust not have made it to one of the replicas yet for some reason?16:38
fungihttps://static.opendev.org/mirror/ceph-deb-octopus/ is working fine16:38
*** dmellado has quit IRC16:39
fungioh! duh16:39
*** dmellado has joined #opendev16:39
fungiapache looks in /var/www/mirror and we create symlinks there, so it's not going to appear on mirror servers until 750519 merges16:40
clarkbya the symlinks expose things16:40
fungi#status log added new mirror.deb-octopus volume mounted at /afs/.openstack.org/mirror/ceph-deb-octopus with replicas and set acls consistent with other reprepro mirrors16:41
openstackstatusfungi: finished logging16:41
Open10K8SHi Team. Please check this PS https://review.opendev.org/751046 .16:42
Open10K8SThank you16:42
*** dmellado has quit IRC16:49
clarkbinfra-root I'm around now and can keep an eye on https://review.opendev.org/#/c/749853/3 if it is approval worthy. Once that lands my next step will be to delete the old server and its volume16:51
*** dmellado has joined #opendev16:55
*** dmellado has quit IRC17:12
openstackgerritMerged openstack/project-config master: Add some more monasca projects in vexxhost tenant  https://review.opendev.org/75104617:12
*** dmellado has joined #opendev17:13
clarkbfungi: frickler have time for https://review.opendev.org/#/c/749777/ and https://review.opendev.org/#/c/749766/ ? Thats setuptools clean up from the earlier situation with the 50 release17:18
fungiclarkb: on a related topic, i've replaced the cinder volumes for nb01 and nb02... should i do the same for nb04 or are we settled on deleting the server?17:25
clarkbI'm good with deleting it based on available disk space17:26
clarkband i think ianw said it should be fine too?17:26
clarkbI'll write up a change to pause it now I guess?17:26
fungithat would be awesome. i'll skip it for now and move its volume to the delete list on the etherpad as a reminder17:26
clarkbsounds good, thanks17:27
*** dmellado has quit IRC17:27
fungi#status log cinder volumes for nb01 and nb02 have been replaced and cleaned up17:27
openstackstatusfungi: finished logging17:27
openstackgerritClark Boylan proposed openstack/project-config master: Pause image builds on nb04.opendev.org  https://review.opendev.org/75106617:31
clarkbI'm going to stop the builder on nb03.openstack.org now just to be sure there is no fallout from that (don't expect any)17:32
clarkb#status log Stopped nodepool-builder on nb03.openstack.org in preparation for its deletion17:33
openstackstatusclarkb: finished logging17:33
mnaserfungi: thank you for the addition17:33
openstackgerritMohammed Naser proposed opendev/system-config master: Add ceph octopus mirrors  https://review.opendev.org/75051917:34
*** dmellado has joined #opendev17:46
fungi#status log cinder volume for wiki-dev has been replaced and cleaned up17:58
openstackstatusfungi: finished logging17:58
funginow we're just down to the volumes for nb04 (soon to be deleted) and wiki.o.o (which will need deeper surgery to solve)17:58
fungiplus a bunch of unused volumes stuck in error_deleting state now, but once i've got the rest taken care of i'll dump that list into the open rax ticket about the maintenance letting them know those are unused17:59
mnaserzuul.opendev.org seems sluggy-ish18:06
mnaserbut i cant nail down a specific reason18:06
mnaserit feels like dns because it responds quickly after a long time18:07
fungispecific views?18:07
fungiahh18:07
clarkbwe did just upgrade the scheduler and the web server (and that included a db update)18:07
fungizuul-web has an entire core pegged, fwiw18:08
fungiaccordnig to top18:08
mnaser`time curl https://zuul.opendev.org/api/tenant/openstack/config-errors` took 6s18:08
mnaser`time curl https://zuul.opendev.org/api/tenant/openstack/build/7e8db9764bc248668a823648913f7403` took 4.66s here so it seems like something is off18:08
fungiyeah, top continues to report a zuul-web process consuming 100-103% cpu18:09
fungithe system itself is still mostly idle because it has 8 processors18:10
fungibut i think zuul-web is single-threaded and so probably starved for cycles18:10
clarkbmemory pressure is fine too (thats a common cause of slowness with zuul if we have a leak)18:10
fungihrm, though no the zuul-web cpu utilization does seem to be getting evenly spread across all the cpus18:11
fungiso it's not that it's eating all of a cpu18:11
clarkbwe did change the prioritization of gearman tasks iirc18:11
*** gmann is now known as gmann_afk18:11
clarkbcould be something like that maybe?18:11
clarkbfwiw I agree my browser seems to exhibit similar18:12
*** andrewbonney has quit IRC18:13
openstackgerritMerged openstack/project-config master: Pause image builds on nb04.opendev.org  https://review.opendev.org/75106618:14
clarkblooking in the zuul web debug log the gearman tasks don't seem to take very long. about 100ms or so each18:14
fungicpu usage also doesn't look particularly higher than it had been before the restart: http://cacti.openstack.org/cacti/graph.php?action=view&local_graph_id=64789&rra_id=all18:14
clarkbthough the openstack tenant requests take longer (its got a lot of stuff going on compared to the others I guess)18:15
fungimnaser: when did you first notice it being slow?18:16
clarkbthere are a lot of requests for /api/tenant/openstack/status18:16
clarkbwhich we should be caching for longer than the period between those requests I thought18:16
mnaserfungi: to be honest, it was a little choppy when i submitted the revision to the ceph patch which means18:16
clarkbI wonder if that is related18:16
fungiwe do seem to have *just* reached full utilization of available ram for cache memory18:16
mnasersince at least 45 minutes18:16
fungiestablished tcp connections is up quite a bit compared to previous days, but that can easily be a symptom of it taking longer to fulfil requests: http://cacti.openstack.org/cacti/graph.php?action=view&local_graph_id=64791&rra_id=all18:18
clarkbthe status response is ~1MB whcih is well below our max cache size of 8MB for memory and 10MB for disk (I think we're using disk?)18:19
clarkbwe aren't logging that we have cache hits like the mirrors do18:20
fungialso the traffic levels on eth0 don't look appreciably higher than usual for a weekday18:20
clarkbso I wonder if that is at least part of the issue (not using the cache)18:20
clarkbthe cache dir has content18:22
clarkbfungi: could it be the .* isn't treated like a regex in our CacheEnable url-string directives?18:26
fungimmm18:26
fungihttp://httpd.apache.org/docs/current/mod/mod_cache.html#cacheenable18:29
fungii don't see anywhere in there indicating you can use a regex, and the examples are all paths, urls or empty strings18:29
fungithere are a couple of host wildcard examples18:30
clarkbI wonder if that is at least part of the issue here (we aren't caching and taking load off of zuul as expected)18:30
fungibut they're fancy syntax, neither traditional regex nor glob18:30
fungilooks like we can put them into a <location> block though18:30
clarkbfungi: maybe we should drop the .* in /static/ and see if that changes anything18:30
clarkbsince its supposed to cache at or below the url-string supplied so the .* in /static/ is not helpful here18:31
fungior locationmatch i guess in this case18:31
clarkband then ya if that makes things happy we can use the location stuff18:32
fungiclarkb: yeah worth a try just making it /static/18:32
fungi"The CacheEnable directive instructs mod_cache to cache urls at or below url-string."18:33
fungiemphasis on "or below"18:33
fungiso the trailing .* is at best redundant18:33
openstackgerritMerged opendev/system-config master: Remove nodepool builder puppetry and nb03.openstack.org  https://review.opendev.org/74985318:33
clarkboh ya we don't use .* on the openstack specific vhost18:34
clarkblet me look at the logs for that vhost18:34
fungiclarkb: but reading between the lines, it looks like if we need to get specific for /api/tenant/.*/status we should probably put that particular cacheenable in a <locationmatch> container18:35
clarkblooking at logs for the openstack vhost we combine the logs into a single log file: zuul-ssl-access.log and I see no cache hits there at all :/18:36
fungido we have mod_cache set to log its info?18:37
clarkbwhat is odd is we seem to have cache content in the cache storage location (but its all hashed paths and I have no idea how to parse that to see what we are caching)18:37
clarkboh maybe that is what we are missing. Let me compare to the mirrors18:37
clarkbya the mirrors use a custome log format18:38
clarkbI'll add that to zuul with the .* stop in /status/18:38
fungialso corvus might be interested in this topic if he's available18:38
fungii guess https://opendev.org/opendev/system-config/src/branch/master/playbooks/roles/zuul-web/templates/zuul.vhost.j2 is where we'll need to patch it when we work out the details18:42
fungi:q18:42
fungiheh, this is not vi18:42
openstackgerritClark Boylan proposed opendev/system-config master: Improve logging and cache config for zuul web proxy  https://review.opendev.org/75107818:43
clarkbfungi: ^ yup18:43
clarkbI think we get better logs then work from there18:43
clarkbI'm going to figure out lunch while that does its thing18:46
mnaserinfra-root: https://review.opendev.org/#/c/750519/ is ready for a +2 so we can get deb octopus mirroring :)18:56
mnasershould be pretty straight forward18:56
fungiso if i `openstack volume attach ...` and the volume state changes to "attaching" for a few seconds and then back to "available" again, where's the best place (as a non-administrator of that cloud) to look for clues as to why it didn't attach?18:59
fungigiven i've already tried dmesg as an obvious first guess, and come up with nothing19:00
mnaserfungi: not much you can do, that seems like a cloud-level issue19:01
*** hashar has joined #opendev19:01
mnaseronce it goes into attaching, that means nova and cinder have started talking to start attaching the port19:01
mnasers/port/volume/19:02
fungiso there's no synchronous attach call which will spit out an actual error or somethingf19:02
mnasernot off the top of my head, it is possible that the instance action log might log something19:02
mnasernova instance-action-list <instance>19:02
fungiit does log the attach volume calls at least19:03
fungiwith corresponding request ids19:04
fungi| attach_volume  | req-a7ec807c-101a-4f27-b630-ae65547dd4a3 | -       | 2020-09-10T18:56:24.000000 |19:04
fungithe message column just contains "-" though19:04
fungioh well, i'll do this the old fashioned way and copy the volume contents to the rootfs, twiddle them into production, detach the old volume, *then* try to attach yet another volume19:06
fungimaybe xen is particular about allowing a very limited number of devices for this old node19:07
fungii know it doesn't allow more than 16 block devices to be attached at once, but this instance is nowhere near that limit19:08
fungiso am strongly suspecting it's related to being booted from a snapshot of a prior instance booted from an ancient "legacy cloud" flavor19:09
fungimaybe that setup only allowed a single volume attachment or something similarly insane19:09
zer0c00lfungi: After adding keep alive, the bot seems to be holding up for 16+ hours. Perhaps wait and see over the weekend?19:11
zer0c00lhttps://review.opendev.org/#/c/750849/19:11
fungizer0c00l: yeah, that sort of thing can definitely be unpredictable, but a very good sign so far!19:13
fungilong ago i worked for an employer whose firewalls had a default tcp idle policy of 120 seconds. had to set ssh keepalive on by default for openssh on my workstation since otherwise my sessions would hang as soon as they hit a lull19:15
fungiit was very frustrating19:15
*** priteau has quit IRC19:16
zer0c00lIt is frustrating! I have seen it miss events and my co-workers complain that it didn't pick up their upstream contributions.19:18
openstackgerritMerged opendev/system-config master: Improve logging and cache config for zuul web proxy  https://review.opendev.org/75107819:51
openstackgerritMerged openstack/project-config master: Retire devstack-plugin-pika project  https://review.opendev.org/74871220:05
clarkbfungi: zuul's new logging seems to show what we assumed20:07
clarkbfungi: there are cache messages for the openstack vhost but not the opendev vhost20:08
clarkbfungi: were you working on a locationmatch yet?20:08
openstackgerritMerged openstack/project-config master: Add openstack/osops  https://review.opendev.org/74983420:18
openstackgerritMerged openstack/project-config master: Set neutron-lib stable ACLs  https://review.opendev.org/75064320:18
openstackgerritClark Boylan proposed opendev/system-config master: Use LocationMatch to cache zuul api statuses  https://review.opendev.org/75109920:18
clarkbfungi: ^ something like that I think?20:18
clarkbthen once thats landed we can reevaluate if zuul is still slow20:18
clarkb(but at least then we should be caching what we expect to be caching taking any unnecessary load off of zuul itself)20:18
openstackgerritAdam Coldrick proposed opendev/storyboard master: Optimise the Story browsing query  https://review.opendev.org/74204620:25
fungiclarkb: oh, sorry, i stepped away to cook dinner, back now and will review asap20:28
openstackgerritAdam Coldrick proposed opendev/storyboard master: Optimise the Story browsing query  https://review.opendev.org/74204620:28
clarkbno worries I was catching up after lunch and figured that was our next step based on what I saw in the logs20:29
clarkbyou might want to check the logs too to confirm (when dinner is sorted)20:29
*** hashar has quit IRC20:33
*** openstackgerrit has quit IRC20:36
fungiclarkb: yeah, logs look right now21:10
fungiexcept that there's little point in setting combined-cache on the http vhost as it's merely a redirect21:10
fungibut it hurts nothing21:11
fungiand i approved the locationmatch change, will keep an eye out21:11
clarkbthanks21:11
mnaserclarkb: do you have a sec to review https://review.opendev.org/#/c/750519/21:19
clarkbya sorry still diggin garound in zuul web logs. I'll look at that now21:23
clarkbfwiw I'm not sure the cache lenght is long enough to help much based on the openstack vhost caching21:24
clarkbbut that vhost may also get a lot less traffic so we'll see when the other change merges21:24
clarkbmnaser: its missing a thing. Left a comment inline21:29
*** openstackgerrit has joined #opendev21:31
openstackgerritMohammed Naser proposed opendev/system-config master: Add ceph octopus mirrors  https://review.opendev.org/75051921:31
mnaserclarkb: ah yes, that's an obvious one.  fixed, thank you21:31
*** rosmaita has joined #opendev21:37
openstackgerritMerged opendev/system-config master: Use LocationMatch to cache zuul api statuses  https://review.opendev.org/75109921:47
*** Goneri has quit IRC21:49
fungiclarkb: ^ that's in place22:13
clarkbhrm still not logging that it is trying to cache statuses22:16
clarkbit does seem to with the openstack vhost though22:18
fungiyeah, confirmed, maybe it didn't read the config?22:19
fungithough it is listing cache info for /static/...22:28
clarkbmaybe I got the regex wrong or didn't location match properly22:28
clarkbshould we try dropping the ^ maybe?22:28
fungiso maybe LocationMatch "^/api/tenant/.*/status" doesn't match what we expect, yeah22:29
fungii'll take a closer look at the locationmatch directive docs22:29
clarkbthanks22:30
fungihttp://httpd.apache.org/docs/current/mod/core.html#locationmatch22:30
fungithat looks like what we have22:30
openstackgerritMerged opendev/system-config master: Add ceph octopus mirrors  https://review.opendev.org/75051922:32
clarkbhrm ya that even includes the ^ root22:41
*** tosky has quit IRC22:42
fungii'm stumped22:46
fungicould we have put a location directive somewhere they get ignored?22:47
*** gmann_afk is now known as gmann22:49
clarkbmaybe it needs to be before the rewrite rule?22:49
clarkbI know rewrite rules themselves are order dependent22:49
clarkbnot sure how that interacts with other directives22:49
clarkboh hey I didn't realize https://review.opendev.org/#/c/749853/3 did end up merging22:52
clarkbianw: if you're around now any last objections to deleting nb03.openstack.org and its cinder volume?22:52
ianwclarkb: hey, yep, looking at some zuul nodejs stuff so that's why i'm quiet :)  no objections to cleaning it up, new server seems to be working great22:53
clarkbgreat, I'll procede with that now as I'm stumped by the zuul web caching thing22:53
clarkbianw: one thing I noticed was I used a different network for the new server than the old server. The old server is on the os-control-network-ipv4 network and new one is on os-control-network (which has ipv6 too)22:57
clarkbianw: does that change anything?22:57
clarkbI guess if ipv6 is an issue we do the netplan hack like we did in vexxhost or rebuild on the -ipv4 network22:58
* clarkb proceeds with deletion22:58
ianwclarkb: my memory is hazy ... i think that maybe we got floating ip's assigned to the other one22:58
ianwkevinz might remember22:58
clarkbhrm we have a floating ip on the current one22:59
clarkbso I expect its fine22:59
fungihopefully the reason was purely historical and no longer relevant23:00
clarkb#status log Deleted nb03.openstack.org. It has been replaced by nb03.opendev.org.23:00
openstackstatusclarkb: finished logging23:01
clarkbianw: have the nlXY records in openstack.org dns been cleaned up yet?23:01
clarkbif not I can clean those up when I do nb0323:01
ianwclarkb: umm, i don't think so.  i did find the old nbxx.openstack.org addresses were redirecting to some exciting new unrelated site when i accidetnally typod that yesterday23:01
*** mlavalle has quit IRC23:02
openstackgerritJeremy Stanley proposed opendev/engagement master: Initial commit  https://review.opendev.org/72929323:03
fungieverybody needs a little excitement these days, i suppose23:04
clarkber nlXY are still openstack23:05
clarkbits nb0X that are not23:05
clarkb(I haven't deleted the wrong records yet thankfully)23:05
clarkband it looks like the nb records have been cleaned23:06
clarkbso I think we're good now23:07
clarkbianw: ^ fyi23:07
ianwright, i thought so, but then i thought i missed something :)23:08
ianwwe did an inplace replacement of the launchers23:08
clarkbianw: also we've paused nb04 so that its images can rotate out23:10
clarkbthen probably next week we can delete it next23:10
fungiclarkb: so back to caching, maybe we need a locationmatch on http://127.0.0.1:9000/api/tenant/.*/status23:13
ianw++23:13
clarkbfungi: or just drop the ^ and it should match a longer prefix that way?23:13
fungiyeah, that probably wouldn't hurt23:14
clarkbwhats weird is the openstack one seems to work23:14
clarkbbut that isn't use a location match because it isn't a regex so maybe order matters for locationmatches23:14
clarkbfungi: maybe we want to hand edit the file and reload really quick to check?23:14
fungii'll try it now, yeah23:15
fungithat doesn't seem to have made any difference, even after a reload23:16
fungireplacing the .* with a specific tenant hasn't either23:18
clarkbfungi: maybe switch back to CacheEnable for the openstack tenant and see if that works?23:18
clarkbif that doesn't work its possible the status isn't cacheable for some reason in that context (but is when using the openstack vhost)23:18
fungiclarkb: tried it and still not getting cached, so i'm starting to think it's something about the response that's making it uncacheable (but then how is it cached through the other vhost?)23:25
clarkbweird23:27
openstackgerritAdam Coldrick proposed opendev/storyboard master: Optimise the Story browsing query  https://review.opendev.org/74204623:34
openstackgerritAdam Coldrick proposed opendev/storyboard master: Unpin webob and hacking to fix compatibility issues  https://review.opendev.org/75111723:34
*** DSpider has quit IRC23:44

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