gokhani | good morning folks, have yo seen this https://bugs.launchpad.net/kuryr-libnetwork/+bug/2048064 ? | 07:48 |
---|---|---|
gokhani | with ovs zun is not working | 07:48 |
noonedeadpunk | gokhani: I think this should have been addressed.... | 07:57 |
noonedeadpunk | oh, well, maybe not and it's different issue I was thinking about | 07:58 |
noonedeadpunk | Yeah, was thinking about this: https://review.opendev.org/c/openstack/kuryr-libnetwork/+/895662 | 07:58 |
gokhani | bug author tried with 2023.1 with OSA | 07:58 |
gokhani | I tried with victoria and got same error. | 07:59 |
noonedeadpunk | gokhani: I think you should be able to change kuryr user to root to workaround the issue | 08:03 |
noonedeadpunk | I've also replied to the bug with the same suggestion | 08:03 |
noonedeadpunk | so worth trying to change zun_kuryr_system_user_name to root for instance | 08:03 |
gokhani | yes I can try this now | 08:16 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Add openstack_resources role skeleton https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/878794 | 08:25 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/2023.2: Skip installing curl for EL https://review.opendev.org/c/openstack/openstack-ansible/+/904845 | 08:32 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/2023.1: Skip installing curl for EL https://review.opendev.org/c/openstack/openstack-ansible/+/904846 | 08:32 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/zed: Skip installing curl for EL https://review.opendev.org/c/openstack/openstack-ansible/+/904847 | 08:33 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Return back /healtcheck URI verification https://review.opendev.org/c/openstack/openstack-ansible/+/904941 | 08:55 |
noonedeadpunk | ^ results in swift failing, as / returns 404 | 09:00 |
opendevreview | Merged openstack/openstack-ansible-ceph_client master: Add backwards compatibility of ceph_components format https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/904499 | 10:27 |
jrosser | good morning | 10:30 |
opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ceph_client stable/2023.2: Add backwards compatibility of ceph_components format https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/904849 | 10:30 |
damiandabrowski | hi! | 10:44 |
noonedeadpunk | o/ | 10:59 |
jrosser | the patch which adds blazar filters shows some missing functionality? | 11:44 |
jrosser | we can enable some set of filters with `enabled_filters = {{ _nova_scheduler_filters | join(',') }}` | 11:45 |
jrosser | but `available_filters` does not seem to get updated to account for `nova_scheduler_extra_filters` | 11:46 |
jrosser | if you want to specify your own like this https://docs.openstack.org/nova/2023.2/admin/scheduling.html#writing-your-own-filter | 11:47 |
noonedeadpunk | jrosser: yeah, I was thinking of adding another variable to manage `available_filters` but then scraped a draft for some reason | 11:59 |
jrosser | it's probably because the default available_filters is all the builtins | 11:59 |
noonedeadpunk | yup | 11:59 |
jrosser | so if thats what you need then it's ok to just adjust `nova_scheduler_extra_filters` | 12:00 |
noonedeadpunk | and then it's also matter of isntalling extra pip package | 12:00 |
noonedeadpunk | if it's about something custom | 12:00 |
noonedeadpunk | Like it's all doable with overrides from one side | 12:01 |
noonedeadpunk | But not neat from another | 12:01 |
noonedeadpunk | If you think it's worth to extend functionality - I'm really fine to push patch for that | 12:01 |
jrosser | it was just a little unusual to see `available_filters` need to be specified away from what seemed to be the default | 12:30 |
jrosser | so i read the docs a bit | 12:30 |
jrosser | i guess if someone has a requirement for a custom filter then we can patch it | 12:30 |
gokhani | noonedeadpunk, zun_kuryr_system_user_name: root, zun_kuryr_system_group_name: root, zun_kuryr_system_home_folder: "/var/lib/kuryr" solved zun bug. | 12:32 |
noonedeadpunk | gokhani: I assume `zun_kuryr_system_home_folder` should be /root | 12:52 |
noonedeadpunk | it was also unusual that value is not a list but multivalue | 12:53 |
noonedeadpunk | so in case of override you'd need to use `? value` to have multiple records | 12:53 |
gokhani | thanks noonedeadpunk | 12:57 |
opendevreview | Merged openstack/openstack-ansible-os_nova master: Improve Blazar integration with Nova https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/904779 | 13:17 |
mgariepy | happy new year everyone ! | 13:38 |
noonedeadpunk | \o/ | 13:40 |
jrosser | so putting docs in the ops repo seems pretty hard as sphinx really wants all the docs to be under one tree | 14:49 |
jrosser | seems that we have a lot of README.rst in there which are only useful when viewed with gitea/github and don't actually get pubished | 14:50 |
noonedeadpunk | we can include README.rst in tree? | 14:50 |
noonedeadpunk | like with literalinclude or smth... | 14:51 |
jrosser | toctree cannot do ../../foo/README.rst | 14:51 |
jrosser | well, or i guess i can put the docs in doc/source/foo/README.rst instead of foo/doc/source/README.rst | 14:53 |
jrosser | up to now all of the directories in the ops repo are pretty self contained | 14:53 |
opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use container setup role from plugins repo https://review.opendev.org/c/openstack/openstack-ansible/+/905004 | 15:39 |
opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: WIP - Add collection to deploy magnum cluster-api with vexxhost driver https://review.opendev.org/c/openstack/openstack-ansible-ops/+/901450 | 16:07 |
opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: WIP - Add collection to deploy magnum cluster-api with vexxhost driver https://review.opendev.org/c/openstack/openstack-ansible-ops/+/901450 | 16:15 |
jrosser | spatel: mgariepy ^ theres a first cut of a pretty complete cluster-api diagram in that patch | 16:16 |
noonedeadpunk | oh, sweeet | 16:17 |
opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: WIP - Add collection to deploy magnum cluster-api with vexxhost driver https://review.opendev.org/c/openstack/openstack-ansible-ops/+/901450 | 16:21 |
jrosser | noonedeadpunk: it's an editable png from draw.io so nice for updatable docs | 16:21 |
noonedeadpunk | Yeah, I think we agreed on draw.io some time back, but likely I missed to document that? | 16:23 |
spatel | cluster-api is good but look like it has minor bugs :( | 16:25 |
spatel | I am planning to go back to Magnum/heat (I don't want to deal with bugs in production) | 16:25 |
jrosser | i think that driver will be deprecated | 16:26 |
noonedeadpunk | Yeah, what is clear about magnum, is that heat driver will go away soonish | 16:38 |
noonedeadpunk | I'm actually not sure if there's any good way to migrate from heat to capi though | 16:38 |
spatel | noonedeadpunk there is no way you can migrate from heat to capi | 17:47 |
opendevreview | Merged openstack/ansible-role-systemd_networkd master: Add support for the ArchLinux distribution https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/903112 | 19:07 |
opendevreview | Merged openstack/openstack-ansible master: Modify RGW client format https://review.opendev.org/c/openstack/openstack-ansible/+/904741 | 19:32 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/2023.2: Modify RGW client format https://review.opendev.org/c/openstack/openstack-ansible/+/904855 | 19:46 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/2023.1: Modify RGW client format https://review.opendev.org/c/openstack/openstack-ansible/+/904856 | 19:46 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/2023.1: Modify RGW client format https://review.opendev.org/c/openstack/openstack-ansible/+/904856 | 19:49 |
opendevreview | Merged openstack/openstack-ansible master: Ensure tempest include and exclude lists all use unique names https://review.opendev.org/c/openstack/openstack-ansible/+/893968 | 22:16 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!