Monday, 2023-07-31

kevkoanyone here ?05:46
kevkofind a bug ...and i think it's impossible to fix because of containerized env :(05:46
opendevreviewBartosz Bezak proposed openstack/kolla-ansible master: Add Pure Storage FlashBlade as Manila backend  https://review.opendev.org/c/openstack/kolla-ansible/+/87984607:14
opendevreviewChristian Berendt proposed openstack/kolla-ansible master: ceilometer: use template module in "Copying over custom pipeline.yaml file"  https://review.opendev.org/c/openstack/kolla-ansible/+/89005607:50
opendevreviewBartosz Bezak proposed openstack/kayobe master: Add STP option for bridge interfaces  https://review.opendev.org/c/openstack/kayobe/+/88985908:07
opendevreviewBartosz Bezak proposed openstack/kayobe master: Add STP option for bridge interfaces  https://review.opendev.org/c/openstack/kayobe/+/88985908:23
mnasiadkakevko: what bug?08:27
fricklerapropos bug and containerized, did anyone ever look into virtlogd? it isn't running in our libvirt container currently, but would be really nice if nova/libvirt could use it for console logging08:31
mnasiadkafrickler: I mentioned that multiple times that it would improve logging - but seems nobody had cycles :)08:52
fricklermnasiadka: ah, sorry, I must have ignored that. guess I'll need to find some cycles for it, then08:56
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/2023.1: Fix the fluentd regexp to collect the logs  https://review.opendev.org/c/openstack/kolla-ansible/+/88453909:42
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: Fail on fluentd log parsing errors  https://review.opendev.org/c/openstack/kolla-ansible/+/74239909:43
kevkomnasiadka: we installed nvme package into nova-compute, and when os-brick is initialized ..also nvmeof driver is initialized ... and there is a line which not working in containerized environment .. https://github.com/openstack/os-brick/blob/bfb55b7b58b4a89a690c9c72d7c0ec45b3b66493/os_brick/initiator/connectors/nvmeof.py#L78611:01
kevkomnasiadka: normally in OS it will return uuid disk which is mounted as /   ...in containerized environment it returns "overlay" ...so function def _get_host_uuid(self)  will fall into except block ....11:03
kevkofor us it is not problem because we are not using it ...but i would like to fix because it's generating error in logs with start of instance ...11:03
kevkoi was wondering that we can create wrapper so openstack code will return proper result even if it is in container ...11:04
kevkowe will just mount /etc/mtab into container to some location ..something as /etc/mtab-docker-host ..and parse from file instead of direct findmnt and then blkid ...what do u think ? 11:05
kevkoOR - we can remove nvme package from images and drop support for it ...11:06
mnasiadkawell, shouldn't that be fixed in os-brick?11:07
kevkomnasiadka: how ? 11:07
kevkoyou are in container ...11:07
kevkoyou can't get information about host os ...11:07
mnasiadkafind more reliable way of getting UUID? :)11:07
kevkohow ? 11:07
kevkoyou are isolated in container :D 11:07
kevkolet me paste it via paste.openstack.org ..11:08
mnasiadkadmidecode still works11:08
kevkow8 a minute11:08
mnasiadkaquestion why the heck they need UUID of the / filesystem :D11:08
kevkowell, i think they are using it for HOST_ID11:08
kevkomnasiadka: this one -> 11:12
kevkohttps://paste.openstack.org/show/b2A1pruYM3N7rbl3rws0/11:12
fricklerto me that sounds like a bad idea to generate a host id, not sure if that currently works with linux, but there have been systems using a shared / via NFS11:25
mnasiadkaisn't dmidecode -s system-uuid better? :)11:28
rockeydmidecode demands /dev/mem and /sys/firmware/dmi mounted, then runned as privileged container11:41
rockeydoesn't sound like a nice workaround, but perhaps run it as an environment variable instead?11:42
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: Remove Glance and Neutron TLS proxies from fluentd input config  https://review.opendev.org/c/openstack/kolla-ansible/+/89008511:42
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: Remove Glance and Neutron TLS proxies from fluentd input config  https://review.opendev.org/c/openstack/kolla-ansible/+/89008511:45
kevkomnasiadka: well, maybe it is working because if uuid is None suuid is returned https://github.com/openstack/os-brick/blob/bfb55b7b58b4a89a690c9c72d7c0ec45b3b66493/os_brick/initiator/connectors/nvmeof.py#L764-L78211:58
mnasiadkakevko: maybe the error message needs to be fixed ;-)12:12
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: fluentd: fix horizon format regex  https://review.opendev.org/c/openstack/kolla-ansible/+/89012813:24
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: fluentd: fix horizon format regex  https://review.opendev.org/c/openstack/kolla-ansible/+/89012813:24
opendevreviewMichal Nasiadka proposed openstack/kolla stable/yoga: opensearch: move to yum/apt repos  https://review.opendev.org/c/openstack/kolla/+/88371613:37
opendevreviewMichal Nasiadka proposed openstack/kolla stable/yoga: opensearch-dashboards: Fix permissions  https://review.opendev.org/c/openstack/kolla/+/88437513:37
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: fluentd: fix horizon format regex  https://review.opendev.org/c/openstack/kolla-ansible/+/89012814:21
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: fluentd: fix horizon format regex  https://review.opendev.org/c/openstack/kolla-ansible/+/89012814:30
opendevreviewMerged openstack/kolla-ansible stable/2023.1: Fix the fluentd regexp to collect the logs  https://review.opendev.org/c/openstack/kolla-ansible/+/88453914:35
opendevreviewSimon Dodsley proposed openstack/kolla-ansible master: Update Pure Storage NVMe Cinder driver  https://review.opendev.org/c/openstack/kolla-ansible/+/87984414:36
opendevreviewSimon Dodsley proposed openstack/kolla-ansible master: Update Pure Storage NVMe Cinder driver  https://review.opendev.org/c/openstack/kolla-ansible/+/87984414:42
opendevreviewSimon Dodsley proposed openstack/kolla-ansible master: Update Pure Storage NVMe Cinder driver  https://review.opendev.org/c/openstack/kolla-ansible/+/87984414:43
spatelmnasiadka what is this option for? - neutron_enable_rolling_upgrade: "yes"15:39
mnasiadkafor doing rolling upgrade which is the default15:39
mnasiadkaif you set that to no, we call legacy_upgrade.yml15:39
mnasiadkawhich is not rolling15:40
spateloh okay so better to keep it yes :)15:40
mnasiadkafrickler, bbezak, mgoddard: maybe we should get rid of those legacy upgrade options?15:46
mnasiadkakevko ^^15:47
opendevreviewMerged openstack/kolla-ansible stable/2023.1: opensearch: alter path after using rpm/deb packaging  https://review.opendev.org/c/openstack/kolla-ansible/+/88980616:29
*** mjknoxie is now known as mjk-17:22
spatelTrying to read the kolla image building code and came across this line - PS1="${_KBOLD}(${KOLLA_SERVICE_NAME})${_KRESET}[\u@\h \W]\\$ "18:38
spatelwhat is KOLLA_SERVICE_NAME coming from?18:38
spateloh wait its ENV variable - KOLLA_SERVICE_NAME=keystone-ssh18:38
spatelgot it 18:39

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