| opendevreview | Merged openstack/ansible-collections-openstack master: Support description in sg-rule creation https://review.opendev.org/c/openstack/ansible-collections-openstack/+/837236 | 09:32 |
|---|---|---|
| opendevreview | Merged openstack/ansible-collections-openstack stable/1.0.0: Temporarily set job openstack-tox-linters-ansible-2.9 to non-voting https://review.opendev.org/c/openstack/ansible-collections-openstack/+/838543 | 09:32 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Dropped broken linter job openstack-tox-linters-ansible-2.9 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/838705 | 09:34 |
| opendevreview | Merged openstack/ansible-collections-openstack stable/1.0.0: Dropped broken linter job openstack-tox-linters-ansible-2.9 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/838705 | 10:16 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Drop username from return values of identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/837484 | 10:36 |
| opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Moves image_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828158 | 11:57 |
| jm1 | gtema: hello :) hope you are doing well. when using .image.create_image() from openstacksdk master against devstack master, i get a strange issue: "On instance['protected']:: Provided object does not match schema 'image': 'False' is not of type 'boolean':" | 12:09 |
| jm1 | gtema: https://147a560d4302a95ea599-b3fb5ecd7a4ff2f6244fae5211e9579f.ssl.cf1.rackcdn.com/837236/5/check/ansible-collections-openstack-functional-devstack/4d75a80/job-output.txt | 12:09 |
| jm1 | gtema: any idea what could be wrong here? | 12:09 |
| jm1 | sshnaidm: fyi ^ | 12:10 |
| jm1 | gtema: yoga sdk against devstack master works fine though. | 12:11 |
| jm1 | gtema: i have no idea what could possible be changed in our call to .image.create_image() | 12:12 |
| sshnaidm | may it be "protected" vs "is_protected"? | 12:30 |
| jm1 | gtema: maybe but "git diff origin/stable/yoga..master openstack/image/v2/image.py" does not reveal any related change. actually, even "git diff origin/stable/train..master openstack/image/v2/image.py" does not show anything special | 12:34 |
| frickler | jm1: this looks related https://review.opendev.org/c/openstack/openstacksdk/+/820926 | 12:52 |
| jm1 | frickler: oh nice, thank you! let me check that | 12:57 |
| opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update routers_info for the new SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/838489 | 13:33 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Move dns zone info to use proxy layer https://review.opendev.org/c/openstack/ansible-collections-openstack/+/833028 | 13:51 |
| ralonsoh | artom, https://review.opendev.org/c/openstack/releases/+/838317 hi, if you have time, please check this patch | 13:57 |
| ralonsoh | I think we can release this version, there is a long diff between now and last version | 13:57 |
| artom | ralonsoh, I think you mean gtema? | 14:02 |
| artom | I'm the other Artom, with an O :) | 14:02 |
| artom | From Nova :) | 14:03 |
| ralonsoh | artom, sorry, right! | 14:03 |
| gtema | I am on PTO this week, sorry. | 14:06 |
| ralonsoh | no problem, thanks! | 14:06 |
| ralonsoh | enjoy | 14:07 |
| opendevreview | Merged openstack/ansible-collections-openstack stable/1.0.0: Drop username from return values of identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/837484 | 16:02 |
| jm1 | frickler: the patch you mentioned is the reason for our issues. actually it is not create_image() which fails but update_image_properties(). previously, one could call update_image_properties(..., protected=False) but since this patch is has to be is_protected instead of protected. Question is, do we want to accept both parameters in update_image_properties()? Then i would submit a patch. If we are fine with breaking backward compatibility ( | 16:19 |
| jm1 | which sdk 1.0.0 does anyway), then i wont ^^ | 16:19 |
| frickler | jm1: if the patch is easy, then go ahead, otherwise best wait for gtema's opinion on this | 16:28 |
| mgagne | May I get this review attention? It's updating Internap auth URL in vendors config. The current URL will be removed in the following days. https://review.opendev.org/c/openstack/openstacksdk/+/838734 | 16:48 |
| opendevreview | Jakob Meng proposed openstack/openstacksdk master: Fix creation of protected image for old user code https://review.opendev.org/c/openstack/openstacksdk/+/838941 | 19:10 |
| jm1 | frickler: the patch is a one liner https://review.opendev.org/c/openstack/openstacksdk/+/838941 | 19:11 |
| jm1 | frickler: but i am not sure if gtema likes to keep backward compat. because he accepted the original patch 🤷 | 19:13 |
| jm1 | frickler: maybe we ask gtema when he is back from PTO | 19:13 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!