Thursday, 2021-07-22

opendevreviewMerged openstack/kolla-ansible master: Fix Masakari host monitor default config  https://review.opendev.org/c/openstack/kolla-ansible/+/79911002:23
oklhostmorning05:18
opendevreviewzhangzetian proposed openstack/kolla stable/wallaby: Get pip freeze  https://review.opendev.org/c/openstack/kolla/+/80170405:23
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/wallaby: Fix Masakari host monitor default config  https://review.opendev.org/c/openstack/kolla-ansible/+/80159506:58
*** rpittau|afk is now known as rpittau07:49
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/ussuri: [docker] Added a new flag to disable default network  https://review.opendev.org/c/openstack/kolla-ansible/+/78120008:09
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/ussuri: Disable docker's ip-forward when iptables disabled  https://review.opendev.org/c/openstack/kolla-ansible/+/80171908:10
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/ussuri: Disable docker's ip-forward when iptables disabled  https://review.opendev.org/c/openstack/kolla-ansible/+/80171908:13
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/victoria: [doc] Fix reno (disable-ip-forward)  https://review.opendev.org/c/openstack/kolla-ansible/+/80171508:18
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/train: [docker] Added a new flag to disable default network  https://review.opendev.org/c/openstack/kolla-ansible/+/78120108:20
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/train: Disable docker's ip-forward when iptables disabled  https://review.opendev.org/c/openstack/kolla-ansible/+/80172008:20
yoctozeptomgoddard: I have a comment regarding https://review.opendev.org/q/Iced014acee7e590c10848e73feca166f48b622dc see it on master; I see you backported but ignored the part about more flags; how do you think we should handle this? I suggest we release with the full flag set rather than doing partial updates; i.e. we could merge this and quickly08:28
yoctozeptofollow-up, wdyt?08:28
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/ussuri: Allow to skip and unset sysctl vars  https://review.opendev.org/c/openstack/kolla-ansible/+/80172108:33
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/ussuri: Add support for changing sysctl.conf path  https://review.opendev.org/c/openstack/kolla-ansible/+/80172208:34
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/ussuri: Allow to skip and unset sysctl vars  https://review.opendev.org/c/openstack/kolla-ansible/+/80172108:34
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/ussuri: Allow user to set sysctl_net_ipv4_tcp_retries2  https://review.opendev.org/c/openstack/kolla-ansible/+/79924308:35
kevkoyoctozepto: hi, you haven't be here for a long time 08:38
mgoddardyoctozepto: feel free to add more flags if you think it's appropriate08:39
mgoddardyoctozepto: the middle one still is true08:43
mgoddardhttps://review.opendev.org/c/openstack/kolla-ansible/+/801267/5/releasenotes/notes/enable-ipxe-cf461344bdb99881.yaml08:44
mgoddardthe first and last, I'm not sure, but I don't feel that the value of the option is in the same tense as the logic of kolla-ansible08:44
opendevreviewMark Goddard proposed openstack/kolla-ansible master: ironic: always enable conductor HTTP server  https://review.opendev.org/c/openstack/kolla-ansible/+/80126708:46
opendevreviewMark Goddard proposed openstack/kolla-ansible master: Fix incorrect config of linuxbridge multiple external networks  https://review.opendev.org/c/openstack/kolla-ansible/+/78799608:52
yoctozeptomgoddard: yeah, the middle can stay; first one is grammar; last one is tricky but one could argue "previously" introduces the implicit past tense; anyhow, you can also write "when ``enable_ironic_ipxe: yes``" and it would convey the same message without the grammar issues08:52
mgoddardI'm not sure it is grammar though, it refers to two different things08:53
mgoddardevent in past, condition08:53
yoctozeptomgoddard: I am very sure about the 1st one; was when was - is when is BUT I don't really mind; the docs fix was more important08:55
opendevreviewMerged openstack/kolla stable/wallaby: gnocchi: add python3-rados to gnocchi-base source image  https://review.opendev.org/c/openstack/kolla/+/80133109:08
haerwumorning09:16
opendevreviewMerged openstack/kolla-ansible stable/train: Fix empty match while setting supported_policy_files  https://review.opendev.org/c/openstack/kolla-ansible/+/79924409:25
opendevreviewMerged openstack/kolla-ansible stable/wallaby: CI: only run cells job on changes to nova role  https://review.opendev.org/c/openstack/kolla-ansible/+/79915209:25
opendevreviewMerged openstack/kolla-ansible stable/victoria: [doc] Fix reno (disable-ip-forward)  https://review.opendev.org/c/openstack/kolla-ansible/+/80171509:25
opendevreviewLin PeiWen proposed openstack/kolla-ansible master: Optimize the outward-rabbitmq and rabbitmq group problems  https://review.opendev.org/c/openstack/kolla-ansible/+/80174709:40
opendevreviewLin PeiWen proposed openstack/kolla-ansible master: Move cephfs deployment to manila-guide.rst  https://review.opendev.org/c/openstack/kolla-ansible/+/80175610:11
opendevreviewMerged openstack/kolla stable/victoria: gnocchi: add python3-rados to gnocchi-base source image  https://review.opendev.org/c/openstack/kolla/+/80133210:20
opendevreviewMerged openstack/kolla stable/ussuri: gnocchi: add python3-rados to gnocchi-base source image  https://review.opendev.org/c/openstack/kolla/+/80133310:20
kevkoguys, did someone see this https://paste.opendev.org/show/807645/ ? 10:25
kevkomariadb upgrade from ussuri->wallaby debian binary 10:25
kevkowithout data it's ok 10:25
yoctozeptokevko: I have not; btw, are you skipping Victoria?10:27
kevkoyoctozepto: yeah, mostly we are doing upgrade +2 releases 10:29
kevkoyoctozepto: previously it was stein -> ussuri 10:29
kevkoyoctozepto: btw, if you remember that strange behaviour of pacemaker I've discussed with you some time ago ... i fixed it here -> https://review.opendev.org/c/openstack/kolla/+/799787  and added you as reviewer also .. could you check when you will have some time ? 10:31
opendevreviewVerification of a change to openstack/kolla failed: Fix prometheus-elasticsearch-exporter container.  https://review.opendev.org/c/openstack/kolla/+/79962010:31
mgoddardkevko: these are offline upgrades?10:32
mgoddardwe don't really support M!=1 for N+M upgrades10:33
kevkomgoddard: yeah I know :), but I am always test it very responsibly10:34
kevkoin fact it's only replace containers ..replace configuration ..and most hurting are sqlalchemy/alembic migrations inside Oopenstack code ...10:35
kevkoand if I am correct .. kolla-ansible -i inventory upgrade -t mariadb is same from ussuri->wallaby as victoria->wallaby ...both ussuri and victoria are buster ..so i suppose there is mariadb from debian/buster 10:36
opendevreviewMark Goddard proposed openstack/kayobe master: Remove iscsi deploy interface  https://review.opendev.org/c/openstack/kayobe/+/80027310:42
mgoddardlooking for +2A on https://review.opendev.org/c/openstack/kolla-ansible/+/801267 and https://review.opendev.org/c/openstack/kolla-ansible/+/801605/10:43
yoctozeptomgoddard: in these dire times I could give you +A to unblock the kayobe gate10:52
yoctozeptohaerwu to the rescue10:53
mgoddardmerci haerwu 10:53
haerwumgoddard: done10:53
haerwuin meantime I am testing a bit newer machine for my tensorflow builds. 56 cores instead of 96 ones and is nearly twice fast :D10:57
mgoddardhaerwu: try a single core machine next?10:57
haerwumgoddard: long time since I saw such10:58
haerwuand still it was wrong arch ;d10:58
opendevreviewwu.chunyang proposed openstack/kolla-ansible master: Fix nova deployment failure when rabbitmq is disabled  https://review.opendev.org/c/openstack/kolla-ansible/+/79985711:03
opendevreviewOle Kleinschmidt proposed openstack/kolla master: adds adjutant support  https://review.opendev.org/c/openstack/kolla/+/80123111:39
opendevreviewMerged openstack/kolla-ansible master: Fix ironic_ipxe healthcheck on Debian/Ubuntu  https://review.opendev.org/c/openstack/kolla-ansible/+/80160511:57
opendevreviewMerged openstack/kolla-ansible master: ironic: always enable conductor HTTP server  https://review.opendev.org/c/openstack/kolla-ansible/+/80126711:57
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/wallaby: Fix ironic_ipxe healthcheck on Debian/Ubuntu  https://review.opendev.org/c/openstack/kolla-ansible/+/80173712:24
parallaxKolla's CI apparently blocked by: Failed to fetch https://packages.grafana.com/oss/deb/dists/stable/main/binary-amd64/Packages.gz  File has unexpected size (25520 != 25511). Mirror sync in progress?12:48
parallaxhopefully it's temporary12:48
oklhosthaerwu: I'm unsure what exactly you mean by gerrit comments? (801231)12:51
haerwuoklhost: look at your commit message as change description. from plain kolla/master view12:52
haerwuoklhost: you described history of changes done in previous reviews. which are useless outside of gerrit12:52
oklhosthaerwu: thx, got your point.12:54
haerwuhttps://chris.beams.io/posts/git-commit/ ;D12:55
opendevreviewMark Goddard proposed openstack/kayobe master: Remove iscsi deploy interface  https://review.opendev.org/c/openstack/kayobe/+/80027312:58
oklhostI vow to improve ;)13:03
haerwuoklhost: thx13:05
yoctozeptooklhost: I accept your vows13:32
jingvarhaerwu: kolla has a lot of conditions like if foo="centos". I just to check ability to use Rocky as base image distro, but it is pain.14:16
haerwujingvar: you can change them into 'if base_package_type == rpm'14:21
haerwujingvar: create a patch, send to us with info that this helps you build images for yet-another-rhel-rebuild14:22
haerwujingvar: we support only one rpm distro so effect will be ~same14:23
yoctozepto++15:00
opendevreviewMark Goddard proposed openstack/kayobe master: Remove iscsi deploy interface  https://review.opendev.org/c/openstack/kayobe/+/80027315:24
mgoddardhaerwu: supporting rocky may become more difficult if stream diverges15:26
kevkoyoctozepto: what about second review about crmadmin ? 15:54
yoctozeptokevko: I have not checked it yet but I guess it's also debian-specific as I did not encounter it on ubuntu when testing15:54
yoctozeptoI can check with CI too15:55
kevkoyoctozepto: it's not debian specific as packages are same 15:57
kevkoyoctozepto: and in both packages are adduser without defining uid ..so it's assigned dynamically ..and because both images are from different base image ..there are differencies in assigned uuids15:58
kevko*is adduser without 15:59
yoctozeptodon't worry, I will check16:00
*** rpittau is now known as rpittau|afk16:02
haerwumgoddard: we do not support rocky. and we will not support it 16:30
haerwumgoddard: my hint was to just make his life easier while we do not have any new distro to support16:31

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