Monday, 2020-06-22

*** ryohayakawa has joined #opendev00:02
*** spotz has joined #opendev00:25
ianwkevinz: i think we might have some launch issues in arm64 cloud00:56
ianwkevinz: http://paste.openstack.org/show/795041/01:10
ianwopenstack.exceptions.SDKException: Error in creating the server (no further information available)01:10
ianwnot very helpful01:10
ianwalthough then there's01:11
ianw2020-06-22 01:07:16,656 ERROR nodepool.NodeLauncher: [e: 6db383fbde354429a12e752df7aec7e9] [node_request: 900-0009569244] [node: 0017298127] Detailed node error: MessagingTimeout01:11
kevinzHi ianw01:32
kevinzYes I see, I will dig to see the problem01:32
*** DSpider has quit IRC01:36
ianwkevinz: cool, LMN, it looks like it's pretty busy launching nodes01:48
*** xiaolin has joined #opendev01:52
kevinzianw: looks recovered02:22
ianwkevinz: excellent :)  thanks, was it a backend issue?02:27
kevinzianw: yes, I see there is a problem with rabbitmq timeout, and I restart all the nova services and rabbitmq02:28
kevinzthen it was recovered...Looks Restarting is the most effiective way :-D02:29
*** sgw has quit IRC03:21
openstackgerritIan Wienand proposed opendev/system-config master: Acutally run system-config arm64 test on an arm64 node  https://review.opendev.org/73528103:30
openstackgerritIan Wienand proposed opendev/system-config master: openafs-client: Use PPA for Xenial ARM64  https://review.opendev.org/73505503:30
openstackgerritIan Wienand proposed opendev/system-config master: openafs-client: Use PPA for Xenial ARM64  https://review.opendev.org/73505504:02
openstackgerritIan Wienand proposed opendev/system-config master: Acutally run system-config arm64 test on an arm64 node  https://review.opendev.org/73528104:02
*** sgw has joined #opendev04:58
*** sgw has quit IRC05:05
*** rpittau|afk is now known as rpittau06:21
openstackgerritIan Wienand proposed openstack/diskimage-builder master: [wip] Fedora 32 support  https://review.opendev.org/73721706:36
*** DSpider has joined #opendev06:47
*** hashar has joined #opendev07:08
*** bhagyashris is now known as bhagyashris|lunc07:27
*** tosky has joined #opendev07:29
*** sshnaidm|off is now known as sshnaidm|ruck07:29
*** lpetrut has joined #opendev07:40
*** factor has quit IRC07:45
*** raukadah is now known as chandankumar07:58
*** moppy has quit IRC08:01
*** moppy has joined #opendev08:03
*** dpawlik6 has quit IRC08:18
*** ykarel is now known as ykarel|lunch08:25
*** bhagyashris|lunc is now known as bhagyashris08:34
openstackgerritJavier Peña proposed opendev/system-config master: Make the base role and playbook compatible with CentOS  https://review.opendev.org/73704308:35
*** dpawlik6 has joined #opendev08:36
openstackgerritCarlos Goncalves proposed openstack/diskimage-builder master: Add support for CentOS 8 Stream  https://review.opendev.org/73408308:37
openstackgerritJesse Pretorius (odyssey4me) proposed zuul/zuul-jobs master: [ensure-python] Remove debian-only note  https://review.opendev.org/73723108:44
*** ykarel|lunch is now known as ykarel08:58
openstackgerritCarlos Goncalves proposed openstack/diskimage-builder master: Download latest CentOS cloud image  https://review.opendev.org/73723709:07
*** ykarel is now known as ykarel|mtg09:07
*** bolg has joined #opendev09:15
openstackgerritJavier Peña proposed opendev/system-config master: [WIP] Make the base role and playbook compatible with CentOS  https://review.opendev.org/73704309:17
*** tkajinam has quit IRC09:21
openstackgerritCarlos Goncalves proposed openstack/diskimage-builder master: Add support for CentOS 8 Stream cloud image  https://review.opendev.org/73724509:42
*** ykarel|mtg is now known as ykarel09:42
openstackgerritJavier Peña proposed opendev/system-config master: [WIP] Make the base role and playbook compatible with CentOS  https://review.opendev.org/73704309:58
*** rpittau is now known as rpittau|bbl10:15
openstackgerritJavier Peña proposed opendev/system-config master: [WIP] Make the base role and playbook compatible with CentOS  https://review.opendev.org/73704310:17
openstackgerritJavier Peña proposed opendev/system-config master: [WIP] Make the base role and playbook compatible with CentOS  https://review.opendev.org/73704310:49
openstackgerritJavier Peña proposed opendev/system-config master: [WIP] Make the base role and playbook compatible with CentOS  https://review.opendev.org/73704311:06
openstackgerritJavier Peña proposed opendev/system-config master: [WIP] Make the base role and playbook compatible with CentOS  https://review.opendev.org/73704311:08
openstackgerritJavier Peña proposed opendev/system-config master: [WIP] Make the base role and playbook compatible with CentOS  https://review.opendev.org/73704311:21
*** rpittau|bbl is now known as rpittau12:22
*** ryohayakawa has quit IRC12:27
*** ysandeep|away is now known as ysandeep|PTO12:38
openstackgerritJavier Peña proposed opendev/system-config master: [WIP] Make the base role and playbook compatible with CentOS  https://review.opendev.org/73704312:38
mnaserinfra-core: got a ptl+1 https://review.opendev.org/#/c/737133/4 on -- would be nice to land that!13:10
fungimnaser: approved, but highlighting config-core in #openstack-infra might be more effective in such cases13:15
mordredyou're a config-core in #openstack-infra13:15
* mordred is enjoying a lovely morning thunderstorm13:18
mordredfungi: a couple of months ago we made the mistake of thinking "wow, it's been uncommonly dry"13:18
openstackgerritJavier Peña proposed opendev/system-config master: [WIP] Make the base role and playbook compatible with CentOS  https://review.opendev.org/73704313:18
mordredthis is no longer true13:19
fungiyeah, it's nice and wet here. has become a struggle to find a dry enough day where there's no standing water in the yard (at least on the side of the seawall where there's not supposed to be water) to be able to push the mower through it13:21
*** hashar is now known as hasharAway13:21
fungii really should do it at least weekly, but recently that's not been entirely possible13:21
openstackgerritMerged openstack/project-config master: Retire the puppet-congress project - Step 1 End Project Gating  https://review.opendev.org/73713313:26
mnaserfungi: yeah i should figure out slowly where to ping more, i always go back and forth :p13:35
fungiheh. me too ;)13:36
mordredfungi: have you considered just getting a scythe?13:42
*** redrobot has joined #opendev13:46
openstackgerritJavier Peña proposed opendev/system-config master: [WIP] Make the base role and playbook compatible with CentOS  https://review.opendev.org/73704313:52
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Add tests for upload-docker-image  https://review.opendev.org/73540213:55
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Fix and test multiarch docker builds in a release pipeline  https://review.opendev.org/73705913:55
openstackgerritMerged openstack/project-config master: Retire the puppet-congress project - Step 3 Remove project  https://review.opendev.org/73714413:58
*** hasharAway is now known as hashar13:58
fungimordred: my reel mower is basically like a steampunk scyth14:03
fungiscythe14:03
fungiclockwork rotary scythe14:05
openstackgerritJavier Peña proposed opendev/system-config master: Make the base role and playbook compatible with CentOS  https://review.opendev.org/73704314:19
*** mlavalle has joined #opendev14:22
corvusmordred, clarkb, fungi: the zk tls change deployed automatically without incident.  there was about a 4 minute outage -- we expected something like that because the quorum was switching to tls, so it couldn't do a rolling restart.14:35
corvusoh sorry a 14m outage14:35
corvus(because i think our playbook does attempt a rolling restart)14:36
funginice!14:37
clarkbyes it restarts them in order14:37
fungiso zk-zk communication is tls now but client communication is still over plaintext?14:37
clarkbnirmally its zero downtime as a result14:37
corvusright, but our tls port is open, so we can write some test scripts to track down that bug14:38
fungiyep, that's where i thought we were, just making sure14:38
clarkbhttp://cacti.openstack.org/cacti/graph.php?action=view&local_graph_id=66633&rra_id=all continues to look good. I think we can land https://review.opendev.org/#/c/737098/ and its parent to upgradegitea to 1.1214:40
openstackgerritJavier Peña proposed opendev/system-config master: Make the base role and playbook compatible with CentOS  https://review.opendev.org/73704314:40
openstackgerritJavier Peña proposed opendev/system-config master: [WIP] Support CentOS for AFS mirror  https://review.opendev.org/73699614:41
clarkbI'll remove gitea01 from the emergency file one those land14:41
fungiclarkb: cool, i was just about to check up on that14:42
corvus+w14:43
clarkbtyty14:43
mordredcorvus: sweet!14:43
fungiand yeah, all the cacti graphs for it look reasonable and not newly resource-constrained14:44
mordredcorvus: and - I'm guessing we're _not_ getting errors with just zk talking to zk14:44
openstackgerritJavier Peña proposed opendev/system-config master: [WIP] Support CentOS for AFS mirror  https://review.opendev.org/73699614:45
corvusmordred: correct; not now, and not during the previous maint, which is why we figured that part was okay.  plus, the bug seemed to be in kazoo.14:45
*** ykarel is now known as ykarel|away14:46
*** sgw1 has joined #opendev14:48
corvusperfect, i reproduced the error with a simple script :)  cc: tobiash14:48
corvusi just walked the nodepool tree14:48
mordredcorvus: EXCELLENT14:49
mordredcorvus: and doing so blows your kazoo connection yeah?14:49
corvusyep14:49
mordredneat14:49
corvusi'll try to characterize this a bit more14:50
mordredso it's an easy test to run without breaking things14:50
corvusyep14:50
mordredthat's excellent14:50
corvusas soon as i do, i'll start reporting on the github issue14:50
clarkbit is always a relief when reproducing a bug like that is easy14:51
tobiashcool14:51
mordrednote to self - when using dib to create an image manually- make sure to include openssh-server element14:52
clarkbmordred: I include infra-package-needs from project-config as it pulls in a few useful things like that14:53
mordredclarkb: yeah - but this is an image I was building not for opendev14:55
*** hashar is now known as hasharAway15:18
AJaegerI have two small infra-manual changes to improve retirement docs - please review https://review.opendev.org/737134 and https://review.opendev.org/73673215:18
openstackgerritRafael Folco proposed openstack/diskimage-builder master: DNM: Debug py3 on dib 7  https://review.opendev.org/73642115:18
*** hrw has quit IRC15:31
*** hasharAway is now known as hashar15:53
openstackgerritMerged zuul/zuul-jobs master: Add tests for upload-docker-image  https://review.opendev.org/73540215:55
openstackgerritMerged zuul/zuul-jobs master: Fix and test multiarch docker builds in a release pipeline  https://review.opendev.org/73705915:55
*** rpittau is now known as rpittau|afk16:01
*** aannuusshhkkaa has joined #opendev16:04
openstackgerritJavier Peña proposed opendev/system-config master: [WIP] Support CentOS for AFS mirror  https://review.opendev.org/73699616:06
clarkbI've removed gitea01 from the emergency file as the two updates there should merge momentarily16:12
*** sshnaidm|ruck is now known as sshnaidm|afk16:13
openstackgerritMerged opendev/system-config master: Update to gitea 1.12.0  https://review.opendev.org/72965916:14
openstackgerritMerged opendev/system-config master: Small repo template cleanups in Gitea  https://review.opendev.org/73709816:14
*** shtepanie has joined #opendev16:27
mordredclarkb: I see 1.12 when I go to opendev.org16:35
clarkbmordred: ya and gitea01 switched to the prod image16:35
mordredclarkb: nova is loading in 1 second for me16:35
clarkbawesome thats down from about 4 seconds before iirc16:37
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Test multiarch release builds  https://review.opendev.org/73731516:39
clarkbthe first set of updates is done16:44
clarkbthe second is about to begin16:44
clarkbthe second basically just cleans up the activity menu as its not quite what we want16:44
* clarkb finds breakfast16:44
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Test multiarch release builds  https://review.opendev.org/73731516:45
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Test multiarch release builds  https://review.opendev.org/73731516:46
*** hashar is now known as hasharAway16:47
mnaserouu16:50
mnaserupgraded gitea, looks nice16:50
*** lpetrut has quit IRC16:50
mnaserdoes indeed seem more responsive too16:51
openstackgerritJavier Peña proposed opendev/system-config master: [WIP] Support CentOS for AFS mirror  https://review.opendev.org/73699616:51
clarkbmnaser: I did non scientific measurements and ya nova root is significantly faster as is neutron16:53
mnaser\o/16:53
* mnaser will see if search has had improvements16:53
fungiyes, much faster16:56
fungimnaser: i'm not sure we expect search to work yet, which is why we've still got hound (codesearch service) up16:57
*** diablo_rojo has joined #opendev16:57
openstackgerritMohammed Naser proposed opendev/project-config master: gate: enqueue if we get Verified+1  https://review.opendev.org/73731916:58
openstackgerritMohammed Naser proposed opendev/project-config master: recheck: allow enqueue-directly-to-gate  https://review.opendev.org/73732016:59
mnaserfungi: yeah, i was thinking more on in-repo search which seems to struggle a tad17:00
mnaserinfra-core: i made one not-very-controversal change (auto enqueue with verified+1) and one that might spark more discussion 'recheck' going straight to gate.  also, i don't know how zuul will decide which pipeline wins when doing a recheck, so there's that to understand .. i think its ok but i split them as to not hinder the other patches progress17:00
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Test multiarch release builds  https://review.opendev.org/73731517:00
*** xiaolin has quit IRC17:02
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Test multiarch release builds  https://review.opendev.org/73731517:03
clarkbmnaser: zuul will enqueue to both pipelines17:04
clarkbthere is no winning an event. each pipeline that matches runs17:04
clarkbthough I guess it says it supercedes check17:06
clarkbwhich may force check back out again if gate enqueues17:06
fungimnaser: has gitea search ever worked for you? for example, https://opendev.org/openstack/nova/search?q=bindep returns zero matches for me while `git grep bindep` in current master branch of that repo gives me 10 matches17:06
clarkbI've confirmed the activity bar is gone now17:07
clarkb#status log Upgraded gitea farm to gitea 1.12.0 with minor local edits17:08
openstackstatusclarkb: finished logging17:08
fungimnaser: also hound can limit searches to specific repos, though you have to use its advanced search options: http://codesearch.openstack.org/?repos=openstack/nova&q=bindep17:10
clarkbmnaser: ya check should be evaluated first because it is first in the config file. The change will be enqueued there. Then gate is evaluated and enqueued and becuase it supercedes check the change will then be removed from check17:10
clarkbwhat isn't clear to me is where do we require a positive verified value before gating17:10
clarkbbecause that would be the thing that interacts with this in unexpected ways17:11
mnaseryeah because there isn't really a lock in place unlike the openstack tenant17:11
mnaserfungi: it has worked _sometimes_, i think it really depends on how the string matches and if its followed by something or if its on its own17:12
clarkboh this is opendev project config then ya I think its all fine. Did you look at how the zuul tenant does it?17:12
clarkbI would set it up the same as zuul and if this is how zuul has done it we should be good17:12
mnaseri thought zuul ran the same project-config as opendev, let me double check17:12
openstackgerritMatthew Thode proposed opendev/glean master: switch glean.sh path in gentoo openrc init  https://review.opendev.org/73732517:15
corvusmnaser: drop the first patch, keep the second, and it will match the zuul tenant and do what you want in all cases17:22
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Test multiarch release builds  https://review.opendev.org/73731517:25
corvusor, actually, i have a suggested revision on the second17:25
openstackgerritMohammed Naser proposed opendev/project-config master: recheck: allow enqueue-directly-to-gate  https://review.opendev.org/73732017:26
mnaseroh, one second17:26
openstackgerritMohammed Naser proposed opendev/project-config master: recheck: allow enqueue-directly-to-gate  https://review.opendev.org/73732017:26
mnasercorvus: sorry missed your comment by a sec, that should be it17:27
openstackgerritMohammed Naser proposed opendev/project-config master: Update promote pipeline precedence  https://review.opendev.org/73733317:29
mnaserand another small thing i just saw ^17:29
corvusmnaser: cool thanks! :)17:33
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Test multiarch release builds  https://review.opendev.org/73731517:39
*** mlavalle has quit IRC17:40
openstackgerritMerged opendev/project-config master: recheck: allow enqueue-directly-to-gate  https://review.opendev.org/73732017:40
openstackgerritMerged opendev/project-config master: Update promote pipeline precedence  https://review.opendev.org/73733317:40
*** DSpider has quit IRC17:44
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Test multiarch release builds  https://review.opendev.org/73731517:46
openstackgerritMatthew Thode proposed opendev/glean master: switch glean.sh path in gentoo openrc init  https://review.opendev.org/73732517:57
mnaserhrm18:03
mnaserhttp://grafana.openstack.org/d/nuvIH5Imk/nodepool-vexxhost18:03
mnaserok i can see the data inside graphite18:08
fungineat, did we merge somethnig to break that dashboard's queries?18:09
mnaseri wonder if we just need to repoint to the right hostname (aka graphite01.opendev.org)18:09
mnaserah, graphite.openstack.org is a CNAME to graphite01.opendev.org18:09
mnaserso that shouldn't be ok18:09
fungiyeah, its been graphite01.opendev.org for over a year18:10
mnasergrafana looks pretty new though18:10
fungiso i wouldn't expect that to have suddenly broken queries18:10
mnaserit lists version 7.0.318:10
mnaserwhich was released june 3rd 202018:11
fungithe grafana server itself hasn't been rebuilt in over a year, but yeah we do seem to be periodically or continuously deploying updates for it18:11
mnaserhttp://grafana.openstack.org/d/MYvSHcSiz/git-load-balancer?orgId=1 these other graphs are working for what it's worth18:11
mnaserfungi: ooooh i know, we started using ca-ymq-1 instead of sjc118:12
mnaserand the 'region' option is empty, so maybe that's it18:12
fungiaha! uep18:12
fungier, yep18:13
mnaserbut, the rackspace one is broken too18:13
mnaserthe selection is empty, so it might be related to that18:13
mnaserso sounds like the query bit is the non-functional part18:14
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Test multiarch release builds  https://review.opendev.org/73731518:14
fungiovh is another where we have multiple regions18:15
mnaserhttp://grafana.openstack.org/d/BhcSH5Iiz/nodepool-ovh?orgId=1 and is broken too, so yeah, likely that is the culprit18:15
fungioh, though some single-region nodepool dashboards are also broken18:17
fungiyeah, i have yet to find a working nodepool dashboard18:17
fungiin fact, the git lb dashboard you linked has quite a few graphs with "no data" too18:19
*** hasharAway has quit IRC18:23
openstackgerritGuillaume Chauvel proposed zuul/zuul-jobs master: prepare-workspace: Add Role Variable in README.rst  https://review.opendev.org/73735218:36
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Test multiarch release builds  https://review.opendev.org/73731518:40
*** DSpider has joined #opendev18:45
fungimnaser: looks like 7.0.0 was released a month ago, though maybe puppet-grafana started pulling 7.x more recently than that18:50
mnaserfungi: i think if someone actually logs into the grafana instance, it might actually spit out some warnings..18:50
fungiexcept our modules.env pins puppet-grafana at v6.0.0 which is over a year old18:51
clarkbthe grafana logs actually aren't really helpful18:57
clarkbits just logging people trying to find urls in there for cgi/php vulns18:58
clarkbI think where we want to start is ask grafana for the dashboard json and from that see what graphite queries it is generating18:58
clarkbwe can probably work backwards from that to see what is breaking18:58
AJaegerfungi, could you put this small infra-manual change on your review queue, please? https://review.opendev.org/73713418:59
fungiclarkb: yeah, i was trying to figure out how to see what queries grafana thinks it's been asked to perform18:59
clarkbfungi: you click on the share button on a dashboard next to the name, then switch to export and json option from there19:00
clarkb"target": "sumSeries(stats.gauges.nodepool.provider.$region.nodes.building)"19:01
*** hashar has joined #opendev19:02
fungiaha, that was not easy to find, thanks ;)19:02
clarkbI expect that we need to provide provider info there along with the region variable?19:02
clarkbthough I'm not super positive of that19:02
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Test multiarch release builds  https://review.opendev.org/73731519:04
clarkbquery: stats.gauges.nodepool.provider.ovh-* is what the grafana config says19:04
clarkbah ok that shows up in the json under different headings19:05
clarkbaha that query is how we get the regions19:06
clarkb"datasource": null <- that is the issue I think19:08
clarkblooking at grafana docs that should be a non null value so that we can lookup the templated variable19:09
AJaegerhtanks, fungi19:12
AJaegerthanks, I mean19:13
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Test multiarch release builds  https://review.opendev.org/73731519:13
fungiclarkb: so this is something which changed with your grafyaml is creating the dashboards, or changed in new grafana?19:13
fungii see we're passing $version = 'latest' in our puppet manifest19:14
clarkbfungi: I don't think we've changed how we create the dashboards, I think this is grafyaml not producing templated values properly19:16
clarkblooking at grafyaml we have a single datasource and we set it to be the default19:16
clarkbwhich I would expect to fix our problem19:16
clarkbbut maybe with templates we have to be explicit about datasources now19:16
clarkbwe can try setting a datasource and see if that fixes it19:16
fungiinterestnig that it just seems to have broken recently though. we haven't altered grafyaml's source code since the beginning of last year19:17
openstackgerritGuillaume Chauvel proposed zuul/zuul-jobs master: prepare-workspace: Add Role Variable in README.rst  https://review.opendev.org/73735219:18
fungior are you saying grafana has gotten more picky about it recently?19:18
openstackgerritClark Boylan proposed openstack/project-config master: Try set explicit datasource on the OVH dashboard  https://review.opendev.org/73736119:19
clarkbfungi: that is my assumption19:19
clarkbwe can see if ^ makes any difference for that dashboard19:19
mordredclarkb: seems reasonable19:21
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Test multiarch release builds  https://review.opendev.org/73731519:25
mordredinfra-root: could I interest anyone in reviewing https://review.opendev.org/#/c/737023/ ? I'd like to run the playbook to get review-test populated ... so it should probably be carefully reviewed19:30
openstackgerritMerged openstack/project-config master: Try set explicit datasource on the OVH dashboard  https://review.opendev.org/73736119:40
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Test multiarch release builds  https://review.opendev.org/73731519:44
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Test multiarch release builds  https://review.opendev.org/73731519:56
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Test multiarch release builds  https://review.opendev.org/73731520:14
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Test multiarch release builds  https://review.opendev.org/73731520:41
clarkbianw: mordred: because I don't quite always get how ansible works, does the condition at https://review.opendev.org/#/c/735055/5/roles/openafs-client/tasks/openafs-client/Debian.yaml check the arch of the target host of the host running ansible?21:11
mordredclarkb: before I look at it - I'm going to put money on "host running ansible"21:12
mordrednope. that would be target host21:12
mordredansible_architecture should be a fact discovered about the remote host21:12
fungiianw: not sure if you saw my ping earlier in #zuul (wrong channel sorry) but i peeked in the rsync mirror logs and vos release for mirror.fedora is running around 6-8 seconds. there's still a fairly sizeable, albeit brief, outbound spike from 01.dfw every two hours on the cacti graph though, and i'm not entirely sure what's causing it:21:13
fungihttp://cacti.openstack.org/cacti/graph.php?action=view&local_graph_id=2362&rra_id=all21:13
mordredclarkb: I left a comment there21:16
*** factor has joined #opendev21:17
clarkbmordred: I like that suggestion, thanks21:17
mordredclarkb, ianw: I think we can do that in a followup - and it might be worth a larger patch21:17
mordredor maybe just a thing we all make a mental note to do as we do things21:17
mordredclarkb: oh good - turns out we're already using that form in many places21:19
clarkbfungi: you reviewed https://review.opendev.org/#/c/735284/ but not its parent depends on. Care to review the depends on too? /me is reviewing both now21:19
openstackgerritMerged openstack/project-config master: [Grafana] Update label of neutron-functional-with-uwsgi job  https://review.opendev.org/73616821:22
fungiclarkb: yeah, i wasn't entirely clear on what the answers were to AJaeger's questions there, so thought it better to hold off reviewing until ianw could respond to them21:22
clarkbah I gave a couple thoughts to those ideas21:23
clarkbbut ya ianw's response would be good too21:23
fungioh, indeed you just did, gertty hadn't synced them for me yet21:24
openstackgerritMonty Taylor proposed opendev/system-config master: Prefix several ansible_ variables with ansible_facts  https://review.opendev.org/73738121:24
mordredclarkb, ianw: ^^21:25
mordredfungi, ianw: while we're fishing for reviews ... https://review.opendev.org/#/c/737023/ could use eyeballs21:26
*** hashar has quit IRC21:27
openstackgerritMerged openstack/project-config master: Add Backport-Candidate label for oslo deliverables  https://review.opendev.org/73409621:28
openstackgerritMerged openstack/project-config master: Add os_adjutant zuul project  https://review.opendev.org/73614521:28
clarkbre grafana I think we are waiting on a puppet pulse now which happens daily?21:29
fungiclarkb: yeah, i've been watching, there's a prod-hourly buildset in progress21:30
clarkbah yup that runs puppet-else21:30
fungii think that needs to run infra-prod-remote-puppet-else21:31
fungiright, that21:31
fungisince this is still deployed with puppetry21:31
openstackgerritMerged opendev/system-config master: openafs-client: Use PPA for Xenial ARM64  https://review.opendev.org/73505521:46
fungiclarkb: 737361 merged over 1.5 hours ago so the last prod-hourly run of infra-prod-remote-puppet-else should have incorporated it, but /etc/grafyaml/config/nodepool-ovh.yaml on grafana.o.o is still showing last modified several months ago21:49
fungiactually merged 2 hours ago (which is still "over 1.5 hours ago" so i wasn't wrong!)21:50
clarkbfungi: there is actually no puppet in syslog21:50
clarkbI wonder if wearen't puppeting like we expect21:50
fungii too am wondering this21:52
fungithough my capacity for wondering about such things is rapidly drawing to a close for today21:52
ianwhey looking22:03
ianwfungi: re the spike -- the actual rsync process is still probably causing that?  just the metadata transfer though22:05
fungimaybe? that's an awful lot of "metadata" though22:09
* fungi checks the actual amount of data transferred22:09
ianwfungi: a fedora run looks like ~ 10-15mb : http://paste.openstack.org/show/795062/22:12
ianwthe others are probably similar ; so ballpark i'd say if it's < 100mb that would explain it22:13
fungithe most recent outbound spike ran from roughly 20:25-20:45 and averaged 44.55Mbps according to cacti though i don't buy the math looking at the data points... still even if it's correct that's still 6.2GiB/sec22:14
fungier, not /sec22:15
fungi6.2GiB total data transferred over the course of that 20 minute period22:15
fungilooking at the data points though i think the average was more like 63.75Mbps22:16
fungier, more like 60-65Mbps. my eyeballs aren't that precise22:17
fungihttp://cacti.openstack.org/cacti/graph.php?action=zoom&local_graph_id=2362&rra_id=5&view_type=&graph_start=1592857482&graph_end=1592858713&graph_height=120&graph_width=500&title_font_size=1222:18
fungiit says Average: 44.55M22:19
fungibit even the lowest data point in that timeframe seems to be higher than its reported average22:19
ianwfungi: i guess we have to multiply out any changes by the number of mirrors, until their cache updates?22:23
fungilikely22:27
fungiwell, divided by the number of read-only volumes i guess22:28
fungier, read-only replicas22:28
fungiassuming they're evenly balanced across the servers, which they likely aren't22:28
ianwhttp://cacti.openstack.org/cacti/graph.php?action=zoom&local_graph_id=3209&rra_id=5&view_type=&graph_start=1592850333&graph_end=1592864733 does look similar in the other diretion22:28
openstackgerritMerged opendev/system-config master: Acutally run system-config arm64 test on an arm64 node  https://review.opendev.org/73528122:31
openstackgerritIan Wienand proposed opendev/bindep master: Cull the test bindep file  https://review.opendev.org/73528222:39
openstackgerritIan Wienand proposed opendev/bindep master: Add centos 8 and focal testing  https://review.opendev.org/73526922:39
ianwfungi: ^ that adds back zip/unzip to test the non-match path as suggested22:39
ianwprobably be good to review that stack as i think the gate is broken22:40
ianwfor bindep22:40
fungiawesome thanks. i'm +2 across the board. if you want to approve with just one reviewer i don't object22:44
clarkbI can look22:45
ianwfungi: https://review.opendev.org/#/c/703055/ to handle different LC_*'s also looks worthwhile22:46
ianwit's probably worth a release too, last one as in august22:49
*** tkajinam has joined #opendev22:51
openstackgerritMerged opendev/bindep master: Use abstracted virtualenv_command from ensure-pip  https://review.opendev.org/73526722:56
openstackgerritMerged opendev/bindep master: Cull the test bindep file  https://review.opendev.org/73528222:56
*** tosky has quit IRC22:59
openstackgerritMerged opendev/bindep master: Add centos 8 and focal testing  https://review.opendev.org/73526923:08
*** pmacdonnell has joined #opendev23:32
openstackgerritMatthew Thode proposed openstack/diskimage-builder master: add musl profile to gentoo  https://review.opendev.org/73739423:38
*** ryohayakawa has joined #opendev23:56

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