| opendevreview | James Denton proposed openstack/openstack-ansible master: Implement OVN inventory changes and deploy by default https://review.opendev.org/c/openstack/openstack-ansible/+/862924 | 01:08 |
|---|---|---|
| opendevreview | James Denton proposed openstack/openstack-ansible master: Implement OVN inventory changes and deploy by default https://review.opendev.org/c/openstack/openstack-ansible/+/862924 | 03:09 |
| opendevreview | James Denton proposed openstack/openstack-ansible master: Implement OVN inventory changes and deploy by default https://review.opendev.org/c/openstack/openstack-ansible/+/862924 | 03:39 |
| *** frenzy_friday|rover is now known as frenzy_friday|rover|doc | 06:56 | |
| *** frenzy_friday|rover|doc is now known as frenzy_friday|rover | 08:17 | |
| noonedeadpunk | jamesdenton: so, either we should install ssl inside neutron-server or we should avoid configuring https://zuul.opendev.org/t/openstack/build/55d001cb429e48b78a5fb5a81e2db53f/log/logs/etc/openstack/aio1_neutron_server_container-8fab4a71/neutron/plugins/ml2/ml2_conf.ini.txt#30-35 Not sure what's correct way though | 09:25 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: add ovn ssl config https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/862403 | 09:25 |
| noonedeadpunk | I'd say basically everywhere where ml2_conf.ini.j2 is placed we should also paste certificates | 09:26 |
| noonedeadpunk | or well. https://opendev.org/openstack/openstack-ansible-os_neutron/src/branch/master/templates/plugins/ml2/ml2_conf.ini.j2#L47 | 09:28 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: Set default plugin type to OVN https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/865961 | 09:28 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-plugins master: Unify vars for glusterfs RHEL variants and remove rocky-8 workaround. https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/866116 | 09:42 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove support for calico ml2 driver. https://review.opendev.org/c/openstack/openstack-ansible/+/866119 | 09:53 |
| opendevreview | Dmitriy 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/+/865566 | 09:58 |
| noonedeadpunk | I just disabled rhel8 jobs for Y on ironic ^ | 09:58 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_neutron master: Remove support for calico ml2 driver. https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/866123 | 10:01 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Disable sahara real tempest tests https://review.opendev.org/c/openstack/openstack-ansible/+/866124 | 10:02 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_sahara master: Update api-paste to enable healthcheck https://review.opendev.org/c/openstack/openstack-ansible-os_sahara/+/864422 | 10:03 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_ironic master: Refactor ironic deploy image handling. https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/863549 | 10:06 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_ironic master: Add variable for user defined list of deploy images https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/865310 | 10:06 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_ironic master: Add variable for user defined list of deploy images https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/865310 | 10:07 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Disable fact variables https://review.opendev.org/c/openstack/openstack-ansible/+/778396 | 10:08 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Switch distro jobs to Jammy https://review.opendev.org/c/openstack/openstack-ansible/+/866126 | 10:11 |
| gokhani | hello folks, when I try to upload large images to glance, firsy I can see that image is saving state and after the x minutes I see that image is lost. Do I need to set timeout at haproxy settings? What is your idea for that situation ? | 10:26 |
| noonedeadpunk | jrosser: small nit in https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/863549/4 | 10:27 |
| noonedeadpunk | gokhani: how large are we talking about ?:) | 10:27 |
| noonedeadpunk | but no, I don't think it has smth to do with haproxy, but can with token lifetime | 10:28 |
| noonedeadpunk | or diskspace on glance container (controller) | 10:28 |
| noonedeadpunk | or uwsgi mode :) | 10:28 |
| gokhani | noonedeadpunk, for example about 500GB, in my env use_uwsgi=false, it is eventlet server | 10:30 |
| noonedeadpunk | well, then I'd say it's diskspace on controller | 10:31 |
| noonedeadpunk | or well | 10:31 |
| noonedeadpunk | depends on how you create the image | 10:31 |
| noonedeadpunk | do you just upload through cli? | 10:31 |
| noonedeadpunk | or create from server/volume? | 10:32 |
| gokhani | when I looked at logs I can oly see store exception invalid literal for int() with base 16 evetlet.wsgi.Chunk read error | 10:32 |
| gokhani | backend is ceph | 10:32 |
| gokhani | I am trying volume upload image | 10:33 |
| noonedeadpunk | I need to refresh in my memory how this process is done in fact, but if I'm not mistaken (I can easily be now), then volume somehow fetched to glance container and then uploaded as image (or smth like that) | 10:34 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_ironic master: Refactor ironic deploy image handling. https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/863549 | 10:36 |
| gokhani | ok it firsly may be make conversions on glance containers and I don't have enough space on glance containers | 10:36 |
| gokhani | I think cache is disable | 10:37 |
| gokhani | and but some of my images uploaded | 10:37 |
| noonedeadpunk | It could be also related to the token lifetime | 10:40 |
| noonedeadpunk | https://docs.openstack.org/cinder/latest/configuration/block-storage/service-token.html | 10:41 |
| noonedeadpunk | Since Yoga we enable service_token_roles and add users to "service" role | 10:43 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_ironic master: Tidy definition of http dir for inspector https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/866132 | 10:46 |
| *** dviroel_ is now known as dviroel | 10:47 | |
| gokhani | noonedeadpunk, I am on vctoria | 10:47 |
| jamesdenton | noonedeadpunk AFAIK they wouldn't be necessary in the neutron server container, so we may need to figure out why they're landing there. northd_container, yes. | 10:53 |
| jamesdenton | but mgariepy would know better | 10:54 |
| noonedeadpunk | well, then we need to change condition here https://opendev.org/openstack/openstack-ansible-os_neutron/src/branch/master/templates/plugins/ml2/ml2_conf.ini.j2#L47 | 10:57 |
| noonedeadpunk | as neutron-server seems to read this file and fully loads it | 10:57 |
| jamesdenton | neutron-server def needs the ml2_conf.ini and those ovn bits, but i'm not sure if the ovn cert/key bits are needed there. | 11:00 |
| noonedeadpunk | well, if it's going to connect to ovn then I guess yes? | 11:02 |
| noonedeadpunk | Maybe not everything though.... | 11:02 |
| jamesdenton | you could be right | 11:02 |
| noonedeadpunk | And likely it looks only for ca-cert or smth | 11:02 |
| noonedeadpunk | well, thinking about that likely you are right :D | 11:03 |
| noonedeadpunk | We don't need actual certs there | 11:03 |
| noonedeadpunk | and ca-cert should come from system ca-store | 11:03 |
| noonedeadpunk | But I have nowhere to test this on | 11:03 |
| jamesdenton | i am not reliable at this time of day | 11:03 |
| noonedeadpunk | yeah, sorry | 11:04 |
| noonedeadpunk | I guess it's super early for you now | 11:04 |
| jamesdenton | the ovn default patch is close, i need to make a change to tempest var | 11:04 |
| noonedeadpunk | tempest_public_net_physical_name ? | 11:04 |
| jamesdenton | yes | 11:04 |
| jamesdenton | change from vlan to flat | 11:04 |
| noonedeadpunk | yeah was just looking at that | 11:04 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Implement OVN inventory changes and deploy by default https://review.opendev.org/c/openstack/openstack-ansible/+/862924 | 11:05 |
| noonedeadpunk | done | 11:05 |
| jamesdenton | thanks :) | 11:05 |
| noonedeadpunk | I will try to play with ovn ssl one wrt neutron-server | 11:06 |
| jamesdenton | now in... < 2 hrs we should know | 11:06 |
| jamesdenton | so, is the issue there the path that was written? | 11:06 |
| noonedeadpunk | but yes, want to wait for CI results first | 11:06 |
| noonedeadpunk | I think yes. Though it might appear that the only thing we need there is path to CA | 11:07 |
| gokhani | noonedeadpunk, upload volume to glance (ceph) is very slow. when I run ceph -s, in output at io side client wr is 8.2 MİB/s. is it expected ? | 11:10 |
| noonedeadpunk | um no idea here tbh | 11:11 |
| noonedeadpunk | I wasn't able to quickly find code that's responsible for that | 11:12 |
| noonedeadpunk | but uploading volume as image is ususally really slow indeed | 11:12 |
| noonedeadpunk | As I guess some staging is used somewhere | 11:12 |
| gokhani | noonedeadpunk, in there I am trying to migrate fc (fiber channel) backend bootable volumes to ceph. so for this , firstly I stop the machine, then create volume snaphot, create volume from this snapshot and on volue upload image. May be my method is wrong for migrating | 11:15 |
| noonedeadpunk | tbh if I were you I would hack my way around. I'm not sure about fc backend and it's storage format (is it raw?), but I would try 2 things: 1. simple volume re-type. might not work though if disk format is not raw for your current backend (though with small hacks in code you may add format conversion) 2. write some migration script that will do rbd import and then make cinder volume manage API call | 11:18 |
| jamesdenton | noonedeadpunk i'll kick off a multi-node lxc-based deploy and see if i can tweak the ovn ssl stuff. | 11:23 |
| gokhani | noonedeadpunk, fc type is hp msa storage, change type is not worked. my image is qcow2 for creating volume | 11:23 |
| noonedeadpunk | Honestly I was thinking just to try this out in CI | 11:24 |
| noonedeadpunk | If we've catched missing certs, I guess we will also catch if these params are even needed | 11:24 |
| opendevreview | Merged openstack/openstack-ansible-lxc_hosts master: Add option to disable lxc interface management https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/861676 | 11:25 |
| jamesdenton | well, if i'm reading the log right, i am not so sure it's missing certs? | 11:25 |
| jamesdenton | this, right? oslo_config.cfg.ConfigFilesNotFoundError: Failed to find some config files: /etc/neutron/neutron.conf,/etc/neutron/plugins/ml2/ml2_conf.ini | 11:26 |
| jamesdenton | oh neveermind | 11:27 |
| jamesdenton | it pays to scroll down | 11:27 |
| jamesdenton | https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/862403/21/templates/plugins/ml2/ml2_conf.ini.j2 | 11:32 |
| jamesdenton | i think those just need to be changed from neutron_conf_version_dir to whatever var specifies "/etc/neutron/" | 11:33 |
| gokhani | noonedeadpunk, I am considering apply your second option with command "rbd -p cinder-volumes --image-format 2 import <your-image-file> <volume-uuid>". I am trying to find how can I reach volume images on storage side | 11:34 |
| noonedeadpunk | um, there's amn api call for that | 11:38 |
| noonedeadpunk | https://docs.openstack.org/api-ref/block-storage/v3/index.html#volume-manage-extension-manageable-volumes | 11:39 |
| gokhani | noonedeadpunk, I tried manageable volumes but on hpe side it is not implemented :( | 12:12 |
| noonedeadpunk | well, it is on ceph side for sure | 12:25 |
| noonedeadpunk | so once you do rbd import you can manage volume and new one will appear in cinder | 12:25 |
| noonedeadpunk | or you will need to mess up in DB manually but it's super tricky | 12:25 |
| gokhani | noonedeadpunk, I think https://ceph.io/en/news/blog/2014/openstack-import-existing-ceph-volumes-in-cinder/ help | 12:30 |
| mgariepy | good morning | 12:51 |
| mgariepy | so ssl ovn stuff ? | 12:52 |
| mgariepy | not sure i follow 100% here. need more coffee. | 12:53 |
| *** frenzy_friday|rover is now known as frenzy_friday|rover|lunch | 12:58 | |
| noonedeadpunk | mgariepy: so question is if neutron-server does not ovn certs to be present or only CA or nothing as CA is in system tryst store anyway | 13:11 |
| mgariepy | it needs the key to talk to ovn, and we could probably use neuytron key directly since it's the same server but we dont have that currently. | 13:13 |
| opendevreview | George Shuklin proposed openstack/openstack-ansible-os_nova master: fix: os_vif_ovs should not be in the middle of libvirt section https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/866159 | 13:13 |
| mgariepy | https://docs.openstack.org/neutron/zed/configuration/ovn.html#ovn | 13:14 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_nova master: fix: os_vif_ovs should not be in the middle of libvirt section https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/866159 | 13:23 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: add ovn ssl config https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/862403 | 13:30 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: Add lxb jobs instead of ovn https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/865973 | 13:31 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: Set default plugin type to OVN https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/865961 | 13:32 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Implement OVN inventory changes and deploy by default https://review.opendev.org/c/openstack/openstack-ansible/+/862924 | 13:32 |
| Mouaa | Hi guys, in distro installation method, OSA use only packages distributed by OS distribution or third party like mariadb? | 13:38 |
| Mouaa | Ubuntu 20.04 doesn't provides mariadb-server-10.4 ! | 13:38 |
| Mouaa | repo.vexxhost.net is down ! CF : https://opendev.org/openstack/openstack-ansible-galera_server/commit/5a19d466920365b50886541cbec7bac80ea66afc | 13:38 |
| jrosser | Mouaa: maridab package is usually taken from the mariadb repo | 13:42 |
| jrosser | Mouaa: which release are you trying to use? | 13:44 |
| jrosser | *of openstack-ansible | 13:44 |
| Mouaa | I'm on ussurri OSA release, OS ubuntu 20.04 | 13:45 |
| noonedeadpunk | Mouaa: ah, yes. We've backported change to cover that | 13:46 |
| Mouaa | I'll testing with the galera repo directly | 13:46 |
| Mouaa | http://mariadb.mirror.globo.tech/repo/10.4/ubuntu/dists/focal/ | 13:46 |
| noonedeadpunk | this one: https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/851439 | 13:46 |
| jrosser | Mouaa: the most recent code on the ussuri branch does not try to use the vexxhost mirror | 13:46 |
| noonedeadpunk | but already existining U deployments were affected by that | 13:47 |
| noonedeadpunk | so yeah, you can bump galera role version to get rid of the issue | 13:48 |
| Mouaa | You're sure? I had started the OSA upgrade from train, now i'm on ussuri release, and i had re boostraped recently all roles.. | 13:48 |
| jrosser | pretty sure | 13:50 |
| *** frenzy_friday|rover|lunch is now known as frenzy_friday|rover | 13:50 | |
| jrosser | remember that ussuri branch is now in "extended maintainance" | 13:50 |
| jrosser | which git reference did you check out openstack-ansible to for your ussuri upgrade? | 13:51 |
| Mouaa | During train to ussuri upgrade, i was on ussuri-em. Now, i switch on origin/stable/ussuri | 13:52 |
| Mouaa | I will check in my notes if i had really reboostrapped all roles, bescause effectively, now the repo is "downloads.mariadb.com" on https://opendev.org/openstack/openstack-ansible-galera_server/src/branch/stable/ussuri/defaults/main.yml | 13:54 |
| jrosser | well ussuri-em is a tag, signalling the point in time that the branch was made EM, tags like that won't move forward with any further changes we land on ussuri | 13:54 |
| jrosser | ussuri-em -> fixed point in history........ origin/stable/ussuri -> always the most recent point on that branch | 13:55 |
| Mouaa | git checkout stable/ussuri | 13:57 |
| Mouaa | Previous HEAD position was 9f1ac5f57 Bump OpenStack-Ansible Ussuri | 13:57 |
| Mouaa | Branch 'stable/ussuri' set up to track remote branch 'stable/ussuri' from 'origin'. | 13:57 |
| jrosser | well whichever way, ansible-role-requirements points to different things depending if you check out ussuri-em or origin/stable/ussuri | 13:59 |
| Mouaa | I had enabled origin/stable/ussuri in OSA and replay scripts/bootstrap-ansible.sh so normally openstack-ansible-galera_server role should be in latest stable/ussuri right? | 14:02 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-lxc_hosts master: Use correct apt repo when ubuntu architecture is not x86_64 https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/866167 | 14:03 |
| jrosser | the ansible roles should be checked out to whatever is specified in ansible-role-requirements.yml | 14:03 |
| spatel | jamesdenton morning! i am looking at your patch https://review.opendev.org/c/openstack/openstack-ansible/+/862924 does it going to impact already running production environment during upgrade? | 14:09 |
| jamesdenton | i would not expect it to, but the custom env.d that was required shouldn't be required any more. i will try and test it | 14:10 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Fix console type selection for aarch64. https://review.opendev.org/c/openstack/openstack-ansible/+/866169 | 14:14 |
| jrosser | noonedeadpunk: do you have a deployment AIO or something since https://review.opendev.org/c/openstack/openstack-ansible-lxc_container_create/+/861391 merged? | 14:20 |
| spatel | jamesdenton why don't we create pre-upgrade-checklist.sh script which will decided if its LXD based install or OVN and then proceed.. just a thought | 14:30 |
| admin1 | is it possible to switch from non-sriov to sr-iov without a change in the ip address ? | 14:41 |
| spatel | admin1 change in ip address? | 14:45 |
| spatel | sriov has nothing to do with IP address | 14:45 |
| admin1 | true | 14:45 |
| jamesdenton | admin1 you mean changing an instance port from tap to vf while keeping the IP? | 14:53 |
| dok53 | Hi all, novnc console not working and getting this in the log on the nova container. Origin header protocol does not match this host. Any suggestions where to look? I've tried a few things but no joy, If I click show console only it says This site can’t provide a secure connection, but, if I remove the s off https I get an attempt at connecting with "something went wrong" | 14:55 |
| opendevreview | George Shuklin proposed openstack/openstack-ansible-os_neutron master: feat: add neutron_ha_vrrp_health_check_interval variable https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/866176 | 14:55 |
| jamesdenton | what version? | 14:55 |
| jrosser | dok53: i think follow what the error message says "Origin header protocol does not match this host." | 14:59 |
| jrosser | probably some URL/FQDN mismatch between how you access the console with your browser and what you have configured in openstack | 15:00 |
| dok53 | Yeah there's a dns entry for the controller but that's just pointing at the IP address it hasn't been configured in the openstack config files but that's where I thought the issue might be. I'll look into it more so, thanks for the confitrmation | 15:01 |
| jrosser | dok53: are you using the FQDN in your openstack_user_config for the external VIP, or the IP? | 15:03 |
| jrosser | dok53: there is a pretty comprehensive set of content security headers set which is why you need to get this correct https://github.com/openstack/openstack-ansible/blob/master/inventory/group_vars/haproxy/haproxy.yml#L46 | 15:05 |
| dok53 | No I'm actually helping someone else with a cluster and they're not using it in the config. After they got the install finished they put in an internal dns entry for their controller. My console worked out of the box every time so I'm just trying to help him. I'll take a look at that so | 15:10 |
| dok53 | thanks | 15:11 |
| jrosser | i think it's needed to do it the other way round really | 15:13 |
| jrosser | noonedeadpunk: I see something a bit odd with erlang versions, it's not matching what we specify in rabbitmq_server defaults/main.yml | 15:16 |
| admin1 | i have seen rabbitmq refusing to install due to versions .. what i do is login and install the latest apt-get install rabbitmq-server and run the playbook again | 15:17 |
| *** dviroel is now known as dviroel|lunch | 15:17 | |
| jrosser | noonedeadpunk: looks like this https://paste.opendev.org/show/brMFO51ou6vQazblH4Hx/ | 15:18 |
| jrosser | it's like the erlang-base package is only in the ubuntu archive, not the cloudsmith one | 15:23 |
| jrosser | but i am looking on aarch64 right now so that could also add wierdness | 15:24 |
| jamesdenton | noonedeadpunk looks like this: https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/862403/21..22/tasks/main.yml, may need to be reverted. Neutron server does want the certs and keys | 15:55 |
| opendevreview | James Denton proposed openstack/openstack-ansible-os_neutron master: add ovn ssl config https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/862403 | 16:04 |
| noonedeadpunk | jamesdenton: now we know :D | 16:06 |
| jamesdenton | :) | 16:08 |
| *** dviroel|lunch is now known as dviroel | 16:19 | |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: Set default plugin type to OVN https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/865961 | 16:39 |
| opendevreview | Bjoern Teipel proposed openstack/openstack-ansible-os_octavia stable/yoga: Adding octavia_provider_network_mtu-parameter parameter https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/865643 | 17:11 |
| opendevreview | Bjoern Teipel proposed openstack/openstack-ansible-os_octavia stable/xena: Adding octavia_provider_network_mtu-parameter parameter https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/865644 | 17:13 |
| opendevreview | Bjoern Teipel proposed openstack/openstack-ansible-os_octavia stable/wallaby: Adding octavia_provider_network_mtu-parameter parameter https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/865645 | 17:13 |
| noonedeadpunk | folks, can we land https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/865566 ? | 17:19 |
| noonedeadpunk | it's needed to unblock upgrade jobs | 17:21 |
| jrosser | noonedeadpunk: you can probably vote yourself as it;s a cherry-pick | 17:28 |
| noonedeadpunk | well... | 17:32 |
| noonedeadpunk | master isn't merged yet... | 17:32 |
| noonedeadpunk | but yeah... | 17:32 |
| jrosser | oh hah yes | 17:32 |
| noonedeadpunk | damn... | 17:34 |
| jrosser | ewwww thats unfortunate | 17:35 |
| noonedeadpunk | btw ovn looks quite good now | 17:38 |
| jrosser | finding oddness with whichever version of the openstack collection we use right now | 17:40 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Run zookeeper installation for validate job https://review.opendev.org/c/openstack/openstack-ansible/+/866047 | 17:40 |
| jrosser | tempest uploading arm images that need properties: {hw_firmware_type: uefi} fails | 17:40 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_octavia master: Add coordination to octavia https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/866058 | 17:42 |
| noonedeadpunk | I think we pushed some fixes for collection specifically regarding image management | 17:43 |
| noonedeadpunk | And master is not tracking this SHA | 17:43 |
| noonedeadpunk | I'm not even sure if change was merged. | 17:44 |
| jrosser | i think this specifically breaks something https://github.com/openstack/openstack-ansible-os_tempest/blob/master/vars/main.yml#L37-L38 | 17:44 |
| jrosser | but i can't find the error message at all with codesearch to see where it comes from | 17:44 |
| jrosser | oh also i have a patch to push to updte to the the latest cirros too | 17:44 |
| noonedeadpunk | It was this https://review.opendev.org/c/openstack/ansible-collections-openstack/+/862565 | 17:44 |
| noonedeadpunk | there was quite some mess in collection regarding image module tbh | 17:45 |
| noonedeadpunk | but not sure it's related though | 17:46 |
| jrosser | well, i commented those properties out of the ansible vars, re-ran it which worked | 17:47 |
| jrosser | and then `openstack image set .......` to fix it up afterwards in the utility host | 17:47 |
| jrosser | more interested if my arm AIO works or not at this point | 17:48 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-tests master: Switch CentOS 8 Stream to 9 Stream https://review.opendev.org/c/openstack/openstack-ansible-tests/+/866197 | 17:52 |
| noonedeadpunk | I've just o_O on https://opendev.org/openstack/openstack-ansible/src/branch/master/playbooks/os-glance-install.yml#L90-L180 | 17:59 |
| noonedeadpunk | or better say /o\ | 17:59 |
| noonedeadpunk | I wonder if there're really good reasons we still do that.... | 18:06 |
| noonedeadpunk | or just scared to touch as "it works:" | 18:08 |
| jrosser | seems to be origins here https://opendev.org/openstack/openstack-ansible/commit/ef12bf04eda9b77f81020c8a1098e04cb53453f7 | 18:09 |
| jrosser | likewise i pretty much dare not touch that | 18:09 |
| noonedeadpunk | yeah, I'm kind of scared to touch as well, as this stuff looks too smart for me to fully understand I guess | 18:20 |
| noonedeadpunk | Just trying to check why distro jobs are failing :D | 18:21 |
| noonedeadpunk | and it's specifically because venv_tag is not defined/exist in localy stored facts | 18:21 |
| jamesdenton | looks like the ovn ssl patches passed | 18:57 |
| mgariepy | yay | 19:04 |
| opendevreview | James Denton proposed openstack/openstack-ansible master: Remove deprecated OVN and Panko services https://review.opendev.org/c/openstack/openstack-ansible/+/866205 | 19:20 |
| opendevreview | James Denton proposed openstack/openstack-ansible master: Implement OVN inventory changes and deploy by default https://review.opendev.org/c/openstack/openstack-ansible/+/862924 | 19:26 |
| opendevreview | James Denton proposed openstack/openstack-ansible master: Implement OVN inventory changes and deploy by default https://review.opendev.org/c/openstack/openstack-ansible/+/862924 | 20:56 |
| *** dviroel is now known as dviroel|afk | 21:04 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!