Wednesday, 2024-11-13

opendevreviewMerged openstack/openstack-ansible master: Add variables to control HSTS records  https://review.opendev.org/c/openstack/openstack-ansible/+/93462000:47
opendevreviewMerged openstack/openstack-ansible master: Use mariadb-admin instead of mysqladmin  https://review.opendev.org/c/openstack/openstack-ansible/+/93443000:47
opendevreviewMerged openstack/openstack-ansible master: Remove usage of mariadb.com infra mirror  https://review.opendev.org/c/openstack/openstack-ansible/+/93403700:47
*** fungi is now known as Guest924901:33
*** kinrui is now known as fungi01:41
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Set Rocky 9 Ceph jobs as voting  https://review.opendev.org/c/openstack/openstack-ansible/+/93359207:32
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server master: Get exact version of installed rabbitmq  https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/93482207:50
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server master: Bump RabbitMQ version to 4.0  https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/93406007:50
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-rabbitmq_server master: Ensure that rabbitmq-erlang and rabbitmq-server repos use the same pins  https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/93372407:55
noonedeadpunksome status update for myself - need to implement rabbitmq cluster bootstrap feature and in haproxy to drop extra_lb_(?tls_)vip_addresses07:57
noonedeadpunkthen with landing mariadb 11.4 and rabbitmq 4.0 I think we can branch?07:58
noonedeadpunkor should we jsut backport things once what we have lands?07:58
noonedeadpunkand I frankly haven't looked if migration to khepri really happens....08:03
andrewbonneyJust checked out deployments as I accidentally moved to 3.13 with 2024.1. Looks like even without the explicit check, because that flag is experimental it doesn't get enabled so there's no issue08:04
noonedeadpunkon 3.13 it's experimental indeed. but on 4.0 it's not anymore08:04
noonedeadpunkso I guess it's jsut migrated during upgrade?08:05
* noonedeadpunk killed all envs with 4.0 now08:05
andrewbonneyYes I suspect so. I'll do another check of that this morning to see08:05
noonedeadpunkif that post-upgrade feature enable is executed :D08:05
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_cinder master: Remove cinder v2 references  https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/93459308:10
jrossero/ morning08:33
noonedeadpunko/08:33
opendevreviewAndrew Bonney proposed openstack/openstack-ansible-os_nova master: nova: template nova configuration directory  https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/93493309:29
opendevreviewAndrew Bonney proposed openstack/openstack-ansible-os_nova master: Load vendor_data.json from versioned path  https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/93493409:29
noonedeadpunkandrewbonney: about that. I think we should take neutron approach instead09:42
noonedeadpunkI was jsut lazy to work on it for other repos09:42
andrewbonneyI was looking at that, but wasn't sure it would actually solve the issue. It still looked like /etc/neutron/ would be missing for a period, but I may be reading it wrong09:42
noonedeadpunkhttps://opendev.org/openstack/openstack-ansible-os_neutron/commit/57638854532690b61c4abc6d735c17f31950b7e609:43
noonedeadpunkso we swap a symlink in handlers: https://opendev.org/openstack/openstack-ansible-os_neutron/src/branch/master/handlers/main.yml#L66-L7709:43
noonedeadpunkwhen service is stopped09:43
andrewbonneyDoes that step also deal with removing the old symlink?09:44
noonedeadpunkyeah, as `force: true` 09:44
noonedeadpunkI'd say this should be done for all services with "smart sources" but neutron was most affected back then09:45
noonedeadpunk(and we didn't care about failed scheduling too much during role runtime)09:46
andrewbonneyOk. Does https://opendev.org/openstack/openstack-ansible-os_neutron/src/branch/master/tasks/neutron_pre_install.yml#L48 not do anything for a symlink then? Wondering why it's there at all if it only runs for source installs09:46
noonedeadpunkbecause link will return `neutron_conf_dir_stat.stat.isdir` as false there09:47
noonedeadpunkI think it was just and upgrade task and probably we can drop it now at all09:48
andrewbonneyOk, I'll have another pass over the Nova role. If that looks reasonable it can be done to others too09:48
noonedeadpunkbut it can be handy if moving from distro setup to source one.09:48
opendevreviewAndrew Bonney proposed openstack/openstack-ansible-os_nova master: Change ordering of /etc/ operations to improve upgrades  https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/93494110:24
opendevreviewMerged openstack/openstack-ansible-rabbitmq_server master: Use rabbitmq_cluster_state task instead of command  https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/93197511:51
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server master: Bump RabbitMQ version to 4.0  https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/93406011:57
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server master: Bump RabbitMQ version to 4.0  https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/93406013:17
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: [doc] Document requirement for RabbitMQ upgrade  https://review.opendev.org/c/openstack/openstack-ansible/+/93482813:18
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server master: Bump RabbitMQ version to 4.0  https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/93406013:19
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server master: Bump RabbitMQ version to 4.0  https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/93406013:27
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server master: Bump RabbitMQ version to 4.0  https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/93406013:29
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server master: Move version pinnings from distro vars to defaults  https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/93496513:33
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server master: Respect `package_state` for the role  https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/93496613:34
opendevreviewAndrew Bonney proposed openstack/openstack-ansible-os_nova master: Change ordering of /etc/ operations to improve upgrades  https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/93494113:49
mgariepywhy is it not happening every time ? https://bugs.launchpad.net/horizon/+bug/204539414:59
jrosserit is very annoying15:02
mgariepyjust once in a few tmies.. (TM)15:03
noonedeadpunkI was never able to catch it outside of CI15:12
noonedeadpunkwas running horizon role quite frequently recently, and it was always successfull15:12
mgariepyit doesnt happen too often juste enough to be annoying15:12
noonedeadpunkyeah15:12
noonedeadpunkjrosser: are you using that to get alternative versions for some things like calico? https://github.com/vexxhost/magnum-cluster-api/pull/426/files17:39
jrossersort of17:40
noonedeadpunkor that's the tooling specifically for images preparation?17:40
jrosserwhat do you need to do?17:40
noonedeadpunkwell, kind of that: https://github.com/vexxhost/magnum-cluster-api/pull/45517:40
noonedeadpunkand it works17:40
noonedeadpunkbut a) I don't like it's hardcoded b) it's failing unit test...17:41
jrosserthis is for workload clusters?17:41
noonedeadpunkyeah17:41
jrosserah ok so we have re-used the image_loader thing to prepare a registry for the control plane side17:41
noonedeadpunkaha17:41
jrosserbut thats cilium, so yes makes sense that calico is for the workload side17:42
jrosserfor our case, the workload side actually does have internet so we have not made a private registry for that17:42
jrosserbut having said that, we do have good tooling now for the control plane side registry17:42
noonedeadpunkgotcha. I was more about - you dynamically load manifests and they are now not flexible enough ,so I was thinking if you was solving same issue17:43
noonedeadpunkas eventually k8s 1.28 needs calico 3.27 ideally17:44
noonedeadpunkok, will try to figure out how to fix the unit test, as you was doing slightly different thing17:44
jrosserwe want to dynamically load manifests yes, so that PR of ours would be applicable for both use cases17:45
noonedeadpunkaha17:45
jrosserbecasue we leverage the same tool to build the registry17:45
jrosserand to cover upgrades you have to load N and N+1 images into the registry17:46
jrosserso it's all a bit wtf17:46
noonedeadpunkbut I somehow was thinking that this not part of the image... but not sure...17:46
jrosserimage? vm image?17:46
noonedeadpunkyeah17:47
noonedeadpunkwhat image is this about? lol17:47
jrosserthe vm images have no container images in them17:47
noonedeadpunkyeah, so they will fetch it afterwards17:48
jrosserright17:48
noonedeadpunkbut I think manifests are placed by the driver inside master node of the created workload cluster?17:48
jrosserimage_loader prepares a registry that has all the container images that the VM images might need17:48
jrossernot really17:48
noonedeadpunkok, yeah. and I'm trying to work with just upstream images for now...17:49
jrosserfor calico i don't exactly know how this works tbh17:49
jrosserbut some things get installed into the vm with charts here https://github.com/vexxhost/magnum-cluster-api/tree/main/magnum_cluster_api/charts17:50
noonedeadpunkok, yeah, I'm not fully into how this all works... Just once tried to define `calico_tag` as a label realized it doesn't work "magically" like with Heat, but fails with `[Errno 2] No such file or directory: '/openstack/venvs/magnum-29.0.0/lib/python3.10/site-packages/magnum_cluster_api/manifests/calico/v3.28.2.yaml'`17:52
noonedeadpunkso still pretty much confusing myself here17:53
jrosseri am not sure that it is possible to take heat concepts across here at all17:53
noonedeadpunkyeah17:53
noonedeadpunkbut defarmation, you know17:54
noonedeadpunk*deformation17:54
jrosserwe do build our own images now17:54
jrosserand it seems ok17:55
noonedeadpunkwas runing heat for years and close to never was looking inside of k8s17:55
noonedeadpunkso was expecting that if label is available - I should be able to set it to value and driver will fetch the proper image for me...17:57
jrosserwell, labels are i think defined in magnum, not the driver18:00
jrossertry setting cidr on master lb fip for example :(18:00
noonedeadpunkthey pretty much passed/handled by driver18:00
noonedeadpunklike LB type - ovn or amphora18:01
noonedeadpunkit's capi that will perform order of LB in octavia18:01
jrosseri have been trying to rotate passwords on bmc this week18:02
jrosseroh what a total nightmare :(18:02
noonedeadpunkhaha, I've heard of such stories from our infra as well lately18:02
noonedeadpunkthey also tried to setup "valid" certs for bmc. through API18:03
jrosseroh interesting18:03
jrosseri bet that was challenging18:03
noonedeadpunkDell iDRAC is full of surprises.18:03
noonedeadpunkabout format, length, random errors, etc...18:03
jrosseroh yeah i am finding similar on supermicro18:04
jrossersome with just broken redfish totally18:04
noonedeadpunkbut I think they ended up with some ansible role which works in most of cases...18:04
jrossersome with pretty much undocumented password complexity rules18:04
noonedeadpunksupermicro was another fun I've heard. different kind of fun, with random reboots18:05
noonedeadpunkoh, yes, password length and complexity - I heard about that as well18:05
noonedeadpunkIt's really a huge shame that in our org there's close to none of upstream contributions culture :(18:06
noonedeadpunkjrosser: arddennis was actually working on tls certs for redfish18:20
noonedeadpunkiirc it had also smth to do with hashi vault usage18:20
jrosseroh that sounds interesting18:20
jrosserpretty much what i would like to do but just did not get to look yet18:21
noonedeadpunkso I kinda have some hope that he might be able to share something somehow18:22
noonedeadpunkbut I have close to no influence on that unfortunatelly18:22
jrosseri was thinking that if i had the hostname/fqdn set correctly on the bmc i could ask it for a CSR, push that through vault and get a certificate out18:23
mgariepybmc truncating password without notice is also fun.18:26
noonedeadpunklike lekystone did a while back ?:)18:28
mgariepynever saw it in keystone when was it ?18:30
arddennisWith the Dell servers I have examples how to do this in Ansible (gen CSR, sign it in vault and upload). I can give examples if needed.18:30
mgariepyi mostly use ldap for auth anyway18:30
noonedeadpunkmgariepy: before or even in 2023.118:32
jrosserarddennis: that would be really interesting - i don't know if it would translate to supermicro or not but just seeing how you handled it with the redfish modules (or had to resort to get_uri?!) would be very helpful18:32
noonedeadpunkand then it started trimming them incorrectly at some point18:32
noonedeadpunkmgariepy: https://review.opendev.org/c/openstack/keystone/+/890936 and https://review.opendev.org/c/openstack/keystone/+/89102418:34
noonedeadpunkso eventually keystone bcrypt, which is default, was always trimming passwords....18:36
jrossernoonedeadpunk: do you use the secrets key/value engine for vault at all/?18:37
mgariepythe bmc was triming a 8 chars irrc lol18:38
noonedeadpunkjrosser: nah, not really.18:38
jrosserah no problem, we have made a 'helper' cli tool for that which is very cool, and i was thinking to open source that18:39
noonedeadpunkfrnakly I had some thoughts/discussions about placing user_secrets there, but for now it's ansible-vault still18:40
jrosseri think we will get to that, just needs some time18:40
noonedeadpunkI also wanna try to play with AWX and osa integration. as when playbooks are in a separate collection....18:43
arddennisjrosser: Here is example: https://paste.openstack.org/show/bH7DKXADlYLkquxvPjoa/ I can share only uri examples. I have a module for the ansible which handles most of this, but in order to share it I need permission from the company I am working for.18:43
noonedeadpunkarddennis: I wonder if it's worth asking, and at least publish that on our github...18:44
noonedeadpunkbut actually that paste looks not afwul18:45
noonedeadpunks/our/company/18:45
jrosseri think what i was surprised at was how quickly the redfish_command module becomes not helpful18:47
arddenniscurrent ansible redfish modules are a bit limited. Look at their source code. It is more like a proxy to API services with filtering on what you can send.18:51
noonedeadpunkI'd guess this how openstack modules might look soon when they'll get generated from openapi18:53
mgariepyhttps://zuul.opendev.org/t/openstack/build/14b37674ed884e3cb5beab65258b43f7/log/logs/host/neutron-server.service.journal-19-15-05.log.txt#674521:06
mgariepysomeone seen that before ? race in ovn/ovsdb ?21:06

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