Monday, 2024-02-05

AravindhI have a question with respect to kolla-build. I'm trying to FWaaS to Kolla source.py and build custom images. I see a flag kolla-build named openstack-release, what is the expected value here stable/2023.1 or just 2023.1.. 05:46
AravindhReason I'm asking is, when I build without this flag, I see an error when its trying to install fwaas package.. collecting pyzmq > 14.3.1 (from neutron-fwaas==18.0.0.0rc2.dev2) cpython version error05:51
Aravindhhttps://pastebin.com/GAiTKvsq05:52
frickleryou should not need to override that at all, instead use the matching kolla stable branch for the release you want to build06:13
opendevreviewRafal Lewandowski proposed openstack/kayobe master: Add Redfish rules to Ironic and Bifrost introspection  https://review.opendev.org/c/openstack/kayobe/+/90277206:42
Aravindhfrickler when its trying to install fwaas, its seems like its hitting a problem with cpython version error06:48
AravindhI installed this same fwaas package in one of the default container kolla builds automatically, its working fine there. Only when I try to build with this version I'm hitting this issue06:53
opendevreviewRoman Krček proposed openstack/kolla-ansible master: Split ipv4 and ipv6 systemctl config  https://review.opendev.org/c/openstack/kolla-ansible/+/90583107:47
mnasiadkamorning08:06
bbezakmrn08:20
opendevreviewBartosz Bezak proposed openstack/kolla stable/2023.2: [follow-up] Use full binary path when invoking ip  https://review.opendev.org/c/openstack/kolla/+/90757508:21
opendevreviewBartosz Bezak proposed openstack/kolla stable/2023.2: [follow-up] Use full binary path when invoking ip  https://review.opendev.org/c/openstack/kolla/+/90757508:22
*** jph0 is now known as jph08:25
kevkomorning08:30
*** ralonsoh_ is now known as ralonsoh08:56
opendevreviewMichal Nasiadka proposed openstack/kolla master: WIP: Switch to Ubuntu 24.04  https://review.opendev.org/c/openstack/kolla/+/90758910:01
opendevreviewMichal Nasiadka proposed openstack/kolla master: WIP: Switch to Ubuntu 24.04  https://review.opendev.org/c/openstack/kolla/+/90758910:07
kevkomnasiadka: https://review.opendev.org/q/topic:%22kolla-horizon-fix%22 << ? 10:17
opendevreviewMerged openstack/kolla stable/2023.2: Use full binary path when invoking ip  https://review.opendev.org/c/openstack/kolla/+/90756310:17
opendevreviewMerged openstack/kolla-ansible stable/zed: Enable instance usage audit only when ceilometer is enabled  https://review.opendev.org/c/openstack/kolla-ansible/+/90689910:22
opendevreviewMerged openstack/kolla-ansible stable/2023.2: Enable instance usage audit only when ceilometer is enabled  https://review.opendev.org/c/openstack/kolla-ansible/+/90689710:22
opendevreviewMerged openstack/kolla-ansible stable/2023.1: Enable instance usage audit only when ceilometer is enabled  https://review.opendev.org/c/openstack/kolla-ansible/+/90689810:22
kevkomnasiadka: done 10:31
kevkomnasiadka: donw10:41
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Rename horizon settings filenames  https://review.opendev.org/c/openstack/kolla-ansible/+/90720310:41
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Rework horizon role to support local_settings.d  https://review.opendev.org/c/openstack/kolla-ansible/+/90634710:41
opendevreviewMartin Hiner proposed openstack/kolla-ansible master: Refactor of getting list of containers that will also prepare function for migration scenario  https://review.opendev.org/c/openstack/kolla-ansible/+/90583710:42
opendevreviewMartin Hiner proposed openstack/kolla-ansible master: Add container engine migration scenario  https://review.opendev.org/c/openstack/kolla-ansible/+/83694110:42
opendevreviewMerged openstack/kayobe master: Ask for confirmation during overcloud deprovision  https://review.opendev.org/c/openstack/kayobe/+/90366510:50
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Rename horizon settings filenames  https://review.opendev.org/c/openstack/kolla-ansible/+/90720310:51
mnasiadkakevko: thanks, will have another look later today - have some time to look at https://review.opendev.org/c/openstack/kolla-ansible/+/905322 ?10:51
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Rework horizon role to support local_settings.d  https://review.opendev.org/c/openstack/kolla-ansible/+/90634710:51
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: [CI] Enable testing horizon  https://review.opendev.org/c/openstack/kolla-ansible/+/90771810:51
kevkomnasiadka: yeah i will check 10:52
opendevreviewSeunghun Lee proposed openstack/kayobe master: Make matching OVS agent independent to OVN if OVS is explicitly enabled  https://review.opendev.org/c/openstack/kayobe/+/90772111:18
opendevreviewMichal Arbet proposed openstack/kolla master: Rework horizon image to support local_settings.d  https://review.opendev.org/c/openstack/kolla/+/90633911:30
opendevreviewWill Szumski proposed openstack/kayobe master: Build letencrypt images  https://review.opendev.org/c/openstack/kayobe/+/90772311:33
opendevreviewWill Szumski proposed openstack/kayobe master: Build letsencrypt images  https://review.opendev.org/c/openstack/kayobe/+/90772311:33
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: openvswitch: add external-ids:hostname  https://review.opendev.org/c/openstack/kolla-ansible/+/90169412:43
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: openvswitch: use Ansible modules to set up bridge  https://review.opendev.org/c/openstack/kolla-ansible/+/90169512:45
opendevreviewVerification of a change to openstack/kolla stable/2023.2 failed: [follow-up] Use full binary path when invoking ip  https://review.opendev.org/c/openstack/kolla/+/90757513:20
kevkomnasiadka: how it can be horizon testing enabled on top of the change ? ..did you get that all CI jobs are passing only because testing horizon is turned off for now ? 13:33
mnasiadkarebase a patch on another patch, never done that?13:34
kevkomnasiadka: what ? if i will push a patch which enable horizon testing ...(and if merged) ..everything start to fail 13:35
mnasiadkapush a patch that fixes horizon and push the patch to enable testing on top of it (in a chain)13:36
kevkomnasiadka: hmm, ok ..this can work 13:36
mnasiadkahttps://review.opendev.org/c/openstack/kolla-ansible/+/907718?tab=change-view-tab-header-zuul-results-summary13:36
mnasiadkathis will fail?13:36
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Rename horizon settings filenames  https://review.opendev.org/c/openstack/kolla-ansible/+/90720313:37
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Rework horizon role to support local_settings.d  https://review.opendev.org/c/openstack/kolla-ansible/+/90634713:37
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: [CI] Enable testing horizon  https://review.opendev.org/c/openstack/kolla-ansible/+/90771813:37
kevkomnasiadka: no, this will pass ..but previous patches will pass because of horizon testing is disabled ...13:38
kevkomnasiadka: actually the last patch will verify previous two ..13:38
kevkomnasiadka: so if it is ok ..i am ok with it 13:38
mnasiadkayes, that's what we usually do13:38
kevkomnasiadka: but first two will pass only because testing is disabled ...13:39
kevkookay13:39
mnasiadkayes, and third one will enable testing and show everything passes - that's fine13:39
kevkookay13:39
opendevreviewBartosz Bezak proposed openstack/kayobe master: Reload NetworkManager on DNS config change  https://review.opendev.org/c/openstack/kayobe/+/90774013:59
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: openvswitch: use Ansible modules to set up bridge  https://review.opendev.org/c/openstack/kolla-ansible/+/90169514:11
opendevreviewVerification of a change to openstack/kolla stable/2023.2 failed: [follow-up] Use full binary path when invoking ip  https://review.opendev.org/c/openstack/kolla/+/90757514:50
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: openvswitch: use Ansible modules to set up bridge  https://review.opendev.org/c/openstack/kolla-ansible/+/90169514:54
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: openvswitch: use Ansible modules to set up bridge  https://review.opendev.org/c/openstack/kolla-ansible/+/90169515:24
opendevreviewRoman Krček proposed openstack/kolla-ansible master: Split ipv4 and ipv6 systemctl config  https://review.opendev.org/c/openstack/kolla-ansible/+/90583115:29
opendevreviewOpenStack Release Bot proposed openstack/ansible-collection-kolla master: reno: Update master for unmaintained/yoga  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/90781416:05
opendevreviewOpenStack Release Bot proposed openstack/kayobe master: reno: Update master for unmaintained/yoga  https://review.opendev.org/c/openstack/kayobe/+/90781916:06
opendevreviewOpenStack Release Bot proposed openstack/kolla-ansible master: reno: Update master for unmaintained/yoga  https://review.opendev.org/c/openstack/kolla-ansible/+/90782316:06
opendevreviewOpenStack Release Bot proposed openstack/kolla master: reno: Update master for unmaintained/yoga  https://review.opendev.org/c/openstack/kolla/+/90782516:07
kevkomnasiadka: priteau: how it is possible that such change https://review.opendev.org/c/openstack/kolla-ansible/+/865555 is merged without release note ? 16:17
priteaukevko: It's a no-op16:18
priteauUpstream default is True16:18
kevkohmm, yeah ..i was just wondering if release note shouldnt be included 16:18
rockeywhat would the release note add/suggest?16:27
*** ravlew is now known as Guest171516:29
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: neutron: rework state volume name  https://review.opendev.org/c/openstack/kolla-ansible/+/90456616:46
priteauWe tend not to add release notes for cleanup changes, as it would add a lot of unnecessary items16:46
SvenKieskewell it depends. if it requires user interaction it should get a release note, "cleanup" is not really a change category I can think in. you usually have: feature, bugfix, refactoring (maybe you mean this by cleanup?), docs only, ci only16:48
SvenKieskefor this case I agree there's not really a need for a reno, because there is no user interaction required16:49
SvenKieskeyou still could argue in favor of a reno for the case a user has set this value to either the default or something else and somehow notices that it doesn't do anything anymore. a reno would make this user faster aware of what changed. still this is only a faster transfer of information. the user can't actually change anything with this information, because the variable has been entirely removed.16:55
opendevreviewMerged openstack/kolla-ansible master: openvswitch: add external-ids:hostname  https://review.opendev.org/c/openstack/kolla-ansible/+/90169416:57
SvenKieskealso I would assume that the upstream project has this covered in it's renos? in that case there is - most of the time - no need to duplicate other project renos in our renos. only for critical stuff imho (user breaking/interaction enforcing stuff).16:58
kevko+116:58
SvenKieskebtw ansible-lint will change it's versioning scheme, guess something will break in the process: https://github.com/ansible/ansible-lint/issues/399616:58
mnasiadkacalendar versioning? whatever :)17:25
fricklerseems to be getting ever more popular these days. or months or seasons or whatever ;)17:29
SvenKieskefunnily enough the facts which are listed as good reasons to use CalVer on the CalVer site all don't apply to ansible-lint - imho :D17:37
SvenKieskehttps://calver.org/#when-to-use-calver17:37
mnasiadkafrickler, kevko, bbezak: I would like to get https://review.opendev.org/c/openstack/kolla-ansible/+/901695 in - that's required for OVN BGP agent work that I signed myself to :)18:32
fricklerunrelated: I think we'll need to squash https://review.opendev.org/c/openstack/ansible-collection-kolla/+/907329 and https://review.opendev.org/c/openstack/ansible-collection-kolla/+/907814 as gate fixes now18:34
mnasiadkaright18:35
opendevreviewMerged openstack/kolla-ansible master: reno: Update master for unmaintained/yoga  https://review.opendev.org/c/openstack/kolla-ansible/+/90782318:36
opendevreviewMerged openstack/kolla master: reno: Update master for unmaintained/yoga  https://review.opendev.org/c/openstack/kolla/+/90782518:36
opendevreviewMichal Nasiadka proposed openstack/kolla master: WIP: Add ovn-bgp-agent / FRR / Horizon BGPVPN dashboard support  https://review.opendev.org/c/openstack/kolla/+/89161718:37
opendevreviewMichal Nasiadka proposed openstack/kolla master: Add ovn-bgp-agent / FRR / Horizon BGPVPN dashboard  https://review.opendev.org/c/openstack/kolla/+/89161718:38
mnasiadkaok, needs a bit more love tomorrow and will move to k-a18:39
opendevreviewMerged openstack/ansible-collection-kolla master: CI: Bump ansible-core and ansible-lint versions  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/90732918:41
opendevreviewMichal Nasiadka proposed openstack/kolla master: openvswitch: Drop ovs_ensure_configured.sh  https://review.opendev.org/c/openstack/kolla/+/90790118:43
atmarkhello, fluentd container for Xena keeps restarting19:54
atmark/opt/td-agent/lib/ruby/2.7.0/rubygems/specification.rb:2247:in `raise_if_conflicts': Unable to activate fluent-plugin-elasticsearch-5.3.0, because faraday-1.10.3 conflicts with faraday (>= 2.0.0), faraday-excon-1.1.0 conflicts with faraday-excon (>= 2.0.0) (Gem::ConflictError)19:54
atmarklaunchpad is down :( 20:15
atmarkcan't view this bug report  https://bugs.launchpad.net/bugs/201955720:16
opendevreviewVerification of a change to openstack/kolla stable/2023.2 failed: [follow-up] Use full binary path when invoking ip  https://review.opendev.org/c/openstack/kolla/+/90757521:53
guesswhat[m]horizon_public_endpoint is setting also port when tls is enabled, this is resulting in cors errors in glance23:28

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