Thursday, 2022-06-16

*** ysandeep|out is now known as ysandeep03:45
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: Support service tokens  https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/84590406:49
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: Support service tokens  https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/84590406:50
noonedeadpunkso. most of upgrade jobs fail on tempest with process being killed06:55
noonedeadpunk /usr/local/bin/openstack-ansible: line 91: 131870 Killed                  ansible-playbook ${VAR1} "${@}"06:55
jrosser_it’s not OOM is it?06:57
noonedeadpunkWell. Interesting is that it's really often ~2h when it's killed06:58
noonedeadpunkbut yeah06:59
noonedeadpunkI think oom06:59
noonedeadpunkI can recall some patches to reduces processes, maybe we should backport smth...07:08
noonedeadpunkok, see cloudkitty and mistral and both backported...07:11
jrosser_neutron too if we make that uwsgi now?07:12
noonedeadpunkI think we have neutron already https://opendev.org/openstack/openstack-ansible/src/branch/master/tests/roles/bootstrap-host/templates/user_variables.aio.yml.j2#L107-L11307:34
noonedeadpunkor not...07:35
noonedeadpunkmaybe that's the reason actually07:36
noonedeadpunkthat's really good point...07:36
jrosser_this https://opendev.org/openstack/openstack-ansible/commit/6eb2dec708b41f3f8b8c61d4a916549205bfb56207:36
opendevreviewJonathan Rosser proposed openstack/openstack-ansible stable/xena: Reduce neutron WSGI workers for CI  https://review.opendev.org/c/openstack/openstack-ansible/+/84611707:37
jrosser_i'm wondering if this is related to the problems we get with ansible 2.13 https://github.com/pallets/jinja/issues/151007:58
noonedeadpunkI haven't looked into 2.13 failures07:58
jrosser_currently the version of jinja2 is held back in ansible-runtime venv by u-c07:59
jrosser_yeah there are bigger things to fix this week i think07:59
jrosser_our bionic jobs look unhappy https://zuul.openstack.org/builds?job_name=openstack-ansible-deploy-aio_metal-ubuntu-bionic&skip=008:22
jrosser_urgh rabbitmq install trouble https://paste.opendev.org/show/bWr1wmqb2v4G4I1z61bh/08:24
noonedeadpunkoh yes08:26
noonedeadpunkI wonder if we should also backport switching repos for rabbit...08:26
jrosser_reformatted https://paste.opendev.org/show/btZwDwdi5Wj6AUj6Pimf/08:26
noonedeadpunkAs this issue is present at least for 3 days08:26
noonedeadpunkhttps://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/826444 <- this one08:27
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server stable/xena: Use cloudsmith repo for rabbit and erlang  https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/84612008:29
noonedeadpunkoh, wait08:30
noonedeadpunkwell, there's totally bionic mentioned 08:33
jrosser_do they have the versions we want though08:33
noonedeadpunkI see it here at least https://packages.erlang-solutions.com/erlang/debian/pool/08:34
jrosser_yes https://cloudsmith.io/~rabbitmq/repos/rabbitmq-server/packages/?q=distribution%3Aubuntu+AND+distribution%3Abionic+AND+version%3A3.9.8-108:35
jrosser_what is actually breaking08:35
noonedeadpunkwell, we don't use cloudsmith repo on Xena I guess08:59
opendevreviewJonathan Rosser proposed openstack/openstack-ansible master: Add mistral-extra to required projects in CI  https://review.opendev.org/c/openstack/openstack-ansible/+/84613508:59
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-os_mistral master: Add mistral-extra in the mistral venv  https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/84404909:00
noonedeadpunkjrosser_: let's also add mistral-extra to https://opendev.org/openstack/openstack-ansible/src/branch/master/playbooks/defaults/repo_packages/openstack_services.yml09:05
jrosser_https://review.opendev.org/c/openstack/openstack-ansible/+/84404809:06
jrosser_something wierd happening with magnum on centos https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/845993/09:07
jrosser_tempest appears to run without failures but gives rc=1 so the task fails09:08
opendevreviewMerged openstack/openstack-ansible-os_cinder master: Forcefully remove Cinder v2 endpoints  https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/84571409:08
noonedeadpunkah, ok09:11
noonedeadpunkregarding magnum - that looks suspicious https://zuul.opendev.org/t/openstack/build/676f976e87b3444399149a402ddb24a3/log/logs/openstack/aio1-utility/tempest_run.log.txt#5109:13
opendevreviewMerged openstack/openstack-ansible-lxc_container_create master: Add ovs support for lxc containers.  https://review.opendev.org/c/openstack/openstack-ansible-lxc_container_create/+/83773409:13
noonedeadpunklike it being killed during execution or smth09:13
noonedeadpunkand same test just fails for centos 9 https://zuul.opendev.org/t/openstack/build/48bf0c53a0cc4294a516aae6d2bcf349/log/logs/openstack/aio1-utility/tempest_run.log.txt#5209:15
jrosser_intereseting09:18
jrosser_magnum API breaks https://zuul.opendev.org/t/openstack/build/676f976e87b3444399149a402ddb24a3/log/logs/host/magnum-api.service.journal-16-14-52.log.txt#2603-261209:21
noonedeadpunkI wonder if thats tempest plugin issue09:23
noonedeadpunknah, it wasn;t changed since 201809:24
opendevreviewMerged openstack/ansible-config_template master: Use ensure SafeDumper used yaml configs  https://review.opendev.org/c/openstack/ansible-config_template/+/83814109:25
opendevreviewMerged openstack/openstack-ansible-plugins master: Use shlex_quote from ansible module_utils rather than SSH plugin  https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/84579209:27
noonedeadpunkHm, according to timestamps `ClusterTemplate fooo could not be found` is different from `OSError: write error`09:29
noonedeadpunkmessages look even more interesting09:32
noonedeadpunkhttps://paste.openstack.org/show/bGPX0X4UqFg2D4Y0r3u5/09:32
noonedeadpunkfeels like oom as well tbh09:32
noonedeadpunkas basically it's glance that appears t obe dead09:33
noonedeadpunkJun 16 01:36:09 aio1 glance-wsgi-api[49493]: 2022-06-16 01:36:09.464 49493 CRITICAL glance [req-f20f2697-06c0-4770-85d0-0e9581626f2a 6493ee21f61e4f03b23f314ce3990822 c239e4619618498ea1c785fa9acb014e - default default] Unhandled error: OSError: write error#0122022-06-16 01:36:09.464 49493 ERROR glance OSError: write error#0122022-06-16 01:36:09.464 49493 ERROR glance 09:33
noonedeadpunkSo OSError that magnum reports is also glance thing09:34
noonedeadpunkand placement reports same...09:34
noonedeadpunkhm09:34
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server stable/wallaby: Use cloudsmith repo for rabbit and erlang  https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/84612209:36
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Reduce m1.magnum flavor  https://review.opendev.org/c/openstack/openstack-ansible/+/84614209:43
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_magnum master: Support service tokens  https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/84603709:44
opendevreviewMerged openstack/openstack-ansible master: Increase manila flavor size  https://review.opendev.org/c/openstack/openstack-ansible/+/84599809:54
opendevreviewMerged openstack/openstack-ansible-os_glance master: Fix glance mounts compatability  https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/84576709:55
opendevreviewMerged openstack/openstack-ansible-os_glance master: Remove mention of haproxy-endpoints role  https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/84579609:57
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Update magnum tags and images for CI  https://review.opendev.org/c/openstack/openstack-ansible/+/84614510:03
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible stable/xena: Increase manila flavor size  https://review.opendev.org/c/openstack/openstack-ansible/+/84612310:04
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_magnum master: Fixed dest typo in config_template  https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/84599310:06
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible stable/xena: Reduce neutron WSGI workers for CI  https://review.opendev.org/c/openstack/openstack-ansible/+/84611710:14
noonedeadpunktbh I'm surprised how good CentOS 9 is passing across roles10:15
jrosser_yes, its worked out ok10:37
opendevreviewMerged openstack/openstack-ansible-plugins master: Add support for role list  https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/84569010:38
jrosser_noonedeadpunk: comment on here https://review.opendev.org/c/openstack/openstack-ansible-os_placement/+/84570810:47
noonedeadpunkyep, thanks:)10:48
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_placement master: Support service tokens  https://review.opendev.org/c/openstack/openstack-ansible-os_placement/+/84570810:48
*** ysandeep is now known as ysandeep|afk11:08
*** dviroel|afk is now known as dviroel11:18
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_murano master: Delegate package import to setup host  https://review.opendev.org/c/openstack/openstack-ansible-os_murano/+/84614711:30
noonedeadpunkhopefully this will unblock murano... ^11:31
noonedeadpunkexactly same issue as was with manilla11:31
noonedeadpunknot sure what to do with magnum though....11:32
noonedeadpunkit just oom-ing11:37
noonedeadpunkoh, well...11:42
noonedeadpunkI think I found wtf11:43
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Limit magnum workers in CI  https://review.opendev.org/c/openstack/openstack-ansible/+/84615111:47
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_magnum master: Control amount of conductor workers  https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/84615311:52
jrosser_^ oh nice find11:57
*** ysandeep|afk is now known as ysandeep12:18
opendevreviewMerged openstack/openstack-ansible-os_trove master: Support service tokens  https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/84604912:23
opendevreviewMerged openstack/openstack-ansible-os_barbican master: Support service tokens  https://review.opendev.org/c/openstack/openstack-ansible-os_barbican/+/84601312:25
opendevreviewMerged openstack/openstack-ansible-os_tacker master: Support service tokens  https://review.opendev.org/c/openstack/openstack-ansible-os_tacker/+/84605012:26
opendevreviewMerged openstack/openstack-ansible-os_blazar master: Support service tokens  https://review.opendev.org/c/openstack/openstack-ansible-os_blazar/+/84601512:26
opendevreviewMerged openstack/openstack-ansible-os_cinder master: Support service tokens  https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/84571712:28
opendevreviewMerged openstack/openstack-ansible-os_designate master: Support service tokens  https://review.opendev.org/c/openstack/openstack-ansible-os_designate/+/84602612:28
opendevreviewMerged openstack/openstack-ansible-os_nova master: Support service tokens  https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/84590312:29
mgariepygood morning12:32
opendevreviewMerged openstack/openstack-ansible-os_heat master: Support service tokens  https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/84602912:34
opendevreviewMerged openstack/openstack-ansible-os_senlin master: Support service tokens  https://review.opendev.org/c/openstack/openstack-ansible-os_senlin/+/84604312:35
jrosser_hello mgariepy 12:36
mgariepyanything urgent to review or fix ?12:37
jrosser_there is this topic https://review.opendev.org/q/topic:osa%252Fservice_tokens though noonedeadpunk has some fixes in-progress for some of it12:38
jrosser_and the remainder of https://review.opendev.org/q/topic:osa%252Fmanage_lb12:39
noonedeadpunkI guess main thing now is backport for neutron process limit12:39
jrosser_looks like 846117 and 846120 are progressing well12:42
noonedeadpunkyup12:43
noonedeadpunknot really for W though...12:43
noonedeadpunkbut anyway12:44
jrosser_looks like a couple of jobs failing to upload logs again12:46
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Install murano client for distro installs  https://review.opendev.org/c/openstack/openstack-ansible/+/84616212:53
noonedeadpunkyup...12:54
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_murano master: Delegate package import to setup host  https://review.opendev.org/c/openstack/openstack-ansible-os_murano/+/84614712:55
noonedeadpunkoh, nice to know that neutron was failing indeed because of missing nova patch12:58
noonedeadpunkfor service_tokens I mean12:59
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_murano master: Support service tokens  https://review.opendev.org/c/openstack/openstack-ansible-os_murano/+/84616313:00
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_murano master: Use global service variables  https://review.opendev.org/c/openstack/openstack-ansible-os_murano/+/84616513:05
opendevreviewMerged openstack/openstack-ansible-os_glance master: Support service tokens  https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/82300913:21
opendevreviewMerged openstack/openstack-ansible master: Bump SHA for pacemaker-corosync role  https://review.opendev.org/c/openstack/openstack-ansible/+/84599413:23
opendevreviewMerged openstack/openstack-ansible-rabbitmq_server stable/xena: Use cloudsmith repo for rabbit and erlang  https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/84612013:38
mgariepywtf :/ https://zuul.opendev.org/t/openstack/build/529bbc921dd046698d1fa30fb7d7a6a7/log/job-output.txt#848313:39
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible stable/xena: Define corosync interface for facts gathering  https://review.opendev.org/c/openstack/openstack-ansible/+/84612413:50
noonedeadpunklooks like just candidate for recheck....13:51
mgariepystill weird.13:52
noonedeadpunkoh yes... it is..13:57
mgariepyleft a comment on : https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/84604613:58
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_zun master: Support service tokens  https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/84604614:02
noonedeadpunkyes, good catch, thanks14:02
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Create  containers for rally/tempest independently  https://review.opendev.org/c/openstack/openstack-ansible/+/84068514:04
*** ysandeep is now known as ysandeep|afk14:08
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Create  containers for rally/tempest independently  https://review.opendev.org/c/openstack/openstack-ansible/+/84068514:15
noonedeadpunkI'm actually not sure how good this idea is overall now tbh ^14:15
noonedeadpunkmaybe we should just leave things as is with weird vars...14:15
noonedeadpunkAs with that we add 2 containers at least, resource consumption with that as well14:16
noonedeadpunkand execution time14:16
noonedeadpunkMaybe I should look better in how to use single container for rally/tempest and if it's possbible...14:18
noonedeadpunkanyway this can be left for Z14:18
*** ysandeep|afk is now known as ysandeep14:58
opendevreviewMerged openstack/openstack-ansible-os_neutron master: Support service tokens  https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/84590415:00
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Fix deprecations handling in journal logs  https://review.opendev.org/c/openstack/openstack-ansible/+/84618915:30
noonedeadpunkWell, I'd say that we have everything required pushed. Hopefully, reducing neutron wsgi processes would help passing upgrade jobs15:39
noonedeadpunkas with that everything should jsut land15:39
mgariepyneutron worker is rechecking :/15:40
noonedeadpunkyeah....15:40
*** ysandeep is now known as ysandeep|out15:58
opendevreviewMerged openstack/openstack-ansible-os_manila master: Delegate share type creation to setup host  https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/84591717:16
mgariepyhttps://zuul.opendev.org/t/openstack/build/0e6a21621f3a40619c174c8f8dff9b30/log/logs/openstack/aio1_galera_container-2ac8c217/mariadb.service.journal-16-35-24.log.txt#278-28017:24
mgariepyc9s error mariadb :/17:25
opendevreviewMerged openstack/openstack-ansible stable/xena: Reduce neutron WSGI workers for CI  https://review.opendev.org/c/openstack/openstack-ansible/+/84611717:47
opendevreviewMerged openstack/ansible-config_template master: Release config-template to 1.3.0  https://review.opendev.org/c/openstack/ansible-config_template/+/84569917:56
noonedeadpunkthanks mgariepy for rechecks) btw murano needs https://review.opendev.org/c/openstack/openstack-ansible/+/846162 and there're bunch of improvements in integrated repo for magnum18:10
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_murano master: Remove mention of haproxy-endpoints role  https://review.opendev.org/c/openstack/openstack-ansible-os_murano/+/84580218:12
noonedeadpunkbtw sorry, I found yours https://review.opendev.org/c/openstack/openstack-ansible-os_murano/+/840729 only when pushed another one :(18:13
mgariepymeh not an issue ;)18:13
mgariepyi didn't babysit it anyway :)18:14
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_magnum master: Support service tokens  https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/84603718:19
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_magnum master: Fixed dest typo in config_template  https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/84599318:20
noonedeadpunkbtw for masakari I beleive we also need to land this https://review.opendev.org/c/openstack/openstack-ansible/+/84612418:21
noonedeadpunkas their upgrade jubs fail here18:22
mgariepyyeah i saw that but i was kinda asking myself if the master one should merge first..18:23
mgariepybut stuff are failing left and right today.. lots of weird issue..18:23
mgariepyi wonder if we should collect : '/var/lib/mysql/mariabackup.backup.log'18:25
mgariepyas it would probably contains a bit of details on the reason why c9s has failed here : https://zuul.opendev.org/t/openstack/build/0e6a21621f3a40619c174c8f8dff9b30/log/logs/openstack/aio1_galera_container-2ac8c217/mariadb.service.journal-16-35-24.log.txt#27518:26
noonedeadpunkah, yes, you;re right....18:34
noonedeadpunkI jsut backported it when there were jsut few jobs left on master....18:34
noonedeadpunkas thought it will just pass...18:35
mgariepyyeah, random fails .18:35
noonedeadpunkwell yes, good point that we should likely collect it. though I didn't thought we run backups in CI...18:36
noonedeadpunkactually should we?18:36
noonedeadpunkAs I guess while backup run, mariadb goes to donor state and haproxy consider such backend as down iirc?18:37
mgariepyin this case trhe issue is the second galera node could receive the data and i have no clue why.18:39
noonedeadpunkah, it's probably not backup but more initial sst sync18:40
noonedeadpunkyeah, makes sense then18:40
noonedeadpunkor well18:40
noonedeadpunknot failure, but that we actually don't run backups and I misread logs :)18:41
noonedeadpunkI really hope that upgrade jobs won't fail now....18:41
mgariepy:/ https://zuul.opendev.org/t/openstack/build/be455b6fd24744e283d79e95e0122fb0/log/job-output.txt#13036 19:06
mgariepywhat to i miss ?19:07
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-os_zun master: Support service tokens  https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/84604619:24
jrosser_^ mgariepy 19:24
mgariepyho lol19:26
mgariepyi should not get up at 5 am too often :/19:27
jrosser_looks like openstacksdk 0.99.0 is now packaged in homebrew which will be fun for anyone using that on a mac to do ansible19:46
opendevreviewMerged openstack/openstack-ansible-os_placement master: Support service tokens  https://review.opendev.org/c/openstack/openstack-ansible-os_placement/+/84570819:58
mgariepynetwork manager seems to be still causing issues for ironic :/20:04
mgariepyon centso 9 stream20:06
spatelI am playing with ceph on single node, to mimic disk i did create loopdevice and put them in LVM so i can create lv but somehow when i run ceph-ansible it print device like this. where that dm-2 and dm-3 coming from? - https://paste.opendev.org/show/bEAp9gVnSF0AGxSZs4dW/20:13
spatelthat dm-2 and dm-3 causing error 20:13
jrosser_spatel: have you looked at how the OSA all-in-one sets up loopback devices for ceph?20:19
spatelnot yet.. i may look at it 20:19
noonedeadpunkdoh....20:22
opendevreviewMarc Gariépy proposed openstack/openstack-ansible master: Disable network manager on centos 9.  https://review.opendev.org/c/openstack/openstack-ansible/+/84622420:25
opendevreviewMarc Gariépy proposed openstack/openstack-ansible master: Stop NetworkManager on centos 9.  https://review.opendev.org/c/openstack/openstack-ansible/+/84622420:25
noonedeadpunkdon't we delete networkmanager somewhere for centos 8?20:29
noonedeadpunklikely not...20:30
mgariepyi don't think so 20:30
mgariepyit might fix ironic on centos 9 .. 20:30
mgariepyconsistency ..  https://zuul.opendev.org/t/openstack/build/f46549448c3847cc9b1d8af18ff604bd/logs https://zuul.opendev.org/t/openstack/build/cbacc51ccfc348358061c9f4f8366b9a20:33
noonedeadpunkmgariepy: well, https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/846033 passed ?20:34
noonedeadpunkwonder how....20:34
mgariepymaybe we should add 846224 as depend .. to see.20:35
noonedeadpunkyeah, I think it's good idea20:36
noonedeadpunkwill you or should  I?20:43
mgariepydo it20:44
mgariepyplease :D20:44
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_ironic master: Allow redhat vars file to cover different RHEL derivatives  https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/84402120:44
mgariepyi think it's a race issue with the dhcp lease not being active when the pkg gets installed.20:45
mgariepythat make it passe.20:45
noonedeadpunkyeah might be...20:45
mgariepyanyway, see you tomorrow 20:45
mgariepymy wife wants me to stop working haha20:46
noonedeadpunkyup:) have a good day :)20:46
mgariepycya 20:46
noonedeadpunkI'm in exact same situation hehe20:46
jrosser_i did the directory creation inside the loop on purpose in here https://review.opendev.org/c/openstack/openstack-ansible/+/84618920:49
jrosser_agree it's ugly though20:49
noonedeadpunkshould I return it back?20:50
noonedeadpunkI just tried to reduce actions inside loop for each log line...20:50
noonedeadpunkAnd indent was wrong there so...20:50
noonedeadpunkAs  I kind of didn't get purpose, except avoid directory creation if no deperecations20:51
jrosser_yeah the logic is broken20:51
jrosser_otherwise we end up with lots and lots of empty dirs to search through20:51
jrosser_originally the only dirs that existed had actual deprectaions in them20:52
noonedeadpunkOk, yes, I guess I just took metal as example...20:52
noonedeadpunkANd based on that thought that meh...20:52
jrosser_maybe we can fix that seperatley with how you also used the fd dict20:52
noonedeadpunklike drop empty dirs afterwards?20:53
noonedeadpunkI think it's doable relatively easily20:55
noonedeadpunkbtw looking at result of https://zuul.opendev.org/t/openstack/build/4faf6a3dd1bb4aaaa572c96a54986337/logs I kind of feel that lxc containers are missing from run at all...21:00
noonedeadpunkor they're just not collected...21:00
noonedeadpunkah, it's just not top level21:01
*** dviroel is now known as dviroel|biab21:07
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Fix deprecations handling in journal logs  https://review.opendev.org/c/openstack/openstack-ansible/+/84618921:27
noonedeadpunkI made some weird thing....21:28
noonedeadpunkbut it worked in sandbox21:29
jrosser_still some oom in telemetry jobs21:34
noonedeadpunkyeah21:34
noonedeadpunkwell. telemetry was always tough to pass because of ooms21:35
noonedeadpunkAnd I guess we could miss some worker setting for smth21:35
jrosser_more things related to memory going on too https://review.opendev.org/c/openstack/nova/+/84620821:36
noonedeadpunkI'm kind of worried about `disabling the per thread arenas` tbh21:40
noonedeadpunk`Choosing a value MALLOC_ARENA_MAX is generally a tradeoff between performance and memory consumption.` 21:43
noonedeadpunkThese folks even did some benchmarking https://devcenter.heroku.com/articles/testing-cedar-14-memory-use21:44
jrosser_i asked someone here about this who knows more python internals than me21:45
jrosser_and they throught it wasnt so bad as there was not really proper threading21:46
jrosser_unless you call out to some external library (numpy being good eample) which might have its own ideas about threads21:46
noonedeadpunkI think we should totally try it out in CI...21:47
noonedeadpunkAnd for prod I guess some rally will be helpfull to understand21:47
jrosser_right - i'm done for today21:48
jrosser_youve done some great debugging / fixing today btw21:48
noonedeadpunkI guess it's only env vars that could be set...21:49
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Reduce memory consumption in CI  https://review.opendev.org/c/openstack/openstack-ansible/+/84622821:58
noonedeadpunklet's see... Hope proxy would work...21:59
jrosser_this may be similar to the ca certs stuff where I think we ended up having to do things in several places22:02
jrosser_like environment, uwsgi and systemd iirc22:03
jrosser_it was useful to look in /proc at the process environments to see if any of it was present in the right way22:06
opendevreviewMerged openstack/openstack-ansible-plugins master: Wait for all volumes when starting glusterd.service  https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/84598722:28
opendevreviewMerged openstack/openstack-ansible-os_manila master: Support service tokens  https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/84604023:02
*** dviroel|biab is now known as dviroel23:07
*** dviroel is now known as dviroel|out23:16
opendevreviewMerged openstack/openstack-ansible-os_manila master: Remove mention of haproxy-endpoints role  https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/84580123:21

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