| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-01-14 | |||
| 17:14:38 | frickler | gtema: yeah, I'm fine with that, I was mainly wanting to see now whether my local testing would also be confirmed by the CI | |
| 17:14:47 | gtema | cool | |
| 17:53:42 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Get rid of tenant_id in the network commands https://review.opendev.org/c/openstack/python-openstackclient/+/824746 | |
| 17:56:05 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Merge feature/r1 branch into master https://review.opendev.org/c/openstack/openstacksdk/+/823961 | |
| 18:39:20 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Switch compute service list, delete and set to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/818434 | |
| 19:15:27 | frickler | gtema: the revert together with your osc patches passes osc unit+func https://review.opendev.org/c/openstack/openstacksdk/+/824741 , with that I'm out for now, feel free to ping if anything else comes up | |
| 21:28:39 | gtema | Cool, thks | |
| #openstack-sdks - 2022-01-15 | |||
| 14:23:11 | opendevreview | Merged openstack/openstacksdk master: Revert "Add "security_group_ids" to Port's query parameters" https://review.opendev.org/c/openstack/openstacksdk/+/824751 | |
| 15:00:14 | opendevreview | Merged openstack/python-openstackclient master: Switch compute service list, delete and set to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/818434 | |
| #openstack-sdks - 2022-01-17 | |||
| 08:16:03 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Ensure flavor attributes are unique https://review.opendev.org/c/openstack/python-openstackclient/+/824865 | |
| 08:17:15 | thomasb06 | gtema: Hello, my script is ready. You only need one commit per service? | |
| 08:17:26 | gtema | yes, please | |
| 08:17:35 | thomasb06 | let's g | |
| 08:17:38 | thomasb06 | *go | |
| 08:19:41 | thomasb06 | For the commit message, something like "Cloud / Baremetal service - reindentation of the docstrings" is ok? | |
| 08:19:50 | gtema | sure | |
| 08:23:23 | isabek | gtema: Hi! Can you please take a look this patches, when you have a time? Thanks in advance! 1) https://review.opendev.org/c/openstack/openstacksdk/+/822305 | |
| 08:23:40 | isabek | 2) https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | |
| 08:24:15 | gtema | unless sdk r1 lands I will not merge anything else - there are quite some merge issues | |
| 08:24:45 | isabek | sdk patch no merge conflict with r1 branch | |
| 08:25:09 | gtema | it's more that | |
| 08:25:34 | thomasb06 | Gerrit's angry again: https://x0.at/onOB.txt | |
| 08:31:25 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Get rid of tenant_id in the network commands https://review.opendev.org/c/openstack/python-openstackclient/+/824746 | |
| 08:33:12 | thomasb06 | Ah, maybe I have a hint | |
| 08:34:00 | gtema | na well - it states explicitly that you have unstaged files in the checkout | |
| 08:36:18 | thomasb06 | of course... Getting rusty | |
| 08:36:35 | thomasb06 | Cloud/Baremetal: https://review.opendev.org/c/openstack/openstacksdk/+/824869 | |
| 08:38:10 | thomasb06 | Cloud / Coe service : https://review.opendev.org/c/openstack/openstacksdk/+/824890 | |
| 08:39:37 | thomasb06 | Cloud / Compute service : https://review.opendev.org/c/openstack/openstacksdk/+/824891 | |
| 08:43:34 | thomasb06 | Cloud / Dns service : https://review.opendev.org/c/openstack/openstacksdk/+/824892 | |
| 08:44:27 | thomasb06 | https://review.opendev.org/c/openstack/openstacksdk/+/824893 Cloud / Floating service | |
| 08:45:34 | thomasb06 | https://review.opendev.org/c/openstack/openstacksdk/+/824894 Cloud / Identity service | |
| 08:46:26 | thomasb06 | https://review.opendev.org/c/openstack/openstacksdk/+/824896 Cloud / Network service | |
| 08:47:20 | thomasb06 | https://review.opendev.org/c/openstack/openstacksdk/+/824897 Cloud / Normalize | |
| 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 | thomasb06 | Ok, understood | |