| noonedeadpunk | good morning | 08:20 |
|---|---|---|
| hamburgler | nononedeadpunk: good morning :) in the process of upgrading to bobcat and have ran into a permissions issue caused by: https://github.com/openstack/openstack-ansible-ceph_client/compare/stable/2023.1...stable/2023.2, https://github.com/openstack/openstack-ansible-ceph_client/compare/stable/2023.1...stable/2023.2#diff-9c58c4e96ba842d81a85a993176608d12cda8f84eef05939360a190f24f7eb29, like 46 of | 08:27 |
| hamburgler | ceph_config_extra, should have owner: root and group: ceph i believe? if not with the new mode on there, my extra confs end up as root:root 640 and can't be read | 08:27 |
| hamburgler | line 46* | 08:27 |
| hamburgler | seems to oddly cause issues with glance as well even though glance has no system user, but cinder too | 08:29 |
| hamburgler | glance has no ceph system user* | 08:29 |
| noonedeadpunk | hamburgler: yeah, I think you're right about root:ceph | 08:36 |
| noonedeadpunk | I kinda wonder if some mistake was made somewhere else as well | 08:37 |
| hamburgler | so far I've been able to run it without issue on the ceph client role other than that one item, I seem to have another issue with glance atm, but will sort out tomorrow :) | 08:40 |
| noonedeadpunk | hamburgler: ok, I will take care of that today then :) | 08:46 |
| noonedeadpunk | but I kinda see that regular ceph.conf has 644 and root:root | 08:47 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-ceph_client master: Align extra conf files mode https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/906030 | 08:55 |
| noonedeadpunk | yeah, reviewing the change made through git blame, seems that it indeed should be jsut that... | 08:57 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: WIP - Add collection to deploy magnum cluster-api with vexxhost driver https://review.opendev.org/c/openstack/openstack-ansible-ops/+/901450 | 10:54 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: WIP - Bootstrapping playbook https://review.opendev.org/c/openstack/openstack-ansible-ops/+/902178 | 10:54 |
| jrosser | should we add octavia to the magnum scenario | 10:55 |
| noonedeadpunk | Well, probably not? | 11:08 |
| noonedeadpunk | I think we're very short on resources for magnum already | 11:08 |
| noonedeadpunk | we totally won't be able to test scenario involving octavia for real | 11:09 |
| noonedeadpunk | (or well, it's a gess, so cross out totally from here) | 11:09 |
| jrosser | i need to extend the os_magnum scenario to include octavia | 11:12 |
| jrosser | well, not exactly | 11:13 |
| jrosser | when the cluster_api job runs it needs to include octavia | 11:14 |
| jrosser | so i can make it so that the job setup includes octavia independantly of whatever SCENARIO does | 11:14 |
| noonedeadpunk | you can just mention octavia in job name for clusterapi job? | 11:16 |
| jrosser | thats OK for CI but not so much for local tests | 11:16 |
| noonedeadpunk | then it will be included in scenario, as scenario is just parsing the job name more or less | 11:16 |
| jrosser | ah i see so instructions can say "use SCENARIO=aio_lxc_magnum_octavia" | 11:17 |
| noonedeadpunk | yup | 11:17 |
| noonedeadpunk | or kinda ./scripts/gate_check_commit.sh aio_lxc_magnum_octavia_capi | 11:17 |
| *** priteau_ is now known as priteau | 12:14 | |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_magnum master: Add job to test Vexxhost cluster API driver https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/905199 | 12:51 |
| mgariepy | https://review.opendev.org/c/openstack/glance_store/+/885581 | 14:10 |
| mgariepy | tada ! | 14:10 |
| noonedeadpunk | niiiiiiiceeeeee | 14:11 |
| mgariepy | yes ! | 14:11 |
| noonedeadpunk | I should have sent you this patch earlier :D | 14:11 |
| mgariepy | now... backport time. lol | 14:11 |
| mgariepy | well didn't had the issue before last week or so lol | 14:11 |
| mgariepy | i don't understand why. | 14:12 |
| noonedeadpunk | so. eventually that worked for you in uWSGI, right? | 14:12 |
| mgariepy | yep | 14:12 |
| mgariepy | i manually patched the code for one container and it is working. | 14:12 |
| noonedeadpunk | So we can drop logic of having non-uwsgi then? | 14:12 |
| mgariepy | once it's all merged i guess | 14:12 |
| noonedeadpunk | Yeah, I also did that but in sandbox, so was not 100% sure | 14:12 |
| mgariepy | but yeah for master | 14:13 |
| noonedeadpunk | cool | 14:13 |
| mgariepy | i'll fork glance and apply the patch | 14:15 |
| mgariepy | for deployment locally on my installation so i can be 100% sure it works correcly. | 14:16 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_glance master: Enable uWSGI usage for Ceph backends https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/906047 | 14:27 |
| hamburgler | noonedeadpunk: thank you :), yeah the regular conf seems to be fine, tried extra configs with root:root 644 now and looks good too | 14:35 |
| jrosser | do we have an example somewhere of using the `openstack` cli in the utility container when the ansible module you want does not exist? | 15:23 |
| noonedeadpunk | yes | 15:30 |
| noonedeadpunk | https://opendev.org/openstack/openstack-ansible-os_cinder/src/branch/master/tasks/cinder_backends.yml#L39 | 15:30 |
| noonedeadpunk | Actually we should drop that as module was created and merged for this specific thing | 15:31 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_glance master: Fix iteration over backends config https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/906048 | 15:50 |
| *** tosky_ is now known as tosky | 16:41 | |
| mgariepy | noonedeadpunk, do you have an idea how to fork glance_store ? how to force glance to use the forked repo ? | 16:47 |
| noonedeadpunk | mgariepy: oh, that's really good question | 16:56 |
| noonedeadpunk | I kinda don't know :D | 16:56 |
| noonedeadpunk | it's comming from upper-constraints | 16:56 |
| noonedeadpunk | So potentially, you need to adjust `glance_pip_packages` | 16:57 |
| noonedeadpunk | by adding there - "git+{{ glance_store_repo }}@{{ glance_store_version }}#egg=glance_store" | 16:58 |
| noonedeadpunk | and then in fork tag glance_store_version with the tag you find on https://releases.openstack.org/constraints/upper/<release> | 16:58 |
| noonedeadpunk | basically drop existing and re-tag in fork | 16:59 |
| noonedeadpunk | or smth like that.... | 16:59 |
| mgariepy | hmm. | 17:02 |
| mgariepy | look like somewhat of a mess. | 17:02 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: Add role to install and run sonobouy k8s validation tests https://review.opendev.org/c/openstack/openstack-ansible-ops/+/906054 | 17:12 |
| jrosser | noonedeadpunk: mgariepy you need to mess with constraints as well https://paste.opendev.org/show/bVQJFgF2c4XqaBageeFV/ | 17:14 |
| noonedeadpunk | well, not unless you re-tag SHA?:) | 17:16 |
| noonedeadpunk | and if that's a fork.... | 17:16 |
| noonedeadpunk | mgariepy: yes, mess | 17:16 |
| jrosser | oh well yeah that | 17:17 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_magnum master: Adopt for usage openstack_resources role https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/901185 | 17:20 |
| noonedeadpunk | I still no idea wtf with TLS jobs ^ | 17:21 |
| jrosser | oh i was going to poke at that as well | 17:24 |
| jrosser | got totally bogged down in the magnum stuff though | 17:24 |
| jrosser | *other magnum stuff | 17:25 |
| noonedeadpunk | I didn't saw issue in my aio | 17:27 |
| noonedeadpunk | *didn't see | 17:27 |
| noonedeadpunk | mnaser: hey there! I was poking around ovn-bgp driver and as you might know it needs FRR to be deployed. And then I kinda recalled about existence of https://opendev.org/vexxhost/ansible-role-frrouting | 17:37 |
| noonedeadpunk | so was wondering if it's doable to propose there any changes or it's worth just forking it elsewhere or any other option you may have in mind? | 17:38 |
| noonedeadpunk | having it under openinfra is handy as potentially I wanted to reffer to it from osa docs as prerequisite | 17:39 |
| jrosser | how does this file get called? https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/878794/33/roles/openstack_resources/tasks/coe_cluster.yml | 17:39 |
| noonedeadpunk | https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/878794/33/roles/openstack_resources/tasks/coe.yml#57 | 17:40 |
| jrosser | 8-O oh i ignored that as i read name: and moved on :) | 17:40 |
| noonedeadpunk | I should have use selectattr there though... | 17:41 |
| noonedeadpunk | hehe, I didn't notice name until you commented | 17:42 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Add openstack_resources role skeleton https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/878794 | 17:45 |
| jrosser | btw in a 32G AIO i get magnum+capi+most basic sonobouy test to pass | 17:47 |
| jrosser | so there is hope for that in a larger CI instance, if it can run within the timeout | 17:48 |
| jrosser | running it locally in a 32G VM and adding +2hours should be able to get full conformance suite | 17:48 |
| jrosser | next i need to look at what we talked about before, to add 'hooks' into OSA to call external playbooks at some points during the run | 17:49 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Add openstack_resources role skeleton https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/878794 | 17:53 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Add openstack_resources role skeleton https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/878794 | 17:59 |
| mgariepy | thanks jrosser i'll test that to see how it goes. | 19:04 |
| opendevreview | Merged openstack/openstack-ansible-os_nova master: Run ceph_client when cinder uses Ceph https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/905741 | 19:16 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_nova stable/2023.2: Run ceph_client when cinder uses Ceph https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/905918 | 19:27 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_nova stable/2023.1: Run ceph_client when cinder uses Ceph https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/905919 | 19:27 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!