Wednesday, 2022-04-27

*** ysandeep|out is now known as ysandeep04:31
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-repo_server master: Add upgrade path from lsyncd to shared filesystem.  https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/83941107:46
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/+/83785907:47
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/+/83758807:47
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/+/83785907:48
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/+/83758807:49
damiandabrowski[m]morning guys, all my tempest patches are rebased and verified now: https://review.opendev.org/q/topic:tempest-damian-2021-12+AND+-is:merged07:59
damiandabrowski[m]btw. it's my last day before the vacation, so let me know if You need anything ;) I will be back on May 10th07:59
opendevreviewJonathan Rosser proposed openstack/openstack-ansible master: WIP - Support Ubuntu 22.04  https://review.opendev.org/c/openstack/openstack-ansible/+/83948308:01
*** ysandeep is now known as ysandeep|lunch08:12
opendevreviewJonathan Rosser proposed openstack/openstack-ansible master: WIP - Support Ubuntu 22.04  https://review.opendev.org/c/openstack/openstack-ansible/+/83948308:18
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-lxc_hosts master: Unify debian and ubuntu cache prep scripts  https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/83883608:35
*** ysandeep|lunch is now known as ysandeep08:52
opendevreviewJonathan Rosser proposed openstack/openstack-ansible master: WIP - Support Ubuntu 22.04  https://review.opendev.org/c/openstack/openstack-ansible/+/83948309:00
opendevreviewJonathan Rosser proposed openstack/openstack-ansible master: WIP - Support Ubuntu 22.04  https://review.opendev.org/c/openstack/openstack-ansible/+/83948309:03
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-openstack_hosts master: Do not install UCA repository for ubuntu 22.04  https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/83883210:38
opendevreviewMerged openstack/ansible-role-uwsgi master: Refactor installation of libpython for debian/ubuntu  https://review.opendev.org/c/openstack/ansible-role-uwsgi/+/83883011:00
*** dviroel|rover|out is now known as dviroel|rover11:16
SiavashSardari@noonedeadpunk I have a question regarding your comment on https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/83938111:17
noonedeadpunkSiavashSardari: yup?11:18
SiavashSardariI get your point on the mentioned topic. actually I had the issue and solved it by https://paste.opendev.org/show/bUWqAApMZBSlCfd1VpRu/ in user_configs11:19
noonedeadpunkoh, ok. I don't think system scopes are really forced on Yoga though. Last ML I read were reffering that things are not ready enough in the release to make final switch so more time to teams were given11:21
noonedeadpunkand saying that I also include that support into openstacksdk was added like month ago to master, and collection is broken with master version of sdk, so active work is done there...11:22
noonedeadpunkSo in fact we can't get proper fix for things as of today afaik11:22
SiavashSardarioh ok then if it got merged there is no need of my patch.11:23
opendevreviewJonathan Rosser proposed openstack/openstack-ansible master: WIP - Support Ubuntu 22.04  https://review.opendev.org/c/openstack/openstack-ansible/+/83948311:24
jrosseransible lint seems kind of sometimes happy / sometimes not happy on my lxc_hosts changes11:30
jrosseroh i see - we have a lot of warnings there normally11:32
opendevreviewJonathan Rosser proposed 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:37
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:38
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:38
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible stable/victoria: Bump OpenStack-Ansible Victoria for EM  https://review.opendev.org/c/openstack/openstack-ansible/+/83951611:59
opendevreviewMarc Gariépy proposed openstack/openstack-ansible-os_nova master: [DNM] test gitreview  https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/83951912:20
jrossernoonedeadpunk: i think we may need the same code path for jammy and centos-9 in the galera role, to install from a distro package12:20
opendevreviewMarc Gariépy proposed openstack/openstack-ansible-specs master: PDF Documentation Build tox target  https://review.opendev.org/c/openstack/openstack-ansible-specs/+/69149912:21
noonedeadpunkyeah, unless maria release new minor version soon12:23
jrossernoonedeadpunk: it's kind of possible now with just OS specific vars, but this fails when there is no point version https://github.com/openstack/openstack-ansible-galera_server/blob/master/tasks/galera_server_setup.yml#L26-L3112:23
jrosserwell still no centos-9 and i made a jira for that months ago12:23
opendevreviewMarc Gariépy proposed openstack/openstack-ansible-specs master: PDF Documentation Build tox target  https://review.opendev.org/c/openstack/openstack-ansible-specs/+/69149912:24
noonedeadpunkthey add new distros only with new releases iirc12:24
noonedeadpunkand they had none since february...12:24
noonedeadpunkbut yeah, in february centos 9 was kind of a thing already...12:25
mgariepyouf. stuping aur pkg :/12:25
mgariepystupid**12:25
jrossernoonedeadpunk: i wondered if you had any ideas for how to handle the case when galera_minor_version is unknown12:26
jrosseror maybe we have to put the value matching the distro package in the OS specific vars, though that will be very fragile12:26
noonedeadpunkjrosser: we don't need that code anymore jsut in case:)12:29
noonedeadpunkor well, I have patch that drops it :)12:29
jrosserooh excellent!12:29
jrossermy 22.04 patch is as far as needing to do *something* for galera12:29
jrosserjust what :)12:30
noonedeadpunkhttps://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/833259/6/tasks/galera_server_setup.yml12:30
noonedeadpunkIt's in a merge conflict....12:30
noonedeadpunkbut it also requires https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/83933812:30
jrosserbackport?12:31
noonedeadpunkyeah, as otherwise upgrade jobs fail because of that tmp stuff12:32
opendevreviewMarc Gariépy proposed openstack/openstack-ansible-specs master: PDF Documentation Build tox target  https://review.opendev.org/c/openstack/openstack-ansible-specs/+/69149912:38
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-galera_server master: Update MariaDB version to 10.6.7  https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/83325912:38
*** ysandeep is now known as ysandeep|afk12:42
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-galera_server master: Add support for Ubuntu 22.04  https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/83952112:53
opendevreviewJonathan Rosser proposed openstack/openstack-ansible master: WIP - Support Ubuntu 22.04  https://review.opendev.org/c/openstack/openstack-ansible/+/83948312:53
jrossernoonedeadpunk: andrewbonney: there was something i wanted a second opinion on - here https://github.com/openstack/openstack-ansible-os_keystone/blob/master/templates/keystone-httpd.conf.j2#L23-L2712:57
jrosseri am thinking that the conditional on line 23 is wrong12:57
noonedeadpunkoh, i was thinking about for a while when reviewing PR12:58
jrosserwhen i run this on 22.04, the x-forwared-for header comes out as http, and everything trying to talk to keystone is broken12:58
jrosserand you can reproduce it with wget easily12:58
jrosseri.e wget gets a 300 redirecting it to http://internal-vip/v312:59
noonedeadpunkthere was weird stuff going on in naming and what these variables represent12:59
jrosserwhich doesnt work, as it should be https://internal-vip/v312:59
noonedeadpunkso that made sense to me at the end12:59
jrosserright, well something is certainly broken for 22.04, where i think we have a newer apache/haproxy/everytjhing12:59
*** ysandeep|afk is now known as ysandeep13:00
jrosserthis is what happens in my AIO https://paste.opendev.org/show/bBYCLpGJQnqJruQi1Alj/13:00
jrosserand the whole thing is pretty broken until i change x-forwarded-for to be https13:01
noonedeadpunksorry can't dig deep into logic again now as meetings from 8-30 till 18-00 with1h break each day this week..13:02
jrosseroh no problem - don't worry13:02
noonedeadpunkto have that said - change there landed recently13:03
noonedeadpunklike weeks ago13:03
jrosseryes https://github.com/openstack/openstack-ansible-os_keystone/commit/6fae2bdade459c85dba55aae64c9f6f4e485a78213:03
noonedeadpunkoh, wait, it was my change...13:03
noonedeadpunkI thought about some another one actually...13:04
jrosseri just can't understand really why it seems for work for focal by not jammy13:04
noonedeadpunkit's still apache 2.4 there?13:04
jrosseryes13:05
jrosserwhen i wget the keystone container it is ok13:05
jrosserbut when i wget the VIP is it not ok13:05
jrosserso it seems to be haproxy13:05
noonedeadpunkthen maybe haproxy does smth differently....13:05
noonedeadpunkas it's smth like 2.2 ?13:05
noonedeadpunkand we had 2.0 for focal13:05
noonedeadpunkand iirc there were pretty big changes...13:06
jrosser2.4.14 in jammy13:06
opendevreviewMarc Gariépy proposed openstack/openstack-ansible-specs master: PDF Documentation Build tox target  https://review.opendev.org/c/openstack/openstack-ansible-specs/+/69149913:06
noonedeadpunkWe can get some syntax deprecated or dropped...13:06
jrosser"The X-Forwarded-Proto (XFP) header is a de-facto standard header for identifying the protocol (HTTP or HTTPS) that a client used to connect to your proxy or load balancer"13:07
jrosserso this is why i think the logic is wrong in os_keystone13:07
jrosserbecasue it accounts for the keystone backend being ssl, not just the VIP13:07
noonedeadpunkyeah, maybe.....13:09
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/+/83785913: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/+/83758813:21
opendevreviewMarc Gariépy proposed openstack/openstack-ansible-specs master: PDF Documentation Build tox target  https://review.opendev.org/c/openstack/openstack-ansible-specs/+/69149913:23
SiavashSardari@jrosser any update on https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/836325? I can confirm there is no cert regeneration for existing deployments.13:24
jrosserSiavashSardari: i have no update on it, but if you've tested it and its OK can you leave a comment on the patch?13:25
SiavashSardarisure13:25
opendevreviewMarc Gariépy proposed openstack/ansible-role-python_venv_build master: Disable setuptools auto discovery  https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/83589213:26
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-repo_server master: Add upgrade path from lsyncd to shared filesystem.  https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/83941113:32
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/+/83785913:32
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/+/83758813:33
opendevreviewMarc Gariépy proposed openstack/openstack-ansible-specs master: PDF Documentation Build tox target  https://review.opendev.org/c/openstack/openstack-ansible-specs/+/69149913:41
opendevreviewMarc Gariépy proposed openstack/openstack-ansible-rsyslog_client master: PDF Documentation Build tox target  https://review.opendev.org/c/openstack/openstack-ansible-rsyslog_client/+/69150113:44
opendevreviewMarc Gariépy proposed openstack/openstack-ansible-specs master: PDF Documentation Build tox target  https://review.opendev.org/c/openstack/openstack-ansible-specs/+/69149913:53
opendevreviewMarc Gariépy proposed openstack/openstack-ansible-rsyslog_client master: PDF Documentation Build tox target  https://review.opendev.org/c/openstack/openstack-ansible-rsyslog_client/+/69150113:59
opendevreviewMarc Gariépy proposed openstack/openstack-ansible-rsyslog_client master: PDF Documentation Build tox target  https://review.opendev.org/c/openstack/openstack-ansible-rsyslog_client/+/69150114:21
mgariepyat some point it will be ok .. lol..14:21
*** ysandeep is now known as ysandeep|out14:21
mgariepyjamesdenton, https://review.opendev.org/q/topic:vpp-deploy are they still needed ?14:24
jamesdentonat this point, probably not. i looked into it just a few weeks ago and it looks like upstream stuff may have been abandoned or at least not updated in a while14:25
jamesdentonso, feel free to close14:25
jamesdentonthanks14:25
mgariepythanks to you :D14:25
opendevreviewMerged openstack/openstack-ansible-os_tempest master: Allow to create only specific tempest resources.  https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/80347714:36
mgariepynoonedeadpunk, for the "setup.cfg: Replace dashes with underscores" from a year ago should we edit or abandon ?14:37
jrosserdidnt we just get rid of a load of setup.py/cfg?14:38
mgariepyhttps://review.opendev.org/c/openstack/ansible-hardening/+/78871514:38
mgariepyhmm . good point. i'll abandon them then,14:39
opendevreviewMerged openstack/openstack-ansible-os_tempest master: Fix hardcoded flavor_ref and flavor_ref_alt  https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/80349214:47
opendevreviewMerged openstack/openstack-ansible-os_tempest master: Add support for both Credential Provider Mechanisms  https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/82540314:47
opendevreviewMerged openstack/openstack-ansible-os_tempest master: Remove unused variables  https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/82540514:47
opendevreviewMerged openstack/openstack-ansible-os_tempest master: Do not store unnecessary sections in tempest.conf  https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/82540714:47
opendevreviewMerged openstack/openstack-ansible-os_tempest master: Fix hardcoded instance_type in [heat_plugin] section  https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/82540814:47
mgariepynice work damiandabrowski[m] :) ^^ 14:48
damiandabrowski[m]finally, thank You! :D 14:48
mgariepyhaha14:48
jrosseropenstack-ansible-deploy-aio_metal-ubuntu-jammy tempest success \o/15:02
jrosserbroken rally though :/15:02
mgariepymerge please : https://review.opendev.org/c/openstack/openstack-ansible-rsyslog_client/+/69150115:05
mgariepyand also : https://review.opendev.org/c/openstack/openstack-ansible-specs/+/69149915:12
opendevreviewJonathan Rosser proposed openstack/openstack-ansible master: Use glusterfs to synchronise repo server contents  https://review.opendev.org/c/openstack/openstack-ansible/+/83758915:13
jrosseri'm sure we've had this before - any ideas? https://paste.opendev.org/show/b0cOy5jUsr61YtLKeMig/15:16
opendevreviewMerged openstack/openstack-ansible-specs master: PDF Documentation Build tox target  https://review.opendev.org/c/openstack/openstack-ansible-specs/+/69149915:22
mgariepydamiandabrowski[m], do you have a minute ? https://review.opendev.org/c/openstack/openstack-ansible-rsyslog_client/+/69150115:23
*** dviroel|rover is now known as dviroel|rover|lunch15:29
opendevreviewJonathan Rosser proposed openstack/openstack-ansible master: WIP - Support Ubuntu 22.04  https://review.opendev.org/c/openstack/openstack-ansible/+/83948315:36
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-lxc_container_create master: Unify variables for all operating systems.  https://review.opendev.org/c/openstack/openstack-ansible-lxc_container_create/+/83916715:55
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-lxc_container_create master: Unify variables for all operating systems.  https://review.opendev.org/c/openstack/openstack-ansible-lxc_container_create/+/83916715:58
*** dviroel|rover|lunch is now known as dviroel|rover16:18
opendevreviewMerged openstack/openstack-ansible-galera_server stable/xena: Use separate tmp directory  https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/83933817:20
damiandabrowski[m]mgariepy: sorry i wasn't available, i reviewed it now 17:51
mgariepyneed to +w also :D17:52
mgariepydamiandabrowski[m], ^^17:53
mgariepyWorkflow +117:53
damiandabrowski[m]ahh that's how it works, i thought it happens automatically :D 17:53
mgariepynow you know the whole magic :D17:54
damiandabrowski[m]jrosser: hmm tried to reproduce it locally: `python-ceilometerclient 2.9.0 requires PrettyTable<0.8,>=0.7, but you have prettytable 2.0.0 which is incompatible.`18:02
damiandabrowski[m]i think we should avoid installing python-ceilometerclient as it's no longer maintained https://opendev.org/openstack/python-ceilometerclient18:02
damiandabrowski[m]unfortunately i need to leave for now18:02
mgariepysee you after your vacations :D18:06
damiandabrowski[m]see You!18:08
opendevreviewMerged openstack/openstack-ansible-rsyslog_client master: PDF Documentation Build tox target  https://review.opendev.org/c/openstack/openstack-ansible-rsyslog_client/+/69150118:12
jrosserfrickler: i got an OSA job to pass on 22.04, the only thing so far i have found is pyCss version not good for py3.1019:00
mgariepyhmm.. approved but not merged.. https://review.opendev.org/c/openstack/openstack-ansible-tests/+/78330419:01
jrosseri think the 4th depends-on is not merged19:03
jrossergiven that it's rocky we can probably abandon those19:03
mgariepyok i'll abandon then19:05
jrosseroh i mean pyScss is not sufficient version for python3.10 19:06
mgariepyi guess stein can also be abandonned ?19:10
mgariepyhttps://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/77947119:10
opendevreviewMerged openstack/openstack-ansible-os_horizon master: Improve defining horizon_lib_dir  https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/82092819:10
jrosserwhere did we EOL up to?19:12
mgariepyfrom this ... newton ? lolhttps://docs.openstack.org/openstack-ansible/latest/19:13
jrosserQ is EM19:14
jrosserwell i'm not sure what we should do19:16
jrossertheres a stage where we noop the CI jobs and can merge fixes if we like with no tests?19:16
mgariepynot sure.19:17
jrosserme neither tbh19:17
mgariepylet's discuss at next meeting19:18
jrosserit is a shame this got left for Z release https://review.opendev.org/c/openstack/requirements/+/83061519:18
jrosserseems like the only issue i have found with Y(-ish) for OSA19:18
jrosseron 22.04 / py3.1019:19
mgariepya lot smoother than other distros..19:20
opendevreviewMerged openstack/openstack-ansible-openstack_hosts master: Add nova dependency repo for distro install  https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/80347519:20
opendevreviewMerged openstack/openstack-ansible-lxc_container_create master: Wait for 2s after restarting systemd-networkd  https://review.opendev.org/c/openstack/openstack-ansible-lxc_container_create/+/83728119:22
jrosseri have a few small patches here to look at https://review.opendev.org/q/topic:osa-jammy19:28
mgariepyrocky doesn't have tar in the containter for https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/83941119:30
mgariepyfor this one: https://review.opendev.org/c/openstack/openstack-ansible/+/83758919:31
jrosserooh nice spot19:40
jrosseri have tried and failed to fix the linters there as well19:40
mgariepyi was tired of looking at really old patch haha so i did decide to take a look at something newer :D19:40
mgariepyjrosser, the linter is the  cmd-instead-of-module for the tar command19:50
mgariepyi think19:51
jrossermgariepy: oh doh! I’ve put the lint skip tag on the wrong task :( that’s why it fails20:49
*** dviroel|rover is now known as dviroel|rover|biab21:12
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-repo_server master: Add upgrade path from lsyncd to shared filesystem.  https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/83941121:12
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/+/83785921:12
*** dviroel|rover|biab is now known as dviroel|rover22:31
*** dviroel|rover is now known as dviroel|rover|out22:54

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