Wednesday, 2022-09-21

congntkevko__: Hi, Can you explain this bug? Thanks00:46
*** matt__ is now known as matfechner03:06
opendevreviewMarcin Juszkiewicz proposed openstack/kolla-ansible master: Fix some issues reported by ansible-lint  https://review.opendev.org/c/openstack/kolla-ansible/+/85857107:08
hrwfrickler, mnasiadka: 5 issues left which may need more ansible knowledge07:09
opendevreviewMarcin Juszkiewicz proposed openstack/kolla-ansible master: Fix some issues reported by ansible-lint  https://review.opendev.org/c/openstack/kolla-ansible/+/85857107:11
mnasiadkamorning07:25
hrwhi mnasiadka 07:28
hrwmnasiadka: mailed centos messaging sig about rmq update08:48
mnasiadkahrw: thanks08:48
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: Fix some issues reported by ansible-lint  https://review.opendev.org/c/openstack/kolla-ansible/+/85857109:09
mnasiadkaWonder I did see only one error locally when running tox -e ansible-lint09:10
mnasiadka*Wonder why09:10
hrwI use 'ansible-lint -v ansible'09:11
mnasiadkaseems that one fix did it10:19
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: Fix init-swift  https://review.opendev.org/c/openstack/kolla-ansible/+/85866910:29
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: Fix init-swift  https://review.opendev.org/c/openstack/kolla-ansible/+/85866910: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
hrwI fixed those11:09
hrwdislike it but even manual suggests to not use copy the way we use11:09
opendevreviewRafal Lewandowski proposed openstack/kolla-ansible master: designate: allow disabling notifications  https://review.opendev.org/c/openstack/kolla-ansible/+/82571811:21
fricklerhmm, 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 well11:24
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: Fix init-swift  https://review.opendev.org/c/openstack/kolla-ansible/+/85866912:12
mnasiadkayoctozepto, frickler: https://review.opendev.org/c/openstack/kolla-ansible/+/858571 this seems to be green12:23
mjknoxiehey 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
opendevreviewJakub Darmach proposed openstack/kolla-ansible master: Keystone OIDC fix for Azure integration  https://review.opendev.org/c/openstack/kolla-ansible/+/85869812:47
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: Fix issues introduced by ansible-lint 6.6.0  https://review.opendev.org/c/openstack/kolla-ansible/+/85857112:48
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: Fix issues introduced by ansible-lint 6.6.0  https://review.opendev.org/c/openstack/kolla-ansible/+/85857112:48
opendevreviewJakub Darmach proposed openstack/kolla-ansible master: Keystone OIDC fix for Azure integration  https://review.opendev.org/c/openstack/kolla-ansible/+/85869812:59
opendevreviewMarcin Juszkiewicz proposed openstack/kolla-ansible master: Fix issues introduced by ansible-lint 6.6.0  https://review.opendev.org/c/openstack/kolla-ansible/+/85857113:10
hrwlet assume that ping knows v4/v613:10
opendevreviewAlex Welsh proposed openstack/kayobe master: added support for ANSIBLE_VAULT_PASSWORD_FILE  https://review.opendev.org/c/openstack/kayobe/+/85870313:15
yoctozeptomjknoxie: 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 well13:38
yoctozeptootoh, this should use the image distro, not the host distro there13:39
yoctozeptoand, ideally, we would migrate all users to lio and abandon tgt13:40
mjknoxieah I see, that makes sense13:47
mjknoxiethanks! 13:47
mnasiadkamgoddard mnasiadka hrw yoctozepto bbezak parallax Fl1nt frickler adrian-a kevko - meeting in 1013:50
mnasiadka#startmeeting kolla14:00
opendevmeetMeeting 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
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
opendevmeetThe meeting name has been set to 'kolla'14:00
mnasiadka#topic rollcall14:00
mnasiadkao/14:00
matfechnero/14:01
opendevreviewRafal Lewandowski proposed openstack/kolla-ansible master: designate: allow disabling notifications  https://review.opendev.org/c/openstack/kolla-ansible/+/82571814:01
frickler\o14:02
yoctozeptoo/14:02
mnasiadka#topic agenda14:03
mnasiadka* Review action items from the last meeting14:03
mnasiadka* CI status14:03
mnasiadka* Release tasks14:03
mnasiadka* Regular stable releases (first meeting in a month)14:03
mnasiadka* Current cycle planning14:03
mnasiadka* Additional agenda (from whiteboard)14:03
mnasiadka* Open discussion14:03
mnasiadka#topic Review action items from the last meeting14:03
mnasiadkahrw work on cron alternative14:03
mnasiadkamnasiadka deprecate Monasca and send mail to ML asking for contributors to step up14:03
mnasiadkahrw probably installed cronie in the image14:03
mnasiadkaI didn't do mine, but will this week14:04
mnasiadka#action mnasiadka deprecate Monasca and send mail to ML asking for contributors to step up14:04
mnasiadka#topic CI Status14:04
mnasiadkaSo, after the Zuul dance - do we have any changes to zuul config that haven't been merged?14:04
mnasiadkaI know about the kayobe one for ussuri, but that branch is long broken there I think14:05
yoctozeptoxena14:05
yoctozeptocentos upgrades no worky14:05
mnasiadkaah right14:06
mnasiadkadhcp not worky after upgrade14:06
yoctozeptoindeed14:07
yoctozeptoany clues>14:07
mnasiadkanope, at least not for now14:07
mnasiadka#topic Release tasks14:07
mnasiadkaI think it's feature freeze week14:07
mnasiadkaYes, R-2 Feature Freeze14:08
hrwo\14:08
mnasiadkaSo what's the list of features that need FFE?14:08
bbezak\o14:08
fricklerforget to mention the linting blocker for CI14:08
mnasiadkaCS9/RL9, OpenSearch, what else?14:09
hrwLE?14:09
fricklerI'd like to get the ovn exporter in, since we already have the image in kolla14:09
mnasiadkafrickler: right, sorry - but that's being worked upon so I assume it's nearly fixed14:09
fricklerand it should not break anything else14:09
mnasiadkafrickler: as long as there are core reviewers happy to review, then I see no issues14:09
yoctozeptosystemd14:11
yoctozeptowe need to merge it finally14:11
mnasiadkahrw: would be nice, but I can't see any reviewers doing any reviews14:11
yoctozeptoI think it is generally fine14:11
mnasiadkalet me make a list on whiteboard14:11
yoctozeptobut I will have more time next 2 weeks14:11
mnasiadkathis cycle is more like a regular cycle, lot of FFEs and nothing working on freeze ;-)14:12
bbezak:)14:12
mnasiadkahttps://etherpad.opendev.org/p/KollaWhiteBoard - L39014:13
yoctozepto:D14:13
hrw#link https://etherpad.opendev.org/p/KollaWhiteBoard#L390 then?14:14
mnasiadkaPlease add yourselves as reviewers there - according to our policy, if there are no two reviewers assigned - FFE is not granted14:14
mnasiadkahrw: thanks for linking :)14:14
mnasiadkaok then, let's move on14:15
mnasiadka#topic Current cycle planning14:15
mnasiadkaSo, let's start with Kolla - CentOS 9 Stream images14:15
mnasiadkaI think it's ready to merge, there will be followups with Grafana and arm64 rabbitmq14:16
mnasiadkaGrafana's repo doesn't support SHA1 gpg although hrw raised the issue long time ago14:16
mnasiadkaI think the best approach for now is disabling gpg check on that repo as a followup14:16
hrwyep14:16
mnasiadka#link https://review.opendev.org/c/openstack/kolla/+/83666414:17
hrwotherwise we are stuck with 7.5 instead of 914:17
hrw(grafana, not cs)14:17
mnasiadkayes, elasticsearch will be ,,fixed'' by opensearch implementation14:17
hrwwe can also mark SHA1 as fine for grafana container only14:17
mnasiadkaAny other core willing to review that change soon-ish?14:18
yoctozeptonext week?14:18
mnasiadkasure- that patch should not change now - I can upload the RL9 patch on top14:19
mnasiadkahrw: will you work on the followup with grafana?14:19
hrwyes14:19
mnasiadkagood, that's more or less covered14:20
yoctozeptothen next week I will review it14:20
yoctozeptoplease stabilise it by then14:20
mnasiadkaok then, next is OpenSearch14:21
mnasiadkaKolla 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
mnasiadkaand then if we enable tls - it gets installed once again from the internet14:22
mnasiadka#url https://review.opendev.org/c/openstack/kolla/+/83037314:22
hrwcan we disable it and enable?14:22
mnasiadkaunfortunately not, opensearch plugin manager only accepts remove and install14:23
yoctozeptolol14:23
yoctozeptois it a newer feature btw?14:23
mnasiadkaI'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
hrwmnasiadka: store binary in image and then install from an image?14:23
mnasiadkasecurity plugin in OpenSearch requires TLS certificates and gives you authentication14:23
mnasiadkahrw: that's the plan14:23
hrw+114:23
mnasiadkaif you don't have a TLS certificate - there's no authentication14:23
mnasiadkabut for ES today we have haproxy based authentication for Kibana14:24
mnasiadkaso nothing changes14:24
mnasiadkaanyway, on track I would say14:24
mnasiadkabut data migration/upgrade from ES to OS will rather be done in Antelope and then backported14:24
hrwno kibana for cs9/rl914:24
hrwunless we do 'no gpg' trick14:25
mnasiadkabasically people will need to redeploy from cs8 to rl914:25
mnasiadkaso wonder if ES will shard with OpenSearch14:25
mnasiadkabut that's a different story14:26
hrw:)14:26
mnasiadkaso then - yoctozepto is telling us systemd patch is more or less ready to go, so ideally we'd need a second reviewer for it14:27
yoctozepto++14:27
fricklerI have that on my list, but I can't promise anything yet14:28
mnasiadkaok14:29
fricklerstill wanted to run in a local setup14:29
mnasiadkaany other features we should discuss?14:29
yoctozeptofrickler: it passed deploy, I wonder more how well it fares in upgrades now14:30
yoctozeptofood for thought when you find the time14:30
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: grafana: ignore SHA1 gpg key on CentOS Stream 9 (and family)  https://review.opendev.org/c/openstack/kolla/+/85873814:31
fricklerack14:31
mnasiadkaok then, no additional agenda14:32
mnasiadka#topic Open discussion14:33
yoctozeptono open discussion from me today either14:33
hrwwho review linters patch?14:34
hrwhttps://review.opendev.org/c/openstack/kolla-ansible/+/85857114:34
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: Fix init-swift  https://review.opendev.org/c/openstack/kolla-ansible/+/85866914:34
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: Fix issues introduced by ansible-lint 6.6.0  https://review.opendev.org/c/openstack/kolla-ansible/+/85857114:34
mnasiadkaok, empty line in j2 file was needed14:35
mnasiadkashould pass now14:35
yoctozeptooh my ansible14:36
mnasiadkaI rebased the swift CI fix on that one14:36
yoctozeptonow we need template instead of templated copy?14:36
mnasiadkaansible-lint says so14:36
yoctozeptomnasiadka: init swift seems to break regular jobs14:36
yoctozeptoso needs work14:36
mnasiadkayoctozepto: will look into that later14:36
mnasiadkaprobably missing variable14:36
yoctozeptolikely14:36
hrwyoctozepto: ansible manual also suggests template if content in copy is used with variables14:37
yoctozeptohrw: thankfully the two ansible knowledge sources agree ;-)14:38
fricklermnasiadka: you removed not only the empty line but also the newline at the end of the first line14:38
fricklerlinter complained about the latter14:38
mnasiadkafrickler: I used Gerrit UI, maybe that's why14:39
fricklerah, possible. let me push another update, then14:39
mnasiadkaanyway, let's progress this and get it merged, without that there's nothing else14:39
mnasiadkaok, I think it's enough for today14:40
mnasiadkathanks for coming14:40
mnasiadka#endmeeting14:40
opendevmeetMeeting ended Wed Sep 21 14:40:05 2022 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:40
opendevmeetMinutes:        https://meetings.opendev.org/meetings/kolla/2022/kolla.2022-09-21-14.00.html14:40
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/kolla/2022/kolla.2022-09-21-14.00.txt14:40
opendevmeetLog:            https://meetings.opendev.org/meetings/kolla/2022/kolla.2022-09-21-14.00.log.html14:40
hrwthx mnasiadka 14:40
yoctozeptothanks mnasiadka 14:41
* yoctozepto off14:41
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: Fix init-swift  https://review.opendev.org/c/openstack/kolla-ansible/+/85866914:41
fricklerhmm, the file doesn't actually have a second line. gerrit is weird14:41
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: Keystone OIDC fix for Azure integration  https://review.opendev.org/c/openstack/kolla-ansible/+/85869814:42
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: Fix init-swift  https://review.opendev.org/c/openstack/kolla-ansible/+/85866914:45
thelounge44https://github.com/openstack/skyline-console/releases/tag/1.0.0.0rc1 interesting, when we can expect support in kolla? thanks17:38
opendevreviewMerged openstack/kolla-ansible master: Fix issues introduced by ansible-lint 6.6.0  https://review.opendev.org/c/openstack/kolla-ansible/+/85857117:40
*** scottsol[m] is now known as scottsol21:57

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