| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2021-10-26 | |||
| 13:22:22 | gtema | once you pushed one change `git reset --hard feature/r1 && git pull` | |
| 13:23:34 | thomasb06 | git pull asks a distant and branch so `git pull origin feature/r1` ? | |
| 13:25:53 | gtema | yes, maybe. I guess I set this in some settings for myself therefore it doesn't ask anymore | |
| 13:34:29 | thomasb06 | one to try: https://review.opendev.org/c/openstack/openstacksdk/+/815503 | |
| 13:51:00 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: Add --security-group to port list https://review.opendev.org/c/openstack/python-openstackclient/+/815506 | |
| 13:59:49 | gtema | thomasb06, would you please also add service title in the commit message title | |
| 14:00:11 | thomasb06 | sure | |
| 14:00:36 | thomasb06 | so "Service title - Reindentation of the docstrings"? | |
| 14:04:35 | thomasb06 | done | |
| 14:07:28 | thomasb06 | in `def unmanage_volume`, there is an embedded docstring | |
| 14:07:44 | thomasb06 | my bad, no worries | |
| 14:48:23 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch server backup to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/814332 | |
| 15:00:46 | opendevreview | Cyril Roelandt proposed openstack/python-openstackclient master: Fix typos https://review.opendev.org/c/openstack/python-openstackclient/+/815510 | |
| #openstack-sdks - 2021-10-27 | |||
| 06:49:25 | thomasb06 | gtema: Hello. Is the commit message what you were expecting: https://review.opendev.org/c/openstack/openstacksdk/+/815503/1 ? | |
| 06:56:50 | gtema | In principle yes, but instead of "service title" there should be "block storage" | |
| 06:57:12 | gtema | And can you please do all versions of same service in single change | |
| 07:13:48 | thomasb06 | Sure, I correct this one with a commit --append | |
| 07:15:01 | thomasb06 | done | |
| 07:16:08 | thomasb06 | ah, there's a v3 for the block storage service... | |
| 07:16:56 | thomasb06 | I add the v3 to the commit now | |
| 07:48:54 | thomasb06 | gtema: at the moment, I'm trying to amend the file block_storage/v3/_proxy.py in the commit 815503, but it never shows in the UI. The procedure is to type `git commit --amend`, then `git add _proxy.py`, and finally `git review -y`? | |
| 07:55:22 | thomasb06 | when I type `git show 894b5450469de600c4a7c7c313c9766de6765fb6`, it looks ok... Let me know if the v3 is missing for you | |
| 07:55:50 | thomasb06 | (by the way, are my commit hash the same as for Gerrit?) | |
| 09:11:29 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Switch server backup to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/814332 | |
| 16:52:24 | opendevreview | Merged openstack/python-openstackclient master: Switch server backup to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/814332 | |
| 18:21:10 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: volume: Add 'block storage cluster *' commands https://review.opendev.org/c/openstack/python-openstackclient/+/795829 | |
| 18:21:10 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: volume: Add 'block storage resource filter list' command https://review.opendev.org/c/openstack/python-openstackclient/+/794645 | |
| 18:26:08 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: volume: Add 'block storage resource filter list' command https://review.opendev.org/c/openstack/python-openstackclient/+/794645 | |
| #openstack-sdks - 2021-10-28 | |||
| 02:22:58 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch openstack server add port to using sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/815769 | |
| 02:42:10 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch openstack server add port/network to using sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/815769 | |
| 06:06:11 | opendevreview | SEUNG TAE KIM proposed openstack/openstacksdk master: Add "limit" string in _query_mapping variable in task.py https://review.opendev.org/c/openstack/openstacksdk/+/814270 | |
| 14:42:05 | thomasb06 | gtema: Hello. The Network service: https://review.opendev.org/c/openstack/openstacksdk/+/815852 | |
| 17:21:23 | gtema | Thomasb, I see those as emails in my inbox, you do not need to mention there except of pointing special attention or not getting review for some time | |
| 22:13:16 | opendevreview | Thrivikram Mudunuri proposed openstack/python-openstackclient master: Switch server image create to SDK. https://review.opendev.org/c/openstack/python-openstackclient/+/815913 | |
| 22:16:45 | opendevreview | Thrivikram Mudunuri proposed openstack/python-openstackclient master: Switch server image create to SDK. https://review.opendev.org/c/openstack/python-openstackclient/+/815913 | |
| 23:36:30 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch command server add volume to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/815919 | |
| 23:57:13 | opendevreview | Thrivikram Mudunuri proposed openstack/python-openstackclient master: Switch server image create to SDK. https://review.opendev.org/c/openstack/python-openstackclient/+/815913 | |
| #openstack-sdks - 2021-10-29 | |||
| 12:31:55 | thomasb06 | gtema: Hello. Clustering: https://review.opendev.org/c/openstack/openstacksdk/+/815957 | |
| 12:32:13 | thomasb06 | (in it's own branch) | |
| #openstack-sdks - 2021-11-01 | |||
| 09:57:10 | opendevreview | Merged openstack/ansible-collections-openstack master: Flavor properties are not deleted on changes and id will stay https://review.opendev.org/c/openstack/ansible-collections-openstack/+/811739 | |
| 11:28:56 | stephenfin | gtema: We're seeing a lot of failures with the nodepool-build-image-siblings job on openstacksdk changes | |
| 11:29:17 | stephenfin | due to an opensuse mirror issue, apparently | |
| 11:46:56 | stephenfin | gtema: Ah, it was fixed on Oct 28 via Ica62392ebf4a665a04cd65458dda9e0a7545ccc8 (nodepool) | |
| 12:16:31 | gtema | stephenfin, good. I'm off today (holiday here), thus will not participate in meeting later today | |
| #openstack-sdks - 2021-11-02 | |||
| 04:56:45 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch server dump create to using sdk https://review.opendev.org/c/openstack/python-openstackclient/+/816296 | |
| 08:47:55 | thomasb06 | gtema: Hello. Shared File System: https://review.opendev.org/c/openstack/openstacksdk/+/816307 | |
| 08:49:42 | thomasb06 | line 111, maybe it's `param` instead of `pram`? | |
| 08:54:53 | gtema | yes, right | |
| 09:01:42 | thomasb06 | ok, I correct it then | |
| 09:07:20 | thomasb06 | DNS: https://review.opendev.org/c/openstack/openstacksdk/+/816308 | |
| 09:43:13 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Fix issue with same host and group names https://review.opendev.org/c/openstack/ansible-collections-openstack/+/816311 | |
| #openstack-sdks - 2021-11-03 | |||
| 10:31:43 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Don't warn if disk overcommit params unset https://review.opendev.org/c/openstack/python-openstackclient/+/816448 | |
| 10:31:49 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Use correct command class for 'show migration' https://review.opendev.org/c/openstack/python-openstackclient/+/816449 | |
| 11:57:52 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Return details of attached volumes https://review.opendev.org/c/openstack/python-openstackclient/+/816491 | |
| 14:31:09 | opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: Keep creating keystone admin endpoint for heat https://review.opendev.org/c/openstack/openstacksdk/+/777343 | |
| 14:34:13 | frickler | gtema: stephenfin: ^^ dependencies finally merged and I added a comment as yoctozepto suggested, please have another look | |
| 14:34:41 | gtema | noted | |
| 22:10:23 | opendevreview | Merged openstack/openstacksdk master: Keep creating keystone admin endpoint for heat https://review.opendev.org/c/openstack/openstacksdk/+/777343 | |
| #openstack-sdks - 2021-11-04 | |||
| 06:59:35 | thomasb06 | gtema: Hello. Object Store: https://review.opendev.org/c/openstack/openstacksdk/+/816619 | |
| 07:02:25 | thomasb06 | line 183 has dashes, does it take stars? | |
| 07:08:13 | gtema | Nope, dashes are correct | |
| 07:21:48 | thomasb06 | ok then | |
| 07:25:22 | thomasb06 | Compute: https://review.opendev.org/c/openstack/openstacksdk/+/816620 | |
| 08:02:24 | thomasb06 | The Compute commit seems to have a serious merge conflict | |
| 09:03:38 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Switch hypervisor operations to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/763414 | |
| 09:55:02 | stephenfin | gtema: Care to send https://review.opendev.org/c/openstack/python-openstackclient/+/813803 and it's wallaby counterpart through | |
| 09:55:03 | stephenfin | ? | |
| 11:06:03 | opendevreview | Merged openstack/ansible-collections-openstack master: Fix issue with same host and group names https://review.opendev.org/c/openstack/ansible-collections-openstack/+/816311 | |
| 11:47:02 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Remove six https://review.opendev.org/c/openstack/keystoneauth/+/816671 | |
| 14:20:48 | opendevreview | Merged openstack/python-openstackclient stable/xena: compute: Fix filtering servers by tags https://review.opendev.org/c/openstack/python-openstackclient/+/813803 | |
| 15:43:14 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch list server volume to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/816716 | |
| 15:46:50 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch list server volume to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/816716 | |
| #openstack-sdks - 2021-11-05 | |||
| 07:15:36 | thomasb06 | gtema: Hello. The new Compute _proxy.py after merging Stephen Finucane changes these days: https://review.opendev.org/c/openstack/openstacksdk/+/816785 | |
| 07:15:52 | thomasb06 | The one from yesterday, I'll abandon it in the UI | |
| 07:19:38 | thomasb06 | The red file is the one on the MAIN branch? It's a mess today... | |
| 07:39:48 | gtema | It's often a mess | |
| 07:40:58 | gtema | Do not really get what you mean "red file" | |
| 07:47:11 | thomasb06 | In the UI, a file has red areas and the other green ones. The red one isn't on feature/r1, is it? | |
| 07:49:31 | gtema | Red is what is deleted, green is what is new | |
| 07:49:47 | gtema | This is a visualization of what has changed in the file | |
| 07:51:06 | thomasb06 | Well, I shouldn't have Stephen changes shown in what I added | |
| 07:55:48 | gtema | it all depends on what is the "base" and which patchsets you compare | |
| 08:08:17 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Temporarily disable deprecation warning https://review.opendev.org/c/openstack/openstacksdk/+/816790 | |
| 08:09:14 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Switch hypervisor operations to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/763414 | |
| 08:21:10 | frickler | gtema: we need the heat-ks-admin fix on the feature branch https://efd23cc4be40a92d4d24-90a9b723857c85e2536fe5d9751b99c8.ssl.cf2.rackcdn.com/797024/20/check/openstacksdk-functional-devstack/7e9e32b/controller/logs/screen-h-eng.txt | |
| 08:21:20 | frickler | cherry-pick: https://review.opendev.org/c/openstack/openstacksdk/+/816667 | |
| 08:21:28 | gtema | ok, thks | |
| 08:55:51 | thomasb06 | Zuul says: `pep8: F811 redefinition of unused 'get_server_console_output' from line 839`. Is it something I can fix? | |
| 09:12:52 | thomasb06 | gtema: it looks like the simpler is to wait for the modification ongoing on the Compute service are done before running my script over it? | |
| 09:39:45 | thomasb06 | Orchestration: https://review.opendev.org/c/openstack/openstacksdk/+/816806 | |
| 09:45:32 | thomasb06 | Image service: https://review.opendev.org/c/openstack/openstacksdk/+/816808 | |
| 09:49:35 | thomasb06 | Workflow service: https://review.opendev.org/c/openstack/openstacksdk/+/816809 | |
| 09:57:02 | thomasb06 | Load balancer: https://review.opendev.org/c/openstack/openstacksdk/+/816809 | |
| 09:58:37 | thomasb06 | Database service: https://review.opendev.org/c/openstack/openstacksdk/+/816812 | |
| 09:59:19 | thomasb06 | This is it for me. Except for the Compute service or a mistake, the _proxy.py files are done | |
| 10:04:34 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Switch hypervisor operations to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/763414 | |
| 13:51:51 | opendevreview | Arkady Shtempler proposed openstack/openstacksdk master: Adds "test_update_zone" test case https://review.opendev.org/c/openstack/openstacksdk/+/816842 | |
| 17:08:19 | opendevreview | SEUNG TAE KIM proposed openstack/python-openstackclient master: Add "hypervisor uptime" command https://review.opendev.org/c/openstack/python-openstackclient/+/816864 | |
| #openstack-sdks - 2021-11-07 | |||
| 13:14:03 | opendevreview | WooJongSeong proposed openstack/python-openstackclient master: impl OSC's cmd hypervisor servers set https://review.opendev.org/c/openstack/python-openstackclient/+/816922 | |
| 23:01:23 | opendevreview | Thrivikram Mudunuri proposed openstack/python-openstackclient master: Switch server image create to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/815913 | |