Friday, 2023-05-26

*** EugenMayer43 is now known as EugenMayer401:26
opendevreviewJohn Garbutt proposed openstack/kolla-ansible master: Ensure we set dns_servers for OVN  https://review.opendev.org/c/openstack/kolla-ansible/+/88446008:22
opendevreviewJohn Garbutt proposed openstack/kolla-ansible master: Ensure we set dns_servers for OVN  https://review.opendev.org/c/openstack/kolla-ansible/+/88446008:23
opendevreviewJohn Garbutt proposed openstack/kolla-ansible master: Ensure we set dns_servers for OVN  https://review.opendev.org/c/openstack/kolla-ansible/+/88446008:36
opendevreviewOpenStack Release Bot proposed openstack/ansible-collection-kolla stable/2023.1: Update .gitreview for stable/2023.1  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88446208:38
opendevreviewOpenStack Release Bot proposed openstack/ansible-collection-kolla stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88446308:38
opendevreviewOpenStack Release Bot proposed openstack/ansible-collection-kolla master: Update master for stable/2023.1  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88446408:38
opendevreviewOpenStack Release Bot proposed openstack/kolla-ansible stable/2023.1: Update .gitreview for stable/2023.1  https://review.opendev.org/c/openstack/kolla-ansible/+/88446508:39
opendevreviewOpenStack Release Bot proposed openstack/kolla-ansible stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1  https://review.opendev.org/c/openstack/kolla-ansible/+/88446608:39
opendevreviewOpenStack Release Bot proposed openstack/kolla-ansible master: Update master for stable/2023.1  https://review.opendev.org/c/openstack/kolla-ansible/+/88446708:39
opendevreviewOpenStack Release Bot proposed openstack/kolla stable/2023.1: Update .gitreview for stable/2023.1  https://review.opendev.org/c/openstack/kolla/+/88446808:40
opendevreviewOpenStack Release Bot proposed openstack/kolla stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1  https://review.opendev.org/c/openstack/kolla/+/88446908:40
opendevreviewOpenStack Release Bot proposed openstack/kolla master: Update master for stable/2023.1  https://review.opendev.org/c/openstack/kolla/+/88447008:40
frickleryay08:41
SvenKieskemgoddard: frickler: maybe you can take a look at https://review.opendev.org/c/openstack/kolla-ansible/+/884460 there is an older abandoned changeset linked where you where involved, maybe we can fix it better this time?08:41
SvenKieskeI don't know enough about ovn dns to have a real opinion there08:42
fricklerSvenKieske: thx for spotting this, commented08:48
mnasiadkafrickler: should we try to merge Mark's patches around MariaDB to get out of the ansible-core 2.14.2 lockout? ;-)08:49
mnasiadkahttps://review.opendev.org/c/openstack/kolla/+/884208 and https://review.opendev.org/c/openstack/kolla-ansible/+/88418208:50
fricklermnasiadka: I didn't look at those yet, trying to finally get some downstream work done this week. also monday is a bank holiday here08:51
fricklernow that we've branched anyway, I guess this can wait a couple of days?08:51
fricklerlike, we won't do the final release until end of next week earliest?08:51
mnasiadkaIn theory 1st of June is the release deadline (final releases)08:52
mnasiadkaBut it can wait until Tuesday I guess08:52
SvenKieskethat's thursday; I had an initial look at the mariadb stuff yesterday, I had the impression it was still being worked on, did that change?08:52
mnasiadkaIt's marked as ready for review and looks like working - we can run a recheck to check if it doesn't fail again08:53
SvenKieskeah right, I see08:53
mnasiadkathe kolla patch is trivial08:53
SvenKieskewill look into it, but have also some meetings today08:53
mnasiadkaso that one might get in and get backported easily08:53
opendevreviewVerification of a change to openstack/kolla-ansible master failed: ceilometer: Use assert on checks for readability  https://review.opendev.org/c/openstack/kolla-ansible/+/86959308:58
opendevreviewMerged openstack/ansible-collection-kolla stable/2023.1: Update .gitreview for stable/2023.1  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88446208:58
opendevreviewMerged openstack/ansible-collection-kolla stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88446308:58
opendevreviewMerged openstack/ansible-collection-kolla master: Update master for stable/2023.1  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88446408:58
opendevreviewJohn Garbutt proposed openstack/kolla-ansible master: Update docs about dns_servers for OVN  https://review.opendev.org/c/openstack/kolla-ansible/+/88446009:08
opendevreviewMatt Crees proposed openstack/kayobe master: 2023.1 RC1 - Sync KA feature flags & inventories  https://review.opendev.org/c/openstack/kayobe/+/88447109:34
grami[m]Hi All, Sorry to be a pain any chance someone could help get this pushed through it's the cherrypick of an already merged in master. https://review.opendev.org/c/openstack/kolla-ansible/+/88391509:44
opendevreviewMerged openstack/kolla-ansible stable/2023.1: Update .gitreview for stable/2023.1  https://review.opendev.org/c/openstack/kolla-ansible/+/88446509:50
opendevreviewMatt Crees proposed openstack/kayobe master: Use OpenStack 2023.1 release  https://review.opendev.org/c/openstack/kayobe/+/88447209:51
opendevreviewMerged openstack/kolla-ansible stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1  https://review.opendev.org/c/openstack/kolla-ansible/+/88446609:54
opendevreviewMerged openstack/kolla-ansible master: Update master for stable/2023.1  https://review.opendev.org/c/openstack/kolla-ansible/+/88446709:54
fricklerSvenKieske: https://review.opendev.org/c/openstack/kolla/+/882924 is a good example to see how a lot of jobs are unstable. would IMO be a good start to look into these failures and try to make things more reliable 09:58
SvenKieskethis reads like a call to action/work, will look into it :D 09:59
opendevreviewMatt Crees proposed openstack/kayobe-config master: 2023.1 RC1 - Sync defaults with Kayobe  https://review.opendev.org/c/openstack/kayobe-config/+/88447310:01
opendevreviewMerged openstack/kolla stable/2023.1: Update .gitreview for stable/2023.1  https://review.opendev.org/c/openstack/kolla/+/88446810:02
opendevreviewMerged openstack/kolla stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1  https://review.opendev.org/c/openstack/kolla/+/88446910:03
opendevreviewMerged openstack/kolla master: Update master for stable/2023.1  https://review.opendev.org/c/openstack/kolla/+/88447010:04
mnasiadkaSvenKieske: rocky ironic inspector failure is a good puzzle10:17
opendevreviewMatt Crees proposed openstack/kayobe-config-dev master: 2023.1 RC1 - Sync defaults with Kayobe  https://review.opendev.org/c/openstack/kayobe-config-dev/+/88447710:18
opendevreviewMatt Crees proposed openstack/kayobe master: 2023.1 RC1 - Sync KA feature flags & inventories  https://review.opendev.org/c/openstack/kayobe/+/88447112:07
opendevreviewMichal Nasiadka proposed openstack/kolla stable/yoga: opensearch: move to yum/apt repos  https://review.opendev.org/c/openstack/kolla/+/88371612:18
opendevreviewMichal Nasiadka proposed openstack/kolla stable/yoga: CI: Fix centos9 builds and drop unnecessary zuul vars  https://review.opendev.org/c/openstack/kolla/+/88448712:27
opendevreviewMichal Nasiadka proposed openstack/kolla stable/yoga: CI: Fix centos9 builds and drop unnecessary zuul vars  https://review.opendev.org/c/openstack/kolla/+/88448712:28
SvenKieskeso e.g. looking at this failed job: https://zuul.opendev.org/t/openstack/build/0cbba93348664af287208b79e1d3fc0c/log/primary/logs/ansible/test-core-openstack it fails during volume creation from image, after over > 240 seconds.  I found this old cinder bug, which seems to indicate that volumes from images are created using a tmp file (wtf?): https://bugs.launchpad.net/nova/+bug/127207612:36
SvenKieskethe test volume seems to be 1GB, I don't know the size of the cirros image from the top of my head, but it should be something below 500MB? that shouldn't run for several minutes, even including a tmp file, no?12:37
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: drop ceph jobs ulimit config for EL9  https://review.opendev.org/c/openstack/kolla-ansible/+/88448812:37
SvenKieskefrickler: do you happen to know what storage backend our CI uses? RBD?12:38
mnasiadkaSvenKieske: cephadm multinode jobs use ceph, single node jobs == no cinder12:43
mnasiadkaSvenKieske: ah, you mean nodepool VMs storage backend? :)12:43
mnasiadkaSvenKieske: https://0668fd1a5cb7e14c0a62-6f4fdaa50c9ffb2ee70643e96aea629f.ssl.cf2.rackcdn.com/884182/5/check/kolla-ansible-ubuntu-cephadm/0cbba93/secondary1/logs/kolla/cinder/cinder-volume.txt - this log shows rbd timeout, so might be our ceph is not functioning properly, although ceph -s claims it's healthy12:47
opendevreviewMatt Crees proposed openstack/kayobe-config-dev master: 2023.1 RC1 - Sync defaults with Kayobe  https://review.opendev.org/c/openstack/kayobe-config-dev/+/88447712:49
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: cephadm - set rbd application on pool creation  https://review.opendev.org/c/openstack/kolla-ansible/+/88448912:49
mnasiadkaah, that failure is volume from image12:56
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: Enable cinder coordination for cephadm jobs  https://review.opendev.org/c/openstack/kolla-ansible/+/88449213:06
opendevreviewMichal Nasiadka proposed openstack/kolla stable/yoga: CI: Fix centos9 builds and drop unnecessary zuul vars  https://review.opendev.org/c/openstack/kolla/+/88448713:07
SvenKieskemnasiadka: thanks for the hints..is that ceph a baremetal install or virtualized itself?13:08
mnasiadkaSvenKieske: we deploy ceph and openstack on three VMs that we get from OpenDev CI, we use cephadm for deploying ceph - you can see the role under roles/cephadm/13:09
SvenKieskeyeah, currently looking at the ceph logs and config13:11
SvenKieskethere's at least a warning: https://zuul.opendev.org/t/openstack/build/0cbba93348664af287208b79e1d3fc0c/log/primary/logs/ceph/ceph_s.txt that 3 pools don't have an application enabled, depending on which this is, this can be harmless.13:12
SvenKieskethere are some interesting errors in the cephadm log, drilling down further13:14
mnasiadkaSvenKieske: I just posted three patches to get the cephadm multinode jobs a bit better 13:24
mnasiadkawell, maybe two 13:25
mnasiadkahttps://review.opendev.org/c/openstack/kolla-ansible/+/884489 and https://review.opendev.org/c/openstack/kolla-ansible/+/88449213:25
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: Enable cinder coordination for cephadm jobs  https://review.opendev.org/c/openstack/kolla-ansible/+/88449213:28
SvenKieskenice13:29
opendevreviewMichal Nasiadka proposed openstack/kolla stable/yoga: opensearch: move to yum/apt repos  https://review.opendev.org/c/openstack/kolla/+/88371613:31
SvenKieskeIn general, speaking from a very high level perspective, I observed that we tend to have a rare(?) failure pattern where we don't iterate over all necessary elements because the lists of these elements are very decentralized organized. so we had - afaik two bugs fixed this week where services where missing from certain ansible loops.13:32
* SvenKieske wonders if we could reorganize the code somewhat to make this stuff more robust, via jinja templates, or if this adds another layer of abstraction which makes everything harder to reason about.13:33
SvenKieskeI'm not yet convinced we can form decent subgroups of services which could be referenced by lists; for this to be worthwhile there needs to be at least one subset of services over which we loop in different code locations.13:35
SvenKieskestuff like this: https://review.opendev.org/c/openstack/kolla-ansible/+/884492/2/tests/run.yml#148; it's of course easy to miss a service file or config dir if you have various lists of services to which different tasks are applied13:37
mnasiadkatrue, that file is not ideal - but we could include files for different scenarios, and that would help with visibility13:39
mnasiadkaand I think we agreed on a goal to make the CI more documented - so that would help13:40
opendevreviewMichal Nasiadka proposed openstack/kolla stable/yoga: CI: Fix centos9 builds and drop unnecessary zuul vars  https://review.opendev.org/c/openstack/kolla/+/88448713:40
opendevreviewMichal Nasiadka proposed openstack/kolla stable/yoga: opensearch: move to yum/apt repos  https://review.opendev.org/c/openstack/kolla/+/88371613:40
opendevreviewSven Kieske proposed openstack/kolla-ansible master: remove unnecessary comments  https://review.opendev.org/c/openstack/kolla-ansible/+/88450313:43
SvenKieskemhm, I don't know this: would it be worthwhile to enable ceph telemetry in our CI ceph? that would depend on how long these clusters typicially live I suppose. if they get immediately deleted there's no point in reporting crashes etc. (the reporting takes some time, but is configurable)13:47
mnasiadkaSvenKieske: they shouldn't live more than 3 hours :)13:54
mnasiadkawell, maybe more on upgrades13:55
guesswhat[m]hello, question, is nested virtualization supported for nova vms ?14:00
mmalchukguesswhat[m] supported if configured14:06
guesswhat[m]mmalchuk: can You point me to the specific config? tried to seach nested in globals.conf, but no luck. Thanks14:06
mmalchukguesswhat[m] this is not the kolla configuration, but the system14:08
mmalchukhttps://docs.openstack.org/devstack/latest/guides/devstack-with-nested-kvm.html14:08
mmalchukas an example ^^^14:09
mmalchukhttps://docs.openstack.org/nova/latest/admin/cpu-models.html#host-passthrough14:11
guesswhat[m]oh, thanks, wondering if triple nested virtualization is nogo... have a lab in kvm where I running openstack node and controllel and the same time I want to use kata containers on some nodes in nova, that would be triple nested virtualization :X14:11
mmalchukabout 'Host passthrough' in nova14:11
SvenKieskenot a no go,  I guess, but should be terribly slow, maybe even unusable. first thing to check if you have nested-vt flags turned on in uefi, it's often disabled by default. it can also be a bit of a security risk; there where some CVE only affecting nested-vt in kvm / linux kernel.14:14
mmalchukguesswhat[m] https://docs.openstack.org/nova/latest/admin/configuration/hypervisor-kvm.html#host-configuration14:14
mmalchukabout nova configuration ^^^14:14
opendevreviewMichal Nasiadka proposed openstack/kolla stable/yoga: CI: Fix centos9 builds and drop unnecessary zuul vars  https://review.opendev.org/c/openstack/kolla/+/88448714:15
mmalchukguesswhat[m] https://docs.openstack.org/nova/latest/admin/configuration/hypervisor-kvm.html#nova-configuration14:15
opendevreviewMichal Nasiadka proposed openstack/kolla stable/yoga: opensearch: move to yum/apt repos  https://review.opendev.org/c/openstack/kolla/+/88371614:15
mmalchukhost-passthrough configuration ^^^14:15
guesswhat[m]Thanks :)14:17
guesswhat[m]Too bad there are not many Nova drivers available, like free esxi is limited to 8vCPU, hyper-v is nogo, xen is not supported 14:18
guesswhat[m]I have 64c/128t CPU server, I am running Proxmox, then 3 VMs, vyos, controller and node ( nested virtualization )14:19
guesswhat[m]But trying Kata containers in Kubernetes deployed on Openstack via ClusterAPI requires baremetal = virtualization, so that would 3rd level14:20
opendevreviewMichal Nasiadka proposed openstack/kolla stable/yoga: CI: Fix centos9 builds and drop unnecessary zuul vars  https://review.opendev.org/c/openstack/kolla/+/88448714:33
opendevreviewPierre Riteau proposed openstack/kayobe master: [DNM] Revert "CI: Enable bare metal testing for Ubuntu upgrades"  https://review.opendev.org/c/openstack/kayobe/+/88452516:23
opendevreviewPierre Riteau proposed openstack/kayobe master: [DNM] Revert "CI: Enable bare metal testing for Ubuntu upgrades"  https://review.opendev.org/c/openstack/kayobe/+/88452518:21
opendevreviewMaksim Malchuk proposed openstack/kolla-ansible master: Fix passwords.yml permissions  https://review.opendev.org/c/openstack/kolla-ansible/+/88210018:55

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