Friday, 2022-11-25

*** frenzy_friday is now known as frenzy_friday|rover07:46
noonedeadpunkSo... For swift we simply need to pass `--os-cacert /etc/ssl/certs/ca-certificates.crt` for it to work08:38
noonedeadpunkMaybe we should specify that for clouds.yaml or smth....08:38
noonedeadpunksourcing openrc also works. 08:41
noonedeadpunkBut for some reason defining cacert in clouds.yaml not...08:42
noonedeadpunkor I'm editing "wrong" clouds.yaml08:43
noonedeadpunkoh, well, it doesn't read clouds.yaml08:49
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Add zookeeper deployment  https://review.opendev.org/c/openstack/openstack-ansible/+/86475009:13
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_ironic master: Pass CA pass to Swift CLI  https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/86565609:57
opendevreviewDmitriy Rabotyagov proposed openstack/ansible-role-zookeeper master: Add SSL support for zookeeper  https://review.opendev.org/c/openstack/ansible-role-zookeeper/+/86544911:01
*** frenzy_friday is now known as frenzy_friday|rover11:10
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_cinder master: Add coordination support  https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/86551811:23
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_designate master: Add coordination support  https://review.opendev.org/c/openstack/openstack-ansible-os_designate/+/86553711:25
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_octavia master: Adopt output structure to new collections version  https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/85507411:57
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_heat stable/yoga: Install git into placement containers  https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/86556412:07
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_heat stable/yoga: Install git into heat containers  https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/86556412:11
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_heat stable/xena: Install git into heat containers  https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/86556512:11
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_ironic stable/yoga: Pass CA pass to Swift CLI  https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/86556613:05
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_horizon master: Install ironic-ui from git source rather than a pip package  https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/86392113:08
noonedeadpunkjrosser: you have quite a few patches in WIP for osa-ironic-tidy topic. Are all of them intended to be in WIP?13:09
opendevreviewMerged openstack/openstack-ansible-ops master: kibana: fix installation on ubuntu jammy and other py3 only systems  https://review.opendev.org/c/openstack/openstack-ansible-ops/+/86425813:31
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-os_ironic master: Allow ironic bmaas network gateway and dns servers to be undefined  https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/86316014:38
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-os_ironic master: Refactor ironic deploy image handling.  https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/86354914:39
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-os_ironic master: Update variables for switchport introspection during inspection  https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/86443714:39
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-os_ironic master: Fix inspector auth config for ironic-api  https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/86439514:39
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-os_ironic master: Remove duplicate creation of nginx config directory  https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/86440314:40
jrossernoonedeadpunk: i think the patch i am most uncertain with is https://review.opendev.org/c/openstack/openstack-ansible/+/86096014:56
jrosseri am concerned that our console handling is not flexible enough, and that the logic here does not help https://github.com/openstack/openstack-ansible/blob/8ceceb4a70e88f87d68dc00920d09f9b176b1371/inventory/group_vars/all/nova.yml#L21-L3814:59
jrossernova console can be spice/vnc/serial, and ironic is serial and maybe also 'graphical' of some kind in the future15:00
jrosseralso we might need to mix nova serial and nova vnc/spice in a multi architecture deployment with x86 and arm computes15:01
jrosseri extended the nova role to install the nova serial proxy in the ironic compute container if needed, so that part is not too troublesome15:10
jrosserhttps://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/86094815:10
jrosserthe difficulty is then having only one choice for nova_console_type when now the deployment may have vnc and serial15:11
jrosserand as i say ironic aside i want to have nova vnc and nova serial st the same time for different host and i'm not sure that is possible right now15:11
opendevreviewMerged openstack/openstack-ansible-os_rally stable/ussuri: Return jobs to voting  https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/86179015:31
opendevreviewMerged openstack/openstack-ansible-os_tempest stable/ussuri: Remove tripleo jobs from stable/ussuri branch  https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/84997315:52
opendevreviewMerged openstack/openstack-ansible-os_aodh master: Install git into aodh containers  https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/86496615:58
*** dviroel is now known as dviroel|lunch15:58
opendevreviewMerged openstack/openstack-ansible-os_swift master: Add support for running object-servers Per Disk  https://review.opendev.org/c/openstack/openstack-ansible-os_swift/+/86468516:28
noonedeadpunkWell, I'm trying to understand that patch, but I think I just don't have any idea about how ironic works :(16:29
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_octavia master: Adding octavia_provider_network_mtu-parameter parameter  https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/86481916:32
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_octavia master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/85765416:32
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_octavia master: Enable /healthcheck for Octavia API  https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/86441916:32
noonedeadpunkthat should unblock octavia https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/855074/316:33
jrossernoonedeadpunk: i emailed you docs for how the ironic console works16:35
noonedeadpunkgot it - will look through during weekends!16:36
jrosserlooks like we never had support for this16:36
noonedeadpunkI won't be surprised tbh16:37
jrosseranyway you will see how it totally reuses all the nova console stuff eventually16:41
jrosseri am wondering if we should simplify all this rather than make more complexity16:44
noonedeadpunkoh, it even does provide url thorugh "console url show"16:46
jrosseri think that is how it joins up to horizon16:47
jrosserjust like nova16:47
noonedeadpunkI'm not sure what you mean under simplification at the moment16:54
jrosseroh well we have quite complicated logic on haproxy to decide on front/back ends for vnc/serial/spice consoles for nova17:01
jrosserand now we want to add serial and maybe more console types for ironic too in the future17:02
jrosserso having one var for nova right now is troublesome https://github.com/openstack/openstack-ansible/blob/8ceceb4a70e88f87d68dc00920d09f9b176b1371/inventory/group_vars/all/nova.yml#L21-L3817:03
jrossercurrently we make a single choice "nova_console_type==foo" and then that means that you can't have other types in parallel17:04
jrosserfeels like it should be more like "are there nova|ironic serial consoles" -> enable the serial proxy part on haproxy17:05
jrosser"are there nova vnc consoles" -> enable the vnc proxy part of haproxy17:06
*** dviroel|lunch is now known as dviroel17:06
jrosserand i don't know how we do "nova_console_type=[vnc,serial]" at haproxy17:08
jrosserand then nova_console_type=vnc for an x86 nova compute at the same time as nova_console_type=serial for an aarch64 nova compute17:09
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_designate master: Fix race condition during designate setup  https://review.opendev.org/c/openstack/openstack-ansible-os_designate/+/86570117:20
noonedeadpunkhm, yes, that's not helpful indeed17:26
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Update ansible-collections-openstack version  https://review.opendev.org/c/openstack/openstack-ansible/+/86455317:27
opendevreviewDamian Dąbrowski proposed openstack/openstack-ansible-haproxy_server master: Change 'vip_bind' to 'vip_address' in templates/service-redirect.j2  https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/86478417:38
opendevreviewDamian Dąbrowski proposed openstack/openstack-ansible master: Do not explicitly define NFS version in mount options  https://review.opendev.org/c/openstack/openstack-ansible/+/86570518:13
damiandabrowskijrosser: sorry, I had a maintenance in one of our regions this week and I missed your message about broken cinder/glance gating. I just pushed a fix18:15
opendevreviewMerged openstack/openstack-ansible stable/yoga: Update self version fetching mechanism  https://review.opendev.org/c/openstack/openstack-ansible/+/86516621:55
opendevreviewMerged openstack/openstack-ansible master: Update ansible-collections-openstack version  https://review.opendev.org/c/openstack/openstack-ansible/+/86455322:54
opendevreviewDamian Dąbrowski proposed openstack/openstack-ansible-haproxy_server master: Change 'vip_bind' to 'vip_address' in templates/service-redirect.j2  https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/86478423:37

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