| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2020-02-05 | |||
| 07:29:49 | openstackgerrit | Merged openstack/python-openstackclient master: Use 'KeyValueAppendAction' from osc-lib https://review.opendev.org/685330 | |
| 07:29:50 | openstackgerrit | Merged openstack/python-openstackclient master: Remove mention of meetings from docs https://review.opendev.org/704494 | |
| 08:45:51 | openstackgerrit | Dominik Stucki proposed openstack/ansible-collections-openstack master: add os_server_tag module https://review.opendev.org/705898 | |
| 09:39:39 | openstackgerrit | Dominik Stucki proposed openstack/ansible-collections-openstack master: add os_server_tag module https://review.opendev.org/705898 | |
| 09:44:59 | openstackgerrit | Dominik Stucki proposed openstack/ansible-collections-openstack master: add os_server_tag module https://review.opendev.org/705898 | |
| 10:39:37 | openstackgerrit | Merged openstack/os-service-types master: [ussuri][goal] Drop python 2.7 support and testing https://review.opendev.org/705815 | |
| 10:40:02 | openstackgerrit | Dominik Stucki proposed openstack/ansible-collections-openstack master: add os_server_tag module https://review.opendev.org/705898 | |
| 12:02:16 | openstackgerrit | Daniel Bengtsson proposed openstack/python-openstackclient master: Stop configuring install_command in tox and stop use pip. https://review.opendev.org/694484 | |
| 15:37:44 | AJaeger | mordred: who can review https://review.opendev.org/686590 as well? Let's wrap this up, please | |
| 15:38:56 | mordred | AJaeger: done | |
| 15:39:21 | AJaeger | thanks, mordred ! | |
| 17:15:09 | dulek | mordred: Hi! Any chance of getting a release with https://review.opendev.org/#/c/704998/? Last fix in this matter broke some assumptions we had in kuryr-kubernetes and we're now working them around by looking for "could not be found" in exception's message. I would enjoy being able to remove that piece. | |
| 17:16:46 | mordred | dulek: yeah - there's two other in the gate, let's let them land then we can cut one | |
| 17:17:12 | dulek | mordred: Thank you, that's really appreciated! :) | |
| 17:21:39 | openstackgerrit | Merged openstack/python-openstackclient master: Complete "Drop python2 support" goal https://review.opendev.org/686590 | |
| 18:08:41 | openstackgerrit | Merged openstack/openstacksdk master: Fix error handling in network trunks operations https://review.opendev.org/705073 | |
| 18:08:45 | openstackgerrit | Merged openstack/openstacksdk master: Fix error handling on add/remove router iface calls https://review.opendev.org/704998 | |
| 18:14:12 | mordred | dulek: ok - I think that's the relevant patches | |
| 18:21:06 | cmurphy | mordred: this might be interesting to you https://bugs.launchpad.net/keystoneauth/+bug/1861695 | |
| 18:21:06 | openstack | Launchpad bug 1861695 in keystoneauth "Programming error choosing an endpoint." [Undecided,New] | |
| 18:23:36 | mordred | dulek: remote: https://review.opendev.org/706076 Release 0.41.0 of openstacksdk | |
| 18:23:39 | mordred | cmurphy: "interesting" | |
| 18:24:40 | mordred | OH GOOD | |
| 18:31:43 | mordred | cmurphy: I have responded - I'm at an impass on further debugging but left a script that should get us an http trace we can look at | |
| 18:32:09 | cmurphy | mordred: tyvm | |
| 18:33:06 | mordred | cmurphy: I'm "excited" to learn what's up | |
| 18:33:20 | cmurphy | :D | |
| #openstack-sdks - 2020-02-06 | |||
| 10:00:51 | openstackgerrit | Vishakha Agarwal proposed openstack/python-openstackclient master: Adding options to user cli https://review.opendev.org/697444 | |
| 10:18:24 | openstackgerrit | Vishakha Agarwal proposed openstack/python-openstackclient master: Adding options to user cli https://review.opendev.org/697444 | |
| 11:27:25 | openstackgerrit | Hervé Beraud proposed openstack/microversion-parse master: adding missing releasenote for the drop of py27 support https://review.opendev.org/706236 | |
| 11:33:37 | openstackgerrit | Hervé Beraud proposed openstack/cliff master: adding missing releasenote for the drop of py27 support https://review.opendev.org/706241 | |
| 12:10:21 | openstackgerrit | Maari Tamm proposed openstack/python-openstackclient master: Add OpenStack Client for Manila docs https://review.opendev.org/706252 | |
| 12:19:44 | maaritamm | vkmc, efried, johnsom, hi! o/ | |
| 12:19:52 | maaritamm | created this one ˆˆ, following the steps provided by efried. Would be nice if someone could check it out :) | |
| 12:20:36 | maaritamm | About creating a doc with the steps I followed, am I correct to think that the place for that would be here : https://docs.openstack.org/python-openstackclient/latest/contributor/plugins.html#changes-to-python-openstackclient ? | |
| 12:59:12 | vkmc | maaritamm++ | |
| 12:59:22 | vkmc | great job, will check out the patch later | |
| 14:32:00 | johnsom | maaritamm Nice, I will have a look today. | |
| 16:04:32 | elmiko | API SIG office hour now open | |
| 17:00:48 | elmiko | have a good weekend all o/ | |
| 17:30:35 | openstackgerrit | Ben Nemec proposed openstack/keystoneauth master: Remove universal wheel configuration https://review.opendev.org/706330 | |
| 17:56:22 | openstackgerrit | Merged openstack/cliff master: adding missing releasenote for the drop of py27 support https://review.opendev.org/706241 | |
| 17:56:46 | openstackgerrit | Merged openstack/microversion-parse master: adding missing releasenote for the drop of py27 support https://review.opendev.org/706236 | |
| 19:16:36 | mordred | cmurphy: the launchpad issue from yesterday turned out to be related to someone trying to use interface: internal and there being no internal endpoint | |
| 20:41:21 | cmurphy | mordred: i saw, do you think the issue should be closed or should ksa handle that better? | |
| 20:41:48 | mordred | it's a good question | |
| 20:41:58 | mordred | cmurphy: ksa probably should produce a better error message | |
| 20:42:24 | mordred | but it's hard to imagine in this specific case what would be the right way to communicate to that person that there is not an internal interface | |
| 20:43:34 | mordred | because we don't necessarily know which filter is the filter that caused us to not find an endpoint, yeah? but maybe just a long message like "could not find endpoint X with these features ... and maybe if we did have matches up further in the loops we could say something like "but we did find {service-type} with ... {blah}" ... but I dunno | |
| 20:43:37 | mordred | cmurphy: what do you think? | |
| 20:46:04 | cmurphy | mordred: yeah something to give a hint other than "programmer error" would be nice | |
| 20:54:55 | mordred | cmurphy: eek. looking at the code right now ... this might not be the most straightforward thing ever | |
| 20:59:49 | cmurphy | mordred: sadpanda | |
| 21:06:08 | mordred | cmurphy: I'm going to stare at it a little more - but oy, did we really need all this complexity? | |
| 21:06:36 | cmurphy | mordred: i guess it seemed like a good idea at the time :P | |
| 21:06:45 | mordred | of course it did | |
| 21:06:58 | mordred | I'd blame kmalloc ... | |
| 23:58:42 | openstackgerrit | Merged openstack/keystoneauth master: Remove universal wheel configuration https://review.opendev.org/706330 | |
| #openstack-sdks - 2020-02-07 | |||
| 13:30:57 | fuhrmannb | I have a question about openstacksdk: I tried to use argparse as options to openstack.connect() as proposed in https://docs.openstack.org/openstacksdk/latest/user/config/using.html#argparse. It is working for environment variables but script arguments does not seem to work. Is there an actual issue about it? | |
| #openstack-sdks - 2020-02-08 | |||
| 10:13:42 | Jeffrey4l | anyone could help me for this ansible patch? https://github.com/ansible/ansible/pull/67025 can not find the real error message in zuul | |
| 10:27:25 | openstackgerrit | Jeffrey Zhang proposed openstack/ansible-collections-openstack master: Support json type for the ironic_node config_drive parameter https://review.opendev.org/706664 | |
| 16:30:43 | openstackgerrit | Radosław Piliszek proposed openstack/js-openstack-lib master: CI: New jobs https://review.opendev.org/702132 | |
| 16:32:23 | openstackgerrit | Radosław Piliszek proposed openstack/js-openstack-lib master: CI: New jobs https://review.opendev.org/702132 | |
| 16:44:14 | openstackgerrit | Radosław Piliszek proposed openstack/js-openstack-lib master: CI: New jobs https://review.opendev.org/702132 | |
| 20:03:01 | openstackgerrit | Radosław Piliszek proposed openstack/js-openstack-lib master: CI: New jobs https://review.opendev.org/702132 | |
| #openstack-sdks - 2020-02-09 | |||
| 07:33:35 | openstackgerrit | Vladimir Ermakov proposed openstack/ansible-collections-openstack master: os_image: Add tags parameter https://review.opendev.org/705343 | |
| 09:16:15 | openstackgerrit | Vladimir Ermakov proposed openstack/ansible-collections-openstack master: os_image: Add tags parameter https://review.opendev.org/705343 | |
| 23:29:36 | sshnaidm | somebody that know sdk better, can you please comment on the patch: https://review.opendev.org/#/c/705343/1/plugins/modules/os_image.py it's about bugs with SDK and tags | |
| #openstack-sdks - 2020-02-10 | |||
| 01:41:03 | adriant | Now... this is a SILLY edge case... the the CLI keystone commands don't handle well the case where someone has a project called "default" in a domain that isn't "Default". Because it seems the project name default is clashing with the ID for the 'Default' domain, so things like role assignment list and such don't work when specifying | |
| 01:41:03 | adriant | "--project-domain" because I guess the ID lookup happens first. | |
| 10:00:17 | openstackgerrit | Lajos Katona proposed openstack/keystoneauth stable/stein: add handling for multiple error returns https://review.opendev.org/706774 | |
| 13:53:03 | sshnaidm | I wonder if anyone from SDK can comment on an issue with tags and update_image_properties - https://review.opendev.org/#/c/705343/1..3/plugins/modules/os_image.py | |
| 14:08:52 | mordred | adriant: wow, that's an amazing edge case | |
| 15:13:17 | KeithMnemonic | anyone feel like getting this through finally https://review.opendev.org/701299 | |
| 15:23:51 | KeithMnemonic | also this one please if someone can pull the trigger https://review.opendev.org/703683 | |
| 15:36:48 | dtantsur | elmiko: hey! at some point (maybe on Thu) we should talk about PTG plans. | |
| 15:37:11 | dtantsur | sshnaidm: btw, have we got a new meeting time for the ansible meeting? | |
| 15:39:34 | sshnaidm | dtantsur, I think yes, we'll have it in Tue 3pm UTC as winning from https://xoyondo.com/dp/ITMGRZSvaZaONcz | |
| 15:39:52 | dtantsur | sshnaidm: have you announced it already? especially if you expect it to happen tomorrow :) | |
| 15:40:13 | sshnaidm | dtantsur, right, I'll announce it today | |
| 15:40:23 | dtantsur | ++ thx | |
| 15:40:27 | sshnaidm | I think all that wanted to vote have already voted | |
| 15:41:02 | dtantsur | I'll be able to attend again \o/ | |
| 15:41:28 | sshnaidm | dtantsur, welcome back! :) | |
| 15:43:15 | sshnaidm | dtantsur, do you know who can help with images work in SDK wrt https://review.opendev.org/#/c/705343/1..3/plugins/modules/os_image.py ? | |
| 15:43:39 | gtema | sshnaidm, what is exactly the question with that? | |
| 15:43:55 | dtantsur | I was about to mention gtema :) | |
| 15:44:02 | sshnaidm | gtema, seems like there is a bug with tagz and update_image_properties call | |
| 15:44:15 | gtema | I was taking care of rework of image and on the other side not happy with overall tag approach of modules | |
| 15:44:47 | gtema | ah, ok | |
| 15:44:52 | sshnaidm | gtema, I wouldn't like to pass workarounds to modules because of SDK bugs, but if no choice.. | |
| 15:44:56 | gtema | will have a look on that tomorrow | |
| 15:45:05 | sshnaidm | gtema, great, appreciate that | |
| 15:45:32 | gtema | btw, just a question in the round: I want to have a separate module for doing tags operations, instead of implementing the whole madness in each module | |
| 15:45:50 | gtema | I was even proposing change in ansible, but it never got merged | |
| 15:45:58 | gtema | what do you think of such approach? | |
| 15:58:52 | sshnaidm | gtema_, I'd say we can try, can you submit an example patch? | |
| 15:59:57 | gtema_ | yes, when will find a time. But sure. The benefit is that in each module we need to repeat some logic otherwise | |
| 16:00:17 | sshnaidm | gtema_, I suppose it should go to modules_utils/openstack.py somewhere? | |
| 16:00:48 | gtema_ | well, this is another option, which is also on my desk | |