Monday, 2022-01-31

opendevreviewBuddhika Sanjeewa proposed openstack/kolla-ansible master: Deploy Zun with Cinder Ceph support  https://review.opendev.org/c/openstack/kolla-ansible/+/82472200:17
opendevreviewImran Hussain proposed openstack/kolla-ansible master: [external-ceph] Use template instead of copy  https://review.opendev.org/c/openstack/kolla-ansible/+/82456609:30
opendevreviewImran Hussain proposed openstack/kolla-ansible master: [external-ceph] Use template instead of copy  https://review.opendev.org/c/openstack/kolla-ansible/+/82456609:30
hrwmorning09:51
mnasiadkamorning10:26
holtgrewemgoddard: thanks for the reply11:06
holtgrewelet me see what I can do11:06
mnasiadkayoctozepto: got some minutes to look into reviewing https://review.opendev.org/c/openstack/kolla/+/826672 ? 11:31
yoctozeptomnasiadka: +211:36
*** cozyurt is now known as Guest127111:43
opendevreviewPierre Riteau proposed openstack/kayobe-config stable/xena: Sync kayobe-config with kayobe changes  https://review.opendev.org/c/openstack/kayobe-config/+/82705911:49
opendevreviewPierre Riteau proposed openstack/kayobe-config stable/wallaby: Sync kayobe-config with kayobe changes  https://review.opendev.org/c/openstack/kayobe-config/+/82706011:49
opendevreviewPierre Riteau proposed openstack/kayobe-config stable/victoria: Sync kayobe-config with kayobe changes  https://review.opendev.org/c/openstack/kayobe-config/+/82706111:50
opendevreviewPierre Riteau proposed openstack/kayobe-config master: Sync kayobe-config with kayobe changes  https://review.opendev.org/c/openstack/kayobe-config/+/82706211:53
yuvalcontinue my question regarding env variable11:54
yuvalthe cinder uses "sudo su" to run specific cmd11:54
yuvalwhen running sudo su - the env variable is different from the "cinder" env variable11:54
yuvalsomeone knows how I can edit the sudo env variable?11:55
yuvalis there a script I can hack before the service is starting?11:55
kevkohi \o/, i wasn't here for a while ..vacationing etc :P 12:09
kevkoyoctozepto: regarding my proxysql patch ..there is a comment why proxysql container has privileged: True , actually i was inspired by haproxy 12:11
rockeyyuval: which command are you referring to?12:11
kevkowhy is haproxy privileged :D ? 12:11
kevkoyuval: just run container with one more ENV ? :D 12:12
yuvalthe os-brick connector check the env variable "VIRTUAL_ENV" while using "sudo su"12:12
yuvalin my cinder and root users I have the variable env12:12
yuvalbut when doing "sudo su" I dont have this env variable12:13
kevkoyuval: docker run ........... -e 'WHATEVER_YOU_WANT_TO_SET=whatever' ? 12:13
yuvalI am using kolla-ansible12:13
rockeyyuval: are you building your own images, or using upstream?12:14
kevkoyeah, an kolla ansible just run docker command :D 12:14
kevkoyou can change kolla-ansible restart handler in your branch locally and add environment: "whatever" 12:14
rockeyif so, you can extend dockerfiles using your own ENV variables12:14
kevkoso when you will run reconfigure ..container will be recreated with whatever you want 12:15
kevkowhere is a problem ? 12:15
yuvalok, thank you I see12:15
rockeyyeah, wondering that too, since "sudo su" isn't runned anywhere, neither kolla nor kolla-ansible12:15
yuval"sudo su" run by cinder12:15
rockeynot within kolla/kolla-ansible source12:16
yuvalyes, yes I agree12:16
rockeyis it recreatable? and how?12:16
yuvali am not sure kolla-ansible/kolla is the problem here actually12:18
yuvalhttps://opendev.org/openstack/os-brick/src/branch/master/os_brick/privileged/__init__.py#L2412:19
yuvalcheck this line - its doing: if os.environ.get('VIRTUAL_ENV'):12:19
kevkoyuval: https://paste.opendev.org/show/812448/12:19
yuvalyes, I managed to add it - also by using group_var/all.yml12:21
yuvalmy issue is that when cinder does check this env variable - its with sudo su - which run with "clean" env12:21
kevkoyuval: where you can see that it's via root ? 12:22
kevkoyuval: do you have some log ? 12:22
yuvalhttps://paste.opendev.org/show/812449/12:25
yuvalI stopped with pdb just before this line12:25
yuvaland check the os.environ12:26
opendevreviewVerification of a change to openstack/kolla stable/train failed: Mitigate two Log4j vulnerabilities in Apache Storm  https://review.opendev.org/c/openstack/kolla/+/82661312:29
rockeyyuval: 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
kevkoyoctozepto: here ? 12:43
yuvalok I found a solution12:51
yuvalin my cinder.conf12:52
yuvalI added:12:52
yuval[privsep_osbrick]12:52
yuvalhelper_command = sudo -E cinder-rootwrap /etc/cinder/rootwrap.conf privsep-helper --config-file /etc/cinder/cinder.conf12:52
yuvalthe important part is the "sudo -E"12:52
yuvalthis is after I added the "VIRUTAL_ENV" to the docker env12:53
yuvalbtw - rocky and kevko - thanks for trying to help13:15
opendevreviewMerged openstack/kolla master: Ensure set_configs sets execute bit on directories  https://review.opendev.org/c/openstack/kolla/+/82667213:40
opendevreviewJakub Darmach proposed openstack/kolla-ansible master: horizon: Support custom themes  https://review.opendev.org/c/openstack/kolla-ansible/+/76136414:16
opendevreviewBuddhika Sanjeewa proposed openstack/kolla-ansible master: Deploy Zun with Cinder Ceph support  https://review.opendev.org/c/openstack/kolla-ansible/+/82472214:33
opendevreviewJakub Darmach proposed openstack/kolla-ansible master: horizon: Support custom themes  https://review.opendev.org/c/openstack/kolla-ansible/+/76136414:34
*** cozyurt is now known as Guest128714:40
opendevreviewChristian Berendt proposed openstack/kolla-ansible master: [ironic] add coordination backend for ironic inspector  https://review.opendev.org/c/openstack/kolla-ansible/+/82711814:57
opendevreviewBuddhika Sanjeewa proposed openstack/kolla-ansible master: Deploy Zun with Cinder Ceph support  https://review.opendev.org/c/openstack/kolla-ansible/+/82472215:00
kevkowhy quay.io/openstack.kolla/debian-binary-horizon:master  is buster ? 15:41
kevkoroot@controller0:/home/ubuntu# docker ps | grep horizon15:41
kevko4db7b789cba0   quay.io/openstack.kolla/debian-binary-horizon:master                              "dumb-init --single-…"   6 minutes ago   Up 3 minutes (healthy)             horizon15:41
kevkoroot@controller0:/home/ubuntu# docker exec -itu root horizon bash -c  "cat /etc/debian_version"15:41
kevko10.915:41
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible master: [WIP] Glance Cinder backend  https://review.opendev.org/c/openstack/kolla-ansible/+/71499916:31
jingvarkevko: I think it too old image, and project moves to source installation type (last modified 10 months ago)16:59
mnasiadkaIt seems we don't have periodic publish jobs for debian-binary, but maybe that's one less to remove ;-)17:17
yoctozeptoyeah, and nobody reported that17:21
yoctozeptoand we last published for victoria17:21
yoctozeptoso no wallaby nor xena17:21
mnasiadkanot that complicated to change ;)17:23
mnasiadkabut debian binary master might be useless...17:26
opendevreviewKonstantin Yarovoy proposed openstack/kolla master: Refactor image module and transition to DockerClient  https://review.opendev.org/c/openstack/kolla/+/82578617:34
jingvarsomeone works with OVN?17:51
jingvarmaybe there is advice about SDN ? ovs+rabbit looks sometimes terrible17:55
jingvarIs it a good idea to use the highest stable version of Kayobe to deploy a lower version of OpenStack?18:14
jingvarI'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 env18:17
opendevreviewMerged openstack/kolla-ansible master: horizon: Support custom themes  https://review.opendev.org/c/openstack/kolla-ansible/+/76136420:11

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