| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-01-18 | |||
| 17:33:20 | stephenfin | diablo_rojo_phone: are you guys meeting? Is there anyone to meet? | |
| 17:34:40 | diablo_rojo_phone | Artem and I are here. | |
| 17:34:50 | diablo_rojo_phone | No students though | |
| 17:34:54 | diablo_rojo_phone | Did I miss something? | |
| 17:35:00 | stephenfin | yeah, they're both sick | |
| 17:38:01 | diablo_rojo_phone | Oh whoops. | |
| 17:38:12 | diablo_rojo_phone | Just Artem and I chatting then lol | |
| 17:38:17 | diablo_rojo_phone | How's your wrist? | |
| 17:42:31 | diablo_rojo_phone | Ah I see the email now. | |
| 18:34:03 | stephenfin | diablo_rojo_phone: All good. Things are staying where they are, so as long as that continues I'm all good :) | |
| 20:39:33 | opendevreview | Thiago Paiva Brito proposed openstack/openstacksdk master: Add update capabilities to Snapshots https://review.opendev.org/c/openstack/openstacksdk/+/778760 | |
| #openstack-sdks - 2022-01-19 | |||
| 10:25:55 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test with SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | |
| 12:11:06 | opendevreview | Merged openstack/osc-lib master: Replace allow_get check with allow_fetch https://review.opendev.org/c/openstack/osc-lib/+/824909 | |
| 13:13:28 | opendevreview | Merged openstack/ansible-collections-openstack master: Fix identity user lookup with a domain https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825117 | |
| 15:48:43 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test with SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | |
| #openstack-sdks - 2022-01-20 | |||
| 09:03:42 | thomasb06 | gtema: do see what the indentation problem is: https://review.opendev.org/c/openstack/openstacksdk/+/824896 ? | |
| 09:05:37 | gtema | the log tells you where it starts failing: create_port, create_router, etc. Similar issue as you had in the baremetal | |
| 09:06:12 | gtema | you reindent "example" inside of the docstrings and it screw the syntax | |
| 09:07:40 | thomasb06 | Zuul says: update_subnet:22: ERROR: Unexpected indentation. | |
| 09:08:05 | thomasb06 | but there's nothing wrong in function update_subnet... | |
| 09:08:09 | gtema | and this is line 22 of the docstring | |
| 09:08:36 | gtema | same issue: whatever is after "For example::" | |
| 09:08:37 | thomasb06 | But in line 22 of the docstring in update_subnet I see nothing wrong | |
| 09:09:28 | gtema | the whole example get's under-indented everywhere | |
| 09:09:36 | thomasb06 | the square bracket is too leftward? | |
| 09:09:42 | gtema | yes | |
| 09:09:42 | thomasb06 | Ok, understood | |
| 09:11:48 | thomasb06 | fixed, thank you | |
| 09:28:10 | isabek | gtema: Hi! When is it planned to land r1 branch to master? | |
| 09:28:36 | gtema | the change is open, will be merged in next days if nobody really objects | |
| 15:58:29 | opendevreview | Dmitry Tantsur proposed openstack/keystoneauth master: User-friendly error message for DiscoveryFailure https://review.opendev.org/c/openstack/keystoneauth/+/825540 | |
| 18:41:25 | opendevreview | Walt proposed openstack/openstacksdk master: Adds volume migrate capability to v3 cinder proxy https://review.opendev.org/c/openstack/openstacksdk/+/825664 | |
| 19:52:12 | opendevreview | Will Szumski proposed openstack/ansible-collections-openstack master: Handle aggregate host list set to None https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825667 | |
| 19:53:03 | opendevreview | Will Szumski proposed openstack/ansible-collections-openstack master: Handle aggregate host list set to None https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825667 | |
| 20:00:23 | opendevreview | Will Szumski proposed openstack/ansible-collections-openstack master: Handle aggregate host list set to None https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825667 | |
| 21:01:22 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| 21:36:07 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| 21:43:41 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| 22:00:15 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| 22:58:54 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| #openstack-sdks - 2022-01-21 | |||
| 10:34:38 | dtantsur | yo, has anyone looked into why the docs job fails on keystoneauth? seems legit to me | |
| 10:34:48 | dtantsur | example: https://zuul.opendev.org/t/openstack/build/ee377460de4f426eba8817dda41d106b | |
| 11:07:13 | gtema | dtantsur: this looks very similar to why I was updating sdk docs with :noindex: (i.e. https://opendev.org/openstack/openstacksdk/raw/branch/master/doc/source/user/proxies/baremetal.rst) | |
| 11:07:31 | gtema | that was some year ago and caused by new version of sphinx or whatever | |
| 11:08:02 | gtema | but here I can't really believe it is also caused by new version of whatever | |
| 11:12:23 | gtema | it looks more that there was no update in a while and thus the issue is only visible now | |
| 11:12:56 | gtema | cause I also see same problem trying to build docs locally from master | |
| 11:14:16 | gtema | especially tricky is that whole api doc is being generated automatically, so somebody need to spend a while on that issue | |
| 11:15:47 | frickler | might be something that thomas could script? | |
| 11:15:58 | gtema | :) | |
| 11:28:12 | frickler | oh wait, that is kind of scripted already. "SPHINX_APIDOC_OPTIONS=noindex,members,undoc-members,show-inheritance" helps https://www.sphinx-doc.org/en/master/man/sphinx-apidoc.html | |
| 12:15:20 | opendevreview | Dr. Jens Harbott proposed openstack/keystoneauth master: Fix docs build https://review.opendev.org/c/openstack/keystoneauth/+/825777 | |
| 12:15:43 | frickler | gtema: dtantsur: ^^ that does it for me locally | |
| 12:16:22 | gtema | awsome, thks | |
| 12:28:59 | opendevreview | Dr. Jens Harbott proposed openstack/keystoneauth master: Fix docs build https://review.opendev.org/c/openstack/keystoneauth/+/825777 | |
| 12:29:53 | frickler | meh, of course it failed for pdfdocs | |
| 12:30:05 | gtema | tja | |
| 13:48:14 | frickler | gtema: now it passed, do you want to review? otherwise I'd self-approve as gate-unblocker | |
| 13:53:03 | gtema | I have no core rights in keystone, left +1 | |
| 13:55:48 | frickler | oh, right, I was somehow thinking that this belonged to sdk with patches being reported here, can't approve myself, either. so let's wait for some keystone ppl to pick it up | |
| 13:56:02 | gtema | thks | |
| 13:56:34 | gtema | I do not actually even know why it is reported here, but it doesn't harm, cause we depend on that heavily | |
| 18:16:01 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| 21:49:33 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| #openstack-sdks - 2022-01-23 | |||
| 18:38:58 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test with SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | |
| #openstack-sdks - 2022-01-24 | |||
| 13:26:41 | isabek | gtema: Hi! On this patch set you said that no need 'project_id' [1]. But according API def we need list by project_id. When I'm removing project_id from query_paramateres, there is key error | |
| 13:26:46 | isabek | 1) https://review.opendev.org/c/openstack/openstacksdk/+/814765/2/openstack/network/v2/local_ip.py#38 | |
| 13:27:47 | gtema | isabek: you should leave simply `, project_id, ...`, but not `project_id='project_id'` | |
| 13:30:00 | gtema | however I was not able to find in the neutron whether this extension will at all support querying by project_id | |
| 13:31:06 | isabek | ohh, ok got it! Thank you! | |
| 13:31:15 | isabek | I will update | |
| 13:36:08 | opendevreview | Mamatisa Nurmatov proposed openstack/openstacksdk master: Add query parameters to local ip https://review.opendev.org/c/openstack/openstacksdk/+/822305 | |
| 13:40:56 | dtantsur | frickler, gtema, I wonder if keystoneauth should move under the SDK or maybe have a shared ownership | |
| 13:41:09 | dtantsur | it seems to be that its main usage is in various clients | |
| 14:09:12 | opendevreview | Merged openstack/openstacksdk master: Merge feature/r1 branch into master https://review.opendev.org/c/openstack/openstacksdk/+/823961 | |
| 14:10:07 | gtema | dtantsur: we may open the discussion on the mailinglist about that | |
| 14:47:08 | thomasb06 | gtema: Hello. This should be it for the Cloud services. All the docstrings seem to pass the tests | |
| 15:01:15 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test CI https://review.opendev.org/c/openstack/ansible-collections-openstack/+/820480 | |
| 17:36:17 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| 20:22:11 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| 20:29:53 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| 23:14:21 | opendevreview | James Kirsch proposed openstack/openstacksdk master: Identity: Add support for system role assignment https://review.opendev.org/c/openstack/openstacksdk/+/826193 | |
| #openstack-sdks - 2022-01-25 | |||
| 02:35:15 | opendevreview | James Kirsch proposed openstack/openstacksdk master: Identity: Add support for system role assignment https://review.opendev.org/c/openstack/openstacksdk/+/826193 | |
| 06:13:07 | opendevreview | Ales Musil proposed openstack/openstacksdk master: network: Fix update of network provider https://review.opendev.org/c/openstack/openstacksdk/+/761933 | |
| 10:39:38 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Fix CI for new openstack SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/826264 | |
| 12:02:06 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test with SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | |
| 12:55:27 | ralonsoh | artom, hi, can you please check https://review.opendev.org/c/openstack/python-openstackclient/+/806016? | |
| 12:55:36 | ralonsoh | thank you in advance | |
| 12:57:04 | ralonsoh | stephenfin, ^^ | |
| 13:08:17 | stephenfin | ralonsoh: I was waiting on a new revision https://review.opendev.org/c/openstack/python-openstackclient/+/806016/7/openstackclient/tests/functional/common/test_quota.py#174 | |
| 13:08:32 | ralonsoh | ok | |
| 13:16:29 | artom | ralonsoh, I'm assuming you mean gtema? :) | |
| 13:16:40 | artom | I'm the *other* Art(e|o)m | |
| 13:19:20 | ralonsoh | artom, sorry!! | |
| 13:52:44 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test with SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | |
| 14:06:32 | artom | ralonsoh, no worries! It's rare that there are multiples of my first name in anglophone spheres, for once I get to enjoy the chaos that it causes ;) | |
| 14:06:48 | ralonsoh | hahaha | |
| 14:24:04 | isabek | gtema, hi! Can you please take a look to this patches [1], [2]? Thanks in advance! | |
| 14:24:05 | isabek | 1) https://review.opendev.org/c/openstack/python-openstackclient/+/805618 2) https://review.opendev.org/c/openstack/openstacksdk/+/822305 | |
| 14:43:05 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test ooo with SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/826296 | |