| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2021-09-30 | |||
| 11:37:28 | opendevreview | Lajos Katona proposed openstack/python-openstackclient master: Add BFD monitor commands https://review.opendev.org/c/openstack/python-openstackclient/+/810286 | |
| 14:27:27 | opendevreview | Pavlo Shchelokovskyy proposed openstack/python-openstackclient master: Add --trusted-image-cert option for server create https://review.opendev.org/c/openstack/python-openstackclient/+/811981 | |
| 16:42:36 | opendevreview | Pavlo Shchelokovskyy proposed openstack/python-openstackclient master: Add --trusted-image-cert option for server create https://review.opendev.org/c/openstack/python-openstackclient/+/811981 | |
| 16:57:11 | opendevreview | Lajos Katona proposed openstack/python-openstackclient master: Add bfd_monitor as optional arg to route https://review.opendev.org/c/openstack/python-openstackclient/+/810370 | |
| #openstack-sdks - 2021-10-01 | |||
| 08:59:16 | opendevreview | Alexandru Verdes proposed openstack/ansible-collections-openstack master: Add client and member listener timeouts for persistence (Ex. SSH) https://review.opendev.org/c/openstack/ansible-collections-openstack/+/812110 | |
| 09:17:56 | opendevreview | Alexandru Verdes proposed openstack/ansible-collections-openstack master: Add client and member listener timeouts for persistence (Ex. SSH) https://review.opendev.org/c/openstack/ansible-collections-openstack/+/812110 | |
| 14:57:00 | noonedeadpunk | hey! I was wondering if there's any good way to import one of the commands to reuse code inside other one? I guess no? | |
| 14:57:20 | noonedeadpunk | regsarding openstackclient | |
| 14:58:35 | gtema | can you be please more precise with your example? what are you thinking of? | |
| 15:03:08 | noonedeadpunk | So, I'm trying to write host-evacuate command for openstackclient. And what I;d ove to do is re-use https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/compute/v2/server.py#L3458 | |
| 15:03:15 | noonedeadpunk | But I guess I can't do that? | |
| 15:03:52 | noonedeadpunk | so the way would be copy/pasting code? Which I'd hate doing... | |
| 15:04:47 | gtema | reuse in the sense of a complete copy or only parts? | |
| 15:05:05 | noonedeadpunk | I guess that was kind of reason why it hasn't been implemented yet... | |
| 15:05:44 | noonedeadpunk | complete copy but pass different parsed_args | |
| 15:06:13 | gtema | and what about the action itself? it currently invoke server.evacuate(xx) | |
| 15:07:09 | noonedeadpunk | yeah, it will be the same. So host evacuate is basically retrieves all servers for hypervisors and iterate on them | |
| 15:07:31 | noonedeadpunk | ie https://github.com/openstack/python-novaclient/blob/8066f8c745054caeb7bd66d4c2be15ab5fac1d1f/novaclient/v2/shell.py#L5190-L5199 | |
| 15:08:41 | gtema | but it is the still not really same. I would suggest just move evacuate implementation into helper function | |
| 15:08:52 | gtema | so that you can also iterate and invoke it | |
| 15:13:17 | noonedeadpunk | Sounds reasonable. And can you kindly point to some helper sample? Can find quickly in tree. Or they are osc_lib or somewhere? | |
| 15:14:22 | gtema | https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/common/sdk_utils.py | |
| 15:14:27 | gtema | whatever is currently in common | |
| 15:14:40 | gtema | but I would rather simply have something in the compute part | |
| 15:14:56 | gtema | https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/network/common.py | |
| 15:16:16 | noonedeadpunk | yeah, great, thanks! | |
| 18:23:05 | opendevreview | Merged openstack/osc-lib stable/xena: Update .gitreview for stable/xena https://review.opendev.org/c/openstack/osc-lib/+/808361 | |
| 18:23:29 | opendevreview | Merged openstack/osc-lib stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena https://review.opendev.org/c/openstack/osc-lib/+/808362 | |
| #openstack-sdks - 2021-10-02 | |||
| 12:53:10 | opendevreview | Pavlo Shchelokovskyy proposed openstack/keystoneauth master: Enforce scope mutual exclusion for system https://review.opendev.org/c/openstack/keystoneauth/+/802683 | |
| 19:54:43 | jsmdk | Hi, I have a long time pending review here https://review.opendev.org/c/openstack/openstacksdk/+/787952. Can someone have a look? | |
| #openstack-sdks - 2021-10-03 | |||
| 12:44:36 | opendevreview | Merged openstack/openstacksdk master: Change image.hw_qemu_guest_agent to be string https://review.opendev.org/c/openstack/openstacksdk/+/808101 | |
| 18:17:12 | opendevreview | ChoiDoa proposed openstack/python-openstackclient stable/wallaby: title: Update Nova CLI decoder document https://review.opendev.org/c/openstack/python-openstackclient/+/812267 | |
| #openstack-sdks - 2021-10-04 | |||
| 07:57:41 | opendevreview | ChoiDoa proposed openstack/python-openstackclient stable/wallaby: Update the Nova CLI decoder document https://review.opendev.org/c/openstack/python-openstackclient/+/812287 | |
| 08:35:14 | opendevreview | ChoiDoa proposed openstack/python-openstackclient stable/wallaby: Update the Nova CLI decoder document. https://review.opendev.org/c/openstack/python-openstackclient/+/812294 | |
| 09:05:11 | opendevreview | ChoiDoa proposed openstack/python-openstackclient stable/wallaby: Update the Nova CLI decoder document https://review.opendev.org/c/openstack/python-openstackclient/+/812305 | |
| 09:12:14 | opendevreview | ChoiDoa proposed openstack/python-openstackclient stable/wallaby: Update the Nova CLI decoder document https://review.opendev.org/c/openstack/python-openstackclient/+/812306 | |
| 09:17:24 | opendevreview | ChoiDoa proposed openstack/python-openstackclient stable/wallaby: Update the Nova CLI decoder document https://review.opendev.org/c/openstack/python-openstackclient/+/812294 | |
| 09:26:27 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Move CI to use Ansible 2.12 version as main https://review.opendev.org/c/openstack/ansible-collections-openstack/+/812308 | |
| 12:06:52 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Move CI to use Ansible 2.12 version as main https://review.opendev.org/c/openstack/ansible-collections-openstack/+/812308 | |
| 15:59:14 | diablo_rojo_phone | gtema: stephenfin as a heads up I am in a meeting that's going long so you might want to get started without me. | |
| 15:59:31 | gtema | ok, thks | |
| 21:20:18 | opendevreview | Ashraf Hasson proposed openstack/ansible-collections-openstack master: Add Neutron RBAC modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/799896 | |
| #openstack-sdks - 2021-10-05 | |||
| 02:30:55 | opendevreview | JIHOJU proposed openstack/python-openstackclient master: Update the Nova CLI docoder document https://review.opendev.org/c/openstack/python-openstackclient/+/812421 | |
| 10:44:27 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Clean up the old jobs for rocky,stein,queens https://review.opendev.org/c/openstack/ansible-collections-openstack/+/812476 | |
| 10:47:57 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Clean up the old jobs for rocky,stein,queens https://review.opendev.org/c/openstack/ansible-collections-openstack/+/812476 | |
| 11:49:04 | opendevreview | Merged openstack/ansible-collections-openstack master: Move CI to use Ansible 2.12 version as main https://review.opendev.org/c/openstack/ansible-collections-openstack/+/812308 | |
| 11:54:44 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Switch hypervisor operations to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/763414 | |
| 13:30:13 | opendevreview | Merged openstack/ansible-collections-openstack master: Clean up the old jobs for rocky,stein,queens https://review.opendev.org/c/openstack/ansible-collections-openstack/+/812476 | |
| 15:44:50 | opendevreview | Merged openstack/ansible-collections-openstack master: Add client and member listener timeouts for persistence (Ex. SSH) https://review.opendev.org/c/openstack/ansible-collections-openstack/+/812110 | |
| 17:29:14 | opendevreview | Rafael Weingartner proposed openstack/os-api-ref master: Fix exception logging https://review.opendev.org/c/openstack/os-api-ref/+/799216 | |
| 17:52:08 | opendevreview | Rafael Weingartner proposed openstack/os-api-ref master: Fix exception logging https://review.opendev.org/c/openstack/os-api-ref/+/799216 | |
| #openstack-sdks - 2021-10-06 | |||
| 08:35:06 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add Neutron RBAC modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/799896 | |
| 10:54:46 | opendevreview | Seunghyun LIm proposed openstack/python-openstackclient master: Remove non-working code after method return. https://review.opendev.org/c/openstack/python-openstackclient/+/812661 | |
| 11:36:48 | opendevreview | ChoiDoa proposed openstack/python-openstackclient master: Update Nova CLI decoder document https://review.opendev.org/c/openstack/python-openstackclient/+/812566 | |
| 11:55:33 | opendevreview | ChoiDoa proposed openstack/python-openstackclient master: Update the Nova CLI decoder document https://review.opendev.org/c/openstack/python-openstackclient/+/812668 | |
| 12:11:07 | opendevreview | Merged openstack/python-openstackclient master: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/c/openstack/python-openstackclient/+/808079 | |
| 12:11:08 | opendevreview | Merged openstack/python-openstackclient master: Fix that the path of functional test https://review.opendev.org/c/openstack/python-openstackclient/+/811008 | |
| 12:12:10 | gtema | I see stephenfin you blowed my attempt to have easy rebase for osc patch ;-) Will try to find time to address your concerns this week. Thks | |
| 13:28:26 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add Neutron RBAC modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/799896 | |
| 13:42:23 | thomasb06 | gtema: awk processes the files line by line... But with a python script, it should be possible to format all the docstrings at once | |
| 13:54:19 | thomasb06 | when there are * bullet points, the indentation should be 8? | |
| 13:55:59 | stephenfin | gtema: :) Sorry! | |
| 15:35:04 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add Neutron RBAC modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/799896 | |
| 15:40:08 | opendevreview | ChoiDoa proposed openstack/python-openstackclient master: Update the Nova CLI decoder document https://review.opendev.org/c/openstack/python-openstackclient/+/812721 | |
| 15:49:23 | seongsoocho | stephenfin: Hi. I'm seongsoo from openstack korea user group. and we are doing mentoring program to contribute openstack. Since it is the first time using gerrit, so several reviews with the same content were created and there were many abandoned ones. Sorry to bother you with the review and thank you :-) | |
| 16:34:54 | stephenfin | seongsoocho: Oh, okay, no problem :) Thank you for the context | |
| 17:32:25 | opendevreview | Merged openstack/python-openstackclient master: Fix typo error in listing server's column name https://review.opendev.org/c/openstack/python-openstackclient/+/806464 | |
| 18:41:51 | opendevreview | Ashraf Hasson proposed openstack/ansible-collections-openstack master: Add Neutron RBAC modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/799896 | |
| 18:43:41 | opendevreview | Ashraf Hasson proposed openstack/ansible-collections-openstack master: Add Neutron RBAC modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/799896 | |
| 22:14:38 | opendevreview | Merged openstack/python-openstackclient master: Add --trusted-image-cert option for server create https://review.opendev.org/c/openstack/python-openstackclient/+/811981 | |
| 22:36:13 | opendevreview | Ashraf Hasson proposed openstack/ansible-collections-openstack master: Add Neutron RBAC modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/799896 | |
| #openstack-sdks - 2021-10-07 | |||
| 05:14:59 | opendevreview | Sven Anders proposed 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 | |
| 07:31:57 | opendevreview | Merged openstack/python-openstackclient master: Remove non-working code after method return. https://review.opendev.org/c/openstack/python-openstackclient/+/812661 | |
| 10:42:08 | opendevreview | ChoiDoa proposed openstack/python-openstackclient master: Update the Nova CLI decoder document https://review.opendev.org/c/openstack/python-openstackclient/+/812721 | |
| 11:06:14 | opendevreview | Merged openstack/python-openstackclient master: Fix the functional-tips tox environment https://review.opendev.org/c/openstack/python-openstackclient/+/790662 | |
| 12:07:20 | opendevreview | JIHOJU proposed openstack/python-openstackclient master: Update the Nova CLI docoder document https://review.opendev.org/c/openstack/python-openstackclient/+/812421 | |
| #openstack-sdks - 2021-10-08 | |||
| 09:06:21 | opendevreview | Merged openstack/python-openstackclient master: Update the Nova CLI decoder document https://review.opendev.org/c/openstack/python-openstackclient/+/812721 | |
| 09:56:32 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Switch hypervisor operations to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/763414 | |
| 10:06:09 | opendevreview | Merged openstack/keystoneauth stable/wallaby: Update .gitreview for stable/wallaby https://review.opendev.org/c/openstack/keystoneauth/+/781867 | |
| 12:01:12 | opendevreview | Merged openstack/keystoneauth stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/keystoneauth/+/781868 | |
| 14:42:57 | opendevreview | Yoon Soo Lim proposed openstack/python-openstackclient master: compute: Add 'hypervisor servers' command https://review.opendev.org/c/openstack/python-openstackclient/+/813196 | |
| 16:12:06 | opendevreview | JIHOJU proposed openstack/python-openstackclient master: Fix bug that readonly option doesn’t setup with volume create. https://review.opendev.org/c/openstack/python-openstackclient/+/811006 | |
| 17:39:13 | opendevreview | Stephen Finucane proposed openstack/osc-lib master: Don't display Munch objects in the output https://review.opendev.org/c/openstack/osc-lib/+/813235 | |
| 17:39:13 | opendevreview | Stephen Finucane proposed openstack/osc-lib master: utils: Remove unnecessary attribute check https://review.opendev.org/c/openstack/osc-lib/+/813236 | |
| 17:45:30 | stephenfin | weird. I pushed patches to openstacksdk but they didn't appear here | |
| 17:47:23 | stephenfin | oh, it's because we're not using the master branch | |
| #openstack-sdks - 2021-10-09 | |||
| 01:52:42 | opendevreview | Yoon Soo Lim proposed openstack/python-openstackclient master: compute: Add 'hypervisor servers' command https://review.opendev.org/c/openstack/python-openstackclient/+/813196 | |
| 04:47:21 | opendevreview | SEUNG TAE KIM proposed openstack/python-openstackclient master: Fix bug that configuration show --mask cannot masking the password & token https://review.opendev.org/c/openstack/python-openstackclient/+/811005 | |
| 04:53:47 | opendevreview | SEUNG TAE KIM proposed openstack/python-openstackclient master: Fix bug that configuration show --mask cannot masking the password & token https://review.opendev.org/c/openstack/python-openstackclient/+/811005 | |
| 04:59:28 | opendevreview | SEUNG TAE KIM proposed openstack/python-openstackclient master: Fix bug that configuration show --mask cannot masking the password & token https://review.opendev.org/c/openstack/python-openstackclient/+/811005 | |
| #openstack-sdks - 2021-10-10 | |||
| 13:48:12 | opendevreview | rura6502 proposed openstack/python-openstackclient master: update cli mapping nova host-meta to host set https://review.opendev.org/c/openstack/python-openstackclient/+/813296 | |
| #openstack-sdks - 2021-10-11 | |||
| 04:12:36 | ryanKor | Hello, I am working on building the glance task commands in openstack with my team members. | |
| 04:12:36 | ryanKor | glance task-create | |
| 04:12:36 | ryanKor | glance task-list | |
| 04:12:36 | ryanKor | glance task-show | |
| 04:12:36 | ryanKor | I would like to inquire about the process of implementing the test case codes about the above ones if there is an order of implementation between them. | |
| 06:03:25 | gtema | Hi ryanKor, the only order is that you need to ensure those operations are supported by OpenStackSDK, since image commands are now being implemented this way directly. | |
| 06:44:48 | opendevreview | JIHOJU proposed openstack/python-openstackclient master: Add more filter option of columns for server list -c COLUMN https://review.opendev.org/c/openstack/python-openstackclient/+/807420 | |
| 08:06:08 | opendevreview | Seunghyun LIm proposed openstack/python-openstackclient master: Add missing document and help message about project option of "image set" https://review.opendev.org/c/openstack/python-openstackclient/+/813393 | |
| 09:19:59 | opendevreview | Seunghyun LIm proposed openstack/python-openstackclient master: Add missing document of an option of "image set" https://review.opendev.org/c/openstack/python-openstackclient/+/813393 | |