| spatel | @nooo | 02:47 |
|---|---|---|
| cnilesh | jrosser++ noonedeadpunk++ thanks | 05:20 |
| noonedeadpunk | mornings | 06:56 |
| noonedeadpunk | jrosser: `defalt` vs `default` everywhere | 06:56 |
| noonedeadpunk | though it should not be needed.... | 06:58 |
| noonedeadpunk | let me check.... | 06:59 |
| noonedeadpunk | let's land https://review.opendev.org/c/openstack/openstack-ansible/+/923632 and https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/923637 to fix zed (https://review.opendev.org/c/openstack/openstack-ansible/+/923619) which would fix 2023.1 | 07:01 |
| jrosser | noonedeadpunk: look at the linter results for the previous patch sets | 07:01 |
| noonedeadpunk | yeah, I'm checking patchset 7 now | 07:01 |
| noonedeadpunk | it was all success, except linters. which is weird | 07:02 |
| noonedeadpunk | or well, depends. have a guess | 07:03 |
| jrosser | if the linter does not load vars, then it will not see a declaration for install_method | 07:03 |
| jrosser | I make another typo :) ? | 07:03 |
| noonedeadpunk | yeah, and for that it should load correct inventory | 07:03 |
| noonedeadpunk | yup | 07:03 |
| noonedeadpunk | but I'd stick with patchset anyway - better to fix linters here, imo | 07:04 |
| jrosser | oh well | 07:05 |
| jrosser | stick with which one? | 07:05 |
| noonedeadpunk | sorry:D 7 | 07:06 |
| jrosser | btw was talking with andrewbonney and it would be super helpful if we can backport the changes to plug-ins repo that affects mcapi | 07:06 |
| jrosser | even if we don’t use them in openstack-ansible repo | 07:07 |
| noonedeadpunk | yeah, sure we can do that | 07:07 |
| jrosser | otherwise we instantly make it not possible to use master version of mcapi on C | 07:07 |
| noonedeadpunk | though - we're branching ops repo as well? | 07:11 |
| noonedeadpunk | so we'd need to backport things there as well? | 07:11 |
| jrosser | for install_defaults I think you are right | 07:16 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Use openstack.osa.install_defaults role instead of vars_files https://review.opendev.org/c/openstack/openstack-ansible/+/923358 | 07:16 |
| noonedeadpunk | this should fix it ^ | 07:17 |
| noonedeadpunk | for linters | 07:17 |
| noonedeadpunk | specifically https://review.opendev.org/c/openstack/openstack-ansible/+/923358/10/.ansible-lint | 07:17 |
| jrosser | oh! TIL :) | 07:18 |
| noonedeadpunk | btw I was wondering recently, why in the world do we install haproxy+keepalived on bare metal instead of containers? | 07:18 |
| noonedeadpunk | just historically? | 07:18 |
| jrosser | I think that because the container restart (that happens quite a bit) during playbooks would be kind of bad | 07:19 |
| jrosser | same for neutron agents I guess | 07:19 |
| noonedeadpunk | and... we need to restart containers that frequently? | 07:20 |
| noonedeadpunk | for neutron agents I actually thought it was related to managing lxb from containers is not great... | 07:21 |
| noonedeadpunk | maybe better with ovs/ovn though, but still | 07:21 |
| noonedeadpunk | ok, but yes, I didn't thought about container restarts | 07:22 |
| noonedeadpunk | ofr linters, probably, there's some way to pass inventory path... but dunno | 07:22 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove remove_container_journal common task file https://review.opendev.org/c/openstack/openstack-ansible/+/923366 | 07:32 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove dynamic-grouping common task file https://review.opendev.org/c/openstack/openstack-ansible/+/923367 | 07:32 |
| opendevreview | Merged openstack/openstack-ansible-openstack_hosts unmaintained/yoga: Allow openstack_hosts role to configure git safe.directory entries https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/923637 | 07:32 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use haproxy_endpoint_manage role from osa collection rather than common-tasks https://review.opendev.org/c/openstack/openstack-ansible/+/923368 | 07:32 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: [doc] Update role maturity matrix https://review.opendev.org/c/openstack/openstack-ansible/+/923712 | 07:50 |
| noonedeadpunk | do you think it's time for us to try to move default of hw_machine_type to q35? meaning: https://docs.openstack.org/nova/latest/admin/hw-machine-type.html | 08:35 |
| jrosser | we do that already for some time as an override | 08:40 |
| noonedeadpunk | I assume you have that for arm/x86 differently? | 09:10 |
| opendevreview | Merged openstack/openstack-ansible-plugins master: Modify install_defaults to be more overridable https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/923390 | 09:10 |
| noonedeadpunk | I'm just starring on this thing for couple of cycles already, but haven't applied it | 09:11 |
| andrewbonney | Yes, we have 'hw_machine_type: "x86_64=q35"' | 09:11 |
| noonedeadpunk | and that looks like a good thing as a default these days... | 09:11 |
| noonedeadpunk | and aarch64=virt? | 09:11 |
| andrewbonney | We don't appear to set anything explicit for aarch64 | 09:12 |
| noonedeadpunk | as I guess you can do just define all arches at once... | 09:12 |
| noonedeadpunk | as, as it's already virt.... | 09:12 |
| noonedeadpunk | makes sense | 09:13 |
| noonedeadpunk | crap, os_ironic patch keeps timing out on rocky upgrade job | 09:23 |
| opendevreview | Merged openstack/openstack-ansible-lxc_container_create stable/2024.1: Respect dhcp_use_routes in lxc_container_networks https://review.opendev.org/c/openstack/openstack-ansible-lxc_container_create/+/923641 | 09:33 |
| jrosser | looks also like the linters still fail | 09:42 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: [doc] Add Code Review policies to contributor documentation https://review.opendev.org/c/openstack/openstack-ansible/+/923725 | 09:53 |
| noonedeadpunk | wtf | 09:54 |
| noonedeadpunk | `Passed: 0 failure(s), 0 warning(s) on 4 files. Last profile that met the validation criteria was 'production'.` was locally | 09:54 |
| noonedeadpunk | https://paste.openstack.org/show/bw2yEzfrIDhYlWD3fjHd/ | 09:56 |
| noonedeadpunk | oh, right | 09:57 |
| noonedeadpunk | it's now `ansible-playbook --syntax-check --list-tasks playbooks/setup-everything.yml` | 09:57 |
| noonedeadpunk | that is reproducible... | 09:57 |
| noonedeadpunk | huh | 10:04 |
| noonedeadpunk | that actually feels like some ansible bug | 10:06 |
| noonedeadpunk | https://paste.openstack.org/show/bFLaOmhckgXj3MXNQSqV/ | 10:06 |
| noonedeadpunk | but, if I uncomment... | 10:06 |
| noonedeadpunk | https://paste.openstack.org/show/bsLDEevwTW50s6fXtz84/ | 10:07 |
| noonedeadpunk | so like it tries to pre-check vars without actually loading them? | 10:07 |
| jrosser | do we run the latest ansible-lint? | 10:08 |
| noonedeadpunk | it's not ansible-lint that's failing now | 10:08 |
| noonedeadpunk | I did just ansible-playbook test.yml | 10:08 |
| noonedeadpunk | I wonder if ansible-core 2.17 covers that... | 10:10 |
| noonedeadpunk | (it does not) | 10:10 |
| noonedeadpunk | actually... include_role works.... | 10:11 |
| noonedeadpunk | ok, so probably your approach with default was good... | 10:19 |
| noonedeadpunk | as indeed, import somehow wants variable in advance.... | 10:19 |
| noonedeadpunk | but then - it also imports wrongly | 10:20 |
| noonedeadpunk | so, inventory/group_vars/ are not loaded/respected | 10:20 |
| noonedeadpunk | https://paste.openstack.org/show/btPzIg6mohQKVZNHH6fc/ | 10:22 |
| noonedeadpunk | so, like... import does not respect group_vars? | 10:27 |
| jrosser | well i had a total difficulty with this the first time round | 10:28 |
| noonedeadpunk | and eventually, the patch works overall _only_ because it's in user_varables: https://zuul.opendev.org/t/openstack/build/a2cd8e25b0fe4d92a3a0154cac7bb412/log/logs/etc/host/openstack_deploy/user_variables.yml.txt#59 | 10:29 |
| noonedeadpunk | as if I will do `ansible-playbook test.yml -e @inventory/group_vars/all/all.yml` - it will work | 10:30 |
| noonedeadpunk | imo, that's an ansible bug.... | 10:30 |
| jrosser | probably worth making an issue if you think so as i have had good luck with replies recently, even if it's "thats not a bug" | 10:31 |
| noonedeadpunk | yeah, making one | 10:33 |
| noonedeadpunk | yeah, our CI is passing _only_ because it's all in extra-vars | 10:37 |
| noonedeadpunk | so indeed, worth adding default for now, I guess... | 10:37 |
| noonedeadpunk | crap | 10:37 |
| noonedeadpunk | https://github.com/ansible/ansible/issues/83555 | 10:59 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Use openstack.osa.install_defaults role instead of vars_files https://review.opendev.org/c/openstack/openstack-ansible/+/923358 | 11:14 |
| noonedeadpunk | ok, I'm out of guesses how to make https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/923594 pass | 12:31 |
| noonedeadpunk | it always timeouts specifically on rocky 2023.1 upgrade | 12:32 |
| noonedeadpunk | pretty much inclined to set this job to NV | 12:32 |
| opendevreview | Merged openstack/openstack-ansible unmaintained/yoga: Skip installing curl for EL https://review.opendev.org/c/openstack/openstack-ansible/+/923632 | 12:39 |
| jrosser | it's pretty unclear to me why the rocky jobs takes sooo much longer consistently than the others | 12:42 |
| noonedeadpunk | they are kinda not on master | 12:45 |
| noonedeadpunk | it's really couple of minutes there | 12:50 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove remove_container_journal common task file https://review.opendev.org/c/openstack/openstack-ansible/+/923366 | 13:52 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove dynamic-grouping common task file https://review.opendev.org/c/openstack/openstack-ansible/+/923367 | 13:58 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use haproxy_endpoint_manage role from osa collection rather than common-tasks https://review.opendev.org/c/openstack/openstack-ansible/+/923368 | 13:58 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: [doc] Add Code Review policies to contributor documentation https://review.opendev.org/c/openstack/openstack-ansible/+/923725 | 13:58 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Add a release note about install_method not respecting group_vars https://review.opendev.org/c/openstack/openstack-ansible/+/923754 | 14:09 |
| opendevreview | Amy Marrich proposed openstack/openstack-ansible master: [doc] Add Code Review policies to contributor documentation https://review.opendev.org/c/openstack/openstack-ansible/+/923725 | 14:58 |
| noonedeadpunk | #startmeeting openstack_ansible_meeting | 15:02 |
| opendevmeet | Meeting started Tue Jul 9 15:02:36 2024 UTC and is due to finish in 60 minutes. The chair is noonedeadpunk. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:02 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:02 |
| opendevmeet | The meeting name has been set to 'openstack_ansible_meeting' | 15:02 |
| noonedeadpunk | #topic rollcall | 15:02 |
| noonedeadpunk | sorry for being late | 15:02 |
| noonedeadpunk | o/ | 15:02 |
| noonedeadpunk | #topic office hours | 15:08 |
| noonedeadpunk | so, I didn't have much progress during this week and was mainly absent previous one | 15:08 |
| noonedeadpunk | one of relatively importanti things - I've decided to write down policies for when we can land a change - please kindly review https://review.opendev.org/c/openstack/openstack-ansible/+/923725 and comment out there | 15:09 |
| NeilHanlon | o/ i'm here | 15:09 |
| noonedeadpunk | as that's kinda project governance thingy | 15:10 |
| NeilHanlon | ++ will review | 15:10 |
| noonedeadpunk | next to that - unmaintained/yoga seems healthy. I clean forgot to update release patch to transition Zed to unmaintained as well | 15:10 |
| jrosser | o/ sorry just in another meeting, here soon | 15:10 |
| noonedeadpunk | sure | 15:11 |
| noonedeadpunk | but now we'd need to land https://review.opendev.org/c/openstack/openstack-ansible/+/923619 to fix 2023.1 upgrade jobs | 15:11 |
| noonedeadpunk | we also should have issued new minor releases last week with regards to CVE-2024-32498 | 15:11 |
| noonedeadpunk | though, I was waiting for os_ironic bugfix to land, though it keeps timeouting on rocky upgrade job | 15:12 |
| NeilHanlon | 🙉 | 15:12 |
| noonedeadpunk | I'm really about to set this job to NV specifically for Ironic... | 15:12 |
| noonedeadpunk | Have zero idea why it's that slow.... | 15:13 |
| jrosser | it is curious indeed | 15:13 |
| noonedeadpunk | to be also fair - upgrade from 2023.1 takes way longer then from 2023.2 | 15:14 |
| noonedeadpunk | and like - 30m longer | 15:14 |
| jrosser | i hope it actually does the upgrade and we did not break that somehow | 15:15 |
| noonedeadpunk | I'm just checking last 4 rechecks for ironic, and for ubuntu upgrade from 2023.2 is 2h 11m; 2h 09m; 2h 15m | 15:15 |
| noonedeadpunk | while from 2023.1 jammy is 2h 49m; 2h 35m; 2h 56m | 15:16 |
| noonedeadpunk | hehe | 15:16 |
| jrosser | like if it upgrades to the same version that would be quicker /o\ | 15:17 |
| noonedeadpunk | I think it does: https://zuul.opendev.org/t/openstack/build/c0680e7fcd124a8ea02ed9a1bfe29ce9/log/job-output.txt#6478-6490 | 15:17 |
| noonedeadpunk | and then https://zuul.opendev.org/t/openstack/build/c0680e7fcd124a8ea02ed9a1bfe29ce9/log/job-output.txt#20066-20285 | 15:18 |
| noonedeadpunk | so we optimized something :D | 15:18 |
| NeilHanlon | :D | 15:19 |
| noonedeadpunk | I can recall you jrosser was puting some effort into that actually | 15:19 |
| NeilHanlon | oh hmmm | 15:19 |
| NeilHanlon | fatal: [localhost]: FAILED! => {"changed": false, "failures": [], "msg": "Depsolve Error occurred: \n Problem: problem with installed package curl-7.76.1-29.el9_4.x86_64\n - package curl-minimal-7.76.1-29.el9_4.x86_64 from baseos conflicts with curl provided by curl-7.76.1-29.el9_4.x86_64 from @System\n - package | 15:19 |
| NeilHanlon | curl-minimal-7.76.1-29.el9_4.x86_64 from baseos conflicts with curl provided by curl-7.76.1-29.el9_4.x86_64 from baseos\n - conflicting requests", "rc": 1, "results": []} | 15:19 |
| NeilHanlon | did we miss a backport? | 15:20 |
| noonedeadpunk | I _think_ it just landed to yoga | 15:20 |
| noonedeadpunk | if you're talking about https://review.opendev.org/c/openstack/openstack-ansible/+/923619 | 15:20 |
| NeilHanlon | yeah | 15:20 |
| noonedeadpunk | it's re-checking right now | 15:20 |
| NeilHanlon | ahh okay | 15:20 |
| NeilHanlon | sorry i misunderstood--thought that was the timeout one | 15:20 |
| noonedeadpunk | doh, it failed :( | 15:21 |
| noonedeadpunk | nah, timeout one is https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/923594 | 15:21 |
| jrosser | that is timeout during log upload? | 15:23 |
| jrosser | i feel like we have been there before | 15:23 |
| noonedeadpunk | nah | 15:23 |
| noonedeadpunk | timeout is actually somewhere between tempest and rally | 15:23 |
| noonedeadpunk | so it's just being slow | 15:23 |
| noonedeadpunk | ok, it succeeded just now in gates | 15:24 |
| NeilHanlon | of course | 15:24 |
| noonedeadpunk | so hopefully it will land now.... | 15:24 |
| NeilHanlon | it's because it was being observed | 15:24 |
| noonedeadpunk | sure, and it just got scared of you :D | 15:24 |
| noonedeadpunk | it always works like that - once you look at a thing it get scared and behaves | 15:25 |
| NeilHanlon | I believe in computer/technology "Mana" | 15:25 |
| NeilHanlon | if you don't have enough mana with something, it won't respect you | 15:25 |
| NeilHanlon | until someone with more mana comes aroud | 15:25 |
| noonedeadpunk | haha, yeah, probably :D | 15:26 |
| noonedeadpunk | so, we still have ansible-core 2.17 and mariadb 11.4 thingy | 15:26 |
| noonedeadpunk | will try to check on mariadb this week | 15:26 |
| jrosser | ansible-core just needs a release note on ceph-ansible | 15:27 |
| jrosser | and a fix for the upgrade issues | 15:27 |
| noonedeadpunk | yeah | 15:27 |
| noonedeadpunk | which maybe worth doing as a follow-up? | 15:27 |
| noonedeadpunk | ok, I guess it's time for another etherpad | 15:27 |
| jrosser | that would be helpful yes | 15:28 |
| noonedeadpunk | #link https://etherpad.opendev.org/p/osa-epoxy-ptg | 15:29 |
| noonedeadpunk | I propose to start building up ptg one right away | 15:30 |
| NeilHanlon | the name i vote for is never chosen... :P | 15:31 |
| noonedeadpunk | frankly - all names were terrible this time | 15:35 |
| noonedeadpunk | imo | 15:35 |
| noonedeadpunk | we have actually 2 more things regarding new ansible-core | 15:37 |
| noonedeadpunk | first - we need to drop `ANSIBLE_COLLECTIONS_PATHS` - this raises the warning in 2.17 | 15:37 |
| noonedeadpunk | https://paste.openstack.org/show/bomMOIV12Nki1ffQNstj/ | 15:37 |
| noonedeadpunk | and the second - kinda time to think about what to do regarding DEFAULT_GATHER_SUBSET | 15:38 |
| noonedeadpunk | as that is /o\ | 15:38 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-openstack_hosts master: Manage apt repositores and keys using deb822_repository module https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/907434 | 15:39 |
| jrosser | i have a patch for the collections path | 15:40 |
| jrosser | https://review.opendev.org/c/openstack/openstack-ansible/+/921928 | 15:41 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server master: Manage apt repositores and keys using deb822_repository module https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/907833 | 15:42 |
| noonedeadpunk | oh nice | 15:42 |
| noonedeadpunk | sorry, I haven't reviewed all things from previous week yet :( | 15:42 |
| noonedeadpunk | huh, I wonder if https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/923403 might help us with facts as well | 15:43 |
| jrosser | yeah i was thinking about something like that too | 15:44 |
| jrosser | though the thing is DEFAULT_GATHER_SUBSET also affects all 3rd party codde we use, which is a super nice side effect | 15:44 |
| noonedeadpunk | yeah | 15:44 |
| noonedeadpunk | I wonder if it's worth rasing within Ansible community | 15:45 |
| noonedeadpunk | and what's the thinking behind that | 15:45 |
| jrosser | but perhaps module_defaults on all playbooks will do the same? | 15:45 |
| noonedeadpunk | maybe they just don't recognize the usecase... | 15:45 |
| jrosser | would need to check if module_defaults will propagate down into a called role | 15:45 |
| noonedeadpunk | that is _very_ annoying.... | 15:46 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible unmaintained/yoga: Bump role SHAs for unamitained/yoga https://review.opendev.org/c/openstack/openstack-ansible/+/923768 | 15:49 |
| noonedeadpunk | this is another thing that's probably needed for Zed upgrade job... | 15:49 |
| noonedeadpunk | as it needs more modern openstack_hosts role | 15:49 |
| jrosser | i'm not sure that the work i did on auto-handling stable/<> vs unmaintained/<> landed in all the branches either | 15:50 |
| jrosser | fixing the unmaintained branches is on my todo list but just no time | 15:50 |
| noonedeadpunk | well, it pulls the right thing on Zed | 15:50 |
| noonedeadpunk | Yoga does not have that patch though | 15:51 |
| noonedeadpunk | but upgrades are NV there anyway | 15:51 |
| noonedeadpunk | actually about that | 15:51 |
| noonedeadpunk | maybe we should just eol everything below Yoga? | 15:51 |
| noonedeadpunk | or Xena | 15:51 |
| noonedeadpunk | (I'd keep Xena maybe...) | 15:52 |
| jrosser | we do still get queries on the ML about upgrades from old stuff | 15:52 |
| jrosser | so yes we should EOL *really* old stuff | 15:52 |
| noonedeadpunk | so we have Victoria, Wallaby, Xena and Yoga | 15:52 |
| noonedeadpunk | like neither of them feel *really* old.... | 15:53 |
| noonedeadpunk | (while they kinda are) | 15:53 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible unmaintained/yoga: Bump role SHAs for unamitained/yoga https://review.opendev.org/c/openstack/openstack-ansible/+/923768 | 15:54 |
| noonedeadpunk | and I highly doubt we'll backport anything below Yoga... | 15:54 |
| noonedeadpunk | Though I know couple of folks are still running Xena... | 15:55 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: Add support for deploying mcapi control plane k8s on debian-12 https://review.opendev.org/c/openstack/openstack-ansible-ops/+/923586 | 16:03 |
| jrosser | NeilHanlon: i have the starting point of making magnum cluster api work on rocky here https://review.opendev.org/c/openstack/openstack-ansible-ops/+/923447/8 | 16:04 |
| NeilHanlon | ooh, nice! | 16:04 |
| jrosser | though if it does not "just work" i'm not sure i have time to chase it | 16:05 |
| jrosser | so just fyi if you're interested in magnum stuff | 16:05 |
| jrosser | it will be kerrnel meets lxc meets bpf meets cilium == brok, most likely | 16:05 |
| NeilHanlon | alright cool, appreciate the heads up | 16:06 |
| noonedeadpunk | #endmeeting | 16:11 |
| opendevmeet | Meeting ended Tue Jul 9 16:11:02 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:11 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/openstack_ansible_meeting/2024/openstack_ansible_meeting.2024-07-09-15.02.html | 16:11 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/openstack_ansible_meeting/2024/openstack_ansible_meeting.2024-07-09-15.02.txt | 16:11 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/openstack_ansible_meeting/2024/openstack_ansible_meeting.2024-07-09-15.02.log.html | 16:11 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use UCA mirror in CI for ubuntu https://review.opendev.org/c/openstack/openstack-ansible/+/923776 | 16:37 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use ceph mirror in CI jobs https://review.opendev.org/c/openstack/openstack-ansible/+/923777 | 16:39 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use UCA mirror in CI for ubuntu https://review.opendev.org/c/openstack/openstack-ansible/+/923776 | 16:43 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use ceph mirror in CI jobs https://review.opendev.org/c/openstack/openstack-ansible/+/923777 | 16:43 |
| noonedeadpunk | btw - another ironic change that waits for merge/backport: https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/923233 | 16:44 |
| jrosser | huh interesting comment about new features in 2.17 on your github issue | 16:50 |
| noonedeadpunk | huh | 17:04 |
| noonedeadpunk | indeed | 17:04 |
| jrosser | oh cool NeilHanlon look it worked \o/ https://zuul.opendev.org/t/openstack/build/0c6e8b4627564ce59d7d28f0c45ce95b/logs | 17:04 |
| jrosser | thats the back-end k8s cluster you need in the openstack control plane working for magnum on rocky | 17:05 |
| jrosser | well i am super happy about that | 17:05 |
| noonedeadpunk | ┳━┳ ヽ(ಠل͜ಠ)ﾉ | 17:06 |
| jrosser | it also worked now on debian-12 | 17:07 |
| noonedeadpunk | there was some patch in WIP for that iirc? | 17:07 |
| jrosser | yeah there are patches in the ops repo for it | 17:08 |
| jrosser | but currently they only get tested in os_magnum as thats where the jobs are defined | 17:08 |
| jrosser | so there is a stack of things in the ops repo here https://review.opendev.org/c/openstack/openstack-ansible-ops/+/923586/3 | 17:10 |
| jrosser | and the tests are here (currently NV but i will change that once it completes) https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/923174 | 17:12 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Add a release note about install_method not respecting group_vars https://review.opendev.org/c/openstack/openstack-ansible/+/923754 | 17:19 |
| opendevreview | Merged openstack/openstack-ansible-os_ironic stable/2024.1: Add new RBAC parameters which are necessary from 2024.1 https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/923594 | 17:31 |
| noonedeadpunk | finally | 17:32 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/2024.1: Bump SHAs for 2024.1 (Caracal) https://review.opendev.org/c/openstack/openstack-ansible/+/923559 | 17:33 |
| noonedeadpunk | crap, I just realized that it was me who didn't review 923233 | 17:34 |
| opendevreview | Merged openstack/openstack-ansible master: Remove use of deprecated ANSIBLE_COLLECTIONS_PATHS https://review.opendev.org/c/openstack/openstack-ansible/+/921928 | 18:12 |
| jrosser | noonedeadpunk: is this right? https://review.opendev.org/c/openstack/openstack-ansible/+/923754/2 | 18:58 |
| jrosser | add `public: true` then abandon? | 18:58 |
| opendevreview | Merged openstack/openstack-ansible master: [doc] Add Code Review policies to contributor documentation https://review.opendev.org/c/openstack/openstack-ansible/+/923725 | 19:01 |
| opendevreview | Merged openstack/openstack-ansible-os_ironic master: Fix Ironic IPA version for 2024.1 https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/923233 | 19:29 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!