opendevreview | Merged openstack/kolla-ansible master: Fix Masakari host monitor default config https://review.opendev.org/c/openstack/kolla-ansible/+/799110 | 02:23 |
---|---|---|
oklhost | morning | 05:18 |
opendevreview | zhangzetian proposed openstack/kolla stable/wallaby: Get pip freeze https://review.opendev.org/c/openstack/kolla/+/801704 | 05:23 |
opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/wallaby: Fix Masakari host monitor default config https://review.opendev.org/c/openstack/kolla-ansible/+/801595 | 06:58 |
*** rpittau|afk is now known as rpittau | 07:49 | |
opendevreview | Radosł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/+/781200 | 08:09 |
opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/ussuri: Disable docker's ip-forward when iptables disabled https://review.opendev.org/c/openstack/kolla-ansible/+/801719 | 08:10 |
opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/ussuri: Disable docker's ip-forward when iptables disabled https://review.opendev.org/c/openstack/kolla-ansible/+/801719 | 08:13 |
opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/victoria: [doc] Fix reno (disable-ip-forward) https://review.opendev.org/c/openstack/kolla-ansible/+/801715 | 08:18 |
opendevreview | Radosł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/+/781201 | 08:20 |
opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/train: Disable docker's ip-forward when iptables disabled https://review.opendev.org/c/openstack/kolla-ansible/+/801720 | 08:20 |
yoctozepto | mgoddard: 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 quickly | 08:28 |
yoctozepto | follow-up, wdyt? | 08:28 |
opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/ussuri: Allow to skip and unset sysctl vars https://review.opendev.org/c/openstack/kolla-ansible/+/801721 | 08:33 |
opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/ussuri: Add support for changing sysctl.conf path https://review.opendev.org/c/openstack/kolla-ansible/+/801722 | 08:34 |
opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/ussuri: Allow to skip and unset sysctl vars https://review.opendev.org/c/openstack/kolla-ansible/+/801721 | 08:34 |
opendevreview | Radosł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/+/799243 | 08:35 |
kevko | yoctozepto: hi, you haven't be here for a long time | 08:38 |
mgoddard | yoctozepto: feel free to add more flags if you think it's appropriate | 08:39 |
mgoddard | yoctozepto: the middle one still is true | 08:43 |
mgoddard | https://review.opendev.org/c/openstack/kolla-ansible/+/801267/5/releasenotes/notes/enable-ipxe-cf461344bdb99881.yaml | 08:44 |
mgoddard | the 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-ansible | 08:44 |
opendevreview | Mark Goddard proposed openstack/kolla-ansible master: ironic: always enable conductor HTTP server https://review.opendev.org/c/openstack/kolla-ansible/+/801267 | 08:46 |
opendevreview | Mark Goddard proposed openstack/kolla-ansible master: Fix incorrect config of linuxbridge multiple external networks https://review.opendev.org/c/openstack/kolla-ansible/+/787996 | 08:52 |
yoctozepto | mgoddard: 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 issues | 08:52 |
mgoddard | I'm not sure it is grammar though, it refers to two different things | 08:53 |
mgoddard | event in past, condition | 08:53 |
yoctozepto | mgoddard: 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 important | 08:55 |
opendevreview | Merged openstack/kolla stable/wallaby: gnocchi: add python3-rados to gnocchi-base source image https://review.opendev.org/c/openstack/kolla/+/801331 | 09:08 |
haerwu | morning | 09:16 |
opendevreview | Merged openstack/kolla-ansible stable/train: Fix empty match while setting supported_policy_files https://review.opendev.org/c/openstack/kolla-ansible/+/799244 | 09:25 |
opendevreview | Merged openstack/kolla-ansible stable/wallaby: CI: only run cells job on changes to nova role https://review.opendev.org/c/openstack/kolla-ansible/+/799152 | 09:25 |
opendevreview | Merged openstack/kolla-ansible stable/victoria: [doc] Fix reno (disable-ip-forward) https://review.opendev.org/c/openstack/kolla-ansible/+/801715 | 09:25 |
opendevreview | Lin PeiWen proposed openstack/kolla-ansible master: Optimize the outward-rabbitmq and rabbitmq group problems https://review.opendev.org/c/openstack/kolla-ansible/+/801747 | 09:40 |
opendevreview | Lin PeiWen proposed openstack/kolla-ansible master: Move cephfs deployment to manila-guide.rst https://review.opendev.org/c/openstack/kolla-ansible/+/801756 | 10:11 |
opendevreview | Merged openstack/kolla stable/victoria: gnocchi: add python3-rados to gnocchi-base source image https://review.opendev.org/c/openstack/kolla/+/801332 | 10:20 |
opendevreview | Merged openstack/kolla stable/ussuri: gnocchi: add python3-rados to gnocchi-base source image https://review.opendev.org/c/openstack/kolla/+/801333 | 10:20 |
kevko | guys, did someone see this https://paste.opendev.org/show/807645/ ? | 10:25 |
kevko | mariadb upgrade from ussuri->wallaby debian binary | 10:25 |
kevko | without data it's ok | 10:25 |
yoctozepto | kevko: I have not; btw, are you skipping Victoria? | 10:27 |
kevko | yoctozepto: yeah, mostly we are doing upgrade +2 releases | 10:29 |
kevko | yoctozepto: previously it was stein -> ussuri | 10:29 |
kevko | yoctozepto: 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 |
opendevreview | Verification of a change to openstack/kolla failed: Fix prometheus-elasticsearch-exporter container. https://review.opendev.org/c/openstack/kolla/+/799620 | 10:31 |
mgoddard | kevko: these are offline upgrades? | 10:32 |
mgoddard | we don't really support M!=1 for N+M upgrades | 10:33 |
kevko | mgoddard: yeah I know :), but I am always test it very responsibly | 10:34 |
kevko | in fact it's only replace containers ..replace configuration ..and most hurting are sqlalchemy/alembic migrations inside Oopenstack code ... | 10:35 |
kevko | and 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 |
opendevreview | Mark Goddard proposed openstack/kayobe master: Remove iscsi deploy interface https://review.opendev.org/c/openstack/kayobe/+/800273 | 10:42 |
mgoddard | looking for +2A on https://review.opendev.org/c/openstack/kolla-ansible/+/801267 and https://review.opendev.org/c/openstack/kolla-ansible/+/801605/ | 10:43 |
yoctozepto | mgoddard: in these dire times I could give you +A to unblock the kayobe gate | 10:52 |
yoctozepto | haerwu to the rescue | 10:53 |
mgoddard | merci haerwu | 10:53 |
haerwu | mgoddard: done | 10:53 |
haerwu | in meantime I am testing a bit newer machine for my tensorflow builds. 56 cores instead of 96 ones and is nearly twice fast :D | 10:57 |
mgoddard | haerwu: try a single core machine next? | 10:57 |
haerwu | mgoddard: long time since I saw such | 10:58 |
haerwu | and still it was wrong arch ;d | 10:58 |
opendevreview | wu.chunyang proposed openstack/kolla-ansible master: Fix nova deployment failure when rabbitmq is disabled https://review.opendev.org/c/openstack/kolla-ansible/+/799857 | 11:03 |
opendevreview | Ole Kleinschmidt proposed openstack/kolla master: adds adjutant support https://review.opendev.org/c/openstack/kolla/+/801231 | 11:39 |
opendevreview | Merged openstack/kolla-ansible master: Fix ironic_ipxe healthcheck on Debian/Ubuntu https://review.opendev.org/c/openstack/kolla-ansible/+/801605 | 11:57 |
opendevreview | Merged openstack/kolla-ansible master: ironic: always enable conductor HTTP server https://review.opendev.org/c/openstack/kolla-ansible/+/801267 | 11:57 |
opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/wallaby: Fix ironic_ipxe healthcheck on Debian/Ubuntu https://review.opendev.org/c/openstack/kolla-ansible/+/801737 | 12:24 |
parallax | Kolla'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 |
parallax | hopefully it's temporary | 12:48 |
oklhost | haerwu: I'm unsure what exactly you mean by gerrit comments? (801231) | 12:51 |
haerwu | oklhost: look at your commit message as change description. from plain kolla/master view | 12:52 |
haerwu | oklhost: you described history of changes done in previous reviews. which are useless outside of gerrit | 12:52 |
oklhost | haerwu: thx, got your point. | 12:54 |
haerwu | https://chris.beams.io/posts/git-commit/ ;D | 12:55 |
opendevreview | Mark Goddard proposed openstack/kayobe master: Remove iscsi deploy interface https://review.opendev.org/c/openstack/kayobe/+/800273 | 12:58 |
oklhost | I vow to improve ;) | 13:03 |
haerwu | oklhost: thx | 13:05 |
yoctozepto | oklhost: I accept your vows | 13:32 |
jingvar | haerwu: 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 |
haerwu | jingvar: you can change them into 'if base_package_type == rpm' | 14:21 |
haerwu | jingvar: create a patch, send to us with info that this helps you build images for yet-another-rhel-rebuild | 14:22 |
haerwu | jingvar: we support only one rpm distro so effect will be ~same | 14:23 |
yoctozepto | ++ | 15:00 |
opendevreview | Mark Goddard proposed openstack/kayobe master: Remove iscsi deploy interface https://review.opendev.org/c/openstack/kayobe/+/800273 | 15:24 |
mgoddard | haerwu: supporting rocky may become more difficult if stream diverges | 15:26 |
kevko | yoctozepto: what about second review about crmadmin ? | 15:54 |
yoctozepto | kevko: I have not checked it yet but I guess it's also debian-specific as I did not encounter it on ubuntu when testing | 15:54 |
yoctozepto | I can check with CI too | 15:55 |
kevko | yoctozepto: it's not debian specific as packages are same | 15:57 |
kevko | yoctozepto: 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 uuids | 15:58 |
kevko | *is adduser without | 15:59 |
yoctozepto | don't worry, I will check | 16:00 |
*** rpittau is now known as rpittau|afk | 16:02 | |
haerwu | mgoddard: we do not support rocky. and we will not support it | 16:30 |
haerwu | mgoddard: my hint was to just make his life easier while we do not have any new distro to support | 16:31 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!