Friday, 2022-05-13

*** Guest0 is now known as prometheanfire00:29
*** dviroel|afk is now known as dviroel|out01:26
*** ysandeep|rover|out is now known as ysandeep|rover04:50
*** ysandeep|rover is now known as ysandeep|rover|lunch07:40
jrosseri made good progress on centos-9 but remember it is completely dependany on https://review.opendev.org/q/topic:osa-gluster08:03
jrosserand try-outs of that would be great08:04
jrosserthe patches failing in that topic are doing so becasue we do not yet merge this https://review.opendev.org/c/openstack/openstack-ansible/+/83758908:05
jrosserand i think it is circular to add another depends-on08:05
noonedeadpunkI will try to spawn up a sandbox during the day if everything remain calm :)08:15
noonedeadpunkI actually want to test out like focal/yammy/centos stack of gluster08:16
* noonedeadpunk wonders what can possibly go wrong there08:16
noonedeadpunkbut first want to look through bugs...08:20
kleiniI am testing currently a minor upgrade in staging from 23.2.0 to 23.2.2. the upgrade guide tells to use "-e rabbitmq_upgrade=true". this seems to cause to shutdown all RabbitMQ instances, which is kind of bad. OSA console log shows to stop [0] first and then [1:]. then second task does not seem to be executed, as OSA console log shows no green or yellow line under that. but all RabbitMQs are then down.08:22
kleiniAm I doing something wrong or is there maybe some problem in the minor upgrade guide or the rabbitmq-server role?08:23
kleiniWhen looking at the tasks/rabbitmq_stopped.yml, there must then be a problem with the rabbitmq_primary_cluster_node variable. Will try to check that.08:26
admin1morning . 08:33
noonedeadpunkkleini: -e rabbitmq_upgrade=true works nicely for me as used it no later then yestarday to have that said. But it could take time to shutdown fully rabit08:38
noonedeadpunkThough you likely don't need to have that flag for minor upgrade (at least for this upgrade)08:39
noonedeadpunkas we haven't changed rabbit version between these releases08:39
jrosserI was going to say rabbit upgrade is not normal for minor releases?08:39
jrosserkleini: are you looking at the major or minor upgrade docs?08:40
noonedeadpunkIt depends I guess? I can recall when we had to bump rabbit version on stable for some OS upgrade purposes? like bionic->focal08:40
noonedeadpunkso minor version upgrade along with rabbit upgrade was required for that08:40
noonedeadpunkSo I'd say it's fine to be in docs in general. And that does't hurt that much, as downtime of rabbit is usually about 30 sec for me08:41
opendevreviewDmitriy Rabotyagov proposed openstack/ansible-role-systemd_networkd master: Update documentation on role requirements  https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/84169108:42
jrossernoonedeadpunk: I think there is a good chance that a test of focal/jammy/centos repo servers might actually do multi-os properly with all the gluster patches applied08:42
jrossermaybe some small bugs but I feel like we should be in pretty good shape for mixed OS / architectures08:43
noonedeadpunkI'm a bit scared to mix gluster versions...08:43
jrosserwell there is that…..08:43
noonedeadpunkbut in theory it should work...08:44
jrosserkind of cool benefit from removing lsyncd really08:44
noonedeadpunkI think we should do milestone release once merge that08:45
jrosserhaving a small ARM repo node just for wheel builds would be great08:45
noonedeadpunkum, and can you build wheels on arm for x86?08:45
jrosseroh well just in addition to existing x8608:46
noonedeadpunkyeah, as our logic won't even pick arm as destination08:46
noonedeadpunk(for x86)08:46
*** ysandeep|rover|lunch is now known as ysandeep|rover08:49
opendevreviewDmitriy Rabotyagov proposed openstack/ansible-role-systemd_service master: Update documentation on role requirements  https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/84169208:50
kleiniokay, will then go without that rabbitmq_upgrade then. thanks. 08:53
opendevreviewDmitriy Rabotyagov proposed openstack/ansible-role-systemd_mount master: Update documentation on role requirements  https://review.opendev.org/c/openstack/ansible-role-systemd_mount/+/84169408:54
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_gnocchi master: Add variables to manage redis url  https://review.opendev.org/c/openstack/openstack-ansible-os_gnocchi/+/84169909:20
admin1help .. i can't seem to figure out this galera error during setup-infrastructure ... i have deleted and created  the containers and retried the steps three times already .. https://gist.githubusercontent.com/a1git/18dd5ca261db0b860cd7ac2e87c0faaa/raw/fd2a248adf0e3c6185bc33e76b9b741b14fbf325/gistfile1.txt10:08
admin1mysql is running in the 1st container 10:10
kleinithat is some haproxy python script failing with "connection refused". Can you connect from that galera container to the haproxy server?10:11
admin1thanks for the hint .. 10:13
admin1indeed c3( the active haproxy) cannot ping the galera container on c1 10:13
admin1some(dept) played with the vlans :D10:13
admin1doh! .. i can ping the ips on br-mgmt from server to server, but not the lxc ips on the same bridge .. 10:16
admin1fixed on a reboot .. could not make sense .. no firewall, ip_forward was set to 1 ..  tcpdump showed the ips coming in the interface .. but the container not responding . 10:44
admin1i am unblocked :) 10:44
admin1me and spatel will be attending the openstack summit in berlin .. anyone else from who might be coming ? 10:45
admin1we can have a coffee /wine time :) 10:46
*** dviroel|out is now known as dviroel11:00
kleinioh, contributor tickets are cheap until Tuesday. have to buy a ticket.11:02
admin1you will be there as well 11:03
opendevreviewDmitriy Rabotyagov proposed openstack/ansible-hardening master: Clean out SSH options we managing  https://review.opendev.org/c/openstack/ansible-hardening/+/84171611:04
noonedeadpunko/ coming11:04
admin1nice .. 11:09
noonedeadpunkhas anybody seen issues with DB performance after upgrade to Xena?12:13
mgariepyhaven't upgraded to X yet.12:34
kleinisorry, still on W12:46
kleinihmm, X is the last one supporting bionic. so either X first and then focal or the other way round...12:48
kleinijrosser, noonedeadpunk: I tested again in staging with rabbitmq_upgrade=true. First all not upgrader RabbitMQ instances are stopped. The the upgrader instance is stopped and upgraded. After upgrade, it does not start any more. In the logs that instance is waiting for Mnesia tables.13:13
spatelkleini did you try to wipe Mnesia?13:15
kleinino, I did not. it helped to run rabbitmqctl force_boot and then another systemd restart13:16
kleinihow do I wipe Mnesia tables?13:16
spatelkleini i did this mostly - rm -rf /var/lib/rabbitmq/mnesia/* 14:15
spatelafter that when you re-build rabbitMQ it will automatically create fresh database. 14:16
*** dviroel is now known as dviroel|lunch15:12
admin1if i get rabbit issues, i normally lxc-containers-destroy , create and then re-run the setup-openstack playbooks 15:55
admin1rabbit issues during planned upgrade i.e 15:55
*** ysandeep|rover is now known as ysandeep|out15:55
*** dviroel|lunch is now known as dviroel16:00
noonedeadpunkNeilHanlon: do you know if it's intended not to have /etc/pam.d/passwd on Rocky?18:13
noonedeadpunkor smth non-default provides that?18:14
noonedeadpunkAs Rocky the only thing that failed https://zuul.opendev.org/t/openstack/build/dfa94b7f08314901a2d0ce46bbcc09a0/log/job-output.txt#534318:14
jrossernoonedeadpunk: i think this is to fix that https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/84035318:18
jrosserif only there wasnt some other brokenness there too :/18:18
noonedeadpunkhm... would passwd provide pam.d plugin? interesting...18:18
jrosseri think mgariepy did some investigation here18:19
jrosserim wondering if it's an actul bug with the nfv repo, or we are just very very unlucky on that job18:20
spateljrosser finally i have created playbook to create glusterfs on infra node for glance storage - https://paste.opendev.org/show/bTBvHD0zlTFfuh2GT6pG/18:26
jrossererrrrm :)18:27
jrosserhttps://review.opendev.org/c/openstack/openstack-ansible-plugins/+/83758218:28
spatel:)18:28
spatelhmm18:28
jrosserwe also added glusterfs support to the systemd_mount role https://review.opendev.org/c/openstack/ansible-role-systemd_mount/+/83775018:30
spatelmy playbook is for very specific task and i don't think anyone else would go on that route :)18:30
damiandabrowski[m]jrosser: I remember about Your gluster patches but didn't have enough time for this yet :/ Today I've started preparing dev environment so hopefully I'll be able to focus on this on Monday18:30
NeilHanlonnoonedeadpunk, jrosser: yeah, passwd in EL is all backed into PAM/authselect18:30
jrosserdamiandabrowski[m]: no problem, all good for next week18:31
spateljrosser i like systemd_mount, let me take that code and apply on my play18:31
NeilHanlon(It's also linked with glibc 🙃)18:31
jrosserspatel: something like this https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/837587/8/tasks/repo_pre_install.yml18:32
jrosserwith data a bit like this https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/837587/8/defaults/main.yml18:32
jrosserbut you need master branch of systemd_mount role for this18:32
jrosserspatel: the neat thing with doing it with systemd_mount is that if somewhere else you need like NFS or something, you just change the data18:33
jrosserrather than having to mess with the playbook18:33
spateltotally! let me see how i can get it work with my playbook 18:34
spatelcurrently i am using fstab but i don't like that18:34
jrosseroh i also made it work in containers if you ever need that, theres trouble with fuse otherwise18:37
spatelwish we can expand to glance also in non-ceph scenario 18:39
spatelI am doing on host machine because its easy 18:40
jrosseryou are kind of in luck for that18:40
jrosserhttps://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/83755018:40
jrosser^ this should let you use whatever systemd_mount can do with glance18:40
jrosserrefactor to make it not specific to NFS18:41
jrosserso i mean that you should easily be able to make glance be a glusterfs client in the next release18:41
jrosserif you put gluster on the host machine that should work out fine with the right data in `glance_remote_client`18:42
spatelnicee! 18:44
spatelthat is neat 18:44
jrosserbtw if you look at what i did in the plugins gluster role, it also handles recovering from a completely destroyed brick18:45
jrossertook a wile to figure that out as theres a bunch of contradictory info out there18:45
spatelwhat do you mean destroyed brick?18:46
jrosserlike if you re-install one of your gluster servers, how to rebuild it an get it back in the cluster18:46
spatellike replacing failed node ?18:46
jrosseryes18:46
spateloh okkk 18:46
jrossermake sure you know about how to find the UUID for each node too :)18:46
spatelUUID for what, sorry i didn't work on gluster in that deep :)18:48
spateldid you set auth etc for gluster volume? 18:50
spateljust incase so someone else can't mount it on LAN18:50
jrosserno, i didnt do anything with auth, but it is on my list of things to look at18:52
noonedeadpunkToday I even failed to go through all recent bugs :facepalm:18:52
noonedeadpunkdue to this weird neutron slowness....18:52
noonedeadpunkI'm really wondering if it only us who face that...18:53
spatelis it related to CI ?18:53
jrossernoonedeadpunk: did you ever manage to use config_template for ini format, just with no [sections] in the file?19:02
jrosserflat key:value structure19:03
noonedeadpunkI can recall talking about it19:03
noonedeadpunkBut seing that for rabbitmq I haven't used it, sounds like no19:04
noonedeadpunkbut not sure if I put any time on that tbh19:05
noonedeadpunkfrom what I see it looks like we'd need to have another config format or some exceptions in place....19:09
noonedeadpunkThe main problem is that to parse config we're parenting ConfigParser19:11
noonedeadpunkWhich can not parse config without sections iirc19:12
noonedeadpunkAnd google offers solutions like that https://stackoverflow.com/questions/2819696/parsing-properties-file-in-python/2819788#2819788 but I'd say we should jsut have another config type....19:14
noonedeadpunkEasy workaround though - to define such config as a dict somewhere in variables, and then you can just combine overrides with default one. and in tempalte only have for loop and {{ key }} = {{ value }}....19:15
jrosserright - seems like an extra type of 'properties' would be the proper name for this19:30
*** dviroel is now known as dviroel|out21:57

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