Tuesday, 2023-04-18

opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/zed: opensearch: default dashboards tag to opensearch_tag  https://review.opendev.org/c/openstack/kolla-ansible/+/88059804:57
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/yoga: opensearch: default dashboards tag to opensearch_tag  https://review.opendev.org/c/openstack/kolla-ansible/+/88059904:58
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/yoga: opensearch: default dashboards tag to opensearch_tag  https://review.opendev.org/c/openstack/kolla-ansible/+/88059905:00
rockeygod morning team05:17
rockeystarting to get real annoyed with reno, do someone have a few moments of time to explain what the f i'm doing wrong?05:18
fricklerrockey: pointing to a review that exposes the issue might be a good first step07:06
mnasiadkamorning07:07
rockeyfrickler: more like general usage, i cannot get a previous post added into releasenotes/notes/ listed when running `reno list/report`07:08
rockeynor get posts handled in multiple release-branches correctly, and it annoyes me out, since i can't find proper documentation or get different outputs when working with the configuration file (releasenotes/config.yaml)07:10
fricklerthat doesn't sound very kolla-specific. though likely reno knowledge is as rare elsewhere in the openstack world nowadays07:10
rockeyyeah, hence the fuzzy question of usage, more than kolla specific07:11
rockeyglola07:12
rockeysorry :)07:12
fricklermnasiadka: any progress on your octavia testing? would be good to get that job working again, also I'm waiting on that with my added debian jobs07:17
mnasiadkafrickler: a bit of progress, but I wanted the code to get better and broke something, will revert to working code and push it today ;-)07:17
opendevreviewVerification of a change to openstack/kolla-ansible stable/zed failed: opensearch: default dashboards tag to opensearch_tag  https://review.opendev.org/c/openstack/kolla-ansible/+/88059809:07
kevkohrw:  replied 09:48
mmalchukmnasiadka could you please merge last patch: https://review.opendev.org/q/topic:fix-merge-action-plugins09:50
mmalchukand review https://review.opendev.org/q/topic:dib-yaml-width09:50
mmalchukthanks09:51
kevkomnasiadka: can we approve this ? https://review.opendev.org/c/openstack/kolla/+/877986 ..it is needed for https://review.opendev.org/c/openstack/kolla-ansible/+/879909 as it is visible here https://review.opendev.org/c/openstack/kolla-ansible/+/880678 (1:1 same patch with [DNM] flag i tested yesterday ) 09:53
kevkothanks 09:54
opendevreviewVerification of a change to openstack/kolla-ansible stable/yoga failed: opensearch: default dashboards tag to opensearch_tag  https://review.opendev.org/c/openstack/kolla-ansible/+/88059909:54
mmalchukmnasiadka the last wallaby patches can be merged:09:59
mmalchukhttps://review.opendev.org/q/project:openstack%252Fkolla+branch:stable%252Fwallaby+status:open09:59
mmalchukhttps://review.opendev.org/q/project:openstack/kayobe+branch:stable/wallaby+status:open10:00
mmalchukthanks10:00
mnasiadkaWallaby is EM, not a priority.10:01
mmalchuknot EM yet) 2 days to go10:05
mnasiadkait's EM for 6 months already10:07
mnasiadkahttps://review.opendev.org/c/openstack/releases/+/86232410:07
mmalchukits formal, agree)10:08
kevkommalchuk: done for k-a, i don't have a power for kayobe :P 10:08
kevkommalchuk: sorry, ^^ kolla10:09
mmalchukthanks10:17
mmalchukkevko please review https://review.opendev.org/q/I4b06eae75bf238f2f093bfb76ba37c7f75dfd61610:18
kevkommalchuk: hmm, this should be btw fixed in merge_configs.py 10:23
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Add support for multiple ceph files  https://review.opendev.org/c/openstack/kolla-ansible/+/87741310:29
mmalchukkevko interesting idea, but better fix in ceph10:37
kevkomnasiadka has a TODO in that file :D 10:38
mmalchukkevko I see, but this is only backports of the note already merged in master11:07
kevkommalchuk: done11:07
mmalchukthx11:08
kevkomnasiadka, bbezak: https://review.opendev.org/c/openstack/kolla-ansible/+/874729 small 11:11
mnasiadkabbezak is on vacation11:12
mnasiadkamaybe mgoddard can have a look11:13
opendevreviewMerged openstack/kolla-ansible stable/xena: Fix merge action plugins verbose output  https://review.opendev.org/c/openstack/kolla-ansible/+/88047711:14
opendevreviewVerification of a change to openstack/kolla master failed: docker: use python3-venv to create virtual env  https://review.opendev.org/c/openstack/kolla/+/87798611:14
kevkoif mgoddard can, it would be cool :) ... 11:28
opendevreviewMerged openstack/kayobe stable/wallaby: Support configuring VLANs with systemd-networkd syntax  https://review.opendev.org/c/openstack/kayobe/+/87776311:28
kevkomnasiadka: btw, should i abandon patches if we know it is obsolete , not going to be merged ? 11:28
kevkomnasiadka:  in kolla it's one page , kolla-ansible has several review pages 11:29
kevkohmm, rocky tests broken :( Failed to download metadata for repo \'baseos\': repomd.xml parser error: Parse error at line: 1 (Extra content at the end of the document\\n)", "exception":11:41
opendevreviewVerification of a change to openstack/kolla-ansible master failed: Fix maximum width of the DIB Multiline-YAML  https://review.opendev.org/c/openstack/kolla-ansible/+/83363311:43
opendevreviewMerged openstack/kolla stable/wallaby: nova-libvirt: Fix for missing libvirt-daemon-driver-nodedev package  https://review.opendev.org/c/openstack/kolla/+/88010011:48
opendevreviewMerged openstack/kolla-ansible stable/zed: Add note about removing leading tabs in ceph.conf files  https://review.opendev.org/c/openstack/kolla-ansible/+/88032011:48
opendevreviewMerged openstack/kolla-ansible stable/yoga: Add note about removing leading tabs in ceph.conf files  https://review.opendev.org/c/openstack/kolla-ansible/+/88032111:48
opendevreviewMerged openstack/kolla-ansible stable/xena: Add note about removing leading tabs in ceph.conf files  https://review.opendev.org/c/openstack/kolla-ansible/+/88032311:48
kevkommalchuk: do you have somewhere rockylinux running :)11:51
mmalchukkevko no11:53
mmalchukI thought it network issue11:53
kevkommalchuk: well, it failed in same way for example here https://review.opendev.org/c/openstack/kolla/+/87798611:54
mmalchukwill try to investigate11:55
mnasiadkait's a mirror issue, depending on the mirror situation should be more or less fixed now - they published broken metadata a couple of hours back12:02
mnasiadkabut works on my local env now12:02
mmalchuktested on rockylinux:9 docker image - dnf update works from my network12:04
mmalchukso I think recheck may be helps12:04
kevkommalchuk: let me know please 12:28
mmalchukkevko I can recheck you change if my passed12:51
mmalchukwell it passed: https://zuul.opendev.org/t/openstack/build/2793fff68e7745c0a0d003f80407267112:52
mmalchukkevko your change rechecked too12:52
kevkommalchuk: btw, use the reason after recheck keyword 12:53
kevkommalchuk: lately they somehow insist on it12:54
mmalchukkevko I know12:57
mmalchukabout bare rechecks12:57
SvenKieskeit's a good practice to analzye why rechecks are needed and optimize the CI system in general (flaky tests, known failing tests, etc)12:57
kevkoSvenKieske: +112:58
opendevreviewVerification of a change to openstack/kolla-ansible master failed: Fix maximum width of the DIB Multiline-YAML  https://review.opendev.org/c/openstack/kolla-ansible/+/83363312:59
fricklermnasiadka: just this patch, seems the new behavior hasn't found a lot of friends elsewhere, either ;) https://review.opendev.org/c/openstack/ansible-collections-openstack/+/87972513:27
opendevreviewWill Szumski proposed openstack/kayobe master: Revert back to michaelrigart.interfaces  https://review.opendev.org/c/openstack/kayobe/+/88074213:29
mnasiadkafrickler: that means once toolbox gets rebuilt after 2.1.0 release, we should be better ;)13:29
fricklerI was hoping we could get your fix merged sooner, but yes, that's the other option13:30
mnasiadkammalchuk: please don't do bare rechecks, "recheck <reason>" is ok, like "recheck rocky9 mirrors problems"13:41
mmalchukok13:48
kevkofolks, how i can build local modified pypi package ? if i tag commit id in git branch and call python3 setup.py sdist ...it still generate oslo.messaging-12.13.1.dev314:18
kevkohttps://peps.python.org/pep-0440/#adding-local-version-identifiers14:20
mnasiadkaand what's wrong with dev3?14:34
kevkohttps://docs.openstack.org/pbr/3.1.0/semver.html section 12 14:34
mnasiadkadev3 is still alphanumeric, unless you're complaining about the precedence thing14:36
kevkomnasiadka: what i am trying is -  1. take some lib  git (oslo.messaging), 2. clone branch for latest tag 12.13.0 , 3. cherry-pick patch what i would like to backport to mo local version modification , 4. tag top of the downstream git branch with tag 12.13.0+ultimum0 for example 5. run python3 setup.py bdist_wheel 6. pbr should work :( .... but it's building .dev version14:37
mnasiadkawell, then blame pbr14:38
kevkobecause i have several patchs i need to deliver into images (upstream merge takes so long ) .. so i would like to build patched packages ..build pypi repo with modifications ...and modify kolla project to be able accept --extra-repo to pypi command ...14:39
kevkomnasiadka: version parse will work ..so ..it's most clean way how to patch images in downstream ....14:42
kevko>>> parse('1.0.0') < parse('1.0.0+ultimum0')14:42
kevkoTrue14:42
kevko>>> parse('1.0.0') <= parse('1.0.0+ultimum0')14:42
kevkoTrue14:42
opendevreviewMerged openstack/kolla-ansible master: Fix maximum width of the DIB Multiline-YAML  https://review.opendev.org/c/openstack/kolla-ansible/+/83363314:51
opendevreviewMaksim Malchuk proposed openstack/kolla-ansible stable/zed: Fix maximum width of the DIB Multiline-YAML  https://review.opendev.org/c/openstack/kolla-ansible/+/88075614:54
opendevreviewMaksim Malchuk proposed openstack/kolla-ansible stable/yoga: Fix maximum width of the DIB Multiline-YAML  https://review.opendev.org/c/openstack/kolla-ansible/+/88075714:54
opendevreviewMaksim Malchuk proposed openstack/kolla-ansible stable/xena: Fix maximum width of the DIB Multiline-YAML  https://review.opendev.org/c/openstack/kolla-ansible/+/88075814:54
mmalchukmnasiadka please merge https://review.opendev.org/c/openstack/kayobe/+/833634 the brother in k-a merged successfully14:57
mmalchukthanks14:58
kevkomnasiadka: pbr abandoned long time ago :( https://review.opendev.org/q/topic:bug%252F145251515:17
opendevreviewMerged openstack/kayobe master: Stop using kolla-ansible bootstrap-servers  https://review.opendev.org/c/openstack/kayobe/+/82499815:50
opendevreviewMerged openstack/kolla master: docker: use python3-venv to create virtual env  https://review.opendev.org/c/openstack/kolla/+/87798615:52
mmalchukkevko change merged15:59
kevkommalchuk: thanks16:14
mmalchukso there was the network issue with mirrors)16:18
mnasiadkanot really network, faulty metadata published on the master mirror, and then it went haywire16:44
mnasiadkakevko: well then, you might want to restore - but I would first make sure you'll get any reviewers ;)16:45
opendevreviewVerification of a change to openstack/kolla-ansible master failed: Fix create sasl account before config file is ready  https://review.opendev.org/c/openstack/kolla-ansible/+/87990917:11
opendevreviewMerged openstack/kolla-ansible master: Fix create sasl account before config file is ready  https://review.opendev.org/c/openstack/kolla-ansible/+/87990918:57
mmalchukkevko congrats19:01
kevkohaha19:01
mmalchukpropose backports19:02
opendevreviewMaksim Malchuk proposed openstack/kolla-ansible stable/zed: Fix create sasl account before config file is ready  https://review.opendev.org/c/openstack/kolla-ansible/+/88077619:26
opendevreviewMaksim Malchuk proposed openstack/kolla-ansible stable/yoga: Fix create sasl account before config file is ready  https://review.opendev.org/c/openstack/kolla-ansible/+/88077719:28
opendevreviewMaksim Malchuk proposed openstack/kolla-ansible stable/xena: Fix create sasl account before config file is ready  https://review.opendev.org/c/openstack/kolla-ansible/+/88077819:29
opendevreviewMerged openstack/kolla-ansible master: Remove RabbitMQ ha-all policy when not required  https://review.opendev.org/c/openstack/kolla-ansible/+/87605320:24

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