| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2021-11-15 | |||
| 19:17:17 | stephenfin | gtema: Okay, almost all things volume attachment-related in the compute proxy layer are broken /o\ I'll try find some time to fix them later in the week | |
| 20:37:58 | gtema | Okay, pretty sad to hear that | |
| #openstack-sdks - 2021-11-16 | |||
| 08:51:11 | ralonsoh | gtema, hello, I would like to ask you about https://review.opendev.org/c/openstack/openstacksdk/+/814813 | |
| 08:51:28 | ralonsoh | maybe I'm not understanding your comment there | |
| 08:51:53 | ralonsoh | what I'm trying to do is to remove any "tenant_id" member from network resources | |
| 08:52:07 | ralonsoh | still present in https://opendev.org/openstack/openstacksdk/src/branch/feature/r1 | |
| 09:07:51 | gtema | yes, I know. What was done there is stop of using those and in case some older client still tries to set it it is will redirected to project_id | |
| 09:08:15 | gtema | the point is that SDK must be backward compatible as long as it is possible due to the fact of being used in older clouds | |
| 09:10:01 | gtema | https://dashboard.tsi-dev.otc-service.com/d/testingAPImonCompute/compute-service-statistics?orgId=1&from=now-7d&to=now | |
| 09:10:12 | gtema | can somebody explain what happened with metadata latencies? | |
| 09:26:36 | ralonsoh | gtema, agree and in those resources you are redirecting tenant_id to project_id | |
| 09:26:52 | ralonsoh | but we don't need the tenant_id member anymore | |
| 09:53:05 | gtema | there might be clients using old sdk which still try to set tenant_id. And we should not leave them | |
| 09:59:13 | ralonsoh | gtema, ok then, thanks for the answer | |
| 11:53:23 | isabek | gtema: Hi! Can you please check updated version: https://review.opendev.org/c/openstack/openstacksdk/+/814765 ? | |
| 11:54:27 | gtema | ok | |
| 11:56:06 | isabek | Thank you! | |
| 16:03:04 | opendevreview | Thrivikram Mudunuri proposed openstack/python-openstackclient master: Switch server image create to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/815913 | |
| 16:57:09 | opendevreview | Thrivikram Mudunuri proposed openstack/python-openstackclient master: Switch server image create to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/815913 | |
| 17:34:57 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch command server add volume to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/815919 | |
| 19:06:19 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch command server add volume to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/815919 | |
| 19:08:59 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch command server remove volume to sdk https://review.opendev.org/c/openstack/python-openstackclient/+/817989 | |
| 22:58:38 | opendevreview | Thrivikram Mudunuri proposed openstack/python-openstackclient master: Switch server pause and server unpause to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/817853 | |
| #openstack-sdks - 2021-11-17 | |||
| 00:23:02 | opendevreview | Thrivikram Mudunuri proposed openstack/python-openstackclient master: Switch server suspend and server resume to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/817860 | |
| 00:33:09 | opendevreview | Thrivikram Mudunuri proposed openstack/python-openstackclient master: Switch server suspend and server resume to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/817860 | |
| 00:35:02 | opendevreview | Thrivikram Mudunuri proposed openstack/python-openstackclient master: Switch server suspend and server resume to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/817860 | |
| 08:02:51 | thomasb06 | gtema: Hello. The Compute service is still messy: https://review.opendev.org/c/openstack/openstacksdk/+/816785 | |
| 08:03:10 | thomasb06 | Should I abandon it, pull the remote and reindent the docstring? | |
| 08:03:53 | thomasb06 | These two services are ready: https://review.opendev.org/c/openstack/openstacksdk/+/816619 and https://review.opendev.org/c/openstack/openstacksdk/+/816307 | |
| 08:05:18 | gtema | yes, this is perhaps the easiest. But when you do new commit simply reuse commit message (including change-id). Then you do not need to abandon/produce new change | |
| 08:12:04 | thomasb06 | from my computer, I don't know how to find the Compute commit to amend it. Is there a command to see the full working tree? | |
| 08:12:58 | gtema | just copy it from the UI: full change description | |
| 08:13:10 | gtema | """Compute - reindentation of the docstrings | |
| 08:13:10 | gtema | Change-Id: Iee76170382faf13c584fa215ef9dbe94de805d32""" | |
| 08:13:39 | thomasb06 | Ah, I add the Change-Id in the commit message? | |
| 08:13:58 | gtema | you just take this full message as the new commit | |
| 08:14:11 | thomasb06 | understood | |
| 08:21:36 | thomasb06 | Here it is: https://review.opendev.org/c/openstack/openstacksdk/+/818202 | |
| 08:39:42 | thomasb06 | Is it ok, it looks weird to have two ChangeIds in the UI? | |
| 08:39:52 | gtema | you should have only one | |
| 08:40:36 | gtema | you produced new change. I think your hook has not properly identified change-id in the commit (maybe missing line break) | |
| 08:41:01 | gtema | in this case just drop the original id in the new change and abandon the old one | |
| 08:46:20 | thomasb06 | you were right, it was just a newline missing. The old one is fine now: https://review.opendev.org/c/openstack/openstacksdk/+/816785 | |
| 08:46:44 | thomasb06 | If it's ok for you, I can delete the new commit? | |
| 08:47:40 | thomasb06 | (which is 818202) | |
| 08:50:34 | gtema | yupp, can be | |
| 08:58:54 | thomasb06 | Unless I forgot some _proxy.py files, this it is. Would you have other docstrings to fix, my script is ready? | |
| 11:42:17 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: image: Remove dead test helper methods https://review.opendev.org/c/openstack/python-openstackclient/+/818223 | |
| 11:42:17 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: image: Remove FakeImage test helper https://review.opendev.org/c/openstack/python-openstackclient/+/818224 | |
| 11:42:18 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Update fake image client in tests https://review.opendev.org/c/openstack/python-openstackclient/+/818225 | |
| 11:42:18 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: image: Sanity check the 'SetImage' command https://review.opendev.org/c/openstack/python-openstackclient/+/818226 | |
| 14:08:28 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch command server add volume to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/815919 | |
| 19:21:11 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch command server add volume to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/815919 | |
| #openstack-sdks - 2021-11-18 | |||
| 01:55:26 | opendevreview | melanie witt proposed openstack/python-openstackclient master: Add 'Host Status' to 'server list --long' with >= v2.16 https://review.opendev.org/c/openstack/python-openstackclient/+/818306 | |
| 03:23:57 | opendevreview | melanie witt proposed openstack/python-openstackclient master: Add 'Host Status' to 'server list --long' with >= v2.16 https://review.opendev.org/c/openstack/python-openstackclient/+/818306 | |
| 05:22:39 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: Support Neutron Local IP CRUD https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | |
| 06:46:53 | isabek | gtema: Good morning! This patches ready for review: https://review.opendev.org/c/openstack/openstacksdk/+/814765 and https://review.opendev.org/c/openstack/python-openstackclient/+/805618 . depends-on patch was merged | |
| 06:47:45 | isabek | Can you please take a look? Thanks a lot! | |
| 16:32:34 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch compute service list, delete and set to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/818434 | |
| 17:11:23 | opendevreview | Rodolfo Alonso proposed openstack/openstacksdk master: Add "check_limit" to network Quota class https://review.opendev.org/c/openstack/openstacksdk/+/806254 | |
| #openstack-sdks - 2021-11-19 | |||
| 00:34:44 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch compute service list, delete and set to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/818434 | |
| 03:25:20 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch compute service list, delete and set to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/818434 | |
| 12:58:09 | opendevreview | Jiri Podivin proposed openstack/cliff master: Removing modindex link from docs https://review.opendev.org/c/openstack/cliff/+/818547 | |
| 14:24:58 | opendevreview | Merged openstack/cliff master: Removing modindex link from docs https://review.opendev.org/c/openstack/cliff/+/818547 | |
| 20:35:14 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch compute service list, delete and set to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/818434 | |
| 21:34:29 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch compute service list, delete and set to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/818434 | |
| 21:37:33 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch compute service list, delete and set to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/818434 | |
| 21:38:02 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch compute service list, delete and set to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/818434 | |
| 22:07:44 | opendevreview | Thrivikram Mudunuri proposed openstack/python-openstackclient master: Switch server pause and server unpause to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/817853 | |
| 22:12:32 | opendevreview | Thrivikram Mudunuri proposed openstack/python-openstackclient master: Switch server suspend and server resume to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/817860 | |
| 22:41:31 | opendevreview | Thrivikram Mudunuri proposed openstack/python-openstackclient master: Switch server suspend and server resume to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/817860 | |
| #openstack-sdks - 2021-11-21 | |||
| 04:44:53 | opendevreview | SaeBom Lee proposed openstack/python-openstackclient master: Fix bug that image creation with volume https://review.opendev.org/c/openstack/python-openstackclient/+/818683 | |
| 05:07:57 | opendevreview | Jieon Lee proposed openstack/python-openstackclient master: Add missing command mapping in nova https://review.opendev.org/c/openstack/python-openstackclient/+/818684 | |
| #openstack-sdks - 2021-11-22 | |||
| 15:54:59 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch command server add volume to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/815919 | |
| 16:01:40 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch command server add volume to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/815919 | |
| 16:07:17 | opendevreview | Daniel Speichert proposed openstack/openstacksdk master: fix: improperly encoded object names https://review.opendev.org/c/openstack/openstacksdk/+/818795 | |
| 16:09:47 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch command server remove volume to sdk https://review.opendev.org/c/openstack/python-openstackclient/+/817989 | |
| 18:14:07 | ivan__ | Hello everyone. I am trying to use an admin account to create a server instance for a given user by supplying user_id. This seems to work unless I also supply the user's ssh key (through argument key_name). The SDK is trying to look that key_name up under the admin’s account rather than under user’s account. Can I tell the create_server functions to use the user’s key? Or, is there a workaround that I can use?. | |
| 18:17:58 | gtema | I guess workaround is to use the plain SDK/create_server call, it doesn't do such validation | |
| 18:20:51 | gtema | I mean conn.compute.create_server | |
| 18:49:17 | ivan__ | gtema: I just tried that, but the same happened... To elaborate a bit, I am using connect_as_project() to try to 'impersonate' the user, but the key_name lookup still happens under admin user (I know this since, if I use a key_name that exists under admin, the server creation succeeds). | |
| 18:57:21 | gtema | Ha, do you mean nova itself does this? | |
| 19:00:26 | gtema | Well, in this case you have no other choice as to physically reconnect as user, alternate you import same key under admin user - it's anyway just a pub key | |
| 19:01:47 | ivan__ | Is there a way to connect as the user without knowing their password? | |
| 19:05:33 | gtema | Depending on particular cloud, but natively no | |
| 19:35:54 | frickler | the connect_as_project() only overrides project_id, but keypairs are user-based ressources in nova. with "openstack keypair list --user demo --debug" you can see the public key though, not sure if the sdk exposes that | |
| 19:55:17 | ivan__ | frickler: SDK exposes the keypair commands for a given user_id, but it does not seem to allow specifying user_id when creating a server for the purposes of looking up the key_name. | |
| 19:56:37 | frickler | ivan__: yes, you would have to fetch the users pubkey and create a keypair with that as your admin user | |
| #openstack-sdks - 2021-11-23 | |||
| 13:53:22 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch command server remove volume to sdk https://review.opendev.org/c/openstack/python-openstackclient/+/817989 | |
| 15:36:08 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch command server remove volume to sdk https://review.opendev.org/c/openstack/python-openstackclient/+/817989 | |
| 17:03:53 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch command server remove volume to sdk https://review.opendev.org/c/openstack/python-openstackclient/+/817989 | |
| 18:24:09 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch command server remove volume to sdk https://review.opendev.org/c/openstack/python-openstackclient/+/817989 | |
| 21:16:22 | opendevreview | Slawek Kaplonski proposed openstack/python-openstackclient master: Allow unset port's host_id https://review.opendev.org/c/openstack/python-openstackclient/+/819024 | |
| #openstack-sdks - 2021-11-24 | |||
| 00:14:29 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch list server volume to sdk https://review.opendev.org/c/openstack/python-openstackclient/+/816716 | |
| 00:15:41 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch list server volume to sdk https://review.opendev.org/c/openstack/python-openstackclient/+/819036 | |
| 00:17:11 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch list server volume to sdk https://review.opendev.org/c/openstack/python-openstackclient/+/816716 | |
| 00:23:46 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch list server volume to sdk https://review.opendev.org/c/openstack/python-openstackclient/+/816716 | |
| 00:28:32 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch list server volume to sdk https://review.opendev.org/c/openstack/python-openstackclient/+/816716 | |
| 00:31:26 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch compute service list, delete and set to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/818434 | |
| 05:51:20 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Temporarily drop aodhclient from doc build https://review.opendev.org/c/openstack/python-openstackclient/+/819055 | |