Monday, 2024-01-08

gokhanigood morning folks, have yo seen this https://bugs.launchpad.net/kuryr-libnetwork/+bug/2048064 ?07:48
gokhaniwith ovs zun is not working07:48
noonedeadpunkgokhani: I think this should have been addressed....07:57
noonedeadpunkoh, well, maybe not and it's different issue I was thinking about07:58
noonedeadpunkYeah, was thinking about this: https://review.opendev.org/c/openstack/kuryr-libnetwork/+/89566207:58
gokhanibug author tried with 2023.1 with OSA07:58
gokhaniI tried with victoria and got same error. 07:59
noonedeadpunkgokhani: I think you should be able to change kuryr user to root to workaround the issue08:03
noonedeadpunkI've also replied to the bug with the same suggestion08:03
noonedeadpunkso worth trying to change zun_kuryr_system_user_name to root for instance08:03
gokhaniyes I can try this now 08:16
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Add openstack_resources role skeleton  https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/87879408:25
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible stable/2023.2: Skip installing curl for EL  https://review.opendev.org/c/openstack/openstack-ansible/+/90484508:32
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible stable/2023.1: Skip installing curl for EL  https://review.opendev.org/c/openstack/openstack-ansible/+/90484608:32
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible stable/zed: Skip installing curl for EL  https://review.opendev.org/c/openstack/openstack-ansible/+/90484708:33
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Return back /healtcheck URI verification  https://review.opendev.org/c/openstack/openstack-ansible/+/90494108:55
noonedeadpunk^ results in swift failing, as / returns 404 09:00
opendevreviewMerged openstack/openstack-ansible-ceph_client master: Add backwards compatibility of ceph_components format  https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/90449910:27
jrossergood morning10:30
opendevreviewJonathan 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/+/90484910:30
damiandabrowskihi!10:44
noonedeadpunko/10:59
jrosserthe patch which adds blazar filters shows some missing functionality?11:44
jrosserwe can enable some set of filters with `enabled_filters = {{ _nova_scheduler_filters | join(',') }}`11:45
jrosserbut `available_filters` does not seem to get updated to account for `nova_scheduler_extra_filters`11:46
jrosserif you want to specify your own like this https://docs.openstack.org/nova/2023.2/admin/scheduling.html#writing-your-own-filter11:47
noonedeadpunkjrosser: yeah, I was thinking of adding another variable to manage `available_filters` but then scraped a draft for some reason11:59
jrosserit's probably because the default available_filters is all the builtins11:59
noonedeadpunkyup11:59
jrosserso if thats what you need then it's ok to just adjust `nova_scheduler_extra_filters`12:00
noonedeadpunkand then it's also matter of isntalling extra pip package12:00
noonedeadpunkif it's about something custom12:00
noonedeadpunkLike it's all doable with overrides from one side12:01
noonedeadpunkBut not neat from another12:01
noonedeadpunkIf you think it's worth to extend functionality - I'm really fine to push patch for that12:01
jrosserit was just a little unusual to see `available_filters` need to be specified away from what seemed to be the default12:30
jrosserso i read the docs a bit12:30
jrosseri guess if someone has a requirement for a custom filter then we can patch it12:30
gokhaninoonedeadpunk, 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
noonedeadpunkgokhani: I assume `zun_kuryr_system_home_folder` should be /root12:52
noonedeadpunkit was also unusual that value is not a list but multivalue12:53
noonedeadpunkso in case of override you'd need to use `? value` to have multiple records12:53
gokhanithanks noonedeadpunk12:57
opendevreviewMerged openstack/openstack-ansible-os_nova master: Improve Blazar integration with Nova  https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/90477913:17
mgariepyhappy new year everyone !13:38
noonedeadpunk\o/13:40
jrosserso putting docs in the ops repo seems pretty hard as sphinx really wants all the docs to be under one tree14:49
jrosserseems that we have a lot of README.rst in there which are only useful when viewed with gitea/github and don't actually get pubished14:50
noonedeadpunkwe can include README.rst in tree?14:50
noonedeadpunklike with literalinclude or smth...14:51
jrossertoctree cannot do ../../foo/README.rst14:51
jrosserwell, or i guess i can put the docs in doc/source/foo/README.rst instead of foo/doc/source/README.rst14:53
jrosserup to now all of the directories in the ops repo are pretty self contained14:53
opendevreviewJonathan Rosser proposed openstack/openstack-ansible master: Use container setup role from plugins repo  https://review.opendev.org/c/openstack/openstack-ansible/+/90500415:39
opendevreviewJonathan 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/+/90145016:07
opendevreviewJonathan 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/+/90145016:15
jrosserspatel: mgariepy ^ theres a first cut of a pretty complete cluster-api diagram in that patch16:16
noonedeadpunkoh, sweeet16:17
opendevreviewJonathan 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/+/90145016:21
jrossernoonedeadpunk: it's an editable png from draw.io so nice for updatable docs16:21
noonedeadpunkYeah, I think we agreed on draw.io some time back, but likely I missed to document that?16:23
spatelcluster-api is good but look like it has minor bugs :(16:25
spatelI am planning to go back to Magnum/heat (I don't want to deal with bugs in production)16:25
jrosseri think that driver will be deprecated16:26
noonedeadpunkYeah, what is clear about magnum, is that heat driver will go away soonish16:38
noonedeadpunkI'm actually not sure if there's any good way to migrate from heat to capi though16:38
spatelnoonedeadpunk there is no way you can migrate from heat to capi 17:47
opendevreviewMerged openstack/ansible-role-systemd_networkd master: Add support for the ArchLinux distribution  https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/90311219:07
opendevreviewMerged openstack/openstack-ansible master: Modify RGW client format  https://review.opendev.org/c/openstack/openstack-ansible/+/90474119:32
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible stable/2023.2: Modify RGW client format  https://review.opendev.org/c/openstack/openstack-ansible/+/90485519:46
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible stable/2023.1: Modify RGW client format  https://review.opendev.org/c/openstack/openstack-ansible/+/90485619:46
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible stable/2023.1: Modify RGW client format  https://review.opendev.org/c/openstack/openstack-ansible/+/90485619:49
opendevreviewMerged openstack/openstack-ansible master: Ensure tempest include and exclude lists all use unique names  https://review.opendev.org/c/openstack/openstack-ansible/+/89396822:16

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