Friday, 2024-01-05

opendevreviewMichal Nasiadka proposed openstack/kolla stable/yoga: [yoga-only]: Use pyroute2 from distro vendors  https://review.opendev.org/c/openstack/kolla/+/90478105:53
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/yoga: CI: Test Nova server resize functionality  https://review.opendev.org/c/openstack/kolla-ansible/+/90428506:06
mnasiadkabtw, I’m off today - but would like to get reviews on 904781 if possible07:57
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Fix long service restarts while using systemd  https://review.opendev.org/c/openstack/kolla-ansible/+/90480507:58
mnasiadkakevko: I meant an explanation why it’s 143 (128 + sigterm) - not the list in commit message :)08:05
mnasiadka(Sorry for maybe being not enough detail in the comment)08:06
mnasiadkaIt’s a bit weird that the list is so limited08:08
mnasiadkaProbably those services do not handle sigterm properly, as in it should exit 0 on sigterm, but they don’t (but still they terminate so it’s not sigkill) - I’ll do some digging on Monday08:15
mnasiadkafrickler: any thoughts? ^^08:15
kevkomnasiadka: check chatgpt answer :D :D ..i've posted into review :D 08:44
kevkomnasiadka: i'm surprised that it is very nice answer :P 08:45
kevkoIn summary, receiving a 143 exit code when stopping a Docker container is normal and indicates that the container was stopped due to receiving a SIGTERM signal. To change this behavior, you'd typically look into how the application inside the container handles signals and whether it can intercept and override the default action for SIGTERM.  <<08:45
kevkoThis is main idea08:45
kevkocheck it in comments 08:46
bbezakmaybe let's add related-bug for now flag08:46
kevkobbezak: why related-bug ? 08:46
bbezakor Partial-Bug - not to close the bug - until we're 100% sure that it can't be made exit 0 for all services08:48
bbezakbut it looks good for now - 143 looks harmless - maybe just not ideal08:49
bbezakas we simply ignored 143 until today :)08:49
kevkobbezak: but I am 100% sure that it can't be made exit 0 for all services ...08:50
kevkobbezak: or do you want to compile your own haproxy for example and rewrite the code ? or openvswitch ? or others ? 08:50
kevkobbezak: well, i mainly testing master and other stable releases on my testing environment ..so i can also see differencies by an eye ...that's the reason why i started to investigate ... in CI it's just not visible 08:51
mnasiadkaLet’s discuss it next week, it’s not urgent - although high priority bug08:55
bbezakyeah, I've noticed it yesterday afternoon in one env on antelope. I agree that we probably won't be able to fix all services. I think partial-bug for now is a good idea08:55
bbezakand let's merge it08:55
bbezakand backport it08:55
kevkobbezak: +1 09:06
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Fix long service restarts while using systemd  https://review.opendev.org/c/openstack/kolla-ansible/+/90480509:07
kevkofg09:07
kevkoanyone for +2 or you want to discuss mnasiadka ? 09:09
opendevreviewMark Goddard proposed openstack/kolla-ansible stable/2023.2: post-2023.1: Remove keystone admin endpoint bits  https://review.opendev.org/c/openstack/kolla-ansible/+/90472409:45
kevkofrickler: how are u changing the status of launchpad bug ? api ? do i have permissions for that for example ? 10:06
fricklerkevko: just via the UI. you'd need to be in some bug manager group on LP for that10:13
opendevreviewDawud proposed openstack/kolla-ansible master: Remove the `grafana` volume  https://review.opendev.org/c/openstack/kolla-ansible/+/89913610:25
kevkofrickler: and should I ? 10:28
opendevreviewMerged openstack/kolla stable/yoga: [yoga-only]: Use pyroute2 from distro vendors  https://review.opendev.org/c/openstack/kolla/+/90478110:29
opendevreviewDawud proposed openstack/kolla-ansible master: Enable HAProxy Prometheus metrics endpoint  https://review.opendev.org/c/openstack/kolla-ansible/+/87711810:36
opendevreviewMark Goddard proposed openstack/kolla-ansible master: cadvisor: Set housekeeping interval to Prometheus scrape interval  https://review.opendev.org/c/openstack/kolla-ansible/+/90482311:03
bbezakhttps://launchpad.net/~kolla-drivers/+members#active11:06
bbezakkevko, frickler11:06
fricklerbest check with mnasiadka as PTL. that list also looks like some cleanup would be useful, but I think only admins can do that11:53
bbezakindeed11:54
opendevreviewPierre Riteau proposed openstack/kolla-ansible master: CI: Test Nova server resize functionality  https://review.opendev.org/c/openstack/kolla-ansible/+/90424912:07
opendevreviewPierre Riteau proposed openstack/kolla-ansible master: CI: Test Nova server resize functionality  https://review.opendev.org/c/openstack/kolla-ansible/+/90424912:07
opendevreviewPierre Riteau proposed openstack/kolla-ansible master: CI: Test Nova server resize functionality  https://review.opendev.org/c/openstack/kolla-ansible/+/90424912:10
bbezakkevko: your're now hereby a member of kolla drivers team in launchpad12:32
kevkobbezak: oh, thanks 12:54
*** hrww is now known as hrw13:19
opendevreviewMerged openstack/kolla-ansible master: Fix long service restarts while using systemd  https://review.opendev.org/c/openstack/kolla-ansible/+/90480513:38
opendevreviewMichal Arbet proposed openstack/kolla master: Drop root privileges for ovn  https://review.opendev.org/c/openstack/kolla/+/66511713:44
opendevreviewMichal Arbet proposed openstack/kolla-ansible stable/2023.2: Fix long service restarts while using systemd  https://review.opendev.org/c/openstack/kolla-ansible/+/90474013:53
opendevreviewMichal Arbet proposed openstack/kolla-ansible stable/2023.1: Fix long service restarts while using systemd  https://review.opendev.org/c/openstack/kolla-ansible/+/90484113:53
opendevreviewPierre Riteau proposed openstack/kolla-ansible master: Configure sftp subsystem on Debian  https://review.opendev.org/c/openstack/kolla-ansible/+/90483814:02
opendevreviewMerged openstack/kayobe stable/2023.2: Ensure git is present before checkout tasks  https://review.opendev.org/c/openstack/kayobe/+/90426514:30
opendevreviewMerged openstack/kayobe stable/2023.1: Ensure git is present before checkout tasks  https://review.opendev.org/c/openstack/kayobe/+/90426614:30
opendevreviewMerged openstack/kayobe stable/zed: Ensure git is present before checkout tasks  https://review.opendev.org/c/openstack/kayobe/+/90426714:30
opendevreviewMerged openstack/kayobe stable/yoga: Ensure git is present before checkout tasks  https://review.opendev.org/c/openstack/kayobe/+/90426814:30
opendevreviewMartin Hiner proposed openstack/kolla-ansible master: Add container engine migration scenario  https://review.opendev.org/c/openstack/kolla-ansible/+/83694115:43
opendevreviewPierre Riteau proposed openstack/kolla-ansible master: [DNM] Dummy change to trigger all CI jobs  https://review.opendev.org/c/openstack/kolla-ansible/+/90486415:45
opendevreviewTadas proposed openstack/kolla master: add: cross-compile support  https://review.opendev.org/c/openstack/kolla/+/88913922:09
opendevreviewTadas proposed openstack/kolla master: add: cross-compile support  https://review.opendev.org/c/openstack/kolla/+/90488022:16
opendevreviewTadas proposed openstack/kolla master: remove IDE leftovers  https://review.opendev.org/c/openstack/kolla/+/90488122:19
opendevreviewTadas proposed openstack/kolla master: remove IDE leftovers  https://review.opendev.org/c/openstack/kolla/+/90488222:30
opendevreviewTadas proposed openstack/kolla master: add: cross-compile support  https://review.opendev.org/c/openstack/kolla/+/90488322:33
opendevreviewTadas proposed openstack/kolla master: add: cross-compile support  https://review.opendev.org/c/openstack/kolla/+/88913922:59

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