opendevreview | Michal Nasiadka proposed openstack/kolla-ansible stable/zed: opensearch: default dashboards tag to opensearch_tag https://review.opendev.org/c/openstack/kolla-ansible/+/880598 | 04:57 |
---|---|---|
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible stable/yoga: opensearch: default dashboards tag to opensearch_tag https://review.opendev.org/c/openstack/kolla-ansible/+/880599 | 04:58 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible stable/yoga: opensearch: default dashboards tag to opensearch_tag https://review.opendev.org/c/openstack/kolla-ansible/+/880599 | 05:00 |
rockey | god morning team | 05:17 |
rockey | starting 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 |
frickler | rockey: pointing to a review that exposes the issue might be a good first step | 07:06 |
mnasiadka | morning | 07:07 |
rockey | frickler: more like general usage, i cannot get a previous post added into releasenotes/notes/ listed when running `reno list/report` | 07:08 |
rockey | nor 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 |
frickler | that doesn't sound very kolla-specific. though likely reno knowledge is as rare elsewhere in the openstack world nowadays | 07:10 |
rockey | yeah, hence the fuzzy question of usage, more than kolla specific | 07:11 |
rockey | glola | 07:12 |
rockey | sorry :) | 07:12 |
frickler | mnasiadka: 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 jobs | 07:17 |
mnasiadka | frickler: 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 |
opendevreview | Verification 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/+/880598 | 09:07 |
kevko | hrw: replied | 09:48 |
mmalchuk | mnasiadka could you please merge last patch: https://review.opendev.org/q/topic:fix-merge-action-plugins | 09:50 |
mmalchuk | and review https://review.opendev.org/q/topic:dib-yaml-width | 09:50 |
mmalchuk | thanks | 09:51 |
kevko | mnasiadka: 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 |
kevko | thanks | 09:54 |
opendevreview | Verification 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/+/880599 | 09:54 |
mmalchuk | mnasiadka the last wallaby patches can be merged: | 09:59 |
mmalchuk | https://review.opendev.org/q/project:openstack%252Fkolla+branch:stable%252Fwallaby+status:open | 09:59 |
mmalchuk | https://review.opendev.org/q/project:openstack/kayobe+branch:stable/wallaby+status:open | 10:00 |
mmalchuk | thanks | 10:00 |
mnasiadka | Wallaby is EM, not a priority. | 10:01 |
mmalchuk | not EM yet) 2 days to go | 10:05 |
mnasiadka | it's EM for 6 months already | 10:07 |
mnasiadka | https://review.opendev.org/c/openstack/releases/+/862324 | 10:07 |
mmalchuk | its formal, agree) | 10:08 |
kevko | mmalchuk: done for k-a, i don't have a power for kayobe :P | 10:08 |
kevko | mmalchuk: sorry, ^^ kolla | 10:09 |
mmalchuk | thanks | 10:17 |
mmalchuk | kevko please review https://review.opendev.org/q/I4b06eae75bf238f2f093bfb76ba37c7f75dfd616 | 10:18 |
kevko | mmalchuk: hmm, this should be btw fixed in merge_configs.py | 10:23 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add support for multiple ceph files https://review.opendev.org/c/openstack/kolla-ansible/+/877413 | 10:29 |
mmalchuk | kevko interesting idea, but better fix in ceph | 10:37 |
kevko | mnasiadka has a TODO in that file :D | 10:38 |
mmalchuk | kevko I see, but this is only backports of the note already merged in master | 11:07 |
kevko | mmalchuk: done | 11:07 |
mmalchuk | thx | 11:08 |
kevko | mnasiadka, bbezak: https://review.opendev.org/c/openstack/kolla-ansible/+/874729 small | 11:11 |
mnasiadka | bbezak is on vacation | 11:12 |
mnasiadka | maybe mgoddard can have a look | 11:13 |
opendevreview | Merged openstack/kolla-ansible stable/xena: Fix merge action plugins verbose output https://review.opendev.org/c/openstack/kolla-ansible/+/880477 | 11:14 |
opendevreview | Verification of a change to openstack/kolla master failed: docker: use python3-venv to create virtual env https://review.opendev.org/c/openstack/kolla/+/877986 | 11:14 |
kevko | if mgoddard can, it would be cool :) ... | 11:28 |
opendevreview | Merged openstack/kayobe stable/wallaby: Support configuring VLANs with systemd-networkd syntax https://review.opendev.org/c/openstack/kayobe/+/877763 | 11:28 |
kevko | mnasiadka: btw, should i abandon patches if we know it is obsolete , not going to be merged ? | 11:28 |
kevko | mnasiadka: in kolla it's one page , kolla-ansible has several review pages | 11:29 |
kevko | hmm, 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 |
opendevreview | Verification 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/+/833633 | 11:43 |
opendevreview | Merged openstack/kolla stable/wallaby: nova-libvirt: Fix for missing libvirt-daemon-driver-nodedev package https://review.opendev.org/c/openstack/kolla/+/880100 | 11:48 |
opendevreview | Merged openstack/kolla-ansible stable/zed: Add note about removing leading tabs in ceph.conf files https://review.opendev.org/c/openstack/kolla-ansible/+/880320 | 11:48 |
opendevreview | Merged openstack/kolla-ansible stable/yoga: Add note about removing leading tabs in ceph.conf files https://review.opendev.org/c/openstack/kolla-ansible/+/880321 | 11:48 |
opendevreview | Merged openstack/kolla-ansible stable/xena: Add note about removing leading tabs in ceph.conf files https://review.opendev.org/c/openstack/kolla-ansible/+/880323 | 11:48 |
kevko | mmalchuk: do you have somewhere rockylinux running :) | 11:51 |
mmalchuk | kevko no | 11:53 |
mmalchuk | I thought it network issue | 11:53 |
kevko | mmalchuk: well, it failed in same way for example here https://review.opendev.org/c/openstack/kolla/+/877986 | 11:54 |
mmalchuk | will try to investigate | 11:55 |
mnasiadka | it's a mirror issue, depending on the mirror situation should be more or less fixed now - they published broken metadata a couple of hours back | 12:02 |
mnasiadka | but works on my local env now | 12:02 |
mmalchuk | tested on rockylinux:9 docker image - dnf update works from my network | 12:04 |
mmalchuk | so I think recheck may be helps | 12:04 |
kevko | mmalchuk: let me know please | 12:28 |
mmalchuk | kevko I can recheck you change if my passed | 12:51 |
mmalchuk | well it passed: https://zuul.opendev.org/t/openstack/build/2793fff68e7745c0a0d003f804072671 | 12:52 |
mmalchuk | kevko your change rechecked too | 12:52 |
kevko | mmalchuk: btw, use the reason after recheck keyword | 12:53 |
kevko | mmalchuk: lately they somehow insist on it | 12:54 |
mmalchuk | kevko I know | 12:57 |
mmalchuk | about bare rechecks | 12:57 |
SvenKieske | it'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 |
kevko | SvenKieske: +1 | 12:58 |
opendevreview | Verification 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/+/833633 | 12:59 |
frickler | mnasiadka: 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/+/879725 | 13:27 |
opendevreview | Will Szumski proposed openstack/kayobe master: Revert back to michaelrigart.interfaces https://review.opendev.org/c/openstack/kayobe/+/880742 | 13:29 |
mnasiadka | frickler: that means once toolbox gets rebuilt after 2.1.0 release, we should be better ;) | 13:29 |
frickler | I was hoping we could get your fix merged sooner, but yes, that's the other option | 13:30 |
mnasiadka | mmalchuk: please don't do bare rechecks, "recheck <reason>" is ok, like "recheck rocky9 mirrors problems" | 13:41 |
mmalchuk | ok | 13:48 |
kevko | folks, 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.dev3 | 14:18 |
kevko | https://peps.python.org/pep-0440/#adding-local-version-identifiers | 14:20 |
mnasiadka | and what's wrong with dev3? | 14:34 |
kevko | https://docs.openstack.org/pbr/3.1.0/semver.html section 12 | 14:34 |
mnasiadka | dev3 is still alphanumeric, unless you're complaining about the precedence thing | 14:36 |
kevko | mnasiadka: 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 version | 14:37 |
mnasiadka | well, then blame pbr | 14:38 |
kevko | because 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 |
kevko | mnasiadka: 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 |
kevko | True | 14:42 |
kevko | >>> parse('1.0.0') <= parse('1.0.0+ultimum0') | 14:42 |
kevko | True | 14:42 |
opendevreview | Merged openstack/kolla-ansible master: Fix maximum width of the DIB Multiline-YAML https://review.opendev.org/c/openstack/kolla-ansible/+/833633 | 14:51 |
opendevreview | Maksim Malchuk proposed openstack/kolla-ansible stable/zed: Fix maximum width of the DIB Multiline-YAML https://review.opendev.org/c/openstack/kolla-ansible/+/880756 | 14:54 |
opendevreview | Maksim Malchuk proposed openstack/kolla-ansible stable/yoga: Fix maximum width of the DIB Multiline-YAML https://review.opendev.org/c/openstack/kolla-ansible/+/880757 | 14:54 |
opendevreview | Maksim Malchuk proposed openstack/kolla-ansible stable/xena: Fix maximum width of the DIB Multiline-YAML https://review.opendev.org/c/openstack/kolla-ansible/+/880758 | 14:54 |
mmalchuk | mnasiadka please merge https://review.opendev.org/c/openstack/kayobe/+/833634 the brother in k-a merged successfully | 14:57 |
mmalchuk | thanks | 14:58 |
kevko | mnasiadka: pbr abandoned long time ago :( https://review.opendev.org/q/topic:bug%252F1452515 | 15:17 |
opendevreview | Merged openstack/kayobe master: Stop using kolla-ansible bootstrap-servers https://review.opendev.org/c/openstack/kayobe/+/824998 | 15:50 |
opendevreview | Merged openstack/kolla master: docker: use python3-venv to create virtual env https://review.opendev.org/c/openstack/kolla/+/877986 | 15:52 |
mmalchuk | kevko change merged | 15:59 |
kevko | mmalchuk: thanks | 16:14 |
mmalchuk | so there was the network issue with mirrors) | 16:18 |
mnasiadka | not really network, faulty metadata published on the master mirror, and then it went haywire | 16:44 |
mnasiadka | kevko: well then, you might want to restore - but I would first make sure you'll get any reviewers ;) | 16:45 |
opendevreview | Verification 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/+/879909 | 17:11 |
opendevreview | Merged openstack/kolla-ansible master: Fix create sasl account before config file is ready https://review.opendev.org/c/openstack/kolla-ansible/+/879909 | 18:57 |
mmalchuk | kevko congrats | 19:01 |
kevko | haha | 19:01 |
mmalchuk | propose backports | 19:02 |
opendevreview | Maksim Malchuk proposed openstack/kolla-ansible stable/zed: Fix create sasl account before config file is ready https://review.opendev.org/c/openstack/kolla-ansible/+/880776 | 19:26 |
opendevreview | Maksim Malchuk proposed openstack/kolla-ansible stable/yoga: Fix create sasl account before config file is ready https://review.opendev.org/c/openstack/kolla-ansible/+/880777 | 19:28 |
opendevreview | Maksim Malchuk proposed openstack/kolla-ansible stable/xena: Fix create sasl account before config file is ready https://review.opendev.org/c/openstack/kolla-ansible/+/880778 | 19:29 |
opendevreview | Merged openstack/kolla-ansible master: Remove RabbitMQ ha-all policy when not required https://review.opendev.org/c/openstack/kolla-ansible/+/876053 | 20:24 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!