| mnasiadka | frickler: it seems validate-all-file.py was not validating block tasks - can you have a look in https://review.opendev.org/c/openstack/kolla-ansible/+/894155 ? | 05:25 |
|---|---|---|
| mnasiadka | I'll rebase it on the fixed code now, so it doesn't fail - but you check the historical run for logs (https://zuul.opendev.org/t/openstack/build/3656d5fa57c34c60aebcfdb8a849dd02) | 05:26 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: add block support to validate-all-file.py https://review.opendev.org/c/openstack/kolla-ansible/+/894155 | 05:26 |
| opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: build: distutils.StrictVersion() is deprecated https://review.opendev.org/c/openstack/kolla/+/886741 | 05:45 |
| opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: base: switch to new Grafana key https://review.opendev.org/c/openstack/kolla/+/886086 | 05:46 |
| opendevreview | Marcin Juszkiewicz proposed openstack/kolla-ansible master: init-runonce: update CirrOS to 0.6.2 https://review.opendev.org/c/openstack/kolla-ansible/+/884849 | 05:46 |
| hrw | rebased | 05:46 |
| frickler | mnasiadka: I don't understand this. since when is "become: true" on the block not working? and if it doesn't work, why isn't it dropped? | 05:48 |
| opendevreview | Merged openstack/kolla-ansible stable/yoga: [yoga only] followup ovn clustering fix https://review.opendev.org/c/openstack/kolla-ansible/+/894194 | 06:37 |
| mnasiadka | frickler: right, missed that - need to update validate-all-file, argh | 06:40 |
| mnasiadka | frickler: need more coffee ;) | 06:59 |
| opendevreview | Dr. Jens Harbott proposed openstack/kolla-ansible master: DNM: Test octavia amp subnet https://review.opendev.org/c/openstack/kolla-ansible/+/894219 | 07:05 |
| frickler | mnasiadka: I wasn't sure if it was me not seeing something important | 07:06 |
| frickler | also fyi ^^ is likely a regression in a-o-c | 07:07 |
| mnasiadka | No breakage no fun ;) | 07:07 |
| frickler | I hope https://review.opendev.org/c/openstack/ansible-collections-openstack/+/894254 is a reproducer | 07:13 |
| frickler | mnasiadka: the fun thing is that this seems to be breaking things only with OVN, while octavia in our CI still runs on OVS | 07:14 |
| Viii | hello, how can I overwrite a newer version of the plugin when creating an image? | 07:19 |
| Viii | "kolla-build.conf" "[neutron-server-plugin-vpnaas-agent]" "type = local" "location=path/neutron-vpnaas-22.1.0.dev3.tar.gz | 07:20 |
| Viii | I have an error while building image | 07:20 |
| Viii | "ERROR: Cannot install neutron-vpnaas 22.0.0.0rc2.dev2 (from /plugins/neutron-vpnaas-22.0.0.0rc2.dev2) and neutron-vpnaas 22.1.0.dev3 (from /plugins/neutron-vpnaas-22.1.0.dev3) because these package versions have conflicting dependencies." | 07:20 |
| mnasiadka | frickler: so you're saying we should have an ovn flavor of the octavia job? | 07:35 |
| mnasiadka | or maybe we need to reorganise the CI jobs and include octavia in the standard ones | 07:36 |
| mmalchuk | Viii use git | 07:38 |
| mmalchuk | type = git | 07:39 |
| mmalchuk | location = http://.... | 07:39 |
| mmalchuk | reference = your_branch | 07:39 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: add block support to validate-all-file.py https://review.opendev.org/c/openstack/kolla-ansible/+/894155 | 07:40 |
| frickler | mnasiadka: maybe we should just switch to OVN everywhere. or maybe next cycle, not sure yet | 07:41 |
| mnasiadka | frickler: first we need a precheck to not break existing ML2/OVS environments, but we have someone working on that in SHPC - once that's done, we could change the default. | 07:42 |
| frickler | are you thinking about migrating OVS to OVN? otherwise the precheck would be easy. and I'm not convinced migration during normal kolla operations is possible, I think it might need to be a dediccated tools | 07:44 |
| Viii | I need to use VPNaaS for OVN | 07:47 |
| frickler | Viii: I don't think that is working yet | 07:48 |
| Viii | version neutron-vpnaas-22.1.0.dev3.tar.gz supports this | 07:48 |
| frickler | with https://review.opendev.org/c/openstack/neutron-vpnaas/+/765353 still open I very much doubt this. anyway there's no support for it in kolla yet I guess | 07:51 |
| Viii | https://pastebin.com/Jd0YjeKn | 07:51 |
| Viii | I know it doesn't support it, so I want to build an image with the new version of the plugin | 07:53 |
| frickler | then what mmalchuk says. with "your_branch=master" I guess | 07:55 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: add block support to validate-all-file.py https://review.opendev.org/c/openstack/kolla-ansible/+/894155 | 08:03 |
| mnasiadka | frickler: fixed ^^ | 08:03 |
| mnasiadka | argh, pep8 | 08:07 |
| frickler | linting the linter failed? ;) | 08:09 |
| mnasiadka | pep8: OK (14.92=setup[12.01]+cmd[2.91] seconds) | 08:12 |
| mnasiadka | should be good now ;) | 08:12 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: add block support to validate-all-file.py https://review.opendev.org/c/openstack/kolla-ansible/+/894155 | 08:12 |
| mnasiadka | I want to backport that all the way to stable/yoga and add a check there for not doing kolla_container_engine - because we backported 5th patch with that and I had to do followup again :D | 08:12 |
| SvenKieske | currently going through the version bumps for bobcat cycle, do we still need CentOS 7 Repositories? https://opendev.org/openstack/kolla/src/branch/master/docker/base/crmsh.repo | 09:27 |
| SvenKieske | they are disabled, can't we delete this from master? | 09:28 |
| mnasiadka | huh | 09:30 |
| mnasiadka | surely we can, question what was installed from that repo | 09:30 |
| SvenKieske | don't know, I don't use EL since...5-8 years? | 09:31 |
| SvenKieske | I'll provide a changeset and we can discuss there with the experts | 09:32 |
| SvenKieske | seemingly was used for corsync/pacemaker ha stuff | 09:32 |
| opendevreview | Sven Kieske proposed openstack/kolla master: delete seemingly unused EL7 repo https://review.opendev.org/c/openstack/kolla/+/894290 | 09:35 |
| SvenKieske | do we run EL7 anywhere still? some old yoga stuff maybe? | 09:36 |
| SvenKieske | okay, this is referenced by the hacluster role, but that role seems to not have been touched since 4 years, and it's still using EL7 packages? does this even work? | 09:40 |
| mnasiadka | no | 09:41 |
| mnasiadka | EL7 was Wallaby I think? | 09:41 |
| SvenKieske | the referenced URL is broken; I doubt this "hacluster" role works at all http://download.opensuse.org/repositories/network:/ha-clustering:/Stable/CentOS_CentOS-7/ is wrong | 09:41 |
| SvenKieske | correct URL: http://download.opensuse.org/repositories/network:/ha-clustering:/Stable/CentOS_7/ | 09:42 |
| mnasiadka | Is that repo file used anywhere? Like do we add it in base image? | 09:42 |
| SvenKieske | should I extend the changeset to delete that role? :D | 09:42 |
| SvenKieske | the base url of that repo is a 404, so I hope we don't use it..investigating | 09:42 |
| mnasiadka | we used that for masakari, but might be crmsh is now in some standard CentOS/Rocky repos | 09:43 |
| SvenKieske | this got introduced here: https://review.opendev.org/c/openstack/kolla/+/668765 | 09:43 |
| mnasiadka | if hacluster containers are buildable, then I guess everything is all right and it's just a dangling file | 09:44 |
| SvenKieske | # FIXME(mgoddard): Not available for CentOS 8 yet. | 09:44 |
| SvenKieske | # 'crmsh.repo', | 09:44 |
| SvenKieske | that file is nowhere else referenced, so I guess it's save to delete | 09:45 |
| SvenKieske | at least not by direct name | 09:45 |
| SvenKieske | okay, nice, moving on, changeset is up for review if anybody cares :) now upgrade those versions.. | 09:46 |
| SvenKieske | mhm, I guess we don't want to switch from mariadb 10.11 to 11.1? | 09:54 |
| SvenKieske | I'll include it, just for the sake of it | 09:54 |
| mnasiadka | mmalchuk: I thought we agreed we could EOL Xena beginning of September? :) | 09:59 |
| SvenKieske | doing this, I'm wondering: why do we mix packages from distributions with packages from upstream on another distribution? wouldn't it be more secure if we - in cases where $dist doesn't offer $version we switch all affected repos to upstream, to get the same compilation flags etc? | 10:00 |
| mnasiadka | SvenKieske: we use mariadb LTS, 11.1 is not LTS, right? | 10:00 |
| mmalchuk | mnasiadka as I said will support Xena until the end | 10:00 |
| SvenKieske | ah only LTS, wasn't aware, makes sense | 10:00 |
| mnasiadka | https://mariadb.com/kb/en/mariadb-server-release-dates/ | 10:00 |
| mnasiadka | yeah, 10.11 is long term stable | 10:00 |
| mmalchuk | mnasiadka agreed, but lets merge fixes until tagged | 10:01 |
| mmalchuk | mnasiadka ok? | 10:01 |
| mnasiadka | sure | 10:01 |
| mnasiadka | bbezak: want to raise xena EOL patch? :) | 10:01 |
| mmalchuk | bbezak want review and merge last Xena patches? please | 10:02 |
| mmalchuk | kolla: https://review.opendev.org/c/openstack/kolla/+/890541 and https://review.opendev.org/c/openstack/kolla/+/893854 | 10:03 |
| mmalchuk | kolla-ansible: https://review.opendev.org/c/openstack/kolla-ansible/+/893422 | 10:03 |
| mmalchuk | https://review.opendev.org/c/openstack/kolla-ansible/+/893954 and https://review.opendev.org/c/openstack/kolla-ansible/+/893441 | 10:04 |
| mmalchuk | thats all | 10:04 |
| mmalchuk | kayobe still lack of review even in master ;( | 10:05 |
| bbezak | sure mnasiadka | 10:08 |
| SvenKieske | oh no, td-agent upstream got renamed to "fluent-package" new LTS Version is 5 instead of 4.. | 10:08 |
| mnasiadka | nice, good bump :) | 10:10 |
| frickler | rocky9 seems to be acting up, if someone wants to take a look https://zuul.opendev.org/t/openstack/build/5efa57a4ffde47518a5a5d37f159d9ca | 10:14 |
| mnasiadka | frickler: that's weird, but it might happen when there are no Nova services registered, I've seen this in the past - maybe we need to have a look | 10:20 |
| opendevreview | Merged openstack/kolla master: Add python 3.10 to setup.cfg metadata https://review.opendev.org/c/openstack/kolla/+/886547 | 10:35 |
| opendevreview | Merged openstack/kolla stable/yoga: Add comprehensive checks for container restarts https://review.opendev.org/c/openstack/kolla/+/892878 | 10:35 |
| opendevreview | Merged openstack/kolla stable/2023.1: Revert "Disable build of Gnocchi containers" https://review.opendev.org/c/openstack/kolla/+/890538 | 11:17 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: bump up ansible-lint https://review.opendev.org/c/openstack/kolla-ansible/+/894295 | 11:20 |
| opendevreview | Merged openstack/kolla-ansible master: Remove duplicate whitespace https://review.opendev.org/c/openstack/kolla-ansible/+/893045 | 11:27 |
| opendevreview | Merged openstack/kolla stable/zed: Revert "Disable build of Gnocchi containers" https://review.opendev.org/c/openstack/kolla/+/890539 | 11:27 |
| opendevreview | Merged openstack/kolla-ansible stable/xena: Use better default bind address for ironic-tftp https://review.opendev.org/c/openstack/kolla-ansible/+/893422 | 11:38 |
| opendevreview | Merged openstack/kolla-ansible stable/xena: Fix designate-producers not running properly https://review.opendev.org/c/openstack/kolla-ansible/+/893441 | 11:38 |
| mmalchuk | mnasiadka you miss the Yoga change https://review.opendev.org/c/openstack/kolla/+/890540 | 11:46 |
| mmalchuk | cores, please +W last two kolla change: https://review.opendev.org/q/project:openstack/kolla+branch:stable/xena+status:open | 11:48 |
| mmalchuk | and the last one in kolla-ansible: https://review.opendev.org/q/project:openstack/kolla-ansible+branch:stable/xena+status:open | 11:50 |
| opendevreview | John Garbutt proposed openstack/kolla-ansible master: WIP: Overide of [default]host in nova-compute-ironic https://review.opendev.org/c/openstack/kolla-ansible/+/894298 | 13:24 |
| opendevreview | Merged openstack/kolla-ansible stable/zed: Use better default bind address for ironic-tftp https://review.opendev.org/c/openstack/kolla-ansible/+/893386 | 13:56 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.1: Use better default bind address for ironic-tftp https://review.opendev.org/c/openstack/kolla-ansible/+/893385 | 13:58 |
| opendevreview | Merged openstack/kolla-ansible stable/xena: Fix typo in endpoint influxdb_internal_endpoint variable https://review.opendev.org/c/openstack/kolla-ansible/+/893954 | 14:00 |
| opendevreview | Doug Szumski proposed openstack/kolla-ansible master: Support customising Nova config at the individual service level https://review.opendev.org/c/openstack/kolla-ansible/+/894298 | 14:15 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: CI: add q35 hardware machine type to tests https://review.opendev.org/c/openstack/kolla-ansible/+/894199 | 14:28 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: Configure Nova libvirt.num_pcie_ports to 16 by default https://review.opendev.org/c/openstack/kolla-ansible/+/768301 | 14:30 |
| opendevreview | Merged openstack/kolla stable/xena: Revert "Disable build of Gnocchi containers" https://review.opendev.org/c/openstack/kolla/+/890541 | 14:41 |
| opendevreview | Will Szumski proposed openstack/kolla-ansible master: Support exposing prometheus_server externally https://review.opendev.org/c/openstack/kolla-ansible/+/828695 | 14:50 |
| opendevreview | Merged openstack/kolla stable/xena: Improve kolla-toolbox offline support. https://review.opendev.org/c/openstack/kolla/+/893854 | 15:00 |
| opendevreview | Will Szumski proposed openstack/kolla-ansible master: Support exposing prometheus_server externally https://review.opendev.org/c/openstack/kolla-ansible/+/828695 | 15:18 |
| opendevreview | Jake Hutchinson proposed openstack/kolla-ansible master: Ironic parameter rework and default NTP server https://review.opendev.org/c/openstack/kolla-ansible/+/893031 | 15:49 |
| opendevreview | Merged openstack/kolla-ansible stable/yoga: Fix typo in endpoint influxdb_internal_endpoint variable https://review.opendev.org/c/openstack/kolla-ansible/+/893953 | 15:56 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: CI: add q35 hardware machine type to tests https://review.opendev.org/c/openstack/kolla-ansible/+/894199 | 16:03 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: Configure Nova libvirt.num_pcie_ports to 16 by default https://review.opendev.org/c/openstack/kolla-ansible/+/768301 | 16:05 |
| spatel | is barbican-ui is supported in kolla-ansible? | 16:13 |
| spatel | mmalchuk ^ | 16:13 |
| spatel | I have seen its not so very curious https://review.opendev.org/c/openstack/kolla-ansible/+/891284 | 16:13 |
| opendevreview | Christian Berendt proposed openstack/kolla-ansible master: senlin: add missing cafile parameter https://review.opendev.org/c/openstack/kolla-ansible/+/894383 | 16:14 |
| opendevreview | Christian Berendt proposed openstack/kolla-ansible master: senlin: add missing cafile parameter https://review.opendev.org/c/openstack/kolla-ansible/+/894383 | 16:24 |
| opendevreview | Will Szumski proposed openstack/kolla-ansible master: Support exposing prometheus_server externally https://review.opendev.org/c/openstack/kolla-ansible/+/828695 | 16:31 |
| opendevreview | Juan Pablo Suazo proposed openstack/kolla-ansible master: Deploy Glance with S3 backend support https://review.opendev.org/c/openstack/kolla-ansible/+/844614 | 16:34 |
| opendevreview | Will Szumski proposed openstack/kolla-ansible master: Support exposing prometheus_server externally https://review.opendev.org/c/openstack/kolla-ansible/+/828695 | 16:45 |
| opendevreview | Will Szumski proposed openstack/kolla-ansible master: Support exposing prometheus_server externally https://review.opendev.org/c/openstack/kolla-ansible/+/828695 | 16:55 |
| opendevreview | Doug Szumski proposed openstack/kolla-ansible master: Support customising Nova config at the individual service level https://review.opendev.org/c/openstack/kolla-ansible/+/894298 | 16:57 |
| atmark | hello, I'm trying to get designate playbook to have designate-mdns service listen on `dns_interface` and `kolla_external_vip_interface` but I'm undefined network. | 18:16 |
| atmark | I'm getting undefined networK* | 18:17 |
| atmark | I have this in my designate.conf | 18:17 |
| atmark | listen = {{ 'dns' | kolla_address | put_address_in_context('url') }}:{{ designate_mdns_port }}, {{ 'kolla_external_vip' | kolla_address | put_address_in_context('url') }}:{{ designate_mdns_port }} | 18:17 |
| atmark | dns_interface defaults to network_interface while kolla_external_vip_interface is mapped to an actual interface | 18:18 |
| atmark | error I'm getting ` FAILED! => {"msg": "Address family undefined for network 'kolla_external_vip' (set 'kolla_external_vip_address_family')"} ` | 18:19 |
| atmark | I figured it out | 18:44 |
| atmark | listen = {{ 'dns' | kolla_address | put_address_in_context('url') }}:{{ designate_mdns_port }}, {{ kolla_external_vip_address }}:{{ designate_mdns_port }} | 18:44 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!