opendevreview | Jiri Podivin proposed openstack/cliff master: Clarification of the algorithm used https://review.opendev.org/c/openstack/cliff/+/862690 | 06:58 |
---|---|---|
opendevreview | Jakob 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/+/856987 | 07:03 |
opendevreview | Jakob 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/+/862741 | 07:23 |
opendevreview | Jiri Podivin proposed openstack/cliff master: Clarification of the algorithm used https://review.opendev.org/c/openstack/cliff/+/862690 | 08:47 |
opendevreview | Jakob 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/+/862741 | 09:10 |
opendevreview | Jakob 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/+/856987 | 10:02 |
opendevreview | Jakob 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/+/856987 | 12:13 |
opendevreview | Jakob 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/+/856987 | 12:13 |
opendevreview | Jakob 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/+/856987 | 12:30 |
opendevreview | Jakob 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/+/856987 | 12:31 |
opendevreview | Jakob 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/+/856987 | 12:34 |
opendevreview | Merged openstack/ansible-collections-openstack master: Always remove temporary files in volume's integration test https://review.opendev.org/c/openstack/ansible-collections-openstack/+/862741 | 15:48 |
opendevreview | Merged 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/+/856987 | 16: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.json | 18: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#585 | 19: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/!