Wednesday, 2022-05-18

*** ysandeep|out is now known as ysandeep|rover06:04
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-lxc_hosts master: Add centos-9 support  https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/84223606:15
*** ysandeep|rover is now known as ysandeep|rover|lunch07:24
noonedeadpunkoh, just realized NeilHanlon already published lxc for centos!07:49
noonedeadpunkthat is awesome, thanks a lot for help!07:49
jrosseri think there is a little more to do, i'm just trying a centos-9 lxc aio with that08:08
jrosserlxc-templates seems to be missing the actual templates08:08
jrosserwhich stacks up with this https://bugzilla.redhat.com/show_bug.cgi?id=186056608:08
jrosseri've dropped the centos template file in by hand copying it from an ubuntu box and it seems to be working just fine08:09
jrossersetup-hosts seems to work08:11
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-lxc_hosts master: Add centos-9 support  https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/84223608:13
jrossernoonedeadpunk: question here - do we really want to uninstall rsync?08:15
jrosserhttps://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/83785908:15
noonedeadpunkI'd say it's good to ensure that rsync is present in post task either way08:22
noonedeadpunkas for rhel it's rsyncd which is quite different from what rsync provides as I got08:23
jrosseroh right good point08:23
jrosserthat will be why it fails only on ubuntu then08:24
jrosserNeilHanlon: i have containers created with you packages which is great, i think we are affected by this https://bugzilla.redhat.com/show_bug.cgi?id=186056608:32
jrosserwhen i drop the missing files into /usr/share/lxc/... it all works. See https://packages.ubuntu.com/jammy/amd64/lxc-templates/filelist08:32
*** ysandeep|rover|lunch is now known as ysandeep|rover08:38
damiandabrowski[m]morning everyone09:24
damiandabrowski[m]jrosser: Yesterday I've tested gluster on focal09:25
damiandabrowski[m]recovery from single/multi node failure and complete outage works fine, recreating repo container and lsync->gluster transition works fine as well09:26
jrosserdamiandabrowski[m]: thats good news09:26
damiandabrowski[m]but I've found one issue, /var/www/repo gluster mount doesn't always come up when restarting container09:26
damiandabrowski[m]https://paste.openstack.org/raw/bZ5GlMKUD0lqzoIthExM/09:26
damiandabrowski[m]tried to play with 'after'/'requires' parameters in systemd mount definition but haven't succeeded :/09:27
jrosseri was thinking that there must be a race condition09:28
jrossereach repo server uses it's own gluster server daemon as the thing to connect to for the mount https://review.opendev.org/c/openstack/openstack-ansible/+/837589/15/inventory/group_vars/repo_all.yml09:29
damiandabrowski[m]yeah i've noticed that, but even when I set 'requires=glusterd.service' it wasn't working, probably because gluster needs few secs to peer?09:31
jrosserdamiandabrowski[m]: omg https://github.com/systemd/systemd/issues/446809:48
damiandabrowski[m]:/09:53
noonedeadpunkugh10:09
opendevreviewMerged openstack/openstack-ansible-repo_server master: Add upgrade path from lsyncd to shared filesystem.  https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/83941111:16
*** dviroel_ is now known as dviroel11:19
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-ceph_client master: Drop default nova client_uuid  https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/84233511:20
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-repo_server master: Remove all code for lsync, rsync and ssh  https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/83758811:39
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-repo_server master: Clean up legacy lsycnd, rsync and ssh key config  https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/83785911:41
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-repo_server master: Clean up legacy lsycnd, rsync and ssh key config  https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/83785911:41
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-repo_server master: Use the same vars file for all versions of centos  https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/84161811:41
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/+/84161911:41
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-ceph_client master: Allow service to be absent  https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/84233911:47
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-ceph_client master: Drop duplicated group creation tasks  https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/84234011:59
noonedeadpunkwhat a mess we have in ceph_client role ....11:59
noonedeadpunkJust tried to use it outside of osa to install ceph clients on hosts....11:59
jrosserthe whole business with getting the keys with several methods is messy12:18
SiavashSardarihey everyone, could you please review these two patches?12:25
SiavashSardarihttps://review.opendev.org/c/openstack/openstack-ansible/+/83774212:25
SiavashSardarihttps://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/83937312:25
jrosserSiavashSardari: there are comments on the os_octavia one already12:28
jrosserand was there a companion part to the first one in lxc_container_create or similar?12:28
SiavashSardari@jrosser, oops I've missed the ocatvia comments. thanks for mentioning that12:33
SiavashSardariand yeah this one is the complementary patch. https://review.opendev.org/c/openstack/openstack-ansible-lxc_container_create/+/83773412:34
*** arxcruz_ is now known as arxcruz12:34
jrosserthere are comments on that lxc_container_create one as well, and an error with default(omit)12:39
SiavashSardari@jrosser, thanks, i will take a look at that.12:55
SiavashSardariis everything alright with gerrit? I'm getting siavash.sardari@review.opendev.org: Permission denied (publickey)12:57
SiavashSardariI'm sure nothing changed on my side. except updating to ubuntu 2212:58
mouaaHi guys. I'm trying to upgrade a cluster from train to wallaby. Upgrade infrastrucure nodes (underlay) from ubuntu 18.04 to 20.04 is planned on ussuri version. The distribution upgrade notice a "repo_container". What is that ? Just in the case of "Repo Server" role usage (not the case here)? Thanks for help/reply13:09
jrossermouaa: do you have something you can show us, either a paste somewhere or a link to a document? I'm not sure exactly what you need13:17
mouaafor example in this document: https://docs.openstack.org/openstack-ansible/victoria/admin/upgrades/distribution-upgrades.html, repo_container is mentioned. I can't find a reference in the code or any other document for this. I don't understand what this refers to13:25
noonedeadpunkmouaa: Just in case - aren't you running tripleo or smth?:)13:28
noonedeadpunkand underlay/overlay is mainly applicable there...13:28
noonedeadpunks/and/as13:28
noonedeadpunkor maybe you're running metal deploy?13:29
noonedeadpunkBasically repo_container is the one, where we are building and storing wheels for packages13:30
mouaanewbee on openstack-ansible, worked before with tripleO, my bad. 13:30
noonedeadpunkif you're running source install with lxc - it's just lxc container with "<control>_repo_*" in it13:31
jrosserlxc-ls will show them, i have one here called aio1_repo_container-638ff2ad for example13:32
mouaaI couldn't find this container on any controller node. I assume this is due to not using the "Repo Server" role? Am I correct?13:35
noonedeadpunkwhat install_method are you using?13:37
noonedeadpunkas in case of distro it might not be needed indeed.13:37
noonedeadpunkbut for source I'd expect deployment to fail without it....13:37
noonedeadpunkbut if you can find container, likely it's even that it's not defined in openstack_user_config13:38
mouaaOk. We use distro install_method13:38
noonedeadpunkthen you shouldn't care about repo container :)13:39
mouaaOk. Thanks you ! 2h for searching what's for13:39
jrosserare you testing these upgrades out on a lab environment first?13:39
mouaayes. Openstack DEV in openstack PROD whit mtu nightmare (ml2)13:40
mouaawith sorry. Vxlan in a vxlan13:41
jrosserwe have much much less test coverage of distro install_method, and i don't think we test upgrades at all for that13:41
NeilHanlonjrosser: interesting.. i will take a peek this morning. i bet i just need to enable some flag13:42
mouaathanks for this warning !13:42
noonedeadpunkwell yes, distro thing is kind of weird, as we don't have much control over packages13:44
jrosserdefault and preferred really is source13:45
jrosserand we talk every PTG about removing distro install, and i think it will go away as OS support is dropped13:45
noonedeadpunkand it becomes more nasty when publisher decides not to have "OS upgrade release"13:46
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-os_nova stable/xena: Pass valid cert regen variable to pki role  https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/84232314:50
opendevreviewJonathan Rosser proposed openstack/ansible-role-pki stable/xena: Fix variable names for certificate CSR fields  https://review.opendev.org/c/openstack/ansible-role-pki/+/84232414:51
opendevreviewJonathan Rosser proposed openstack/ansible-role-pki stable/wallaby: Fix variable names for certificate CSR fields  https://review.opendev.org/c/openstack/ansible-role-pki/+/84232514:51
opendevreviewMerged openstack/openstack-ansible master: Define tempest network features for ovs/ovn  https://review.opendev.org/c/openstack/openstack-ansible/+/84215314:59
opendevreviewMerged openstack/openstack-ansible master: [doc] Be even more explicit about lb_vip_address  https://review.opendev.org/c/openstack/openstack-ansible/+/84213214:59
opendevreviewMerged openstack/openstack-ansible master: haproxy: fix csp for glance image uploads via horizon  https://review.opendev.org/c/openstack/openstack-ansible/+/84211115:00
opendevreviewMerged openstack/openstack-ansible stable/wallaby: Limit mistral worker  https://review.opendev.org/c/openstack/openstack-ansible/+/84194215:00
opendevreviewJonathan Rosser proposed openstack/openstack-ansible stable/xena: [doc] Be even more explicit about lb_vip_address  https://review.opendev.org/c/openstack/openstack-ansible/+/84232715:08
opendevreviewJonathan Rosser proposed openstack/openstack-ansible stable/wallaby: [doc] Be even more explicit about lb_vip_address  https://review.opendev.org/c/openstack/openstack-ansible/+/84232815:09
opendevreviewJonathan Rosser proposed openstack/openstack-ansible stable/victoria: [doc] Be even more explicit about lb_vip_address  https://review.opendev.org/c/openstack/openstack-ansible/+/84232915:09
*** ysandeep|rover is now known as ysandeep|dinner15:17
*** dviroel is now known as dviroel|lunch15:39
*** ysandeep|dinner is now known as ysandeep15:52
*** ysandeep is now known as ysandeep|out15:57
opendevreviewJonathan Rosser proposed openstack/openstack-ansible master: Bump ansible-core to 2.13.0  https://review.opendev.org/c/openstack/openstack-ansible/+/84238716:20
*** dviroel|lunch is now known as dviroel16:31
opendevreviewMerged openstack/openstack-ansible stable/xena: [doc] Be even more explicit about lb_vip_address  https://review.opendev.org/c/openstack/openstack-ansible/+/84232716:32
spatelI have keepalived question, any problem if i keep state: BACKUP on both node? 16:32
jrosserthat means neither has the VIP - do i misunderstand?16:36
spatelI have set both node to be BACKUP and things working fine.. 16:36
spatelBACKUP state is just to tell when keepalived come online it doesn't need election and act like BACKUP (pre define) 16:37
jrosseroh, ok16:37
spatelBut if both are BACKUP then they do election and highest priority win 16:37
jrosserdoesnt this all have reasonable defaults anyway16:38
opendevreviewJames Denton proposed openstack/openstack-ansible-os_neutron master: Enable FDB extension for OVS Agent when using SR-IOV  https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/84238916:38
spatelI believe so. 16:38
opendevreviewMerged openstack/openstack-ansible stable/wallaby: [doc] Be even more explicit about lb_vip_address  https://review.opendev.org/c/openstack/openstack-ansible/+/84232816:38
opendevreviewMerged openstack/openstack-ansible stable/victoria: [doc] Be even more explicit about lb_vip_address  https://review.opendev.org/c/openstack/openstack-ansible/+/84232916:38
spatelJust trying to create my keepalived server for firewall and thinking to keep both BACKUP and let election decide who will be master 16:39
jrosseroh, i thought you meant for openstack-ansible use of keepalived16:40
spatelI was also looking at openstack-ansible config to see what options we are setting up and found we predefined MASTER and BACKUP16:41
spatelTrying to understand any downside of setting up both BACKUP :)16:41
jrosserthis patch is cursed, more failures https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/84035316:50
mgariepyhmm. sad. 16:57
jrossergood news is ansible 2.13.0 appears to print more output for templating errors https://paste.opendev.org/show/bGJTSS3N0ZYx0xvsMN1K/17:01
jrosserbad news is i dont understand it :)17:02
jrosseri expect this is "Increase minimum jinja2 dependency to 3.0+"17:09
opendevreviewDmitriy Rabotyagov proposed openstack/ansible-role-systemd_mount master: Do not report changed for shell  https://review.opendev.org/c/openstack/ansible-role-systemd_mount/+/84241417:09
mgariepyanyone performed the bionic > focal upgrade?17:21
damiandabrowski[m]i did17:22
mgariepyhow does the repo stuff went ?17:22
mgariepyon Ussuri ?17:22
damiandabrowski[m]Victoria17:24
damiandabrowski[m]I filled one bug: https://bugs.launchpad.net/openstack-ansible/+bug/196453517:24
damiandabrowski[m]and i remember fixing one more thing but i'm not sure if that fix was backported, let me find it17:24
mgariepyi was thinking of doing a quick/dirty hack to only do-release upgrade a repo container to accomodate a new host.. 17:25
damiandabrowski[m]https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/82927017:26
damiandabrowski[m]it was backported up to victoria17:26
damiandabrowski[m]hmm haven't tried this do-release-upgrade hack :D 17:27
mgariepyi did it once or twice before.. lol17:27
opendevreviewMerged openstack/openstack-ansible-os_octavia master: Updated from OpenStack Ansible Tests  https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/83571717:28
mgariepylast time was to install 7 or 8 nodes.. and didn't want to have to re-upgrade them a couple weeks after the initial installation..17:28
mgariepythis time i only have one to do.17:28
damiandabrowski[m]I am glad it's already behind me and all our regions are running focal now... :D 17:31
damiandabrowski[m]spent a lot time reprovisioning servers, performing firmware upgrade, evacuating vms etc...17:31
mgariepydo you have local storagE?17:32
damiandabrowski[m]only on few computes, it's ceph in 95%17:32
mgariepyok17:39
mgariepyi don't have local storage on a new cluster and migrating vms is much more fun.17:40
mgariepyalso jammy is comming soon ! :P haha17:42
damiandabrowski[m]not sure if I should be happy about it or just start crying... :D 17:49
mgariepyhaha.17:50
mgariepythe worst one was xenial to bionic.17:50
mgariepyimo.17:50
opendevreviewMerged openstack/ansible-role-pki stable/wallaby: Fix variable names for certificate CSR fields  https://review.opendev.org/c/openstack/ansible-role-pki/+/84232517:51
mgariepysome package are named by the os version and pkg-1.x-xenial is higher than pkg-1.x-bionic17:51
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-ceph_client master: Use global package_state  https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/84242618:45
noonedeadpunkmgariepy: you just don't know how jammy upgrade would look like :p18:46
noonedeadpunkBut with shared fs instead of lsyncd.... It looks very promising18:46
mgariepyit will be awesome 18:47
mgariepyit's always a pain.18:47
mgariepythe memcached stuff is is not fun at all either.18:47
opendevreviewMerged openstack/openstack-ansible master: Use glusterfs to synchronise repo server contents  https://review.opendev.org/c/openstack/openstack-ansible/+/83758918:50
opendevreviewMerged openstack/openstack-ansible-os_nova stable/xena: Pass valid cert regen variable to pki role  https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/84232318:53
jrossernoonedeadpunk: I wonder if we should try to drop using ansible_sshd role18:57
jrosserit breaks quite badly on ansible 2.13.0 and I was very surprised how gross the template is18:58
jrosserhttps://github.com/willshersystems/ansible-sshd/blob/master/templates/sshd_config.j218:59
SiavashSardariapparently openssh 8.8 dropped the support of rsa signatures using sha1, which still is used by gerrit. in case anyone wants to upgrade your ssh client this link may save you from some hair pulling over why git review stops working :D 19:01
SiavashSardarihttps://confluence.atlassian.com/bitbucketserverkb/ssh-rsa-key-rejected-with-message-no-mutual-signature-algorithm-1026057701.html19:01
*** dviroel is now known as dviroel|out20:28
NeilHanlonjrosser: looked into the templates thing. it appears the upstream package changed at some point to stop providing a lot of templates. I'll just add some back in :) 22:16

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