15:01:29 <gagehugo> #startmeeting openstack-helm 15:01:29 <opendevmeet> Meeting started Tue Jul 27 15:01:29 2021 UTC and is due to finish in 60 minutes. The chair is gagehugo. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:01:29 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:01:29 <opendevmeet> The meeting name has been set to 'openstack_helm' 15:01:38 <gagehugo> cool that works 15:01:41 <gagehugo> o/ 15:01:44 <reddy1> o/ 15:01:45 <sangeet> o/ 15:05:15 <reddy1> I'd like to discuss: https://review.opendev.org/c/openstack/openstack-helm/+/801043 15:06:02 <reddy1> The PS calculates the workers for nova services and there are similar PS for other services. 15:06:17 <gagehugo> yeah I saw that 15:06:42 <gagehugo> #topic worker count nova 15:06:52 <gagehugo> #link https://review.opendev.org/c/openstack/openstack-helm/+/801043 15:07:21 <reddy1> I'm of the opinion that we can set the workers to 1 and let the tenant scale the replicas 15:07:40 <lamt> \o sorry. I was running late 15:07:52 <puppetsock> reddy1: a long time ago thats where i was too 15:08:02 <puppetsock> but its an approach that leads to a lot of overhead 15:08:29 <puppetsock> 32 worker pods of 1 thread each != 1 pod of 32 threads (and the linear between them) 15:08:53 <puppetsock> that said, this sort of automagic worker calc, also makes me a bit worried 15:08:57 <sangeet> I agree .. it should not be auto. Can this be overwritten? 15:09:21 <puppetsock> the only time i think it really adds value is when you have nodes of diff cpu thread counts in your cluster 15:09:37 <puppetsock> in which case, you'll have some pods running x threads and others running y 15:09:37 <sangeet> I mean if it is auto, we can still overwrite it, correct 15:09:46 <reddy1> I agree with the auto calculation as well. Moreover, nproc returns the number of cpus on the pods, not from the pod resource limits 15:10:05 <reddy1> * Moreover, nproc returns the number of cpus on the host, not from the pod resource limits 15:11:31 <puppetsock> is the rationalle behind this ps in the commit somewhere 15:11:52 <puppetsock> i think that would be fantastic to have, it may be trying to solve a issue we have not thought of 15:11:59 <gagehugo> puppetsock: I asked why in the ps I believe 15:12:09 <miniroy> o/ sorry I was running late also 15:12:16 <gagehugo> I am dubious about hard-coding n/4 into the chart as "auto" 15:12:29 <gagehugo> what if we wanna do n/2? or n/8? 15:12:33 <puppetsock> i see your comment gagehugo 15:12:45 <puppetsock> id push to the an answer to your 1st q in your comment 15:12:59 <reddy1> @Sangeet, yes auto can be overwritten but having this as default may not be optimal solution 15:13:07 <lamt> miniroy: I thought you saw issues before with this autoscaling? 15:13:19 <lamt> something about talking to rabbit? 15:13:48 <lamt> s/autoscaling/# of worker threads/ 15:14:52 <gagehugo> I don't think this change will do what they think it will 15:15:17 <puppetsock> gagehugo: i assume you thinking that its intended to allow 'hands free' deployment of osh 15:15:20 <miniroy> lamt: yeah there is still that rabbitmq issue when nova osapi workers are set to > 1 15:15:24 <puppetsock> and have it right size to the env? 15:15:34 <gagehugo> puppetsock: yeah, I am assuming that is the intended goal 15:15:45 <puppetsock> in which case i agree 15:15:51 <puppetsock> i think to reddy1's point above 15:15:59 <puppetsock> it should scale to the resources on the pod 15:16:06 <puppetsock> rather than the underlying host in that case 15:16:27 <puppetsock> eg - if you allocate 2vcpu you get a multiple of threads 15:16:53 <puppetsock> would also be much easier/cleaner to implement 15:17:00 <puppetsock> not requireing init containers etc 15:17:12 <puppetsock> but could be done purely through the config template 15:19:18 <gagehugo> yeah 15:20:32 <gagehugo> but also miniroy's point still stands (for nova anyway) 15:20:48 <gagehugo> bumping the worker count has unintended side-effects with oslo.messaging 15:22:17 <gagehugo> There is also a ps for cinder as well for this 15:22:19 <gagehugo> #link https://review.opendev.org/c/openstack/openstack-helm/+/802015 15:23:32 <puppetsock> id provide some fb in the reviews with a summary or link to this meeting 15:24:09 <gagehugo> sure 15:24:26 <puppetsock> also miniroy it may be worth updating the docs, or putting a comment in the values yaml 15:24:32 <puppetsock> for what you have observed 15:24:43 <puppetsock> when workers > 1 15:25:14 <miniroy> puppetsock: yeah will probably do that, we were just noticing and investigating this issue recently 15:25:24 <puppetsock> nice 15:25:51 <miniroy> seeing lots of connection timeout messages in rabbitmq when more than 1 worker is defined 15:26:17 <miniroy> seems something is interfering with the heartbeat message is we are guessing but not overly certain on the reason 15:27:59 <gagehugo> yeah, we might wanna add a comment then if that turns out to be an issue 15:28:37 <gagehugo> #topic Yoga PTG 15:28:59 <gagehugo> So the sign up for timeslots for the next PTG is available, does this timeslot work for everyone? 15:29:08 <gagehugo> It's what we've done in the past 15:29:16 <gagehugo> just use the meeting time now instead for the PTG 15:29:36 <miniroy> works for me 15:32:27 <gagehugo> We can verify next week as well and get signed up 15:32:37 <gagehugo> #topic open discussion 15:32:44 <gagehugo> Anyone have anything else they wanna discuss? 15:33:32 <reddy1> Thank you for the discussion around the workers count. Appreciate it. 15:41:00 <opendevreview> Merged openstack/openstack-helm-infra master: [ceph-osd] Mount /var/crash inside ceph-osd pods https://review.opendev.org/c/openstack/openstack-helm-infra/+/801536 18:12:08 <opendevreview> Gage Hugo proposed openstack/openstack-helm master: Use bitnami memcached chart https://review.opendev.org/c/openstack/openstack-helm/+/802461 18:13:24 <opendevreview> chinasubbareddy mallavarapu proposed openstack/openstack-helm master: [OSH] Add make utlity to list of required tools https://review.opendev.org/c/openstack/openstack-helm/+/802595 18:17:33 <opendevreview> Gage Hugo proposed openstack/openstack-helm master: Use bitnami memcached chart https://review.opendev.org/c/openstack/openstack-helm/+/802461 18:26:05 <opendevreview> chinasubbareddy mallavarapu proposed openstack/openstack-helm-infra master: [ceph-provisioner] Add ceph mon v2 port for ceph csi provisioner https://review.opendev.org/c/openstack/openstack-helm-infra/+/802597 18:34:05 <opendevreview> Nafiz Haider proposed openstack/openstack-helm master: [nova-novnc] Mount rabbitmq tls certs for nova-novnc https://review.opendev.org/c/openstack/openstack-helm/+/802598 18:50:48 <opendevreview> Christopher Straut proposed openstack/openstack-helm-images master: Add retry around Nagios Request https://review.opendev.org/c/openstack/openstack-helm-images/+/802600 18:52:26 <opendevreview> Nafiz Haider proposed openstack/openstack-helm master: [nova-novnc] Mount rabbitmq tls certs for nova-novnc https://review.opendev.org/c/openstack/openstack-helm/+/802598 18:52:31 <opendevreview> Sangeet Gupta proposed openstack/openstack-helm-infra master: cert-rotation: New chart for certificate rotation https://review.opendev.org/c/openstack/openstack-helm-infra/+/788061 19:01:05 <opendevreview> chinasubbareddy mallavarapu proposed openstack/openstack-helm-infra master: [ceph-provisioner] Add check for empty ceph endpoint https://review.opendev.org/c/openstack/openstack-helm-infra/+/802605 19:02:28 <opendevreview> Sangeet Gupta proposed openstack/openstack-helm-infra master: cert-rotation: New chart for certificate rotation https://review.opendev.org/c/openstack/openstack-helm-infra/+/788061 19:17:11 <opendevreview> chinasubbareddy mallavarapu proposed openstack/openstack-helm-infra master: [ceph-provisioner] Add check for empty ceph endpoint https://review.opendev.org/c/openstack/openstack-helm-infra/+/802605 19:19:25 <opendevreview> Christopher Straut proposed openstack/openstack-helm-images master: Add retry around Nagios Request https://review.opendev.org/c/openstack/openstack-helm-images/+/802600 19:29:21 <opendevreview> Nafiz Haider proposed openstack/openstack-helm master: [nova-novnc] Mount rabbitmq tls cert for nova-novnc communication https://review.opendev.org/c/openstack/openstack-helm/+/802598 20:20:02 <opendevreview> Sangeet Gupta proposed openstack/openstack-helm-infra master: Helm-Toolkit: Make Rabbit-init job more robust https://review.opendev.org/c/openstack/openstack-helm-infra/+/800097 21:02:41 <opendevreview> Nafiz Haider proposed openstack/openstack-helm master: Mount rabbitmq tls cert for nova-novnc communication https://review.opendev.org/c/openstack/openstack-helm/+/802598 03:49:25 <opendevreview> zhen proposed openstack/openstack-helm-infra master: Modify the rbac_role tpl to make secrets accessible https://review.opendev.org/c/openstack/openstack-helm-infra/+/802434 07:05:01 <opendevreview> Oleksandr Kozachenko proposed openstack/openstack-helm master: add monasca(WIP) https://review.opendev.org/c/openstack/openstack-helm/+/771465 07:08:50 <opendevreview> Oleksandr Kozachenko proposed openstack/openstack-helm master: add monasca(WIP) https://review.opendev.org/c/openstack/openstack-helm/+/771465 08:01:30 <opendevreview> Oleksandr Kozachenko proposed openstack/openstack-helm master: add monasca(WIP) https://review.opendev.org/c/openstack/openstack-helm/+/771465 08:45:25 <opendevreview> Oleksandr Kozachenko proposed openstack/openstack-helm master: add monasca(WIP) https://review.opendev.org/c/openstack/openstack-helm/+/771465 09:16:53 <opendevreview> Susanta gautam proposed openstack/openstack-helm master: Set default workers to one by default https://review.opendev.org/c/openstack/openstack-helm/+/802672 09:38:21 <opendevreview> Oleksandr Kozachenko proposed openstack/openstack-helm master: add monasca(WIP) https://review.opendev.org/c/openstack/openstack-helm/+/771465 10:12:57 <opendevreview> Susanta gautam proposed openstack/openstack-helm master: Set wsgi daemon workers count to 1 in default https://review.opendev.org/c/openstack/openstack-helm/+/802677 11:38:19 <opendevreview> chinasubbareddy mallavarapu proposed openstack/openstack-helm master: [OSH] Add make utlity to list of required tools https://review.opendev.org/c/openstack/openstack-helm/+/802595 11:58:26 <opendevreview> Oleksandr Kozachenko proposed openstack/openstack-helm master: add monasca(WIP) https://review.opendev.org/c/openstack/openstack-helm/+/771465 14:41:42 <opendevreview> Gage Hugo proposed openstack/openstack-helm master: Test gate https://review.opendev.org/c/openstack/openstack-helm/+/802474 14:50:19 <opendevreview> Gage Hugo proposed openstack/openstack-helm master: Test gate https://review.opendev.org/c/openstack/openstack-helm/+/802474 14:53:30 <mnaser> gagehugo: do you know the reasoning behind using a single ip address for memcache instead of something like setting up a sts and providing the ip of all N pods (like rabbbit) 14:53:37 <opendevreview> Cliff Parsons proposed openstack/openstack-helm-infra master: Limit Ceph Provisioner Container Security Contexts https://review.opendev.org/c/openstack/openstack-helm-infra/+/802734 14:54:00 <mnaser> i'm seeing issues in horizon where the session acts weird because we're using cached_db and one pod might actually not have the same context as others because the cache is built as a write through 14:54:17 <opendevreview> Jagan Mohan Kavva proposed openstack/openstack-helm master: [WIP] Fix neutron privileged issues https://review.opendev.org/c/openstack/openstack-helm/+/802735 14:54:26 <mnaser> if there's a need to have a single host, maybe we can add `mcrouter` .. 14:54:52 <gagehugo> I believe we just ran one pod for memcached to avoid issues like that 14:55:41 <opendevreview> Cliff Parsons proposed openstack/openstack-helm-infra master: Limit Ceph Provisioner Container Security Contexts https://review.opendev.org/c/openstack/openstack-helm-infra/+/802734 14:56:02 <puppetsock> mcrouter is what you gonna need if you want a cluster of memcaches 14:56:16 <puppetsock> for us untill now not been an issue 14:56:47 <puppetsock> as we only use it for keystone tokens, consoleauth, and some light caching of horizon with the db backend 14:57:20 <mnaser> gagehugo, puppetsock: yeah im thinking of bringing it down to a single replica. horizon is having problems because cached_db is a write-through cache so things might not show up on subsequent requests 14:57:32 <puppetsock> yup 14:57:41 <puppetsock> id say single replica is all we support for now 14:57:54 <puppetsock> though you can run different ones for each function 14:58:06 <mnaser> and we had some issues with keystone / oauth2 as well, but i think dropping to one replica and assuming k8s will do the ha of it i guess 14:58:09 <puppetsock> ie - a dedicated instance for horizon sessions etc 14:58:37 <puppetsock> exclusing horizon, this is exaclty how i viewed it 14:59:00 <puppetsock> 99% of openstack is well behaved and if memcached is down, just treats it as a cache miss 14:59:14 <puppetsock> so letting k8s keep a sinlge pod up, was fine 14:59:34 <mnaser> yeah i think i agree with that approach 14:59:35 <puppetsock> though bear in mind our clusters though very numerious, probably much lower traffic than a public facing service 14:59:56 <mnaser> it's probably still enough for a single memcached instance to handle imho even in much bigger public facing 15:00:15 <mnaser> cpu usage is minimal from what i see, and we have enough memory in our controllers to have a bigger cache 15:00:47 <mnaser> okay thanks for that insight 15:04:17 <opendevreview> Merged openstack/openstack-helm-infra master: Helm-Toolkit: Make Rabbit-init job more robust https://review.opendev.org/c/openstack/openstack-helm-infra/+/800097 15:05:41 <opendevreview> Gage Hugo proposed openstack/openstack-helm master: Test gate https://review.opendev.org/c/openstack/openstack-helm/+/802474 15:06:13 <opendevreview> Gage Hugo proposed openstack/openstack-helm master: Test gate https://review.opendev.org/c/openstack/openstack-helm/+/802474 15:23:32 <opendevreview> Gage Hugo proposed openstack/openstack-helm master: Test gate https://review.opendev.org/c/openstack/openstack-helm/+/802474 15:35:33 <opendevreview> Gage Hugo proposed openstack/openstack-helm master: Test gate https://review.opendev.org/c/openstack/openstack-helm/+/802474 16:07:21 <opendevreview> Gage Hugo proposed openstack/openstack-helm master: Test gate https://review.opendev.org/c/openstack/openstack-helm/+/802474 16:13:58 <mnaser> folks -- i've been trying to fix loci's gate for a while with a few attempts but im not able to figure it out :( 16:14:11 <mnaser> for some reason, it works in our local ci (and i bet yours works too), but it is an error with uwsgi 16:14:23 <mnaser> there was some opensuse related stuff that i dropped to speed it up, but nada 16:14:59 <mnaser> https://review.opendev.org/c/openstack/loci/+/801526 16:15:09 <mnaser> even osh images works? so i have no idea why its borken 16:15:32 <mnaser> 'Wheel failed to build\\n', 'uwsgi\\n', '\\x1b[91m+ exit 1\\n\\x1b[0m', 16:17:25 <gagehugo> hmm 16:23:07 <mnaser> it's also only focal that is failing to build 16:23:11 <mnaser> bionic / centos is ok 16:23:18 <mnaser> actually thats probably why we're not catching it 16:23:21 <mnaser> we're building bionic 16:24:23 <mnaser> focal uses python 3.8 and bionic uses 3.6 i wonder if that has to do with it 16:25:46 <mnaser> trying `pip wheel uwsgi` in a docker container for ubuntu:focal here 16:26:16 <mnaser> and it worked perfectly :X 16:28:25 <mnaser> `pip wheel --constraint https://raw.githubusercontent.com/openstack/requirements/master/upper-constraints.txt uwsgi` also works :L 16:28:49 <opendevreview> Gage Hugo proposed openstack/openstack-helm master: Remove indonesian translations https://review.opendev.org/c/openstack/openstack-helm/+/802474 16:29:47 <opendevreview> Gage Hugo proposed openstack/openstack-helm master: Remove indonesian translations https://review.opendev.org/c/openstack/openstack-helm/+/802474 16:30:14 <gagehugo> miniroy lamt https://review.opendev.org/c/openstack/openstack-helm/+/802474 16:55:19 <opendevreview> Merged openstack/openstack-helm master: Remove indonesian translations https://review.opendev.org/c/openstack/openstack-helm/+/802474 16:57:19 <opendevreview> Jagan Mohan Kavva proposed openstack/openstack-helm-infra master: [WIP] Fix ceph-osd privileged issues https://review.opendev.org/c/openstack/openstack-helm-infra/+/802759 17:07:29 <opendevreview> Nafiz Haider proposed openstack/openstack-helm master: Mount rabbitmq tls cert for nova-novnc communication https://review.opendev.org/c/openstack/openstack-helm/+/802598 18:13:38 <opendevreview> chinasubbareddy mallavarapu proposed openstack/openstack-helm-infra master: [ceph-provisioner] Add check for empty ceph endpoint https://review.opendev.org/c/openstack/openstack-helm-infra/+/802605 18:36:09 <opendevreview> chinasubbareddy mallavarapu proposed openstack/openstack-helm master: [OSH] Add make utlity to list of required tools https://review.opendev.org/c/openstack/openstack-helm/+/802595 18:36:43 <chinasubbareddy> can i please get reviews on this ps ? https://review.opendev.org/c/openstack/openstack-helm/+/802595 19:20:40 <opendevreview> Jagan Mohan Kavva proposed openstack/openstack-helm master: [WIP] Fix cinder privileged issues https://review.opendev.org/c/openstack/openstack-helm/+/802797 19:28:46 <opendevreview> Jagan Mohan Kavva proposed openstack/openstack-helm master: [WIP] Fix nova-compute privileged issues https://review.opendev.org/c/openstack/openstack-helm/+/802799 19:33:21 <opendevreview> Oleksandr Kozachenko proposed openstack/openstack-helm master: add monasca(WIP) https://review.opendev.org/c/openstack/openstack-helm/+/771465 19:33:23 <opendevreview> Merged openstack/openstack-helm master: Mount rabbitmq tls cert for nova-novnc communication https://review.opendev.org/c/openstack/openstack-helm/+/802598 19:33:25 <opendevreview> Merged openstack/openstack-helm master: [OSH] Add make utlity to list of required tools https://review.opendev.org/c/openstack/openstack-helm/+/802595 03:08:34 <opendevreview> zhen proposed openstack/openstack-helm-infra master: Modify the rbac_role to make secrets accessible https://review.opendev.org/c/openstack/openstack-helm-infra/+/802434 03:48:35 <opendevreview> Merged openstack/openstack-helm master: Added post-install and post-upgrade hook for jobs. https://review.opendev.org/c/openstack/openstack-helm/+/782409 03:48:36 <opendevreview> Merged openstack/openstack-helm master: Remove member bootstrap logic https://review.opendev.org/c/openstack/openstack-helm/+/801015 04:01:55 <opendevreview> zhen proposed openstack/openstack-helm-infra master: Fix problems in commit 'Realize libvirt SSL' https://review.opendev.org/c/openstack/openstack-helm-infra/+/802822 07:38:28 <opendevreview> OpenStack Proposal Bot proposed openstack/openstack-helm master: Imported Translations from Zanata https://review.opendev.org/c/openstack/openstack-helm/+/802832 12:23:27 <opendevreview> chinasubbareddy mallavarapu proposed openstack/openstack-helm-infra master: [ceph-provisioner] Add check for empty ceph endpoint https://review.opendev.org/c/openstack/openstack-helm-infra/+/802605 13:46:07 <opendevreview> Gage Hugo proposed openstack/openstack-helm-images master: Added Victoria & Wallaby images https://review.opendev.org/c/openstack/openstack-helm-images/+/786637 14:54:52 <opendevreview> Cliff Parsons proposed openstack/openstack-helm-infra master: Limit Ceph Provisioner Container Security Contexts https://review.opendev.org/c/openstack/openstack-helm-infra/+/802734 15:04:20 <opendevreview> diwakar thyagaraj proposed openstack/openstack-helm-infra master: [WIP] Add Apparmor to Calico https://review.opendev.org/c/openstack/openstack-helm-infra/+/743050 15:23:25 <opendevreview> Frank Ritchie proposed openstack/openstack-helm-infra master: [WIP] Remove locked aliases https://review.opendev.org/c/openstack/openstack-helm-infra/+/802906 15:53:14 <opendevreview> Merged openstack/openstack-helm-infra master: [ceph-provisioner] Add check for empty ceph endpoint https://review.opendev.org/c/openstack/openstack-helm-infra/+/802605 17:42:47 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: Added Victoria & Wallaby images https://review.opendev.org/c/openstack/openstack-helm-images/+/786637 18:30:55 <opendevreview> Gage Hugo proposed openstack/openstack-helm master: Modify key repo file permissions https://review.opendev.org/c/openstack/openstack-helm/+/799101 18:36:03 <opendevreview> Cliff Parsons proposed openstack/openstack-helm-infra master: Limit Ceph Provisioner Container Security Contexts https://review.opendev.org/c/openstack/openstack-helm-infra/+/802734 18:36:05 <opendevreview> Maximilian Weiss proposed openstack/openstack-helm-infra master: Revoke connect from PUBLIC on postgres dbs https://review.opendev.org/c/openstack/openstack-helm-infra/+/802925 18:36:17 <opendevreview> Gage Hugo proposed openstack/openstack-helm master: Modify key repo file permissions https://review.opendev.org/c/openstack/openstack-helm/+/799101 18:36:35 <opendevreview> Gage Hugo proposed openstack/openstack-helm master: Modify key repo file permissions https://review.opendev.org/c/openstack/openstack-helm/+/799101 18:38:11 <opendevreview> Gage Hugo proposed openstack/openstack-helm master: Modify key repo file permissions https://review.opendev.org/c/openstack/openstack-helm/+/799101 18:44:44 <opendevreview> Maximilian Weiss proposed openstack/openstack-helm-infra master: Revoke connect on PUBLIC role in postgres dbs https://review.opendev.org/c/openstack/openstack-helm-infra/+/802925 20:01:06 <opendevreview> Gage Hugo proposed openstack/openstack-helm master: Modify key repo file permissions https://review.opendev.org/c/openstack/openstack-helm/+/799101 20:23:05 <opendevreview> Cliff Parsons proposed openstack/openstack-helm-infra master: Limit Ceph OSD Container Security Contexts https://review.opendev.org/c/openstack/openstack-helm-infra/+/802959 20:24:53 <opendevreview> Cliff Parsons proposed openstack/openstack-helm-infra master: Limit Ceph OSD Container Security Contexts https://review.opendev.org/c/openstack/openstack-helm-infra/+/802959 20:25:43 <opendevreview> Cliff Parsons proposed openstack/openstack-helm-infra master: Limit Ceph Provisioner Container Security Contexts https://review.opendev.org/c/openstack/openstack-helm-infra/+/802734 22:12:43 <opendevreview> Gage Hugo proposed openstack/openstack-helm master: Use bitnami memcached chart https://review.opendev.org/c/openstack/openstack-helm/+/802461 22:28:27 <opendevreview> Gage Hugo proposed openstack/openstack-helm master: Modify key repo file permissions https://review.opendev.org/c/openstack/openstack-helm/+/799101 22:28:32 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: Added Victoria & Wallaby images https://review.opendev.org/c/openstack/openstack-helm-images/+/786637 22:28:40 <opendevreview> Gage Hugo proposed openstack/openstack-helm master: Modify key repo file permissions https://review.opendev.org/c/openstack/openstack-helm/+/799101 03:19:00 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm master: Add victoria and wallaby compute-kit and cinder jobs https://review.opendev.org/c/openstack/openstack-helm/+/802984 03:21:52 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: Added Victoria & Wallaby images https://review.opendev.org/c/openstack/openstack-helm-images/+/786637 03:24:12 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: Added Victoria & Wallaby images https://review.opendev.org/c/openstack/openstack-helm-images/+/786637 04:13:36 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: Added Victoria & Wallaby images https://review.opendev.org/c/openstack/openstack-helm-images/+/786637 04:16:05 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: Added Victoria & Wallaby images https://review.opendev.org/c/openstack/openstack-helm-images/+/786637 04:23:30 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm master: Add victoria and wallaby compute-kit and cinder jobs https://review.opendev.org/c/openstack/openstack-helm/+/802984 04:26:01 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: Added Victoria & Wallaby images https://review.opendev.org/c/openstack/openstack-helm-images/+/786637 14:20:08 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm master: Add victoria and wallaby compute-kit and cinder jobs https://review.opendev.org/c/openstack/openstack-helm/+/802984 14:28:48 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm master: Add victoria and wallaby compute-kit and cinder jobs https://review.opendev.org/c/openstack/openstack-helm/+/802984 14:37:46 <opendevreview> Darren DeJaeger proposed openstack/openstack-helm-infra master: Update mon-check with latest monmap outputs https://review.opendev.org/c/openstack/openstack-helm-infra/+/803044 14:47:52 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm master: Add victoria and wallaby compute-kit and cinder jobs https://review.opendev.org/c/openstack/openstack-helm/+/802984 14:52:05 <opendevreview> Darren DeJaeger proposed openstack/openstack-helm-infra master: Update mon-check with latest monmap outputs https://review.opendev.org/c/openstack/openstack-helm-infra/+/803044 16:24:29 <opendevreview> Darren DeJaeger proposed openstack/openstack-helm-infra master: Update mon-check with latest monmap outputs https://review.opendev.org/c/openstack/openstack-helm-infra/+/803044 16:30:05 <opendevreview> Darren DeJaeger proposed openstack/openstack-helm-infra master: Update mon-check with latest monmap outputs https://review.opendev.org/c/openstack/openstack-helm-infra/+/803044 17:03:43 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm master: Add victoria and wallaby compute-kit and cinder jobs https://review.opendev.org/c/openstack/openstack-helm/+/802984 17:21:26 <opendevreview> Maximilian Weiss proposed openstack/openstack-helm-infra master: Revoke all privileges for PUBLIC role in postgres dbs https://review.opendev.org/c/openstack/openstack-helm-infra/+/802925 19:02:31 <opendevreview> Christopher Straut proposed openstack/openstack-helm-images master: Add retry around Nagios Request https://review.opendev.org/c/openstack/openstack-helm-images/+/802600 20:10:30 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm master: Add victoria and wallaby compute-kit and cinder jobs https://review.opendev.org/c/openstack/openstack-helm/+/802984 20:21:17 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm master: Add victoria and wallaby compute-kit and cinder jobs https://review.opendev.org/c/openstack/openstack-helm/+/802984 22:37:17 <opendevreview> Merged openstack/openstack-helm-infra master: Limit Ceph Provisioner Container Security Contexts https://review.opendev.org/c/openstack/openstack-helm-infra/+/802734 22:44:19 <opendevreview> Merged openstack/openstack-helm-infra master: Limit Ceph OSD Container Security Contexts https://review.opendev.org/c/openstack/openstack-helm-infra/+/802959 22:53:05 <opendevreview> Merged openstack/openstack-helm-infra master: Revoke all privileges for PUBLIC role in postgres dbs https://review.opendev.org/c/openstack/openstack-helm-infra/+/802925 03:12:51 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-infra master: Use focal libvirt image for victoria and wallaby https://review.opendev.org/c/openstack/openstack-helm-infra/+/803095 03:13:46 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: Added Victoria & Wallaby images https://review.opendev.org/c/openstack/openstack-helm-images/+/786637 03:22:38 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm master: Add victoria and wallaby compute-kit and cinder jobs https://review.opendev.org/c/openstack/openstack-helm/+/802984 03:24:42 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm master: Add victoria and wallaby compute-kit and cinder jobs https://review.opendev.org/c/openstack/openstack-helm/+/802984 16:58:21 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm master: Add victoria and wallaby compute-kit and cinder jobs https://review.opendev.org/c/openstack/openstack-helm/+/802984 20:42:16 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm master: Add victoria and wallaby compute-kit and cinder jobs https://review.opendev.org/c/openstack/openstack-helm/+/802984 20:54:13 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-infra master: Use focal libvirt image for victoria and wallaby https://review.opendev.org/c/openstack/openstack-helm-infra/+/803095 23:09:40 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm master: Add victoria and wallaby compute-kit and cinder jobs https://review.opendev.org/c/openstack/openstack-helm/+/802984 02:32:32 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm master: Add victoria and wallaby compute-kit and cinder jobs https://review.opendev.org/c/openstack/openstack-helm/+/802984 02:33:45 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm master: Add victoria and wallaby compute-kit and cinder jobs https://review.opendev.org/c/openstack/openstack-helm/+/802984 07:28:05 <opendevreview> Phil Sphicas proposed openstack/openstack-helm-infra master: namespace-config: Grant access to existing PSP https://review.opendev.org/c/openstack/openstack-helm-infra/+/803114 01:36:57 <opendevreview> Phil Sphicas proposed openstack/openstack-helm-infra master: namespace-config: Grant access to existing PSP https://review.opendev.org/c/openstack/openstack-helm-infra/+/803114 02:10:42 <opendevreview> zhen proposed openstack/openstack-helm-infra master: Modify the rbac_role to make secrets accessible https://review.opendev.org/c/openstack/openstack-helm-infra/+/802434 08:03:30 <opendevreview> Eric Xie proposed openstack/openstack-helm master: Add missing slash https://review.opendev.org/c/openstack/openstack-helm/+/803149 10:39:37 <sgautam> Hello, As some of the modules have the different process count, i mean like for neutron, https://opendev.org/openstack/openstack-helm/src/branch/master/neutron/values.yaml#L1880-L1881 we have 1 and 4. I think it will be good to have the workers count consistent across all the modules with the default value of 1. What do you think? Can you please give suggestion, as i am not sure about it as it does not have functionality 10:39:37 <sgautam> impact. 12:58:46 <opendevreview> Darren DeJaeger proposed openstack/openstack-helm master: Nova bootstrap job efficiency https://review.opendev.org/c/openstack/openstack-helm/+/803204 13:00:10 <opendevreview> Darren DeJaeger proposed openstack/openstack-helm master: Nova bootstrap job efficiency https://review.opendev.org/c/openstack/openstack-helm/+/803204 13:16:40 <opendevreview> chinasubbareddy mallavarapu proposed openstack/openstack-helm-infra master: Update mon-check with latest monmap outputs https://review.opendev.org/c/openstack/openstack-helm-infra/+/803044 14:15:41 <opendevreview> Darren DeJaeger proposed openstack/openstack-helm master: Nova bootstrap job efficiency https://review.opendev.org/c/openstack/openstack-helm/+/803204 14:32:32 <opendevreview> Darren DeJaeger proposed openstack/openstack-helm master: Nova bootstrap job efficiency https://review.opendev.org/c/openstack/openstack-helm/+/803204 14:53:42 <opendevreview> Gage Hugo proposed openstack/openstack-helm master: Modify key repo file permissions https://review.opendev.org/c/openstack/openstack-helm/+/799101 15:06:23 <gagehugo> sgautam: 1 is probably fine 15:46:09 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm master: Add support for Victoria and Wallaby https://review.opendev.org/c/openstack/openstack-helm/+/802984 15:46:39 <opendevreview> Merged openstack/openstack-helm-infra master: Update mon-check with latest monmap outputs https://review.opendev.org/c/openstack/openstack-helm-infra/+/803044 17:11:40 <opendevreview> Merged openstack/openstack-helm-images master: Add retry around Nagios Request https://review.opendev.org/c/openstack/openstack-helm-images/+/802600 17:39:48 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: Fix periodic pipeline and run ussuri test deployments https://review.opendev.org/c/openstack/openstack-helm-images/+/803239 18:36:32 <opendevreview> Merged openstack/openstack-helm-infra master: Use focal libvirt image for victoria and wallaby https://review.opendev.org/c/openstack/openstack-helm-infra/+/803095 18:39:28 <opendevreview> Jagan Mohan Kavva proposed openstack/openstack-helm-infra master: [WIP] Fix elasticsearch privileged issues https://review.opendev.org/c/openstack/openstack-helm-infra/+/803247 18:50:42 <opendevreview> Jagan Mohan Kavva proposed openstack/openstack-helm-infra master: [WIP] Fix libvirt privileged issues https://review.opendev.org/c/openstack/openstack-helm-infra/+/803249 19:07:33 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: Fix periodic pipeline and run Ussuri test deployments https://review.opendev.org/c/openstack/openstack-helm-images/+/803239 19:44:06 <opendevreview> Chi Lo proposed openstack/openstack-helm-infra master: Add Alertmanager dashboard to Grafana https://review.opendev.org/c/openstack/openstack-helm-infra/+/803258 19:52:31 <opendevreview> Chi Lo proposed openstack/openstack-helm-infra master: Add Alertmanager dashboard to Grafana https://review.opendev.org/c/openstack/openstack-helm-infra/+/803258 19:52:53 <opendevreview> Darren DeJaeger proposed openstack/openstack-helm master: Nova bootstrap job efficiency https://review.opendev.org/c/openstack/openstack-helm/+/803204 20:17:53 <opendevreview> Chi Lo proposed openstack/openstack-helm-infra master: Add Alertmanager dashboard to Grafana https://review.opendev.org/c/openstack/openstack-helm-infra/+/803258 23:05:58 <opendevreview> Chi Lo proposed openstack/openstack-helm-infra master: Add Alertmanager dashboard to Grafana https://review.opendev.org/c/openstack/openstack-helm-infra/+/803258 23:59:31 <opendevreview> Merged openstack/openstack-helm master: Add support for Victoria and Wallaby https://review.opendev.org/c/openstack/openstack-helm/+/802984 00:44:58 <opendevreview> Merged openstack/openstack-helm-infra master: namespace-config: Grant access to existing PSP https://review.opendev.org/c/openstack/openstack-helm-infra/+/803114 01:11:49 <opendevreview> Darren DeJaeger proposed openstack/openstack-helm master: Nova bootstrap job efficiency https://review.opendev.org/c/openstack/openstack-helm/+/803204 01:19:40 <opendevreview> Darren DeJaeger proposed openstack/openstack-helm master: Nova bootstrap job efficiency https://review.opendev.org/c/openstack/openstack-helm/+/803204 01:33:02 <opendevreview> zhou jielei proposed openstack/openstack-helm master: add new glance multi_backend support. The old glance_store.stores and store.default_store was be deprecated and remove form rocky. So this is a update for config file. https://review.opendev.org/c/openstack/openstack-helm/+/803298 01:37:30 <opendevreview> zhou jielei proposed openstack/openstack-helm master: fix some bug https://review.opendev.org/c/openstack/openstack-helm/+/803299 01:46:56 <opendevreview> Jagan Mohan Kavva proposed openstack/openstack-helm-infra master: [WIP] Fix node-problem-detector privileged issues https://review.opendev.org/c/openstack/openstack-helm-infra/+/803300 01:54:05 <opendevreview> zhou jielei proposed openstack/openstack-helm master: add new glance multi_backend support. The old glance_store.stores and store.default_store was be deprecated and remove form rocky. So this is a update for config file. https://review.opendev.org/c/openstack/openstack-helm/+/803298 02:03:20 <opendevreview> Merged openstack/openstack-helm-images master: Added Victoria & Wallaby images https://review.opendev.org/c/openstack/openstack-helm-images/+/786637 02:17:45 <opendevreview> Merged openstack/openstack-helm-images master: Fix periodic pipeline and run Ussuri test deployments https://review.opendev.org/c/openstack/openstack-helm-images/+/803239 02:18:46 <opendevreview> zhou jielei proposed openstack/openstack-helm master: add multi backend support for after Train. The old glance_store.stores and store.default_store was be deprecated and remove form rocky. So this is an update for config file. https://review.opendev.org/c/openstack/openstack-helm/+/803302 03:08:51 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: [test] https://review.opendev.org/c/openstack/openstack-helm-images/+/803303 03:12:49 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: [test] https://review.opendev.org/c/openstack/openstack-helm-images/+/803303 03:20:27 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: [test] https://review.opendev.org/c/openstack/openstack-helm-images/+/803303 03:27:14 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: [test] https://review.opendev.org/c/openstack/openstack-helm-images/+/803303 03:31:27 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: [test] https://review.opendev.org/c/openstack/openstack-helm-images/+/803303 03:36:18 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: [test] https://review.opendev.org/c/openstack/openstack-helm-images/+/803303 03:46:25 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: [test] https://review.opendev.org/c/openstack/openstack-helm-images/+/803303 03:54:35 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: [test] https://review.opendev.org/c/openstack/openstack-helm-images/+/803303 04:12:03 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: [test] https://review.opendev.org/c/openstack/openstack-helm-images/+/803303 04:25:50 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: [test] https://review.opendev.org/c/openstack/openstack-helm-images/+/803303 04:32:30 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: [test] https://review.opendev.org/c/openstack/openstack-helm-images/+/803303 05:38:44 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: [test] https://review.opendev.org/c/openstack/openstack-helm-images/+/803303 07:51:37 <opendevreview> OpenStack Proposal Bot proposed openstack/openstack-helm master: Imported Translations from Zanata https://review.opendev.org/c/openstack/openstack-helm/+/802832 08:10:36 <opendevreview> Susanta gautam proposed openstack/openstack-helm master: Set wsgi daemon workers count to 1 in default https://review.opendev.org/c/openstack/openstack-helm/+/802677 08:16:19 <opendevreview> Susanta gautam proposed openstack/openstack-helm master: Change default workers for rpc_workers to one https://review.opendev.org/c/openstack/openstack-helm/+/802455 08:19:24 <opendevreview> Susanta gautam proposed openstack/openstack-helm master: Set default workers to one by default https://review.opendev.org/c/openstack/openstack-helm/+/802672 14:42:07 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: Update octopus ceph version to the one available in repos https://review.opendev.org/c/openstack/openstack-helm-images/+/803356 14:42:33 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: Update octopus ceph version to the one available in repos https://review.opendev.org/c/openstack/openstack-helm-images/+/803356 14:50:32 <opendevreview> Andrii Ostapenko proposed openstack/openstack-helm-images master: Update octopus ceph version to 15.2.13 available in repos https://review.opendev.org/c/openstack/openstack-helm-images/+/803356 15:03:07 <opendevmeet> gagehugo: Error: Can't start another meeting, one is in progress. Use #endmeeting first. 15:03:14 <gagehugo> que? 15:03:17 <gagehugo> #endmeeting