| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2021-12-14 | |||
| 08:18:40 | thomasb06 | By the way, in identity/v3/_proxy.py line 1591, it seems a piece is missing proper: `:class:`~openstack.identity.v3.federation_protocol.` should probably be `:class:`~openstack.identity.v3.federation_protocol.FederationProtocol`? | |
| 08:20:55 | thomasb06 | same line 1545 | |
| 08:24:08 | frickler | thomasb06: yes, both are missing the "FederationProtocol` instance." part | |
| 08:27:10 | thomasb06 | fixed, thanks | |
| 08:27:41 | thomasb06 | gtema: (or maybe it was RussianFederationProtocol, hehe...) | |
| 08:38:41 | thomasb06 | The Identity service: https://review.opendev.org/c/openstack/openstacksdk/+/821668 | |
| 08:49:13 | thomasb06 | The Instance HA service: https://review.opendev.org/c/openstack/openstacksdk/+/821674 | |
| 08:50:15 | thomasb06 | According to my script, this is it for the :class: links | |
| 15:55:52 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Switch add fixed IP to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820050 | |
| 15:55:53 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Return information about fixed IP https://review.opendev.org/c/openstack/python-openstackclient/+/821728 | |
| 16:04:53 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Switch remove fixed IP to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820420 | |
| 16:44:01 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: compute: Fix 'update_volume_attachment' https://review.opendev.org/c/openstack/openstacksdk/+/821733 | |
| 16:44:26 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Switch server volume update to sdk https://review.opendev.org/c/openstack/python-openstackclient/+/821734 | |
| 16:44:26 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Switch list server volume to sdk https://review.opendev.org/c/openstack/python-openstackclient/+/816716 | |
| 21:22:42 | dking | Does the python-openstackclient happen to have any option to set a timeout or a maximum amount of time before it stops trying to connect? | |
| #openstack-sdks - 2021-12-15 | |||
| 05:02:04 | frickler | dking: I have been wondering about that, too, seems at least by default it doesn't | |
| 05:28:48 | gtema | dking, frickler: In the clouds.yaml you can set api_timeout per each connection | |
| 05:34:20 | dking | gtema: I thought that clouds.yaml usually just matched existing command line options. Is there a related command line option for that? Also, is that documented anywhere? I'm not seeing api_timeout listed. | |
| 05:34:45 | dking | I'm checking in the online man page, but also https://docs.openstack.org/python-openstackclient/latest/cli/man/openstack.html and https://docs.openstack.org/python-openstackclient/latest/configuration/index.html and I don't see that option. | |
| 05:35:08 | gtema | No, that one is not listed and coming from SDK reading the config | |
| 05:37:17 | gtema | Moreover you'll not find it in any SDK docs sadly, but here it is in the code: https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/config/cloud_region.py#L636 | |
| 05:44:44 | dking | gtema: Thanks for pointing that out. However, from looking at the code, that seems like maybe it's a timeout for the keystone session itself. I'm referring to the actual request to the API. Specifically, I'm connecting to Ironic with "openstack baremetal node ..." commands, but when Ironic-API gets bogged down, it causes some of my scripts using it to get stuck. | |
| 05:46:46 | dking | I'm probably going to have to just specifically wrap the openstack command in the timeout command, but I was hoping that there was something built in for that. If it's not there, perhaps that would be a good option to add? | |
| 05:46:52 | gtema | That one is used as a timeout parameter on that python requests | |
| 06:01:41 | frickler | gtema: oh, wow, I just found _fix_backwards_api_timeout() and that is really ... surprising. thx mordred ;) | |
| 08:47:48 | isabek | Hi openstackclient cores! If you have some time, can you please take a look https://review.opendev.org/c/openstack/python-openstackclient/+/805618 ? | |
| 12:36:37 | opendevreview | Merged openstack/osc-lib master: Update master for stable/xena https://review.opendev.org/c/openstack/osc-lib/+/808366 | |
| 13:27:43 | opendevreview | Merged openstack/osc-lib master: Add Python3 yoga unit tests https://review.opendev.org/c/openstack/osc-lib/+/808368 | |
| 16:54:36 | dking | gtema: So, I decided to actually try the timeout, and unless I'm doing something wrong, it doesn't seem to work. Here's the clouds.yaml file: https://paste.opendev.org/show/811703/ | |
| 16:55:54 | dking | I added an endpoint override so that I could be sure that the file is being read and also it provided a handy way to break the requests which is the failure scenario to need the timeout. Then, I added api_timeout, and it still seems to take just as long to fail. | |
| 17:00:08 | gtema | It works for me definitely, so you might be doing something wrong | |
| 17:44:21 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Fix weird option definition for 'server ssh' https://review.opendev.org/c/openstack/python-openstackclient/+/821867 | |
| 17:44:21 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Pass through args to ssh https://review.opendev.org/c/openstack/python-openstackclient/+/402530 | |
| #openstack-sdks - 2021-12-16 | |||
| 12:38:46 | frickler | gtema: stephenfin: https://review.opendev.org/c/openstack/python-openstackclient/+/815032 causes a regression, 5.7.0 shows the location Munch noise again for neutron objects | |
| 12:45:45 | frickler | ah, because it simply dropped my beautiful special case https://review.opendev.org/c/openstack/python-openstackclient/+/815032/1/openstackclient/network/sdk_utils.py#b43 | |
| 12:46:01 | frickler | :-( | |
| 12:47:04 | frickler | cf. https://review.opendev.org/c/openstack/python-openstackclient/+/679445 | |
| 12:49:18 | gtema | weird, but this beautiful special case is now in osc_lib as well | |
| 12:49:54 | gtema | ah, now I got it, except this `if isinstnace` | |
| 12:51:08 | gtema | location should be filtered as in https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/image/v2/image.py#L104 | |
| 12:59:42 | frickler | gtema: hmm, o.k., that would be a larger patch. I was thinking to simply just repropose my original patch against osc-lib | |
| 13:00:36 | gtema | Can be, but that will take longer | |
| 13:02:03 | gtema | I am not 100% confident on any of both options. Feel that handling things in osc would be a more "proper" way | |
| 13:05:09 | gtema | And we will get rid of munch as such also - SDK will return pure SDK resources | |
| 13:06:15 | frickler | yeah, you got me convinced, will do a proper osc patch | |
| 13:08:46 | gtema | Thanks a lot, will process it fast, promise | |
| 14:21:19 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: Support Neutron Local IP CRUD https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | |
| 15:51:06 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Add location to fake network objects https://review.opendev.org/c/openstack/python-openstackclient/+/821996 | |
| 15:51:07 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Hide location columns for network objects https://review.opendev.org/c/openstack/python-openstackclient/+/821997 | |
| 15:51:53 | frickler | gtema: ^^ it will fail in this order, but I wanted to show that the modified tests work and make sure that this doesn't regress yet again | |
| 15:52:24 | frickler | we can either merge or revert the order on the test results are in, up to you | |
| 15:52:35 | frickler | s/on/once/ | |
| 15:57:53 | ricolin | Hey guys in the previous openstack pain point discussion, we have targeted OSC and OpenstackSDK as one of general pain points | |
| 15:58:08 | ricolin | * https://etherpad.opendev.org/p/pain-point-elimination#L176 | |
| 15:58:08 | ricolin | see: * https://etherpad.opendev.org/p/pain-point-elimination#L25 | |
| 15:59:03 | ricolin | As one of TC's action items, I'm looking for more feedback for them and hopefully also someone to volunteer to champion the community-goal to help investigate whether it's ready to be a goal | |
| 15:59:21 | ricolin | gtema, thoughts?:) | |
| 16:11:31 | gtema | ricolin, I'm surprised it is now all of a sudden pain point after years of my and other's attempts to increase participation. There's even TC resolution landed "recently" | |
| 16:13:02 | gtema | Me and stephenfin are doing our best on trying to improve things (including involving students together with diablo_rojo), but unless thing change and more people will really participate nothing will change | |
| 16:13:44 | gtema | And no communuty-goal will help. Few years ago I tried this already but faced quite a resistance from community on that | |
| 16:51:45 | ricolin | gtema, agree with you on the part that if nothing ever changed since Stein, them to make a goal will be horrifying idea. | |
| 16:53:14 | ricolin | Do think to investigate and see if that's still the case is important | |
| 16:55:05 | ricolin | not plan to circle back to the long discussion years ago, but do wondering if some resistance reason like glance support still remains | |
| 17:00:51 | gtema | Yeah, some sort of pre heck might be helpful | |
| 17:07:16 | ricolin | I guess it can be tracking with the previous discussion in gerrit and ML and recheck on the blockers: https://review.opendev.org/c/openstack/governance/+/639376 http://lists.openstack.org/pipermail/openstack-discuss/2019-August/008601.html | |
| 18:59:51 | opendevreview | Will Szumski proposed openstack/ansible-collections-openstack master: quota: Adds metadata_items parameter https://review.opendev.org/c/openstack/ansible-collections-openstack/+/822037 | |
| 19:33:05 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Hide location columns for network objects https://review.opendev.org/c/openstack/python-openstackclient/+/821997 | |
| 19:33:30 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Add location to fake network objects https://review.opendev.org/c/openstack/python-openstackclient/+/821996 | |
| 19:56:54 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Move queens job to experimental https://review.opendev.org/c/openstack/ansible-collections-openstack/+/822042 | |
| 20:09:34 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Replace victoria job by xena one https://review.opendev.org/c/openstack/ansible-collections-openstack/+/822043 | |
| #openstack-sdks - 2021-12-17 | |||
| 03:02:55 | opendevreview | Merged openstack/ansible-collections-openstack master: Move queens job to experimental https://review.opendev.org/c/openstack/ansible-collections-openstack/+/822042 | |
| 08:38:48 | thomasb06 | gtema: Hello. These 3 commits didn't build properly, but I'm not sure it was me: | |
| 08:38:49 | thomasb06 | https://review.opendev.org/c/openstack/openstacksdk/+/814395 | |
| 08:38:58 | thomasb06 | https://review.opendev.org/c/openstack/openstacksdk/+/821656 | |
| 08:39:08 | thomasb06 | https://review.opendev.org/c/openstack/openstacksdk/+/821674 | |
| 08:52:05 | frickler | thomasb06: the first one has still genuine docs build issues, two more missing blank lines. the other errors are unrelated, let's see how the recheck goes | |
| 09:25:00 | thomasb06 | frickler: ah, finally I found the extra __`__ that was causing the issue in the first one... Thank you | |
| 10:18:10 | frickler | thomasb06: the rechecks didn't help, there are still issues with pypi. I'll recheck once those are fixed | |
| 11:33:34 | thomasb06 | frickler: no worries... As long as I didn't break anything, it's the usual workflow | |
| 11:34:40 | thomasb06 | By the way, would you have something else to give to me? The pace is ok for me so maybe I can do something else to give a hand | |
| 22:40:08 | opendevreview | Pierre Riteau proposed openstack/ansible-collections-openstack master: Remove failing job from gate https://review.opendev.org/c/openstack/ansible-collections-openstack/+/822211 | |
| 22:53:55 | opendevreview | Merged openstack/ansible-collections-openstack master: Remove failing job from gate https://review.opendev.org/c/openstack/ansible-collections-openstack/+/822211 | |
| #openstack-sdks - 2021-12-18 | |||
| 20:21:43 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test CI https://review.opendev.org/c/openstack/ansible-collections-openstack/+/820480 | |
| 20:23:42 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Leave queens job only in experimental https://review.opendev.org/c/openstack/ansible-collections-openstack/+/822228 | |
| 23:18:45 | opendevreview | Merged openstack/ansible-collections-openstack master: quota: Adds metadata_items parameter https://review.opendev.org/c/openstack/ansible-collections-openstack/+/822037 | |
| 23:25:14 | opendevreview | Merged openstack/ansible-collections-openstack master: Leave queens job only in experimental https://review.opendev.org/c/openstack/ansible-collections-openstack/+/822228 | |
| #openstack-sdks - 2021-12-19 | |||
| 06:01:16 | opendevreview | Takashi Natsume proposed openstack/openstacksdk master: Fix misuse of assertTrue https://review.opendev.org/c/openstack/openstacksdk/+/822254 | |
| #openstack-sdks - 2021-12-20 | |||
| 05:11:35 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test CI https://review.opendev.org/c/openstack/ansible-collections-openstack/+/820480 | |
| 07:45:06 | opendevreview | Slawek Kaplonski proposed openstack/python-openstackclient master: Add new job osc-functional-devstack-neutron-ovs https://review.opendev.org/c/openstack/python-openstackclient/+/793143 | |
| 10:25:15 | opendevreview | Mamatisa Nurmatov proposed openstack/openstacksdk master: Add query parameters to local ip https://review.opendev.org/c/openstack/openstacksdk/+/822305 | |
| 11:33:59 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: compute: Fix '*volume_attachment' proxy methods https://review.opendev.org/c/openstack/openstacksdk/+/821733 | |
| 12:18:29 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Handle overridden 'verify_delete' (kw)args https://review.opendev.org/c/openstack/openstacksdk/+/822311 | |
| 12:18:29 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: compute: Fix '*volume_attachment' proxy methods https://review.opendev.org/c/openstack/openstacksdk/+/821733 | |
| 12:54:39 | opendevreview | Mamatisa Nurmatov proposed openstack/openstacksdk master: Add query parameters to local ip https://review.opendev.org/c/openstack/openstacksdk/+/822305 | |
| 13:00:06 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: compute: Add functional tests for volume attachments https://review.opendev.org/c/openstack/openstacksdk/+/822314 | |
| 13:00:07 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: volume: Trivial docstring fixes to 'wait_for_status' https://review.opendev.org/c/openstack/openstacksdk/+/822315 | |
| 14:41:14 | opendevreview | Merged openstack/python-openstackclient master: Hide location columns for network objects https://review.opendev.org/c/openstack/python-openstackclient/+/821997 | |
| 14:41:18 | opendevreview | Merged openstack/python-openstackclient master: Add location to fake network objects https://review.opendev.org/c/openstack/python-openstackclient/+/821996 | |
| 14:41:26 | opendevreview | Merged openstack/openstacksdk master: Fix misuse of assertTrue https://review.opendev.org/c/openstack/openstacksdk/+/822254 | |
| 15:21:20 | opendevreview | Jan Hartkopf proposed openstack/openstacksdk master: fix creation of protected image https://review.opendev.org/c/openstack/openstacksdk/+/820926 | |