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 | 03:49 |
---|---|---|
opendevreview | Oleksandr Kozachenko proposed openstack/openstack-helm master: add monasca(WIP) https://review.opendev.org/c/openstack/openstack-helm/+/771465 | 07:05 |
opendevreview | Oleksandr Kozachenko proposed openstack/openstack-helm master: add monasca(WIP) https://review.opendev.org/c/openstack/openstack-helm/+/771465 | 07:08 |
opendevreview | Oleksandr Kozachenko proposed openstack/openstack-helm master: add monasca(WIP) https://review.opendev.org/c/openstack/openstack-helm/+/771465 | 08:01 |
opendevreview | Oleksandr Kozachenko proposed openstack/openstack-helm master: add monasca(WIP) https://review.opendev.org/c/openstack/openstack-helm/+/771465 | 08:45 |
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:16 |
opendevreview | Oleksandr Kozachenko proposed openstack/openstack-helm master: add monasca(WIP) https://review.opendev.org/c/openstack/openstack-helm/+/771465 | 09:38 |
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 | 10:12 |
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:38 |
opendevreview | Oleksandr Kozachenko proposed openstack/openstack-helm master: add monasca(WIP) https://review.opendev.org/c/openstack/openstack-helm/+/771465 | 11:58 |
*** reddy is now known as Guest2573 | 13:33 | |
opendevreview | Gage Hugo proposed openstack/openstack-helm master: Test gate https://review.opendev.org/c/openstack/openstack-helm/+/802474 | 14:41 |
opendevreview | Gage Hugo proposed openstack/openstack-helm master: Test gate https://review.opendev.org/c/openstack/openstack-helm/+/802474 | 14:50 |
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 |
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:53 |
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 |
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 |
mnaser | if there's a need to have a single host, maybe we can add `mcrouter` .. | 14:54 |
gagehugo | I believe we just ran one pod for memcached to avoid issues like that | 14:54 |
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:55 |
puppetsock | mcrouter is what you gonna need if you want a cluster of memcaches | 14:56 |
puppetsock | for us untill now not been an issue | 14:56 |
puppetsock | as we only use it for keystone tokens, consoleauth, and some light caching of horizon with the db backend | 14:56 |
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 |
puppetsock | yup | 14:57 |
puppetsock | id say single replica is all we support for now | 14:57 |
puppetsock | though you can run different ones for each function | 14:57 |
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 |
puppetsock | ie - a dedicated instance for horizon sessions etc | 14:58 |
puppetsock | exclusing horizon, this is exaclty how i viewed it | 14:58 |
puppetsock | 99% of openstack is well behaved and if memcached is down, just treats it as a cache miss | 14:59 |
puppetsock | so letting k8s keep a sinlge pod up, was fine | 14:59 |
mnaser | yeah i think i agree with that approach | 14:59 |
puppetsock | though bear in mind our clusters though very numerious, probably much lower traffic than a public facing service | 14:59 |
mnaser | it's probably still enough for a single memcached instance to handle imho even in much bigger public facing | 14:59 |
mnaser | cpu usage is minimal from what i see, and we have enough memory in our controllers to have a bigger cache | 15:00 |
mnaser | okay thanks for that insight | 15:00 |
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:04 |
opendevreview | Gage Hugo proposed openstack/openstack-helm master: Test gate https://review.opendev.org/c/openstack/openstack-helm/+/802474 | 15:05 |
opendevreview | Gage Hugo proposed openstack/openstack-helm master: Test gate https://review.opendev.org/c/openstack/openstack-helm/+/802474 | 15:06 |
opendevreview | Gage Hugo proposed openstack/openstack-helm master: Test gate https://review.opendev.org/c/openstack/openstack-helm/+/802474 | 15:23 |
opendevreview | Gage Hugo proposed openstack/openstack-helm master: Test gate https://review.opendev.org/c/openstack/openstack-helm/+/802474 | 15:35 |
opendevreview | Gage Hugo proposed openstack/openstack-helm master: Test gate https://review.opendev.org/c/openstack/openstack-helm/+/802474 | 16:07 |
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:13 |
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 |
mnaser | there was some opensuse related stuff that i dropped to speed it up, but nada | 16:14 |
mnaser | https://review.opendev.org/c/openstack/loci/+/801526 | 16:14 |
mnaser | even osh images works? so i have no idea why its borken | 16:15 |
mnaser | 'Wheel failed to build\\n', 'uwsgi\\n', '\\x1b[91m+ exit 1\\n\\x1b[0m', | 16:15 |
gagehugo | hmm | 16:17 |
mnaser | it's also only focal that is failing to build | 16:23 |
mnaser | bionic / centos is ok | 16:23 |
mnaser | actually thats probably why we're not catching it | 16:23 |
mnaser | we're building bionic | 16:23 |
mnaser | focal uses python 3.8 and bionic uses 3.6 i wonder if that has to do with it | 16:24 |
mnaser | trying `pip wheel uwsgi` in a docker container for ubuntu:focal here | 16:25 |
mnaser | and it worked perfectly :X | 16:26 |
mnaser | `pip wheel --constraint https://raw.githubusercontent.com/openstack/requirements/master/upper-constraints.txt uwsgi` also works :L | 16:28 |
opendevreview | Gage Hugo proposed openstack/openstack-helm master: Remove indonesian translations https://review.opendev.org/c/openstack/openstack-helm/+/802474 | 16:28 |
opendevreview | Gage Hugo proposed openstack/openstack-helm master: Remove indonesian translations https://review.opendev.org/c/openstack/openstack-helm/+/802474 | 16:29 |
gagehugo | miniroy lamt https://review.opendev.org/c/openstack/openstack-helm/+/802474 | 16:30 |
opendevreview | Merged openstack/openstack-helm master: Remove indonesian translations https://review.opendev.org/c/openstack/openstack-helm/+/802474 | 16:55 |
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 | 16:57 |
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 | 17:07 |
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:13 |
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 |
chinasubbareddy | can i please get reviews on this ps ? https://review.opendev.org/c/openstack/openstack-helm/+/802595 | 18:36 |
opendevreview | Jagan Mohan Kavva proposed openstack/openstack-helm master: [WIP] Fix cinder privileged issues https://review.opendev.org/c/openstack/openstack-helm/+/802797 | 19:20 |
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:28 |
opendevreview | Oleksandr Kozachenko proposed openstack/openstack-helm master: add monasca(WIP) https://review.opendev.org/c/openstack/openstack-helm/+/771465 | 19:33 |
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 |
opendevreview | Merged openstack/openstack-helm master: [OSH] Add make utlity to list of required tools https://review.opendev.org/c/openstack/openstack-helm/+/802595 | 19:33 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!