Thursday, 2022-10-27

opendevreviewJiri Podivin proposed openstack/cliff master: Clarification of the algorithm used  https://review.opendev.org/c/openstack/cliff/+/86269006:58
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Run Ansible devel branch job on Python 3.10  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85698707:03
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Always remove temporary files in volume's integration test  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/86274107:23
opendevreviewJiri Podivin proposed openstack/cliff master: Clarification of the algorithm used  https://review.opendev.org/c/openstack/cliff/+/86269008:47
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Always remove temporary files in volume's integration test  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/86274109:10
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Run Ansible devel branch job on Python 3.10  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85698710:02
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Run Ansible devel branch job on Python 3.10  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85698712:13
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Run Ansible devel branch job on Python 3.10  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85698712:13
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Run *-devstack-ansible-devel job on Ubuntu 22.04 LTS (Jammy Jellyfish)  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85698712:30
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Run *-devstack-ansible-devel job on Ubuntu 22.04 LTS (Jammy Jellyfish)  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85698712:31
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Run *-devstack-ansible-devel job on Ubuntu 22.04 LTS (Jammy Jellyfish)  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85698712:34
opendevreviewMerged openstack/ansible-collections-openstack master: Always remove temporary files in volume's integration test  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/86274115:48
opendevreviewMerged openstack/ansible-collections-openstack master: Run *-devstack-ansible-devel job on Ubuntu 22.04 LTS (Jammy Jellyfish)  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85698716:02
ard_hey, I am working on a https://review.opendev.org/c/openstack/ansible-collections-openstack/+/862565 and have several question which I can't answer myself. According to https://docs.openstack.org/api-ref/image/v2/index.html?expanded=update-image-detail there is a set of image properties which can't be updated and this will result in 403 reply. The set I believe is here: https://github.com/openstack/glance/blob/master/etc/schema-image.json18:36
ard_ and this does include kernel_id and ramdisk_id . At the same time version of the script tries to change them if such params defined and differs from the present image. 18:36
ard_at the same time min_disk and min_ram are not tried to be updated. Am I missing something? 18:38
ard_jm1: regarding image.update_image with id and image object. If you send image object request.body being produced with {'op': 'remove', 'path': '/stores'} and lots of properties which aren't planned to touch. Because they are correct in existing image. openstacksdk==0.102.0 So my change has this strange workaround https://review.opendev.org/c/openstack/ansible-collections-openstack/+/862565/8/plugins/modules/image.py#58519:37
ard_Here is a little bit more details: https://paste.opendev.org/show/817361/ So does it means that update args should contain all properties to not attempt removing them? Then the logic in the module is not correct. Otherwise there is an issue in sdk version .0.102.0 19:53

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