Tuesday, 2023-03-21

*** EugenMayer43 is now known as EugenMayer406:56
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: Pin zun jobs to Docker 20  https://review.opendev.org/c/openstack/kolla-ansible/+/87388807:33
opendevreviewRobin Klostermeyer proposed openstack/kolla-ansible master: Add ironic-prometheus-exporter  https://review.opendev.org/c/openstack/kolla-ansible/+/87438007:49
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: Pin zun jobs to Docker 20  https://review.opendev.org/c/openstack/kolla-ansible/+/87388808:34
opendevreviewMatt Crees proposed openstack/kolla-ansible master: Set RabbitMQ ha-promote-on-shutdown=always  https://review.opendev.org/c/openstack/kolla-ansible/+/87286309:08
opendevreviewMatt Crees proposed openstack/kolla-ansible master: Add flags for RabbitMQ message TTL & queue expiry  https://review.opendev.org/c/openstack/kolla-ansible/+/87242709:11
opendevreviewMatt Crees proposed openstack/kolla-ansible master: Set RabbitMQ message TTL and queue expiry  https://review.opendev.org/c/openstack/kolla-ansible/+/82219109:11
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: neutron: Add neutron-ovn-agent support  https://review.opendev.org/c/openstack/kolla-ansible/+/87129710:28
mnasiadkakevko: care for another look in https://review.opendev.org/c/openstack/kolla-ansible/+/875729 ?10:30
hrwmnasiadka: opinion on https://review.opendev.org/c/openstack/kolla/+/878007 or https://review.opendev.org/c/openstack/kolla/+/877986 10:37
hrwmnasiadka: first one fixes neutron-mlnx-agent in one way, second drops virtualenv in favour or 'python3 -m venv'10:37
hrws/or/of/10:37
mnasiadkahrw: first - if it fixes, then fine by me, I have no clue if anybody uses this image ;-)10:39
hrwmnasiadka: the question is: pyzmq 19 or pyzmq 21 ;D10:39
mnasiadkagoing to virtualenv makes sense, kolla-ansible and kayobe moved that way in zed10:39
mnasiadkawell, if 21 works, fine by me10:39
Fl1ntHi folks!10:40
Fl1ntI'm working to prepare all my patches for contribution with Designate role, however, I've a question regarding this one: https://opendev.org/openstack/kolla-ansible/src/branch/master/ansible/roles/designate/tasks/update_pools.yml#L410:42
Fl1ntBasically, this doesn't work10:43
Fl1ntas designate-manage command exit with 1 instead of 0 and without any error10:43
Fl1ntthe fact that the service doesn't return the appropriate return code isn't kolla-ansible related10:44
Fl1ntBUT10:44
Fl1ntthe command itself isn't the appropriated one10:44
Fl1ntas10:44
Fl1ntwe either want to do: designate-manage pool update10:44
Fl1ntor10:44
Fl1ntdesignate-manage pool update --delete10:44
Fl1ntthe path we use is the default one10:44
Fl1ntand it's currently hardcoded on k-a10:45
Fl1ntso pretty irrelevant to me10:45
Fl1ntadditionally that's what cause the error with bad return code10:45
Fl1ntso the question is, do I patch it with "designate-manage pool update" (only add diff between existing pools and newly found) or do I use "designate-manage pool update --delete" (delete all pools existing in DB but not on file)?10:47
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: Test haproxy single external frontend  https://review.opendev.org/c/openstack/kolla-ansible/+/84123910:48
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: Test haproxy single external frontend  https://review.opendev.org/c/openstack/kolla-ansible/+/84123910:48
opendevreviewMerged openstack/kolla master: Add Dockerfile for neutron-ovn-agent  https://review.opendev.org/c/openstack/kolla/+/87079311:00
opendevreviewMichal Nasiadka proposed openstack/kolla master: Revert "Pin setuptools=67.2.*"  https://review.opendev.org/c/openstack/kolla/+/87646011:08
Fl1ntWhere are located the tests units on k-a ??11:08
opendevreviewGaël THEROND proposed openstack/kolla-ansible master: Fix improper designate-manage command usage.  https://review.opendev.org/c/openstack/kolla-ansible/+/87806311:10
Fl1ntmnasiadka, how can I tag a bug report as duplicate?11:15
mnasiadkaFl1nt: which one?11:15
Fl1ntfound out ^^ nvm11:16
Fl1nthttps://bugs.launchpad.net/kolla-ansible/+bug/1943891 duplicate of 201229211:17
kevkomnasiadka: i wanted to give +2 for opensearch but didn't test installation ...but i can't see any issue there ..i am going to give +212:41
mnasiadkakevko: thanks12:41
kevkomnasiadka: what about that ceph multiple configs ? 12:42
mnasiadkakevko: I can have a look later today12:43
kevkomnasiadka: i've contacted juan pablo and he is saying his patch is near merge ... i don't care ..we can merge it and i can rebase ..12:43
kevkobecause my patch is also solving different keys, glance, cinder, nova ..while his is only copying ceph.conf ..no different keys ..nothing 12:44
mnasiadkakevko: understood, will have a look12:47
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: neutron: Add neutron-ovn-agent support  https://review.opendev.org/c/openstack/kolla-ansible/+/87129712:48
kevkomnasiadka: thanks12:52
opendevreviewMerged openstack/kolla-ansible master: Pin zun jobs to Docker 20  https://review.opendev.org/c/openstack/kolla-ansible/+/87388813:10
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/zed: Pin zun jobs to Docker 20  https://review.opendev.org/c/openstack/kolla-ansible/+/87568713:14
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/yoga: Pin zun jobs to Docker 20  https://review.opendev.org/c/openstack/kolla-ansible/+/87568813:14
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Add support for multiple ceph files  https://review.opendev.org/c/openstack/kolla-ansible/+/87741313:36
opendevreviewGaël THEROND proposed openstack/kolla-ansible master: Fix improper designate-manage command usage.  https://review.opendev.org/c/openstack/kolla-ansible/+/87806313:36
Fl1nt@kevko, What are you solving in here on CEPH? you already can have multiple ceph config13:45
Fl1ntTo be honest, using different key per services doesn't matter with Ceph as you'll need nova to access glance/cinder rbd pool and cinder access glance pools too. So in the end we did kept one key generated per service, but ended up having nova to access all RBD.13:47
Fl1ntpools13:47
opendevreviewMerged openstack/kolla-ansible stable/yoga: Add OpenSearch  https://review.opendev.org/c/openstack/kolla-ansible/+/87572913:50
kevkoFl1nt: i am not sure if I understand 14:02
kevkoFl1nt: ok, imagine you have cinder-volume which is connecting to several ceph backends ..how is it possible with current kolla-ansible code ? 14:03
kevkoFl1nt: next, imagine you have several availability_zones and you want to mount cinder-volume->ceph-az1->compute-az1 ...you need to provide different ceph.conf for different set of hosts 14:04
kevkoFl1nt: Our new customer wanted to have , 3 availability zones, in every availability zone is one ceph, cinder-volume configured every az, every compute host from availability_zone mount cinder-volume from their AZ 14:06
kevkoFl1nt: so you need several ceph.confs ... okay, you can use same keyrings for every ceph .. but you can have it different (implemented in patch) ...they also using glance multistore ..so every instance from every AZ contacting glance from specific AZ (speed)14:07
kevkoFl1nt: AZs can be configured via config overrides ... multiple ceph.confs you cannot ... (follow up will be support AZ out-of-the-box )14:08
kevkoFl1nt: listening 14:10
Fl1ntkevko, ok got it, wasn't thinking about multiple Ceph cluster but multiple backends, that you can already do.14:17
Fl1ntfor the AZ you don't need multiple conf, you just need a correct cluster crushmap14:17
Fl1ntBut all in all kevko, your customer needs is ok, and the patch is ok.14:19
Fl1ntI've my cluster multi AZ, all my cinder volumes are restricted to AZ (no cross AZ allowed) and my cinder-volumes hosted on any compute node, so it means each time I spawn a new AZ, I join 300 new cinder-volumes, but I've only one ceph.conf for all cinder-volumes services as the crushmap do the job14:22
Fl1ntand same things for glance as it's either swift through RGW on multisite/multizone with different EC Profile and policy or a Ceph RBD Pool with different EC profile.14:23
kevkoFl1nt: yes, -> https://paste.openstack.org/show/bpr5R6W9gwwF1d9dfPJt/  better explanation maybe14:48
kevkoFl1nt: last line in paste means cinder az_name in config and neutron agents ...14:49
kevkoFl1nt: follow-up patch for AZs i wanted to implement as host variable in inventory which will be rendered in configs and if enabled host agregates will be created , set azs and add hosts ...14:50
Fl1ntHonestly a better support for AZs would be a bliss yes. 14:59
Fl1nt for now I'm having your exact diagram under kolla-ansible, just leveraging the crushmap.15:01
opendevreviewMerged openstack/kolla master: gnocchi: bump to 4.5.0  https://review.opendev.org/c/openstack/kolla/+/87781515:09
opendevreviewMichal Nasiadka proposed openstack/kayobe stable/yoga: Support OpenSearch configuration  https://review.opendev.org/c/openstack/kayobe/+/87764915:19
opendevreviewMartin Hiner proposed openstack/ansible-collection-kolla master: Adding podman support  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/85224015:26
opendevreviewMerged openstack/kolla master: base: make curl retry on all errors  https://review.opendev.org/c/openstack/kolla/+/87472315:30
guesswhat[m]Anyone can help me integrate cinder backend for glance ?16:10
opendevreviewMichal Nasiadka proposed openstack/kayobe master: Fix opensearch container builds  https://review.opendev.org/c/openstack/kayobe/+/87812316:15
opendevreviewMichal Nasiadka proposed openstack/kayobe stable/yoga: Support OpenSearch configuration  https://review.opendev.org/c/openstack/kayobe/+/87764916:18
opendevreviewMichal Nasiadka proposed openstack/kayobe stable/yoga: Support OpenSearch configuration  https://review.opendev.org/c/openstack/kayobe/+/87764916:20
opendevreviewMerged openstack/kolla-ansible stable/zed: Pin zun jobs to Docker 20  https://review.opendev.org/c/openstack/kolla-ansible/+/87568716:48

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