Friday, 2024-02-23

opendevreviewPierre Riteau proposed openstack/kayobe master: Rephrase kolla_build_neutron_ovs release note  https://review.opendev.org/c/openstack/kayobe/+/90947308:05
mnasiadkasupamatt: and keystone points they are experimental? ;-)08:35
mnasiadkasupamatt: https://docs.openstack.org/keystone/latest/admin/unified-limits.html08:35
jovialDon't suppose I could get some reviews on: https://review.opendev.org/c/openstack/kolla/+/909865. I can't use Depends-On in kayobe to verify it in CI, but I have tested locally and it seems to do the trick.09:23
mnasiadkakevko, frickler: can you help in ^^?09:35
fricklermnasiadka: commented09:41
mnasiadkafrickler: thanks, replied09:46
fricklermnasiadka: ack, thx09:49
jovialthanks all :)09:50
opendevreviewAlex Welsh proposed openstack/kayobe master: CI: Add SLURP upgrade jobs  https://review.opendev.org/c/openstack/kayobe/+/90986309:57
opendevreviewMatúš Jenča proposed openstack/kolla-ansible master: Add backend TLS between MariaDB and ProxySQL  https://review.opendev.org/c/openstack/kolla-ansible/+/90991210:00
opendevreviewWill Szumski proposed openstack/kolla-ansible master: Adds feature flag for ironic-inspector in bifrost  https://review.opendev.org/c/openstack/kolla-ansible/+/90991010:04
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: Run ML2/OVS agents processes in separate containers  https://review.opendev.org/c/openstack/kolla-ansible/+/86478010:04
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: Run ML2/OVS agents processes in separate containers  https://review.opendev.org/c/openstack/kolla-ansible/+/86478010:06
SvenKieskeI'm not that familiar with bifrost, does it support HA deployments? I don't want another instance of https://bugs.launchpad.net/kolla-ansible/+bug/2054705 happening10:26
SvenKiesketl;dr: don't run ironic-inspector in HA mode (experimental/not really working) and don't use the rabbitmq transport_url, despite being upstreams default.10:27
jovialSvenKieske, I am not aware that it supports a HA mode of deployment. It is essentially all the services (database, dnsmasq, ironic, etc) in one container10:29
SvenKieskeokay, I just didn't have an intense look at the bifrost role, so it's all a SPOF? :D good to know :)10:30
SvenKieskeat least the inspector should work then10:31
SvenKieskebut I'm curious how the insepctor is configured in bifrost, actually.10:31
SvenKieske*inspector10:31
SvenKieskeah I realize https://review.opendev.org/c/openstack/kolla/+/909865 is only part one? at least I would expect a second change actually configuring it in k-a? is there something up for that already?10:32
jovialWe've often thought about replacing it it with standalone Ironic deploy by K-A but have never got round to it :-D10:32
jovialIndeed, I've added a flag to enable it here: https://review.opendev.org/c/openstack/kolla-ansible/+/909910. Not strictly necessary as you add this in your bifrost.yml customizations.10:33
SvenKieskemhm, so we rely on the default configuration for the inspector and don't change anything there? it might be worth to fix the transport_url for that case as well, because the default seems broken10:35
opendevreviewMatt Crees proposed openstack/kolla-ansible stable/2023.1: Add precheck for RabbitMQ quorum queues  https://review.opendev.org/c/openstack/kolla-ansible/+/90996710:36
opendevreviewMatt Crees proposed openstack/kolla-ansible stable/2023.1: Rework quorum queues precheck  https://review.opendev.org/c/openstack/kolla-ansible/+/90996810:36
opendevreviewMatt Crees proposed openstack/kolla-ansible stable/2023.1: RabbitMQ: correct docs on Quorum Queue migrations  https://review.opendev.org/c/openstack/kolla-ansible/+/90996910:36
jovialSvenKieske, is there a bug for the transport_url that I could look at?10:37
SvenKieskeyes, I linked it above (https://bugs.launchpad.net/kolla-ansible/+bug/2054705) and I also just now replied in your k-a review with more details. had a longer chat with the ironic people yesterday about this10:40
jovialsorry, missed that, thanks10:41
SvenKieskeno problem :)10:41
jovialSvenKieske, Seems like we have rpc_transport = None in bifrost (https://github.com/openstack/bifrost/blob/7a6af41a265cb641f5900afe1ab8b8abb31dc2c0/playbooks/roles/bifrost-ironic-install/templates/ironic.conf.j2#L36) is that how we et away with the default transport url?10:49
opendevreviewMatt Crees proposed openstack/kolla-ansible master: CI: Only migrate RMQ queues during SLURP  https://review.opendev.org/c/openstack/kolla-ansible/+/90997110:49
jovialget*10:50
jovialSorry, wrong config file :facepalm:10:51
jovialI do see fake here: https://github.com/openstack/bifrost/blob/7a6af41a265cb641f5900afe1ab8b8abb31dc2c0/playbooks/roles/bifrost-ironic-install/templates/ironic-inspector.conf.j2#L1810:52
SvenKieske"none" might be a little brittle, to quote oslo docs: "If no transport URL is provided, the URL in the notifications section of the config file will be used. If that URL is also absent, the same transport as specified in the user’s default section will be used."10:57
SvenKieskeI take it, "none" means providing no transport url and not a transport url that has the string "none"? ;)10:58
SvenKieskesource: https://docs.openstack.org/oslo.messaging/latest/reference/transport.html10:58
SvenKieske"None" seems to be oslos default: https://docs.openstack.org/oslo.messaging/latest/reference/transport.html#oslo_messaging.TransportURL but ironics default is "rabbit://"10:59
SvenKieskeironic devs told me to use explicitly "fake://" I don't know if this is better/safer, but I would maybe suggest to do what the ironic devs recommend here. not sure though ¯\_(ツ)_/¯11:00
SvenKieskeI guess it can't hurt.11:01
SvenKieskemhm, I don't get the oslo docs tbh, the class doc clearly states the default transport is none, but the default config docs clearly state the default is "rabbit", guess I'll read some source code..11:07
SvenKieskemaybe the default config just overrides the default class implementation, why you would do such a destinction is beyond me.11:08
SvenKieskeah, good old factory methods okay, so the default config is right, it's "rabbit://" per default (in oslo.messaging).11:11
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: cinder: Add prechecks and docs for HA/clustering  https://review.opendev.org/c/openstack/kolla-ansible/+/90997411:14
PrzemekKWhen is planned finish this https://review.opendev.org/q/topic:%22frr-ovn-bgp-agent%22 . Is it safe to introduce it to ubuntu environment ? What about OVN VPNaaS11:15
SvenKieskeI'll head over to ironic dev room again I guess.. or does there happen to be someone around who actually knows something about oslo drivers? Because the oslo docs clearly state, that you should not use the "fake://" driver in prod.11:15
SvenKieskePrzemekK: I guess you have to ask the developer, this is marked "work in progress" and I have no authority over the devs in question. Don't know when they finish their work. :)11:17
SvenKieskeI guess you could reach out to EPG Group, where they seem to be employed? :)11:18
opendevreviewMichal Nasiadka proposed openstack/kolla master: WIP: Add support for rpm to repos.yaml  https://review.opendev.org/c/openstack/kolla/+/90987911:24
mnasiadkaRMQ mirror broken, how nice11:30
mnasiadkaand 3.13.0 is out - https://github.com/rabbitmq/rabbitmq-server/discussions/1059511:31
mnasiadkacoincidence? ;-)11:31
opendevreviewVerification of a change to openstack/kolla master failed: Install ironic-inspector in bifrost  https://review.opendev.org/c/openstack/kolla/+/90986511:50
opendevreviewMichal Nasiadka proposed openstack/kolla master: WIP: Add support for rpm to repos.yaml  https://review.opendev.org/c/openstack/kolla/+/90987912:02
opendevreviewRussell Tweed proposed openstack/kolla-ansible master: Avoids loop where 'murano_api_listen_port' is referenced to itself. Set to 'murano_api_port' instead.  https://review.opendev.org/c/openstack/kolla-ansible/+/90991912:12
kevkomorning :) 12:13
kevkoSorry, I am little bit unresponsibe today ... preparing for upgrade from xena -> yoga at evening :) 12:18
mnasiadkadoing an upgrade on Friday? that's surely unresponsible :)12:20
kevkomnasiadka: Banks usually do the upgrades on fridays :) 12:38
opendevreviewWill Szumski proposed openstack/kayobe master: Support configuring persistent systemd journal  https://review.opendev.org/c/openstack/kayobe/+/90968512:41
opendevreviewWill Szumski proposed openstack/kayobe master: WIP: Add podman support  https://review.opendev.org/c/openstack/kayobe/+/90968612:41
opendevreviewWill Szumski proposed openstack/kayobe master: WIP: Add podman support  https://review.opendev.org/c/openstack/kayobe/+/90968612:54
opendevreviewMatúš Jenča proposed openstack/kolla-ansible master: Add backend TLS between MariaDB and ProxySQL  https://review.opendev.org/c/openstack/kolla-ansible/+/90991213:08
opendevreviewMatúš Jenča proposed openstack/kolla-ansible master: Add backend TLS between MariaDB and ProxySQL  https://review.opendev.org/c/openstack/kolla-ansible/+/90991213:13
supamattmnasiadka: I'm not sure that's it's expiremental anymore, it's recommended to enable in Bobcat+ for Nova13:21
supamattmnasiadka: https://docs.openstack.org/nova/latest/admin/unified-limits.html13:21
opendevreviewRussell Tweed proposed openstack/kolla-ansible master: Avoids loop where 'murano_api_listen_port' is referenced to itself.  https://review.opendev.org/c/openstack/kolla-ansible/+/90991913:45
SvenKieskeregarding the unified limits api, it's marked as experimental since stein release: https://docs.openstack.org/keystone/stein/admin/unified-limits.html probably worth it to ask over in #openstack-keystone if this is just a docs bug? supamatt, mnasiadka?13:50
mnasiadkasupamatt: to put it in normal words, it’s not on our priority list, but if you want to work on it (to make that easily enabled in Kolla) - feel free to submit patches14:17
opendevreviewMerged openstack/kayobe stable/2023.1: Add missing space in help output  https://review.opendev.org/c/openstack/kayobe/+/90924714:44
kevkomnasiadka: I have a little problem :) - don't say that we are using weird deployment ->  we are using centos libvirt images ... now I found that xena spinned instance can't be migrated to centos libvirt - even if there is same version but different package revision  - (10 - 22 or something like that - maintainers just patched libvirt because of14:47
kevkosome bug ..but broke live migration :D )14:47
kevko*migrated to centos libvirts yoga ...14:47
opendevreviewWill Szumski proposed openstack/kayobe master: WIP: Add podman support  https://review.opendev.org/c/openstack/kayobe/+/90968614:50
kevkoRight now I am going to check if rocky will help ... if it will work ..it should be good replacement for centos based image right ? 14:51
mnasiadkayup15:34
mnasiadkasurely better than centos stream I guess15:34
opendevreviewMerged openstack/kayobe stable/2023.2: Add missing space in help output  https://review.opendev.org/c/openstack/kayobe/+/90924615:54
kevkomnasiadka: yep .. but they removed VGA model qxl :D 16:14
admin1what port is skyline on if both horizon and skyline are enabled ? 16:14
kevkomnasiadka: so i need to switch to novnc16:14
mnasiadkakevko: yes, that's unavoidable16:14
mnasiadkaadmin1: I'm not sure that's supported16:15
admin1isn't it a good idea to support it in alternate port so the deployers can try/test it side by side ? 16:16
kevkomnasiadka: hmmmmm, but even if i switched to novnc ..i can't migrate :'( 16:17
kevkoon test 16:17
kevkoi probably have to hack db :) 16:21
opendevreviewMichal Nasiadka proposed openstack/kolla master: rabbitmq: Mark unbuildable on Debian/Ubuntu  https://review.opendev.org/c/openstack/kolla/+/91000616:23
mnasiadkaadmin1: it's a fantastic idea, are you willing to contribute patches to make it work?16:23
admin1i am taking a look 16:25
admin1i want to try skyline and looks like kolla has it solved :) 16:27
kevkoadmin1: pity is that skyline is just fancy  nice frontend (which still don't work) ...but backend is still python :( 16:28
admin1"(which still don't work) " -- my idea was to see if it works :) 16:29
kevkokevko:  it would be fine to send calls from dashboard directly to API 16:29
kevkoadmin1: it work ..but horizon is still better I would say 16:29
admin1noted 16:31
admin1inst lb member role enough on a project to create lbs ? 16:38
admin1isn't   16:38
admin1need more roles to create lb ? 16:38
opendevreviewWill Szumski proposed openstack/kolla-ansible master: Support mode in named volumes when using podman  https://review.opendev.org/c/openstack/kolla-ansible/+/91001617:08
opendevreviewWill Szumski proposed openstack/kayobe-config-dev master: Replace docker group with container-engine  https://review.opendev.org/c/openstack/kayobe-config-dev/+/91001717:23
opendevreviewWill Szumski proposed openstack/kayobe master: Support configuring persistent systemd journal  https://review.opendev.org/c/openstack/kayobe/+/90968517:25
opendevreviewWill Szumski proposed openstack/kayobe master: WIP: Add podman support  https://review.opendev.org/c/openstack/kayobe/+/90968617:25
opendevreviewWill Szumski proposed openstack/kayobe master: WIP: Add podman support  https://review.opendev.org/c/openstack/kayobe/+/90968618:09
opendevreviewWill Szumski proposed openstack/kayobe master: WIP: Add podman support  https://review.opendev.org/c/openstack/kayobe/+/90968619:07
opendevreviewMichal Nasiadka proposed openstack/kolla master: DNM: test rmq  https://review.opendev.org/c/openstack/kolla/+/91000619:09
spatelI was say try skyline in parallel and submit bugs as much as possible :) 21:20
spatelIn my environment I am running skyline as a instance inside openstack :)  It doesn't create any conflict that way. I wish there is a ansible variable to specify port for skyline in global.yml 21:22
opendevreviewChristopher Spivey proposed openstack/kolla-ansible master: Added Variables for Cyborg API and Internal Endpoint addresses for references Had issue where enabling Cyborg caused the Kolla to not read defaults correctly. Added the lines to make it enable properly using the kolla_internal_fqdn variable and such.  https://review.opendev.org/c/openstack/kolla-ansible/+/91003323:36

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