Wednesday, 2022-12-14

opendevreviewJames Denton proposed openstack/openstack-ansible master: [WIP] Update documentation for LXC/metal and LXB/OVS/OVN  https://review.opendev.org/c/openstack/openstack-ansible/+/86757701:28
jrossermorning08:25
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-os_ironic master: Update ironic documentation  https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/86754708:39
noonedeadpunko/09:05
jrossermorning09:36
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Bump OpenStack-Ansible for Zed release  https://review.opendev.org/c/openstack/openstack-ansible/+/86761809:50
noonedeadpunkI guess it's time I've reviewed PKI patches again....09:52
noonedeadpunkOk, so https://review.opendev.org/c/openstack/ansible-role-pki/+/867542 seems quite good regrdless of others I would say09:58
opendevreviewDmitriy Rabotyagov proposed openstack/ansible-role-pki master: Ensure CA privatekey permissions  https://review.opendev.org/c/openstack/ansible-role-pki/+/86755510:03
noonedeadpunkjrosser: your idea with file seems quite good as well regardless of everything else ^10:04
opendevreviewDmitriy Rabotyagov proposed openstack/ansible-role-pki master: Allow to define mode and ownership for CA private keys  https://review.opendev.org/c/openstack/ansible-role-pki/+/86755310:27
noonedeadpunkmeh, everything looks good to me 10:35
noonedeadpunkOnly not sure about https://review.opendev.org/c/openstack/ansible-role-pki/+/867549/ indeed. But it's also quite fair I would say10:36
noonedeadpunkBut can become too complicated in the future10:36
noonedeadpunkSo maybe worth avoiding it indeed10:36
*** dviroel|out is now known as dviroel|rover11:12
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-os_ironic master: Update ironic documentation  https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/86754711:50
kleiniIs it true, that ceph_client/tasks/ceph_auth_extra_compute.yml is only meant to be working, if OSA has access to mon hosts of Ceph? I defined keyrings with keyring_src and keyring_dest and that only works with Glance and Cinder but not Nova. Would it make sense to extend ceph_auth_extra_compute.yml to support hat? I don't have access to mon hosts of Ceph instances.12:22
noonedeadpunkUm, I'm not sure. I had setup without access to mon hosts only in multinode sandbox, but it worked nicely12:41
opendevreviewMerged openstack/ansible-role-pki master: Ensure CA privatekey permissions  https://review.opendev.org/c/openstack/ansible-role-pki/+/86755512:42
noonedeadpunkalso looking at https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/866974/1/tasks/ceph_auth_extra_compute.yml now - file: absent should not fail if file doesn't exist, should it?12:42
noonedeadpunkalso I haven't seen failures in that sandbox....12:43
noonedeadpunkhm.... Seems smth off with neutron on metal :(13:00
noonedeadpunkhttps://zuul.opendev.org/t/openstack/build/e56e5d93df0043089e47447039d966a7/log/logs/host/neutron-server.service.journal-11-08-38.log.txt#245313:00
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Bump OpenStack-Ansible for Zed release  https://review.opendev.org/c/openstack/openstack-ansible/+/86761813:03
noonedeadpunkdamn it :(13:10
noonedeadpunkand how it happens that lxc works at the same time....13:11
opendevreviewDamian Dąbrowski proposed openstack/ansible-role-pki stable/zed: Ensure CA privatekey permissions  https://review.opendev.org/c/openstack/ansible-role-pki/+/86763113:12
mgariepyit's weird.13:12
opendevreviewDamian Dąbrowski proposed openstack/ansible-role-pki stable/yoga: Ensure CA privatekey permissions  https://review.opendev.org/c/openstack/ansible-role-pki/+/86763213:13
opendevreviewDamian Dąbrowski proposed openstack/ansible-role-pki stable/xena: Ensure CA privatekey permissions  https://review.opendev.org/c/openstack/ansible-role-pki/+/86763313:13
opendevreviewDamian Dąbrowski proposed openstack/ansible-role-pki stable/wallaby: Ensure CA privatekey permissions  https://review.opendev.org/c/openstack/ansible-role-pki/+/86763413:14
jamesdentonthere are some ovn-related errors in neutron-server log, and "connection dropped (Protocol error)" in the ovn logs13:42
mgariepyalmost 10 minutes before ?13:59
noonedeadpunkand eventually SQL query is exactly what timeouts in role14:00
jamesdentontrue14:00
jamesdentoni've got an AIO spinning up, might be able to replicate soon14:00
mgariepywhy does the instances info seems to stops at 10:25 ?14:04
mgariepyhttps://6f6f67e9fba8f2d009b9-fd1d23de4951783ce4905cf96230b70e.ssl.cf2.rackcdn.com/867618/1/check/openstack-ansible-deploy-aio_metal-ubuntu-jammy/e56e5d9/logs/openstack/instance-info/index.html14:05
mgariepywe do have logs in the services for up to 11:08 at least14:05
kleininoonedeadpunk: https://paste.opendev.org/show/b6mTLzi6IfUaxtfj74hi/ <- this is my current configuration for two ceph instances without having access to mon hosts. everything needs to be migrated: images and volumes14:14
kleinitherefore I have two extra confs, one for glance and one for cinder/Nova14:15
kleinithe extra ceph conf for glance does not have a secret_uuid and therefore those two cleanup tasks of ceph_auth_extra_compute.yml fail14:16
noonedeadpunkwell, I used ceph cluster_name to separate these things14:16
kleiniso, you have an easier configuration for this scenario?14:16
noonedeadpunkSo I set `ceph_cluster_name` for group_vars14:16
noonedeadpunkwell, my scenario was different clusters per AZ.14:17
noonedeadpunkThough I had https://opendev.org/openstack/openstack-ansible-ceph_client/commit/b3e7560e8022384b8269d1b380aae9602cea182414:17
kleiniI need two Ceph instances in the same AZ. We need to migrate all images and volumes from "old" Ceph to new one.14:17
noonedeadpunkIt doesn't really matter I guess, as it's more about path of configs that are expected14:18
noonedeadpunkAs I placed ceph_cluster_name per az groups, but you can in cinder_all and glance_all14:19
kleiniI don't see, that there will be two secrets registered in libvirt. As said: some part of volumes is in Ceph A and some in Ceph B. There can be VMs on the same compute host using maybe even two volumes whereof one is one Ceph A and one on Ceph B.14:21
kleiniI looked all the way through ceph_auth_extra_compute.yml (on W and master) where a secret is created from keyring_src or keyring_dest in ceph_extra_confs and that is what I need if I don't have access to mon hosts to fetch/create keyrings there14:24
kleiniI need Ceph A and B completely working in Glance, Cinder and Nova as I need a longer time frame to be able to migrate all images and volumes in Ceph. This works fine for Glance and Cinder with the above configuration but I am currently missing the secret of Ceph B with ceph2-cinder user in libvirt on the compute node.14:35
kleininoonedeadpunk: any thoughs on this? If not, I would try to extend ceph_auth_extra_compute.yml to add the ceph2-cinder secret to libvirt according to this configuration: https://paste.opendev.org/show/b6mTLzi6IfUaxtfj74hi/14:53
noonedeadpunkjamesdenton: that looks like neutron bug to me. as once I've roledback SHA CI is happy15:03
jamesdentonahh, interesting15:04
noonedeadpunkkleini: ah, ok, now I see what you're doing15:05
noonedeadpunkyeah, I think now we assume that only one ceph cluster is used per service (not multiple ones for same service)15:05
noonedeadpunkand interesting indeed, that lxc jobs passed, to it don't like ovn with api on same place kind of15:13
jamesdentonnoonedeadpunk FWIW - i performed a local aio_metal deployment with 867618 and it seemed to work OK15:14
noonedeadpunkhuh15:15
noonedeadpunkbut well - what SHA was there? 15:15
noonedeadpunkAs I've rolled back now to previous neutron version15:15
jamesdentonhow long ago?15:15
noonedeadpunkquite a while...15:16
noonedeadpunk`neutron_git_install_branch: 6927dfbb3e52ba298928362da5cce32d49b1e1f8` is what's broken in CI15:16
noonedeadpunk`neutron_git_install_branch: add538d7ff26d843fd43ca85f7dca9385dc3ecc1` is passing15:16
noonedeadpunkChanged at 2pm UTC15:17
jamesdentonahh gotcha, yeah i've got add538d7ff26d843fd43ca85f7dca9385dc3ecc115:17
jamesdentonwell ,then nevermind :)15:17
jamesdentoni missed your rollback15:17
*** dviroel|rover is now known as dviroel|rover|lunch15:52
noonedeadpunkwe need to merge https://review.opendev.org/c/openstack/openstack-ansible/+/867618/2 sooner better16:15
jamesdentonahh another zuul ui change16:24
mgariepywhy do they mode the Backport stuff lol16:30
*** dviroel|rover|lunch is now known as dviroel|rover16:38
noonedeadpunkI guess you meant gerrit :)16:40
mgariepymove** bacport candiate box is right were the +w was..16:47
jamesdentonyes, gerrit, sorry16:55
noonedeadpunkI more wonder about switch of +W and +V as they're even not alphabetic now16:58
noonedeadpunkAs for backport - since it's not required for merge, it's hidden at all until it has some vote16:58
opendevreviewMerged openstack/ansible-role-pki master: Backup CA key and certs by default  https://review.opendev.org/c/openstack/ansible-role-pki/+/86754218:36
noonedeadpunkfwiw I don't see keystone failures anymore20:29
noonedeadpunknot sure it was haproxy or workers fix, but it's not an issue anymore from what I see20:29
jrosseroh so close on merging the release patch20:46
*** tosky_ is now known as tosky21:12
*** dviroel|rover is now known as dviroel|out21:31
noonedeadpunkyup(21:37

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