Wednesday, 2022-05-25

*** dviroel|out is now known as dviroel00:21
*** dviroel is now known as dviroel|out01:24
*** ysandeep|out is now known as ysandeep|rover04:35
*** ysandeep|rover is now known as ysandeep|rover|brb05:25
*** ysandeep|rover|brb is now known as ysandeep|rover05:32
noonedeadpunkhuh. And on Pypi package not available anymore https://pypi.org/project/ctx/07:11
noonedeadpunkI _think_ nothing from openstack should use that as it's not in allowable requirements https://opendev.org/openstack/requirements/src/branch/master/global-requirements.txt07:14
noonedeadpunkbut that's crazy...07:14
noonedeadpunkjrosser: I bet we were landing some patch to fix package pinning for ceph_client lately?07:14
jrosserwe did on master, but I think what was wrong there was a little different07:15
noonedeadpunkit wasn't ceph likely, it was smth else...07:16
jrosserI think I put a link up there ^^^^07:17
noonedeadpunkah07:18
noonedeadpunkyes, fair07:18
noonedeadpunkso basically when we get to ceph_client role indeed we already have thing installed07:19
noonedeadpunkI _guess_ we can backport that?07:21
jrossermaybe :/07:22
jrosserit was pretty late yesterday and i wasnt sure if this was the same situation as we had on master07:26
jrosserand also why we are not failing in the wallaby CI job like spatel saw locally07:26
jrosserbut i could believe that being because in the AIO everything is collapsed on one node07:27
jrosseryes, so backporting that patch would stop UCA being the installed in place of the distro package07:33
jrosserso we would end up with a (install from distro during nova) -> (upgrade to whatever is in ceph.com with ceph_client)07:34
jrosserrather than (install from UCA during nova) -> (downgrade to whatever is in ceph.com with ceph_client)07:34
*** ysandeep|rover is now known as ysandeep|rover|lunch07:44
noonedeadpunkI can't recall how much disabling recommends was breaking stuff07:48
noonedeadpunkNot very, right?07:48
kleiniI am on W. https://jira.mariadb.org/browse/MDEV-27459 that bug seems to hit me more and more. If some Galera nodes crashes, systemd restarts it, it joins and the donor for the sync remains in donor state. first node out of the cluster. and this goes on, until quorum is not possible any more. Manually restarting then is really hard. Bug is fixed with Galera 26.4.11 but W runs 26.4.9. What would be best option to 08:02
kleinimitigate that problem?08:02
noonedeadpunkone option is to override mariadb version in variables. I think we stopped hitting it when set galera_wsrep_slave_threads: 1, but I won't put money that it solved this issue as well08:07
noonedeadpunkAt least I can't recall hitting it afterwards08:08
kleiniWill have an eye on that.08:13
opendevreviewJonathan Rosser proposed openstack/openstack-ansible master: Allow lxc container backing store to be defined as part of the scenario  https://review.opendev.org/c/openstack/openstack-ansible/+/84301108:31
opendevreviewJonathan Rosser proposed openstack/openstack-ansible master: Add zuul jobs and template for testing different LXC backing stores  https://review.opendev.org/c/openstack/openstack-ansible/+/84302708:31
*** ysandeep|rover|lunch is now known as ysandeep|rover08:33
noonedeadpunkbtw I'd say nginx uid/gid are quite consistant from what I see in regions08:49
jrosseryeah, i was just starting to look at making it a fixed value08:51
jrosserit's probably variable between distros though?08:51
jrosseri'm expecting it to be in 100-499 range according to LSB08:52
noonedeadpunkI wonder if we should just set owner/group here https://opendev.org/openstack/openstack-ansible/src/branch/master/inventory/group_vars/repo_all.yml#L28-L3308:56
noonedeadpunkI checked several regions, and we have 998:33 everywhere. Could be cross-distro issue, yes, but in CI we totally use same one...08:58
noonedeadpunkok, I will leave it if you're already watching.08:58
noonedeadpunkupgrade jobs always fail just in case - it's not accidental failures just in case09:02
noonedeadpunk(for infra)09:02
jrosserdo we know why? /me not looked yet09:03
jrosseralso i'm not sure we can set the user/group on a mount?09:03
noonedeadpunkYeah, we can09:05
noonedeadpunkor well09:05
noonedeadpunkhttps://opendev.org/openstack/ansible-role-systemd_mount/src/branch/master/tasks/systemd_mounts.yml#L31-L3209:05
noonedeadpunkbut directory in upgrade should already exist...09:06
jrosserisnt this a bit like nfs where the mount is pretty much 'by root' and the permissions are dealt with on each client09:06
jrosserunlike smb for example where the mount is 'by user'09:06
noonedeadpunktbh don't know a lot about gluster09:12
noonedeadpunkut yes, I think you're right and it's alike nfs09:13
noonedeadpunkbut on NFS you need to set bunch of options for mount to get things working properly09:14
noonedeadpunkand eventually things fail when we're trying to override files that were created on another host09:16
*** arxcruz is now known as arxcruz|off10:52
*** dviroel|out is now known as dviroel11:30
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-repo_server master: Run content restore only on first host  https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/84326711:58
jrosserthere are quite some patches to merge in the repo_server if anyone has time https://review.opendev.org/q/project:openstack%252Fopenstack-ansible-repo_server12:19
jrosserit would be nice to get those in as I think there is some more refactoring to do to make it simpler12:19
mgariepyi'll take a look.12:19
jrosseri would like to combine tasks/repo_pre_install.yml and tasks/repo_install.yml in particular12:20
mgariepyrocky8 still failing :(12:22
mgariepyhttps://zuul.opendev.org/t/openstack/build/005ef281d2fb478ebf1acf1142196888/log/job-output.txt#959912:22
mgariepyjrosser, commented on : https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/84161912:29
jrosserdid you press reply? not sure i see it?12:30
mgariepyLOL12:30
mgariepycode 18.. haha12:30
jrosserah yes, we should remove that12:32
*** ysandeep|rover is now known as ysandeep|rover|afk12:41
mgariepynoonedeadpunk, why did you remove W on :https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/837588 ?12:43
noonedeadpunkbecause of dependant patch not merged yet12:44
noonedeadpunkand then zuul will invalidate V+112:44
mgariepyok12:44
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-repo_server master: Use distro packages for nginx on centos.  https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/84161912:48
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-repo_server master: Use distro packages for nginx on centos.  https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/84161912:55
spatelmorning folks12:58
opendevreviewJonathan Rosser proposed openstack/openstack-ansible master: Allow lxc container backing store to be defined as part of the scenario  https://review.opendev.org/c/openstack/openstack-ansible/+/84301113:04
opendevreviewJonathan Rosser proposed openstack/openstack-ansible master: Add zuul jobs and template for testing different LXC backing stores  https://review.opendev.org/c/openstack/openstack-ansible/+/84302713:04
jrosserhmmm no ARA report on an infra job13:06
jrosserdamiandabrowski[m]: did you look at the infra jobs upgrade errors at all? (stale file handle whilst untar the old repo data)13:10
spateljrosser i have added my comments here - https://bugs.launchpad.net/openstack-ansible/+bug/197564313:15
damiandabrowski[m]jrosser: this one? https://zuul.opendev.org/t/openstack/build/7e33eaf8cfb5445c92bcbe38647b792d/log/job-output.txt#19556-1955813:16
damiandabrowski[m]i saw this error in zuul but didn't have to to look more into this :/ 13:16
damiandabrowski[m]but I'm curious about `Cannot utime: Stale file handle`13:17
jrosseryeah thats the one13:17
damiandabrowski[m]maybe this one will solve it? https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/84326713:17
jrosseri have a bunch of meetings shortly but might be able to look at it later13:17
mgariepygluster mirror on rocky doens't work because of the relase and something else..13:17
jrosserit is rather unfortunate that https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/843267 patch does not run the infra upgrade job13:18
jrosseri expect that is how we have merged a broken thing13:18
jrossermgariepy: i think i fixed the gluster/rocky thing https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/84299913:20
jrossermore test coverage gaps there though :/13:20
mgariepyhttps://paste.openstack.org/show/b5iG0k49RRDi5qLYVFNg/13:23
jrosserthat looks like you don't have  842999 applied13:25
jrosserthe mirrorlist should be "release=8-stream"13:26
jrosser*should contain13:26
mgariepyok.13:26
jrosserjust for added awkwardness if you're applying that by hand it needs to be done in /etc/ansible/ansible_collections/......13:27
jrosserwhich isn't a git repo due to `ansible-galaxy` $reasons13:27
mgariepyi was testing in a container with dnf command directly .13:27
mgariepy;)13:27
jrosserahha13:28
jrosserif you hack the repo file and change the `release` string it should work13:28
mgariepyi was looing at 842571 .. now it has 2 recheck :)13:28
*** ysandeep|rover|afk is now known as ysandeep|rover13:31
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-repo_server master: Add infra upgrade jobs  https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/84328413:37
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-plugins master: Use the integrated repo infra jobs for testing  https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/84328613:42
Adri2000hello noonedeadpunk - I'm trying to debug some issues I have with Swift playbooks and ended up on this old patch from you: https://review.opendev.org/c/openstack/openstack-ansible-os_swift/+/765354 - I see a a change in that patch that I'm not sure was intended... before, it copied rings to both /etc/swift/ and /etc/swift/ring_build_files/; now it only copies rings to13:42
Adri2000/etc/swift/; was that the intention?13:42
noonedeadpunkAdri2000: I guess I just missed that. It wasn't intention13:45
Adri2000ok, I *think* that because of that we get md5sum mismatches when we add/remove nodes to the rings... but I'll need a few days before I can confirm this and test a fix. unless you or someone else is confident about a fix and wants to push it, I'll come back later and certainly push a patch. thanks for your reply :)13:54
noonedeadpunkAdri2000: but I'm not sure why they're needed there?13:56
noonedeadpunk` md5sum mismatches` ok, yes, that would explain13:56
noonedeadpunkI bet there was some fresh bug report related to that as well13:56
noonedeadpunkhttps://bugs.launchpad.net/openstack-ansible/+bug/197304513:57
noonedeadpunkI feel we're doing smth quite weird about ring files verification...13:59
noonedeadpunkanyway13:59
noonedeadpunkWill just fix what I broke14:00
Adri2000this bug report looks similar indeed14:02
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_swift master: Sync rings to all swift hosts to build_files  https://review.opendev.org/c/openstack/openstack-ansible-os_swift/+/84328914:04
noonedeadpunkAdri2000: you should check if that fix ^14:10
noonedeadpunkbut not sure about that.... huh14:12
Adri2000noonedeadpunk: unfortunately can't test right now, will but share some feedback asap. thanks for pushing this.14:22
noonedeadpunkthe logic we're verifying rings is really weird and makes limited sense to me. Need to look for some better solution. As verifying against local files makes more sense to me...14:31
noonedeadpunkBut need to reproduce env first14:32
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-repo_server master: Add infra upgrade jobs  https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/84328414:47
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-repo_server master: Add infra upgrade jobs  https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/84328415:03
*** dviroel is now known as dviroel|lunch15:09
opendevreviewMerged openstack/openstack-ansible-repo_server master: Run content restore only on first host  https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/84326715:58
*** dviroel|lunch is now known as dviroel16:16
*** ysandeep|rover is now known as ysandeep|out16:59
opendevreviewJonathan Rosser proposed openstack/openstack-ansible master: WIP - Centos-9 Stream support  https://review.opendev.org/c/openstack/openstack-ansible/+/82341717:38
PraveenkakkeraHello Every one17:55
PraveenkakkeraGreeting for the day17:56
PraveenkakkeraHello Any one is avalible to Help me17:56
PraveenkakkeraI Am trying to Deploy the openstack wallaby on centos 8 with help on openstack Ansible17:56
PraveenkakkeraI am copleated till openstack-ansible setup-hosts.yml17:56
Praveenkakkerawhen i am running openstack-ansible setup-infrastructure.yml  deployment is getting failed on the galara cluster setuo17:56
Praveenkakkerawhen i tried deep my lxc container in infra 1 is unable to communicate to the Infra 2 and infra3 containers17:56
Praveenkakkeraany one can help me on this please17:56
noonedeadpunkPraveenkakkera didn't give us any chance to help...18:24
jrossercentos-9 unblocker https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/84298818:28
jrosserhmm these infra upgrade jobs are still failing https://zuul.opendev.org/t/openstack/build/13c907fac7c6440a9d5131aa306ef8cd/log/job-output.txt#1928718:29
opendevreviewMerged openstack/openstack-ansible master: Update repo verification file URI  https://review.opendev.org/c/openstack/openstack-ansible/+/84257118:36
jrosserstale file handle means that the inode is no longer there18:37
jrosseri really wonder if there is a race condition, it's via fuse.....18:37
opendevreviewMerged openstack/openstack-ansible-repo_server master: Add infra upgrade jobs  https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/84328418:39
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: [doc] Mark Bionic and Buster as unsupported for Y  https://review.opendev.org/c/openstack/openstack-ansible/+/84334019:06
jrossercool \o/ https://review.opendev.org/c/openstack/openstack-ansible/+/82341719:41
jrosserhow much of centos-9 do we dare add to * CI jobs this close to release..... i guess we need to support it for Y?19:42
mgariepyGood job !19:45
noonedeadpunkjrosser: yep, we need it for Y as in Z we should drop C8 I believe19:52
noonedeadpunkAs I don't see how we can have libvirt binding for py38 there19:53
opendevreviewJonathan Rosser proposed openstack/openstack-ansible master: Add Centos-9 Stream support  https://review.opendev.org/c/openstack/openstack-ansible/+/82341719:54
jrossertheres a couple of outstanding depends-on https://review.opendev.org/q/topic:osa%252Fcentos-919:55
noonedeadpunk"couple" :D19:56
noonedeadpunkI think upgrade jobs should be fixed now, so we should re-check some19:56
noonedeadpunkfwiw I was pretty annoyed by answers in #maria for CentOS 9. Was abount to ping monty directly....19:57
jrosserright - that was pretty uncool19:59
jrosseri've seen some unpleasantness in #maria as well when people turn up asking beginner questions20:00
mgariepyJust read the channel and yep it's not cool ..20:01
opendevreviewMerged openstack/openstack-ansible-tests stable/wallaby: Fix rich version for repos that don't have test-requirements  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/84317520:01
*** dviroel is now known as dviroel|afk20:07
jrosseri expect we still have a big bunch of work to do on the minor roles once we merge centos-920:14
jrosserthere is still something wrong with the infra upgrade jobs https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/837588/2821:22
*** tosky_ is now known as tosky21:49
opendevreviewMerged openstack/openstack-ansible-lxc_hosts master: Add centos-9 support  https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/84223622:51
opendevreviewMerged openstack/openstack-ansible-galera_server master: Add support for centos-9  https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/82398323:35

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