congnt | kevko__: Hi, Can you explain this bug? Thanks | 00:46 |
---|---|---|
*** matt__ is now known as matfechner | 03:06 | |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla-ansible master: Fix some issues reported by ansible-lint https://review.opendev.org/c/openstack/kolla-ansible/+/858571 | 07:08 |
hrw | frickler, mnasiadka: 5 issues left which may need more ansible knowledge | 07:09 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla-ansible master: Fix some issues reported by ansible-lint https://review.opendev.org/c/openstack/kolla-ansible/+/858571 | 07:11 |
mnasiadka | morning | 07:25 |
hrw | hi mnasiadka | 07:28 |
hrw | mnasiadka: mailed centos messaging sig about rmq update | 08:48 |
mnasiadka | hrw: thanks | 08:48 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Fix some issues reported by ansible-lint https://review.opendev.org/c/openstack/kolla-ansible/+/858571 | 09:09 |
mnasiadka | Wonder I did see only one error locally when running tox -e ansible-lint | 09:10 |
mnasiadka | *Wonder why | 09:10 |
hrw | I use 'ansible-lint -v ansible' | 09:11 |
mnasiadka | seems that one fix did it | 10:19 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Fix init-swift https://review.opendev.org/c/openstack/kolla-ansible/+/858669 | 10:29 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Fix init-swift https://review.opendev.org/c/openstack/kolla-ansible/+/858669 | 10:32 |
frickler | "template-instead-of-copy: Templated files should use template instead of copy" I don't think I agree on this, if the content is only a single variable. can we exclude this instead? | 11:06 |
hrw | I fixed those | 11:09 |
hrw | dislike it but even manual suggests to not use copy the way we use | 11:09 |
opendevreview | Rafal Lewandowski proposed openstack/kolla-ansible master: designate: allow disabling notifications https://review.opendev.org/c/openstack/kolla-ansible/+/825718 | 11:21 |
frickler | hmm, o.k., "Using a variable in the content field will result in unpredictable output.", sounds more like a bug than a feature to me, but well | 11:24 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Fix init-swift https://review.opendev.org/c/openstack/kolla-ansible/+/858669 | 12:12 |
mnasiadka | yoctozepto, frickler: https://review.opendev.org/c/openstack/kolla-ansible/+/858571 this seems to be green | 12:23 |
mjknoxie | hey yoctozepto, thanks for the feedback on centos. I am just using kolla-ansible as an all-in-one with LVM backed cinder (which needs iSCSI), centos is in the config, so I can assume that centos will be removed? | 12:26 |
opendevreview | Jakub Darmach proposed openstack/kolla-ansible master: Keystone OIDC fix for Azure integration https://review.opendev.org/c/openstack/kolla-ansible/+/858698 | 12:47 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Fix issues introduced by ansible-lint 6.6.0 https://review.opendev.org/c/openstack/kolla-ansible/+/858571 | 12:48 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Fix issues introduced by ansible-lint 6.6.0 https://review.opendev.org/c/openstack/kolla-ansible/+/858571 | 12:48 |
opendevreview | Jakub Darmach proposed openstack/kolla-ansible master: Keystone OIDC fix for Azure integration https://review.opendev.org/c/openstack/kolla-ansible/+/858698 | 12:59 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla-ansible master: Fix issues introduced by ansible-lint 6.6.0 https://review.opendev.org/c/openstack/kolla-ansible/+/858571 | 13:10 |
hrw | let assume that ping knows v4/v6 | 13:10 |
opendevreview | Alex Welsh proposed openstack/kayobe master: added support for ANSIBLE_VAULT_PASSWORD_FILE https://review.opendev.org/c/openstack/kayobe/+/858703 | 13:15 |
yoctozepto | mjknoxie: kolla-ansible decides whether to try tgtd or lio based on the host distribution; on rhel systems lio is used; you are probably using centos containers on a non-centos distro (guessing ubuntu or debian) which is not supported well | 13:38 |
yoctozepto | otoh, this should use the image distro, not the host distro there | 13:39 |
yoctozepto | and, ideally, we would migrate all users to lio and abandon tgt | 13:40 |
mjknoxie | ah I see, that makes sense | 13:47 |
mjknoxie | thanks! | 13:47 |
mnasiadka | mgoddard mnasiadka hrw yoctozepto bbezak parallax Fl1nt frickler adrian-a kevko - meeting in 10 | 13:50 |
mnasiadka | #startmeeting kolla | 14:00 |
opendevmeet | Meeting started Wed Sep 21 14:00:52 2022 UTC and is due to finish in 60 minutes. The chair is mnasiadka. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
opendevmeet | The meeting name has been set to 'kolla' | 14:00 |
mnasiadka | #topic rollcall | 14:00 |
mnasiadka | o/ | 14:00 |
matfechner | o/ | 14:01 |
opendevreview | Rafal Lewandowski proposed openstack/kolla-ansible master: designate: allow disabling notifications https://review.opendev.org/c/openstack/kolla-ansible/+/825718 | 14:01 |
frickler | \o | 14:02 |
yoctozepto | o/ | 14:02 |
mnasiadka | #topic agenda | 14:03 |
mnasiadka | * Review action items from the last meeting | 14:03 |
mnasiadka | * CI status | 14:03 |
mnasiadka | * Release tasks | 14:03 |
mnasiadka | * Regular stable releases (first meeting in a month) | 14:03 |
mnasiadka | * Current cycle planning | 14:03 |
mnasiadka | * Additional agenda (from whiteboard) | 14:03 |
mnasiadka | * Open discussion | 14:03 |
mnasiadka | #topic Review action items from the last meeting | 14:03 |
mnasiadka | hrw work on cron alternative | 14:03 |
mnasiadka | mnasiadka deprecate Monasca and send mail to ML asking for contributors to step up | 14:03 |
mnasiadka | hrw probably installed cronie in the image | 14:03 |
mnasiadka | I didn't do mine, but will this week | 14:04 |
mnasiadka | #action mnasiadka deprecate Monasca and send mail to ML asking for contributors to step up | 14:04 |
mnasiadka | #topic CI Status | 14:04 |
mnasiadka | So, after the Zuul dance - do we have any changes to zuul config that haven't been merged? | 14:04 |
mnasiadka | I know about the kayobe one for ussuri, but that branch is long broken there I think | 14:05 |
yoctozepto | xena | 14:05 |
yoctozepto | centos upgrades no worky | 14:05 |
mnasiadka | ah right | 14:06 |
mnasiadka | dhcp not worky after upgrade | 14:06 |
yoctozepto | indeed | 14:07 |
yoctozepto | any clues> | 14:07 |
mnasiadka | nope, at least not for now | 14:07 |
mnasiadka | #topic Release tasks | 14:07 |
mnasiadka | I think it's feature freeze week | 14:07 |
mnasiadka | Yes, R-2 Feature Freeze | 14:08 |
hrw | o\ | 14:08 |
mnasiadka | So what's the list of features that need FFE? | 14:08 |
bbezak | \o | 14:08 |
frickler | forget to mention the linting blocker for CI | 14:08 |
mnasiadka | CS9/RL9, OpenSearch, what else? | 14:09 |
hrw | LE? | 14:09 |
frickler | I'd like to get the ovn exporter in, since we already have the image in kolla | 14:09 |
mnasiadka | frickler: right, sorry - but that's being worked upon so I assume it's nearly fixed | 14:09 |
frickler | and it should not break anything else | 14:09 |
mnasiadka | frickler: as long as there are core reviewers happy to review, then I see no issues | 14:09 |
yoctozepto | systemd | 14:11 |
yoctozepto | we need to merge it finally | 14:11 |
mnasiadka | hrw: would be nice, but I can't see any reviewers doing any reviews | 14:11 |
yoctozepto | I think it is generally fine | 14:11 |
mnasiadka | let me make a list on whiteboard | 14:11 |
yoctozepto | but I will have more time next 2 weeks | 14:11 |
mnasiadka | this cycle is more like a regular cycle, lot of FFEs and nothing working on freeze ;-) | 14:12 |
bbezak | :) | 14:12 |
mnasiadka | https://etherpad.opendev.org/p/KollaWhiteBoard - L390 | 14:13 |
yoctozepto | :D | 14:13 |
hrw | #link https://etherpad.opendev.org/p/KollaWhiteBoard#L390 then? | 14:14 |
mnasiadka | Please add yourselves as reviewers there - according to our policy, if there are no two reviewers assigned - FFE is not granted | 14:14 |
mnasiadka | hrw: thanks for linking :) | 14:14 |
mnasiadka | ok then, let's move on | 14:15 |
mnasiadka | #topic Current cycle planning | 14:15 |
mnasiadka | So, let's start with Kolla - CentOS 9 Stream images | 14:15 |
mnasiadka | I think it's ready to merge, there will be followups with Grafana and arm64 rabbitmq | 14:16 |
mnasiadka | Grafana's repo doesn't support SHA1 gpg although hrw raised the issue long time ago | 14:16 |
mnasiadka | I think the best approach for now is disabling gpg check on that repo as a followup | 14:16 |
hrw | yep | 14:16 |
mnasiadka | #link https://review.opendev.org/c/openstack/kolla/+/836664 | 14:17 |
hrw | otherwise we are stuck with 7.5 instead of 9 | 14:17 |
hrw | (grafana, not cs) | 14:17 |
mnasiadka | yes, elasticsearch will be ,,fixed'' by opensearch implementation | 14:17 |
hrw | we can also mark SHA1 as fine for grafana container only | 14:17 |
mnasiadka | Any other core willing to review that change soon-ish? | 14:18 |
yoctozepto | next week? | 14:18 |
mnasiadka | sure- that patch should not change now - I can upload the RL9 patch on top | 14:19 |
mnasiadka | hrw: will you work on the followup with grafana? | 14:19 |
hrw | yes | 14:19 |
mnasiadka | good, that's more or less covered | 14:20 |
yoctozepto | then next week I will review it | 14:20 |
yoctozepto | please stabilise it by then | 14:20 |
mnasiadka | ok then, next is OpenSearch | 14:21 |
mnasiadka | Kolla part is building and functional, the only thing I dislike is the removal of the security plugin (as in it gets removed from the running container) | 14:22 |
mnasiadka | and then if we enable tls - it gets installed once again from the internet | 14:22 |
mnasiadka | #url https://review.opendev.org/c/openstack/kolla/+/830373 | 14:22 |
hrw | can we disable it and enable? | 14:22 |
mnasiadka | unfortunately not, opensearch plugin manager only accepts remove and install | 14:23 |
yoctozepto | lol | 14:23 |
yoctozepto | is it a newer feature btw? | 14:23 |
mnasiadka | I'll try to find a way to store it in some directory so we don't need internet access to enable it back :) | 14:23 |
hrw | mnasiadka: store binary in image and then install from an image? | 14:23 |
mnasiadka | security plugin in OpenSearch requires TLS certificates and gives you authentication | 14:23 |
mnasiadka | hrw: that's the plan | 14:23 |
hrw | +1 | 14:23 |
mnasiadka | if you don't have a TLS certificate - there's no authentication | 14:23 |
mnasiadka | but for ES today we have haproxy based authentication for Kibana | 14:24 |
mnasiadka | so nothing changes | 14:24 |
mnasiadka | anyway, on track I would say | 14:24 |
mnasiadka | but data migration/upgrade from ES to OS will rather be done in Antelope and then backported | 14:24 |
hrw | no kibana for cs9/rl9 | 14:24 |
hrw | unless we do 'no gpg' trick | 14:25 |
mnasiadka | basically people will need to redeploy from cs8 to rl9 | 14:25 |
mnasiadka | so wonder if ES will shard with OpenSearch | 14:25 |
mnasiadka | but that's a different story | 14:26 |
hrw | :) | 14:26 |
mnasiadka | so then - yoctozepto is telling us systemd patch is more or less ready to go, so ideally we'd need a second reviewer for it | 14:27 |
yoctozepto | ++ | 14:27 |
frickler | I have that on my list, but I can't promise anything yet | 14:28 |
mnasiadka | ok | 14:29 |
frickler | still wanted to run in a local setup | 14:29 |
mnasiadka | any other features we should discuss? | 14:29 |
yoctozepto | frickler: it passed deploy, I wonder more how well it fares in upgrades now | 14:30 |
yoctozepto | food for thought when you find the time | 14:30 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: grafana: ignore SHA1 gpg key on CentOS Stream 9 (and family) https://review.opendev.org/c/openstack/kolla/+/858738 | 14:31 |
frickler | ack | 14:31 |
mnasiadka | ok then, no additional agenda | 14:32 |
mnasiadka | #topic Open discussion | 14:33 |
yoctozepto | no open discussion from me today either | 14:33 |
hrw | who review linters patch? | 14:34 |
hrw | https://review.opendev.org/c/openstack/kolla-ansible/+/858571 | 14:34 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Fix init-swift https://review.opendev.org/c/openstack/kolla-ansible/+/858669 | 14:34 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Fix issues introduced by ansible-lint 6.6.0 https://review.opendev.org/c/openstack/kolla-ansible/+/858571 | 14:34 |
mnasiadka | ok, empty line in j2 file was needed | 14:35 |
mnasiadka | should pass now | 14:35 |
yoctozepto | oh my ansible | 14:36 |
mnasiadka | I rebased the swift CI fix on that one | 14:36 |
yoctozepto | now we need template instead of templated copy? | 14:36 |
mnasiadka | ansible-lint says so | 14:36 |
yoctozepto | mnasiadka: init swift seems to break regular jobs | 14:36 |
yoctozepto | so needs work | 14:36 |
mnasiadka | yoctozepto: will look into that later | 14:36 |
mnasiadka | probably missing variable | 14:36 |
yoctozepto | likely | 14:36 |
hrw | yoctozepto: ansible manual also suggests template if content in copy is used with variables | 14:37 |
yoctozepto | hrw: thankfully the two ansible knowledge sources agree ;-) | 14:38 |
frickler | mnasiadka: you removed not only the empty line but also the newline at the end of the first line | 14:38 |
frickler | linter complained about the latter | 14:38 |
mnasiadka | frickler: I used Gerrit UI, maybe that's why | 14:39 |
frickler | ah, possible. let me push another update, then | 14:39 |
mnasiadka | anyway, let's progress this and get it merged, without that there's nothing else | 14:39 |
mnasiadka | ok, I think it's enough for today | 14:40 |
mnasiadka | thanks for coming | 14:40 |
mnasiadka | #endmeeting | 14:40 |
opendevmeet | Meeting ended Wed Sep 21 14:40:05 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:40 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/kolla/2022/kolla.2022-09-21-14.00.html | 14:40 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/kolla/2022/kolla.2022-09-21-14.00.txt | 14:40 |
opendevmeet | Log: https://meetings.opendev.org/meetings/kolla/2022/kolla.2022-09-21-14.00.log.html | 14:40 |
hrw | thx mnasiadka | 14:40 |
yoctozepto | thanks mnasiadka | 14:41 |
* yoctozepto off | 14:41 | |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Fix init-swift https://review.opendev.org/c/openstack/kolla-ansible/+/858669 | 14:41 |
frickler | hmm, the file doesn't actually have a second line. gerrit is weird | 14:41 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Keystone OIDC fix for Azure integration https://review.opendev.org/c/openstack/kolla-ansible/+/858698 | 14:42 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Fix init-swift https://review.opendev.org/c/openstack/kolla-ansible/+/858669 | 14:45 |
thelounge44 | https://github.com/openstack/skyline-console/releases/tag/1.0.0.0rc1 interesting, when we can expect support in kolla? thanks | 17:38 |
opendevreview | Merged openstack/kolla-ansible master: Fix issues introduced by ansible-lint 6.6.0 https://review.opendev.org/c/openstack/kolla-ansible/+/858571 | 17:40 |
*** scottsol[m] is now known as scottsol | 21:57 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!