| noonedeadpunk | jrosser: well, capi 0.29.0 is not building properly, as pretty much expected: https://zuul.opendev.org/t/openstack/build/d29defe6767c48dcbeee3c8101731daa/log/job-output.txt#19518 | 12:09 |
|---|---|---|
| noonedeadpunk | apparently we need to add some dependencies for it to succeed | 12:09 |
| opendevreview | Merged openstack/openstack-ansible-ops master: Define clusterctl_version explicitly https://review.opendev.org/c/openstack/openstack-ansible-ops/+/941114 | 14:38 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-lxc_hosts stable/2024.2: Use distro-specific SSH service name https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/945229 | 15:30 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-lxc_hosts stable/2024.2: Use distro-specific SSH service name https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/945229 | 15:30 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-lxc_hosts stable/2024.2: Use distro-specific SSH service name https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/945229 | 15:31 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Skip image upload if checksum is provided https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/945248 | 18:50 |
| noonedeadpunk | jrosser: this is smth we've discussed a while ago about image upload - just managed to look into this (finally) ^ | 18:50 |
| noonedeadpunk | like... 2 cycles ago... | 18:50 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Skip image upload if checksum is provided https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/945248 | 18:51 |
| opendevreview | Merged openstack/openstack-ansible-ops master: Store clusterctl and magnum-system namespace dump if cluster create fails https://review.opendev.org/c/openstack/openstack-ansible-ops/+/936626 | 18:51 |
| jrosser | noonedeadpunk: we also had the situation in CI jobs | 18:58 |
| jrosser | where the might be a directory (like cache directory location) that already has downloaded images in, which do need uploading to glance | 18:58 |
| jrosser | but don't need downloading from the source | 18:59 |
| noonedeadpunk | I think this should be fine? | 19:04 |
| noonedeadpunk | oh, right, I forgot to remove images_with_url as it's not used anywhere now :) | 19:05 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Skip image upload if checksum is provided https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/945248 | 19:05 |
| noonedeadpunk | as initially I wanted to skip uploadf attempt only for ones with url | 19:05 |
| noonedeadpunk | and which are in local cache - let the module decide their destiny | 19:06 |
| noonedeadpunk | but then realized there's no reason to separate these 2 usecases as checksum is kinda reliable enough to not bother with rest | 19:06 |
| noonedeadpunk | so for this usecase we just define `filename` instead of url | 19:09 |
| jrosser | so its like wanting to define the url to cirros image for example | 19:29 |
| jrosser | then by magic in a zuul job that is already on the disk so we don't download it again, but just put into glance | 19:29 |
| jrosser | but in a local AIO it should download it as there won't be an on-disk copy | 19:29 |
| noonedeadpunk | I wonder if we can just override the variable as a whole in zuul? or like define some another var? | 20:12 |
| noonedeadpunk | just to offload this part from the role to variable defenition? | 20:12 |
| noonedeadpunk | or you wanna detect "file://" as the url? | 20:16 |
| noonedeadpunk | or well, detect in url and the pass either url or path depending on the regex? | 20:17 |
| jrosser | well i don't know | 20:17 |
| jrosser | i'd also like to avoid repeatedly downloading for no reason | 20:18 |
| noonedeadpunk | yes, that is actually what https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/945248 aims to do | 20:18 |
| noonedeadpunk | I can check what can be done wrt file:// as a url | 20:19 |
| noonedeadpunk | but it will only wortk if checksum is supplied | 20:21 |
| jrosser | magnum is a similar case but for that the image is not on the disk | 20:28 |
| jrosser | in that case you'd want to download once then cache | 20:29 |
| jrosser | (really sorry but i've not managed to look at this code again for a really long time) | 20:29 |
| opendevreview | Merged openstack/openstack-ansible-lxc_hosts stable/2024.2: Use distro-specific SSH service name https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/945229 | 21:00 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!