opendevreview | Buddhika Sanjeewa proposed openstack/kolla-ansible master: Deploy Zun with Cinder Ceph support https://review.opendev.org/c/openstack/kolla-ansible/+/824722 | 00:17 |
---|---|---|
opendevreview | Imran Hussain proposed openstack/kolla-ansible master: [external-ceph] Use template instead of copy https://review.opendev.org/c/openstack/kolla-ansible/+/824566 | 09:30 |
opendevreview | Imran Hussain proposed openstack/kolla-ansible master: [external-ceph] Use template instead of copy https://review.opendev.org/c/openstack/kolla-ansible/+/824566 | 09:30 |
hrw | morning | 09:51 |
mnasiadka | morning | 10:26 |
holtgrewe | mgoddard: thanks for the reply | 11:06 |
holtgrewe | let me see what I can do | 11:06 |
mnasiadka | yoctozepto: got some minutes to look into reviewing https://review.opendev.org/c/openstack/kolla/+/826672 ? | 11:31 |
yoctozepto | mnasiadka: +2 | 11:36 |
*** cozyurt is now known as Guest1271 | 11:43 | |
opendevreview | Pierre Riteau proposed openstack/kayobe-config stable/xena: Sync kayobe-config with kayobe changes https://review.opendev.org/c/openstack/kayobe-config/+/827059 | 11:49 |
opendevreview | Pierre Riteau proposed openstack/kayobe-config stable/wallaby: Sync kayobe-config with kayobe changes https://review.opendev.org/c/openstack/kayobe-config/+/827060 | 11:49 |
opendevreview | Pierre Riteau proposed openstack/kayobe-config stable/victoria: Sync kayobe-config with kayobe changes https://review.opendev.org/c/openstack/kayobe-config/+/827061 | 11:50 |
opendevreview | Pierre Riteau proposed openstack/kayobe-config master: Sync kayobe-config with kayobe changes https://review.opendev.org/c/openstack/kayobe-config/+/827062 | 11:53 |
yuval | continue my question regarding env variable | 11:54 |
yuval | the cinder uses "sudo su" to run specific cmd | 11:54 |
yuval | when running sudo su - the env variable is different from the "cinder" env variable | 11:54 |
yuval | someone knows how I can edit the sudo env variable? | 11:55 |
yuval | is there a script I can hack before the service is starting? | 11:55 |
kevko | hi \o/, i wasn't here for a while ..vacationing etc :P | 12:09 |
kevko | yoctozepto: regarding my proxysql patch ..there is a comment why proxysql container has privileged: True , actually i was inspired by haproxy | 12:11 |
rockey | yuval: which command are you referring to? | 12:11 |
kevko | why is haproxy privileged :D ? | 12:11 |
kevko | yuval: just run container with one more ENV ? :D | 12:12 |
yuval | the os-brick connector check the env variable "VIRTUAL_ENV" while using "sudo su" | 12:12 |
yuval | in my cinder and root users I have the variable env | 12:12 |
yuval | but when doing "sudo su" I dont have this env variable | 12:13 |
kevko | yuval: docker run ........... -e 'WHATEVER_YOU_WANT_TO_SET=whatever' ? | 12:13 |
yuval | I am using kolla-ansible | 12:13 |
rockey | yuval: are you building your own images, or using upstream? | 12:14 |
kevko | yeah, an kolla ansible just run docker command :D | 12:14 |
kevko | you can change kolla-ansible restart handler in your branch locally and add environment: "whatever" | 12:14 |
rockey | if so, you can extend dockerfiles using your own ENV variables | 12:14 |
kevko | so when you will run reconfigure ..container will be recreated with whatever you want | 12:15 |
kevko | where is a problem ? | 12:15 |
yuval | ok, thank you I see | 12:15 |
rockey | yeah, wondering that too, since "sudo su" isn't runned anywhere, neither kolla nor kolla-ansible | 12:15 |
yuval | "sudo su" run by cinder | 12:15 |
rockey | not within kolla/kolla-ansible source | 12:16 |
yuval | yes, yes I agree | 12:16 |
rockey | is it recreatable? and how? | 12:16 |
yuval | i am not sure kolla-ansible/kolla is the problem here actually | 12:18 |
yuval | https://opendev.org/openstack/os-brick/src/branch/master/os_brick/privileged/__init__.py#L24 | 12:19 |
yuval | check this line - its doing: if os.environ.get('VIRTUAL_ENV'): | 12:19 |
kevko | yuval: https://paste.opendev.org/show/812448/ | 12:19 |
yuval | yes, I managed to add it - also by using group_var/all.yml | 12:21 |
yuval | my issue is that when cinder does check this env variable - its with sudo su - which run with "clean" env | 12:21 |
kevko | yuval: where you can see that it's via root ? | 12:22 |
kevko | yuval: do you have some log ? | 12:22 |
yuval | https://paste.opendev.org/show/812449/ | 12:25 |
yuval | I stopped with pdb just before this line | 12:25 |
yuval | and check the os.environ | 12:26 |
opendevreview | Verification of a change to openstack/kolla stable/train failed: Mitigate two Log4j vulnerabilities in Apache Storm https://review.opendev.org/c/openstack/kolla/+/826613 | 12:29 |
rockey | yuval: seems like an issue with os-brick rather than K-A, try kevko's solution or open up a bug with cinder/os-brick team? | 12:32 |
kevko | yoctozepto: here ? | 12:43 |
yuval | ok I found a solution | 12:51 |
yuval | in my cinder.conf | 12:52 |
yuval | I added: | 12:52 |
yuval | [privsep_osbrick] | 12:52 |
yuval | helper_command = sudo -E cinder-rootwrap /etc/cinder/rootwrap.conf privsep-helper --config-file /etc/cinder/cinder.conf | 12:52 |
yuval | the important part is the "sudo -E" | 12:52 |
yuval | this is after I added the "VIRUTAL_ENV" to the docker env | 12:53 |
yuval | btw - rocky and kevko - thanks for trying to help | 13:15 |
opendevreview | Merged openstack/kolla master: Ensure set_configs sets execute bit on directories https://review.opendev.org/c/openstack/kolla/+/826672 | 13:40 |
opendevreview | Jakub Darmach proposed openstack/kolla-ansible master: horizon: Support custom themes https://review.opendev.org/c/openstack/kolla-ansible/+/761364 | 14:16 |
opendevreview | Buddhika Sanjeewa proposed openstack/kolla-ansible master: Deploy Zun with Cinder Ceph support https://review.opendev.org/c/openstack/kolla-ansible/+/824722 | 14:33 |
opendevreview | Jakub Darmach proposed openstack/kolla-ansible master: horizon: Support custom themes https://review.opendev.org/c/openstack/kolla-ansible/+/761364 | 14:34 |
*** cozyurt is now known as Guest1287 | 14:40 | |
opendevreview | Christian Berendt proposed openstack/kolla-ansible master: [ironic] add coordination backend for ironic inspector https://review.opendev.org/c/openstack/kolla-ansible/+/827118 | 14:57 |
opendevreview | Buddhika Sanjeewa proposed openstack/kolla-ansible master: Deploy Zun with Cinder Ceph support https://review.opendev.org/c/openstack/kolla-ansible/+/824722 | 15:00 |
kevko | why quay.io/openstack.kolla/debian-binary-horizon:master is buster ? | 15:41 |
kevko | root@controller0:/home/ubuntu# docker ps | grep horizon | 15:41 |
kevko | 4db7b789cba0 quay.io/openstack.kolla/debian-binary-horizon:master "dumb-init --single-…" 6 minutes ago Up 3 minutes (healthy) horizon | 15:41 |
kevko | root@controller0:/home/ubuntu# docker exec -itu root horizon bash -c "cat /etc/debian_version" | 15:41 |
kevko | 10.9 | 15:41 |
opendevreview | Radosław Piliszek proposed openstack/kolla-ansible master: [WIP] Glance Cinder backend https://review.opendev.org/c/openstack/kolla-ansible/+/714999 | 16:31 |
jingvar | kevko: I think it too old image, and project moves to source installation type (last modified 10 months ago) | 16:59 |
mnasiadka | It seems we don't have periodic publish jobs for debian-binary, but maybe that's one less to remove ;-) | 17:17 |
yoctozepto | yeah, and nobody reported that | 17:21 |
yoctozepto | and we last published for victoria | 17:21 |
yoctozepto | so no wallaby nor xena | 17:21 |
mnasiadka | not that complicated to change ;) | 17:23 |
mnasiadka | but debian binary master might be useless... | 17:26 |
opendevreview | Konstantin Yarovoy proposed openstack/kolla master: Refactor image module and transition to DockerClient https://review.opendev.org/c/openstack/kolla/+/825786 | 17:34 |
jingvar | someone works with OVN? | 17:51 |
jingvar | maybe there is advice about SDN ? ovs+rabbit looks sometimes terrible | 17:55 |
jingvar | Is it a good idea to use the highest stable version of Kayobe to deploy a lower version of OpenStack? | 18:14 |
jingvar | I've got an issue with merging environments variables on Wallaby, Xena works, it is not critical, but I have to search for and move some vars from global to env | 18:17 |
opendevreview | Merged openstack/kolla-ansible master: horizon: Support custom themes https://review.opendev.org/c/openstack/kolla-ansible/+/761364 | 20:11 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!