| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-01-17 | |||
| 08:48:03 | thomasb06 | https://review.opendev.org/c/openstack/openstacksdk/+/824898 Cloud / Security group service | |
| 08:48:51 | thomasb06 | https://review.opendev.org/c/openstack/openstacksdk/+/824900 Cloud / Utils service | |
| 08:49:02 | thomasb06 | Except for errors, this should be it | |
| 09:28:38 | opendevreview | Merged openstack/python-openstackclient master: Skip original_name from flavor info https://review.opendev.org/c/openstack/python-openstackclient/+/824733 | |
| 09:40:10 | isabek | gtema: Thanks got it. I will update patches with tenant_id deprecation and will wait for r1 branch merge to master | |
| 10:02:44 | opendevreview | Artem Goncharov proposed openstack/osc-lib master: Replace allow_get check with allow_fetch https://review.opendev.org/c/openstack/osc-lib/+/824909 | |
| 10:03:20 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Merge feature/r1 branch into master https://review.opendev.org/c/openstack/openstacksdk/+/823961 | |
| 10:33:41 | opendevreview | Artem Goncharov proposed openstack/osc-lib master: Replace allow_get check with allow_fetch https://review.opendev.org/c/openstack/osc-lib/+/824909 | |
| 10:34:40 | opendevreview | Artem Goncharov proposed openstack/osc-lib master: Replace allow_get check with try block https://review.opendev.org/c/openstack/osc-lib/+/824909 | |
| 10:39:51 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: Support Neutron Local IP CRUD https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | |
| 13:58:12 | opendevreview | Dr. Jens Harbott proposed openstack/osc-lib master: Replace allow_get check with try block https://review.opendev.org/c/openstack/osc-lib/+/824909 | |
| 14:19:45 | opendevreview | Artem Goncharov proposed openstack/osc-lib master: Replace allow_get check with allow_fetch https://review.opendev.org/c/openstack/osc-lib/+/824909 | |
| 15:34:57 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Merge feature/r1 branch into master https://review.opendev.org/c/openstack/openstacksdk/+/823961 | |
| 16:29:05 | opendevreview | Ivan ROGER proposed openstack/ansible-collections-openstack master: Fix identity user lookup with a domain https://review.opendev.org/c/openstack/ansible-collections-openstack/+/824993 | |
| 17:39:21 | opendevreview | Merged openstack/python-openstackclient master: Get rid of tenant_id in the network commands https://review.opendev.org/c/openstack/python-openstackclient/+/824746 | |
| #openstack-sdks - 2022-01-18 | |||
| 10:02:43 | opendevreview | Jan Hartkopf proposed openstack/openstacksdk master: fix creation of protected image https://review.opendev.org/c/openstack/openstacksdk/+/820926 | |
| 12:38:48 | thomasb06 | gtema: do you see anything wrong in the `register_machine` function? There seem to be an error of block quoting: https://zuul.opendev.org/t/openstack/build/d412329810e941cabf4b7c4794cb5781 | |
| 12:40:28 | gtema | looking | |
| 12:43:43 | thomasb06 | The error is triggered during the Coe compilation while the Baremetal one went well | |
| 12:50:43 | gtema | mostly you will need to "rebase" on top of parent change every time you update previous commit | |
| 12:51:06 | gtema | alternatively (preferred) you break chain by rebasing each change on top of base branch | |
| 12:53:16 | thomasb06 | Wow... How can I do this now? Move head backward to the Coe commit and pull? | |
| 12:53:45 | gtema | in the UI | |
| 12:54:11 | gtema | go to rebase and select " Rebase on top of the feature/r1 branch (breaks relation chain) " | |
| 12:54:14 | gtema | and this for every change you created | |
| 12:54:28 | thomasb06 | In the Coe commit? Let me check | |
| 12:55:16 | thomasb06 | Found. This is it? | |
| 12:56:04 | thomasb06 | Before, I need to 'recheck' maybe? | |
| 12:56:12 | gtema | no | |
| 12:56:25 | gtema | this will trigger it automatically | |
| 12:56:46 | thomasb06 | Ah, you did it. This is why it didn't work, there was an error message | |
| 12:57:00 | thomasb06 | For the next ones, I'll know. Thanks | |
| 12:57:28 | gtema | I rebased still on the latest version of the previous change (updated underlaying ver) | |
| 12:57:39 | gtema | but as I meant - best is to break dependency | |
| 13:01:08 | thomasb06 | Understood | |
| 13:21:21 | opendevreview | Pierre Riteau proposed openstack/ansible-collections-openstack master: Add support for project parent https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825108 | |
| 14:17:21 | thomasb06 | gtema: same for the Compute service. So I click on "Rebase on top of the feature/r1 branch (breaks relation chain)"? | |
| 14:17:55 | gtema | on every chain, otherwise you will always need to rebase them once you update bottom change | |
| 14:19:24 | thomasb06 | alright | |
| 14:58:13 | opendevreview | Ivan ROGER proposed openstack/ansible-collections-openstack master: Fix identity user lookup with a domain https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825117 | |
| 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 | |