Wednesday, 2022-07-27

*** rlandy is now known as rlandy|out00:08
opendevreviewIan Wienand proposed zuul/zuul-jobs master: ensure-kubernetes: install cri-dockerd  https://review.opendev.org/c/zuul/zuul-jobs/+/85109300:19
opendevreviewIan Wienand proposed zuul/zuul-jobs master: ensure-kubernetes: install cri-dockerd  https://review.opendev.org/c/zuul/zuul-jobs/+/85109300:30
opendevreviewIan Wienand proposed zuul/zuul-jobs master: ensure-kubernetes: install cri-dockerd; fix networking  https://review.opendev.org/c/zuul/zuul-jobs/+/85109300:54
opendevreviewIan Wienand proposed zuul/zuul-jobs master: test/upload-pypi : install venv  https://review.opendev.org/c/zuul/zuul-jobs/+/85108900:54
opendevreviewIan Wienand proposed zuul/zuul-jobs master: setup.py: override modules to stop auto-detection  https://review.opendev.org/c/zuul/zuul-jobs/+/85108800:54
opendevreviewIan Wienand proposed zuul/zuul-jobs master: linters: fix spaces between filters  https://review.opendev.org/c/zuul/zuul-jobs/+/85101400:54
opendevreviewIan Wienand proposed zuul/zuul-jobs master: linters: add names to blocks  https://review.opendev.org/c/zuul/zuul-jobs/+/85101500:54
opendevreviewIan Wienand proposed zuul/zuul-jobs master: linters: rename loop variable  https://review.opendev.org/c/zuul/zuul-jobs/+/85101600:54
opendevreviewIan Wienand proposed zuul/zuul-jobs master: linters: update to ansible-lint 6  https://review.opendev.org/c/zuul/zuul-jobs/+/85101700:54
opendevreviewIan Wienand proposed zuul/zuul-jobs master: setup.py: override modules to stop auto-detection  https://review.opendev.org/c/zuul/zuul-jobs/+/85108801:14
opendevreviewIan Wienand proposed zuul/zuul-jobs master: linters: fix spaces between filters  https://review.opendev.org/c/zuul/zuul-jobs/+/85101401:14
opendevreviewIan Wienand proposed zuul/zuul-jobs master: linters: add names to blocks  https://review.opendev.org/c/zuul/zuul-jobs/+/85101501:14
opendevreviewIan Wienand proposed zuul/zuul-jobs master: linters: rename loop variable  https://review.opendev.org/c/zuul/zuul-jobs/+/85101601:14
opendevreviewIan Wienand proposed zuul/zuul-jobs master: linters: update to ansible-lint 6  https://review.opendev.org/c/zuul/zuul-jobs/+/85101701:14
*** ysandeep|out is now known as ysandeep01:19
tonybHi All, I'm to use hound/codesearch to, well search the code, but on various stable branches.  I poked around in opendev/system-config but I can't find the hound config to see which refs are indexed.02:25
fungitonyb: it's only indexing (and from what i understand, only capable of indexing) the master branches of repositories02:28
tonybfungi: Based on: https://github.com/hound-search/hound/pull/345 I *think* it can index non-default branches but I can't tell if it can do multiple branches.02:30
tonybfungi: I guess I'll have to cobble something together locally to answer the requirements question(s) I have02:31
fungitonyb: yeah, last time we looked into it, we could set it to index a different branch than master, but then it wouldn't index master02:38
tonybI can forsee multiple instances running for codesearch.opendev.org/{main,zed,yoga,wallaby}/......02:39
tonybI've very far from serious02:39
fungiyeah, that would be kinda rough02:42
tonybfor now: for d in ~/projects/OIF/openstack/* ; do git -C $d grep --color=never -E '(${pypi_library})' origin/stable/wallaby -- \*requirements.txt 2>/dev/null| sed -e "s/^/$d:/g"; done03:09
tonybwill have to do03:10
opendevreviewIan Wienand proposed zuul/zuul-jobs master: [wip] upload-git-mirror : testing  https://review.opendev.org/c/zuul/zuul-jobs/+/85114703:26
opendevreviewIan Wienand proposed zuul/zuul-jobs master: [wip] upload-git-mirror : testing  https://review.opendev.org/c/zuul/zuul-jobs/+/85114703:59
opendevreviewIan Wienand proposed zuul/zuul-jobs master: [wip] upload-git-mirror : testing  https://review.opendev.org/c/zuul/zuul-jobs/+/85114704:23
opendevreviewIan Wienand proposed zuul/zuul-jobs master: [wip] upload-git-mirror : testing  https://review.opendev.org/c/zuul/zuul-jobs/+/85114704:29
opendevreviewIan Wienand proposed zuul/zuul-jobs master: upload-git-mirror: fix ssh key newline  https://review.opendev.org/c/zuul/zuul-jobs/+/85114704:42
opendevreviewIan Wienand proposed zuul/zuul-jobs master: linters: fix spaces between filters  https://review.opendev.org/c/zuul/zuul-jobs/+/85101404:42
opendevreviewIan Wienand proposed zuul/zuul-jobs master: linters: add names to blocks  https://review.opendev.org/c/zuul/zuul-jobs/+/85101504:42
opendevreviewIan Wienand proposed zuul/zuul-jobs master: linters: rename loop variable  https://review.opendev.org/c/zuul/zuul-jobs/+/85101604:42
opendevreviewIan Wienand proposed zuul/zuul-jobs master: linters: update to ansible-lint 6  https://review.opendev.org/c/zuul/zuul-jobs/+/85101704:42
opendevreviewIan Wienand proposed openstack/project-config master: proposal updates: fix typo in release match  https://review.opendev.org/c/openstack/project-config/+/85115604:50
opendevreviewMerged openstack/project-config master: proposal updates: fix typo in release match  https://review.opendev.org/c/openstack/project-config/+/85115605:11
*** ysandeep is now known as ysandeep|afk06:08
opendevreviewIan Wienand proposed zuul/zuul-jobs master: linters: rename loop variable  https://review.opendev.org/c/zuul/zuul-jobs/+/85101607:20
opendevreviewIan Wienand proposed zuul/zuul-jobs master: linters: fix spaces between filters  https://review.opendev.org/c/zuul/zuul-jobs/+/85101407:20
opendevreviewIan Wienand proposed zuul/zuul-jobs master: linters: add names to blocks  https://review.opendev.org/c/zuul/zuul-jobs/+/85101507:20
opendevreviewIan Wienand proposed zuul/zuul-jobs master: linters: update to ansible-lint 6  https://review.opendev.org/c/zuul/zuul-jobs/+/85101707:20
opendevreviewIan Wienand proposed zuul/zuul-jobs master: emit-job-header: noqa on error ignore  https://review.opendev.org/c/zuul/zuul-jobs/+/85116307:20
opendevreviewIan Wienand proposed zuul/zuul-jobs master: ansible-lint: disable progressive mode  https://review.opendev.org/c/zuul/zuul-jobs/+/85116407:20
*** jpena|off is now known as jpena07:35
*** ysandeep|afk is now known as ysandeep|lunch08:00
*** undefined is now known as Guest617608:19
*** undefined is now known as Guest617809:23
hrwmorning09:29
hrwis someone using centos-stream-9 nodes?09:29
hrwas I start to wonder why kolla-ansible CI job OOMs on cs9 while it works on Debian and Ubuntu09:33
ianwhrw: they do get some use; i would say that you might like to look at the dstat service run by devstack as a monitoring tool09:38
hrwianw: I booted local cs9 node and deployed. rabbitmq container eats 25% of ram..09:38
*** ysandeep|lunch is now known as ysandeep10:01
*** rlandy|out is now known as rlandy10:34
*** dviroel|out is now known as dviroel11:35
jrosserhrw: fwiw we see OOM all the time on openstack-ansible jobs for centos-8/9-stream where the debian/ubuntu is fine11:46
hrwjrosser: thanks11:46
jrosserit is one of the things we just can't really get much grip on understanding11:46
jrosserwe've turned all the threads/processes down to minimum, in fact too much because thats now causing other types of error11:47
*** ysandeep is now known as ysandeep|break12:19
fungihrw: jrosser: do we maybe set up swap differently between them?12:31
fungialso be aware, in last week's tc meeting there seemed to be consensus that centos stream isn't a stable enough for pre-merge testing in openstack and folks were going to work on dropping the requirement to just daily periodic jobs12:32
jrosserhmm well unfortunatley in a deployment tool a large proportion of time is spent wrangling the differences between distro X and Y, so thats a lot harder12:34
jrosserand looking at the osa dstat setup seems this has bitrotted and needs some tlc12:38
hrwfungi: we (kolla) plan to go rockylinux9 but it is not there yet12:39
fungijrosser: "this" being swap setup?12:40
jrosseroh no, something has changed with dstat and the way we run it makes an error rather than a useful report12:41
fungioh, got it12:41
fungithought you meant dstat was telling you something other than "dstat broke" ;)12:42
jrosserahhhh if only :)12:42
jrosserwhat is certainly true is we have struggled with these OOM for quite some time now12:43
fungiyeah, without some profiling to tell you what's eating all your ram or happening to spot some inconsistency like less/no swap on centos nodes, it's hard to guess12:44
jrosserit's revealed a few places we forgot to turn down the threads/processes, fixing those helped some jobs. but it still feels too tight12:44
fungialso this may be another sign of openstack needing to go on a diet. as time goes on, we seem to be making it less memory-efficient rather than more12:45
jrosserpart of the trouble with really being aggressive on threads/processes seems to be that a long running request can take longer than the haproxy healthcheck interval, and if that doesnt get serviced then everything comes crashing down12:46
*** ysandeep|break is now known as ysandeep12:58
*** rlandy is now known as rlandy|biab13:17
*** rlandy|biab is now known as rlandy13:33
*** dasm|off is now known as dasm14:42
*** ysandeep is now known as ysandeep|out14:54
*** dviroel is now known as dviroel|lunch14:57
*** Guest6178 is now known as rcastillo15:11
clarkbjrosser: hrw: iirc frickler found newer qemu is very memory inefficient and had to work around that on debian jobs with devsatck15:40
clarkbit allocates a ton of memory even for tiny VMs or something. I would check to see if that is an issue on centos 9 stream and if so look to see what was done on debian to address it15:40
clarkbjrosser: re dstat it is provided by https://pcp.io/ now and pcp relies on a bunch of daemons managed by systemd and half the time the services don't start properly on ubuntu.15:50
clarkbreally unfortunate that dstat proper died as it was much simpler15:50
*** dviroel|lunch is now known as dviroel15:52
jrosserthere are some zuul roles for this i think, i probably have to look at retrofitting what they do into OSA jobs15:52
hrwclarkb: rabbitmq allocated lot of memory. like 13x more15:57
hrwwill look into that later today/tomorrow15:57
*** marios is now known as marios|out16:04
opendevreviewClark Boylan proposed openstack/project-config master: Set ensure_pip_virtualenv_command in Focal proposal jobs  https://review.opendev.org/c/openstack/project-config/+/85123916:10
corvusi'm looking into the stuck queue (zuul tenant, gate pipeline)16:26
*** jpena is now known as jpena|off16:32
opendevreviewAlex Kavanagh proposed openstack/project-config master: Complete charm-cinder-dell-emc-powerstore config  https://review.opendev.org/c/openstack/project-config/+/85124216:40
opendevreviewMerged openstack/project-config master: Complete charm-cinder-dell-emc-powerstore config  https://review.opendev.org/c/openstack/project-config/+/85124217:00
corvusi identified the problem and have a fix in progress.  i'm going to restart zuul02 which should correct the current issue17:27
corvus#status log restarted zuul02 to resolve conflicting change key error17:29
opendevstatuscorvus: finished logging17:29
fungithanks for finding/fixing!17:57
fungipip 22.2.1 is out, maybe watch out for new issues around setuptools' vendored copy of distutils: https://pip.pypa.io/en/stable/news/#v22-2-118:22
opendevreviewClark Boylan proposed opendev/system-config master: WIP Add a mailman3 list server  https://review.opendev.org/c/opendev/system-config/+/85124818:37
clarkbfungi: ^ that is a very early mostly outline/sketch of what things will look like. It doesn't actually install any mailman3 or apache stuff yet. But it should run a test and ensure our LE stuff and docker install works on jammy18:37
clarkband have all the other files and bits we'll be modifying to add in the mailman3 stuff18:38
clarkbI think I'm switching to the laptop after lunch as well so pushing what i have now makes it easy to pull it on the other machine :)18:38
fungithanks! i'm hacking on the pvmove for the afs and backup servers in preparation for the upcoming block device maintenance, but will take a look once i have these underway18:38
clarkbthe parent chagne for that should be mergeable though.18:40
fungiafs01.ord has a root screen session with a `pvmove /dev/xvdb1 /dev/xvdc1` now underway18:42
fungiwith a goal of freeing up the afs01.ord.openstack.org/main01 volume for deletion18:43
fungibackup01.ord has two volumes to move off of, so it'll probably take a bit longer18:43
fungibackup01.ord has a root screen session with a `pvmove /dev/xvdb /dev/xvdc1` now underway for the main-202010 vg (and yes, the old one is a raw device while the new one is a partition, slowly getting this server consistent with the others)18:52
fungiall these volumes are 1tb, so pvmove will take a while18:52
fungiclarkb: where's the magic that installs the distro release-specific sources.list files?19:09
fungioh, found it i think19:09
fungiplaybooks/roles/base/repos/tasks/Debian.yaml also covers ubuntu, in contrast to its name19:10
fungiyeah, the "Replace sources.list file" task in there is the magic i was looking for19:13
fungisystem-config-run-mirror-arm64 isn't really exercising the ubuntu-jammy-arm64 addition in 851094 but i don't suppose that's especially important for now as this is mainly in service of getting mailman running which we'll do on amd64 anyway19:17
Clark[m]Ya sorry, stepped away to figure out lunch. Ya this is early bootstrapping. There is an arm base job which should exercise jammy though19:17
fungisystem-config-run-base-arm64? that wasn't run19:19
fungioh, wait...19:19
funginevermind, i was looking right at it19:20
fungii forgot there's more than one buildset19:20
fungipvmove on the afs server and first round on the backup server are nearly done20:09
opendevreviewClark Boylan proposed opendev/system-config master: WIP Add a mailman3 list server  https://review.opendev.org/c/opendev/system-config/+/85124820:22
fungifirst old volume detached from backup01.ord and cleaned up, second volume attached and prepped and now `pvmove /dev/xvdf /dev/xvdb1` is underway there (yes another move from a raw disk to a partition)20:23
fungiand only a few percent left on the afs server20:23
opendevreviewClark Boylan proposed opendev/system-config master: WIP Add a mailman3 list server  https://review.opendev.org/c/opendev/system-config/+/85124820:52
*** timburke_ is now known as timburke20:58
opendevreviewMerged openstack/project-config master: Set ensure_pip_virtualenv_command in Focal proposal jobs  https://review.opendev.org/c/openstack/project-config/+/85123921:01
*** dasm is now known as dasm|off21:02
fungiold afs cinder volume has been cleaned up now, just waiting on the second backup volume pvmove to complete21:06
opendevreviewMerged zuul/zuul-jobs master: upload-artifactory: fix version comparision  https://review.opendev.org/c/zuul/zuul-jobs/+/85108721:22
clarkbI've got a weird ansible templating issue in the mailman3 change. I have a lists99.opendev.org.yaml.j2 template file for the host_vars. But that needs to be sent through raw without substituting variables and we need to preserve the existing raw block there. It seems we can't do nested raw blocks because it ends at the first endraw?21:24
clarkbI guess I could modify the setup base playbook to copy that file directly without interpolating vars21:24
opendevreviewClark Boylan proposed opendev/system-config master: WIP Add a mailman3 list server  https://review.opendev.org/c/opendev/system-config/+/85124821:31
clarkbthat seems extra verbose but I can't figure out how to make jinja2 happy and we don't actually need to substitute any values there21:31
fungiyeah, that's unfortunately probably the simplest hackaround21:32
opendevreviewIan Wienand proposed zuul/zuul-jobs master: ensure-kubernetes: pull cri-dockerd systemd from tag  https://review.opendev.org/c/zuul/zuul-jobs/+/85126321:41
corvusif you want to take a break and see something cool, load up https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_a5a/851107/2/check/zuul-build-dashboard-opendev/a5a80be/npm/html/ then click on openstack projects, then opendev/system-config then enter "master" into the branch selection at the bottom of the page21:48
corvus(that will totally be deep-linkable when it lands, but you have to go through the steps manually since it's a preview build)21:48
corvus(i'm kind of thinking the arrows should go the other way, ie, show the dependency relationship rather than the job flow.  i think that will make it easier to read)21:55
opendevreviewClark Boylan proposed opendev/system-config master: WIP Add a mailman3 list server  https://review.opendev.org/c/opendev/system-config/+/85124821:58
clarkbcorvus: scrolling works oddly on that graphic (it does zoom rather than shift up and down). But I agree pretty neat22:00
corvusclarkb: with a scroll wheel?22:00
clarkbcorvus: two finger scrolling on a touchpad so ya should be the same as a scroll wheel I think22:00
clarkbalso ++ to changing arrow direction22:01
corvusneat.  i've been zooming with double-click and shift-double-click.  left-click should pan it.  we might be able to prevent it from acting on the scroll wheel/touchpad22:01
corvus(i guess it depends on what people expect; i don't have a scroll wheel/touchpad so i have no expectation :)22:02
clarkbI think scrollwheel should scroll the page and keep zoom level the same. But ya I can see an argument for either method22:02
*** rlandy is now known as rlandy|bbl22:06
corvuswe can see at a glance that nothing is using 3.8 -- are we keeping those images around for uwsgi?22:10
clarkbya I think mostly because at one time other people were using them and zuul only recently got off of 3.822:11
clarkbwe likely can clean up 3.8 at this point22:11
ianwvery cool.  yeah the scroll did get me in that when it comes up the mouse is over the image and scrolling zooms it in and out22:20
ianwso it's like the page scrolls down until it hits the image, then starts zooming the image22:21
ianwi guess with something like google maps, it's a "full page" app or whatever so doesn't deal with scrollbars with zooming22:21
opendevreviewClark Boylan proposed opendev/system-config master: WIP Add a mailman3 list server  https://review.opendev.org/c/opendev/system-config/+/85124822:24
opendevreviewClark Boylan proposed opendev/system-config master: Ensure /etc/apt/sources.list.d exists  https://review.opendev.org/c/opendev/system-config/+/85126622:24
opendevreviewIan Wienand proposed zuul/zuul-jobs master: ensure-kubernetes: pull cri-dockerd systemd from tag  https://review.opendev.org/c/zuul/zuul-jobs/+/85126322:26
*** dviroel is now known as dviroel|afk22:36
opendevreviewClark Boylan proposed opendev/system-config master: WIP Add a mailman3 list server  https://review.opendev.org/c/opendev/system-config/+/85124823:20
clarkbok I think that might actually pass now so all the framework stuff is in place. I'll pick up on adding in service bits tomorrow23:20
corvusany preference between these 3 styles?  forw-RL https://imgur.com/Ii3unWe.png forw-LR https://imgur.com/n30UqR1.png back-LR https://imgur.com/tYSEfGK.png23:24
corvus(those all have the arrows in the dependency direction -- so the content is the same, just different layout)23:25
corvus(i think i like back-LR which is the same layout as the current change except the arrows are on the other ends of the edge lines)23:26
clarkbI don't like the second one. Don't really have a preference between the first and third. I think that is my read from left to right bias23:26
corvusyeah, i think i like back-lr (third) because you sort of read time from left to right on that, then go backwards to follow the dependency arrows.23:27
ianw3 seems most "right" to me also23:28
ianwi could see an argument for the arrows going the other way to23:29
ianwon 323:29
corvuskk done23:31
corvusi think it's the dashed line that clinches it for me with the arrows.  i was explaining what the arrows and dashed line meant to someone and said "they represent hard or soft dependencies.  but the dependencies go the other way."23:33
opendevreviewIan Wienand proposed zuul/zuul-jobs master: upload-git-mirror: no_log around key writing  https://review.opendev.org/c/zuul/zuul-jobs/+/85126923:33
fungisecond backup01.ord volume has been swapped out and cleaned up. i'll close out the maintenance ticket now that all the volumes they warned us would be disrupted no longer exist23:50
opendevreviewMerged zuul/zuul-jobs master: ensure-kubernetes: install cri-dockerd; fix networking  https://review.opendev.org/c/zuul/zuul-jobs/+/85109323:53
opendevreviewMerged zuul/zuul-jobs master: test/upload-pypi : install venv  https://review.opendev.org/c/zuul/zuul-jobs/+/85108923:53
opendevreviewMerged zuul/zuul-jobs master: setup.py: override modules to stop auto-detection  https://review.opendev.org/c/zuul/zuul-jobs/+/85108823:53
opendevreviewMerged zuul/zuul-jobs master: upload-git-mirror: fix ssh key newline  https://review.opendev.org/c/zuul/zuul-jobs/+/85114723:53
opendevreviewMerged zuul/zuul-jobs master: linters: rename loop variable  https://review.opendev.org/c/zuul/zuul-jobs/+/85101623:53

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