| headphoneJames | I did another pass at adding system role to sdk. I integrated feedback from the initial patch and built it on the feature/r1 branch. | 02:11 |
|---|---|---|
| headphoneJames | https://review.opendev.org/c/openstack/openstacksdk/+/824470/ | 02:12 |
| *** mnasiadka_ is now known as mnasiadka | 08:43 | |
| *** sshnaidm|afk is now known as sshnaidm | 08:49 | |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Release 1.6.0 version https://review.opendev.org/c/openstack/ansible-collections-openstack/+/824559 | 12:46 |
| opendevreview | Merged openstack/ansible-collections-openstack master: Release 1.6.0 version https://review.opendev.org/c/openstack/ansible-collections-openstack/+/824559 | 13:42 |
| *** sshnaidm is now known as sshnaidm|afk | 16:18 | |
| ralonsoh | stephenfin, hi! do you have a couple of mins for https://review.opendev.org/c/openstack/openstacksdk/+/804979? | 17:13 |
| ralonsoh | thanks in advance! | 17:13 |
| outbrito | Howdy folks! | 17:22 |
| outbrito | gtema, I just got some time to check on your comment on this patch: https://review.opendev.org/c/openstack/openstacksdk/+/777801/3#message-1d4c80ba284da4feaf2da6497fa8fa5db833024f | 17:22 |
| outbrito | Do you know what is the plan to merge those changes on feature/r1 on master? | 17:23 |
| gtema | R1 to master merge is in progress. I need to analyse test failures which are most likely due to merge issues | 19:14 |
| frickler | gtema: is that for https://review.opendev.org/c/openstack/openstacksdk/+/823961 ? I saw that, but was wondering if that was just an intermediate test. will you postpone the cache rework, then? | 19:55 |
| frickler | I can help looking at the failures if that's the only thing left | 19:55 |
| opendevreview | Merged openstack/openstacksdk master: Add "security_group_ids" to Port's query parameters https://review.opendev.org/c/openstack/openstacksdk/+/804979 | 20:13 |
| opendevreview | Merged openstack/python-openstackclient master: Add --security-group to port list https://review.opendev.org/c/openstack/python-openstackclient/+/815506 | 20:13 |
| outbrito | gtema, nice, so I'll abandon that change and rebase the other ones | 20:14 |
| frickler | gtema: stephenfin: finding 1: https://review.opendev.org/c/openstack/openstacksdk/+/813233 is broken, allow_get is still being used in osc_lib. it is strange that the osc test only fails for flavors, because all resources seem to have "name" appearing twice in keys() after that | 21:19 |
| frickler | reverting that and adding original_name to hidden_columns for flavors fixes the osc unit tests for me | 21:22 |
| frickler | tomorrow I'll take a look at the tenant_id things | 21:22 |
| frickler | this is the reference in osc-lib fwiw https://opendev.org/openstack/osc-lib/src/branch/master/osc_lib/utils/__init__.py#L726 | 21:24 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!