| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2021-07-13 | |||
| 19:15:22 | diablo_rojo_phone | stephenfin: thanks! | |
| 19:33:49 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: volume: Add 'volume transfer request create --(no-)snapshots' option https://review.opendev.org/c/openstack/python-openstackclient/+/800715 | |
| #openstack-sdks - 2021-07-14 | |||
| 15:09:58 | gshippey | any possibility on getting feedback on https://review.opendev.org/c/openstack/openstacksdk/+/799508? Looking to add support for keystone service providers, first time contributing to this project | |
| 17:00:12 | gouthamr | gtema diablo_rojo_phone: have you folks had any further thoughts on adding one/more manila reviewers? | |
| 17:01:33 | gtema | Guys, you need to start reviewing patches yourself. If we add you as reviewers, but you do not review - why's that? | |
| 17:01:46 | gtema | From manilla patches only one was reviewed by you | |
| 17:03:22 | gouthamr | gtema: ? sorry, i perhaps missed review notifications | |
| 17:04:14 | gtema | From around 10 manila patches only one was finally approved by you. Others had some iterations, but are now not reviewed | |
| 17:04:20 | gouthamr | yes | |
| 17:04:47 | gtema | And this is normally a signal for me when I see you approve the patch | |
| 17:04:59 | gouthamr | i've been looking at these; i can add my comments there | |
| 17:08:35 | gouthamr | so if you see a +1 from one of us, you're looking to treat that as a +2? | |
| 17:09:24 | gouthamr | i'm okay with that - just want to know what the operating procedure should be around here :) | |
| 17:10:17 | gouthamr | i'd love to see https://review.opendev.org/c/openstack/openstacksdk/+/796040 merge for one - its been worked on for a while | |
| 17:41:04 | gouthamr | gtema: hey you might know the answer to my prior queries to this channel - i'd like to specify a microversion for a particular SDK call; is this possible today? | |
| 17:41:29 | gouthamr | i tried doing something like this: https://review.opendev.org/c/openstack/openstacksdk/+/799208/2/openstack/tests/functional/shared_file_system/base.py | |
| 17:41:34 | gouthamr | but it didn't seem to have an effect | |
| #openstack-sdks - 2021-07-15 | |||
| 11:33:08 | opendevreview | Takashi Kajinami proposed openstack/openstacksdk master: Replace deprecated inspect.getargspec https://review.opendev.org/c/openstack/openstacksdk/+/800918 | |
| 11:37:54 | opendevreview | Takashi Kajinami proposed openstack/openstacksdk master: Replace deprecated inspect.getargspec https://review.opendev.org/c/openstack/openstacksdk/+/800918 | |
| #openstack-sdks - 2021-07-17 | |||
| 03:14:52 | opendevreview | Cuong proposed openstack/ansible-collections-openstack master: Add protocol listener octavia https://review.opendev.org/c/openstack/ansible-collections-openstack/+/801176 | |
| #openstack-sdks - 2021-07-19 | |||
| 11:08:45 | opendevreview | Jesper Schmitz Mouridsen proposed openstack/ansible-collections-openstack master: Wait for pool to be active and online https://review.opendev.org/c/openstack/ansible-collections-openstack/+/801286 | |
| 13:28:45 | opendevreview | Merged openstack/ansible-collections-openstack master: Add protocol listener octavia https://review.opendev.org/c/openstack/ansible-collections-openstack/+/801176 | |
| 20:39:11 | opendevreview | Uemit Seren proposed openstack/ansible-collections-openstack master: Add extra_specs attribute to network and router https://review.opendev.org/c/openstack/ansible-collections-openstack/+/800492 | |
| #openstack-sdks - 2021-07-20 | |||
| 09:26:46 | opendevreview | Uemit Seren proposed openstack/ansible-collections-openstack master: Add extra_specs attribute to network and router https://review.opendev.org/c/openstack/ansible-collections-openstack/+/800492 | |
| 16:42:34 | mchlumsky | Hi! :) | |
| 16:42:34 | mchlumsky | I am using the openstacksdk and I am trying to list all the users in a particular group (in keystone). There's an endpoint for it (https://docs.openstack.org/api-ref/identity/v3/index.html?expanded=id627-detail#list-users-in-group) but I can't find an equivalent in the openstacksdk. Did I miss it somehow? | |
| 17:06:03 | gtema | mchlumsky sadly right now it's not implemented, but it is rather easy to do so. I'll look if I find time doing that shortly | |
| 17:06:26 | opendevreview | Merged openstack/cliff master: Add conflict_handler parameter as attribut in Command class https://review.opendev.org/c/openstack/cliff/+/794792 | |
| 17:33:47 | mchlumsky | gtema thank you! :) | |
| 21:17:53 | opendevreview | Uemit Seren proposed openstack/openstacksdk master: Add extra params to create_router & create_network https://review.opendev.org/c/openstack/openstacksdk/+/801535 | |
| #openstack-sdks - 2021-07-22 | |||
| 14:27:17 | opendevreview | Balazs Gibizer proposed openstack/openstacksdk master: Remove block storage v2 support https://review.opendev.org/c/openstack/openstacksdk/+/801864 | |
| 14:36:32 | opendevreview | Balazs Gibizer proposed openstack/openstacksdk master: Remove block storage v2 func tests https://review.opendev.org/c/openstack/openstacksdk/+/801864 | |
| 14:40:33 | gibi | hi! you might noticed the openstacksdk-functional-devstack job started to fail since cinder merged the v2 api removal in https://review.opendev.org/c/openstack/cinder/+/792299 | |
| 14:40:54 | gibi | I proposed the removal of the v2 func test from the job https://review.opendev.org/c/openstack/openstacksdk/+/801864 to unblock the gate | |
| 14:57:05 | gmann | gibi: also we should have test v2 as separate job and default jobs test to v3 only. that is how we did in tempest integration jobs | |
| 14:59:08 | gtema | SDK has a contract to work on older clouds (not only for OSC, but also ansible collections, Zuul, etc). Dropping v2 support is definitely not the way we will go. Dropping tests seems ok, but I will check for a possibility to check whether specific version is available (in some cases we have skipTest if service is not available) | |
| 15:01:46 | gibi | gtema: ack. I've limited experience in sdk so I follow any lead. I just wanted to unblock the gate | |
| 15:02:16 | gtema | clear. If I am not able to find alternative solution in 1-2 hours I will approve the patch | |
| 15:02:27 | gibi | gtema: thanks. I appreciate it! | |
| 15:02:35 | gtema | then we can revert the tests with some logic | |
| 15:06:35 | gmann | gtema: gibi we can skip in v2 test in job | |
| 15:06:45 | gmann | I can try after TC meeting | |
| 15:07:14 | gibi | gmann: feel free to take over my patch I will probably disappeare in a hour or so | |
| 15:07:41 | gmann | gibi: I can do in separate and then we can discuss best way to remove v2 in your patch ? | |
| 15:07:59 | gibi | that works for me too | |
| 15:41:43 | opendevreview | Ghanshyam proposed openstack/openstacksdk master: Skip block_storage.v2 test in functional job https://review.opendev.org/c/openstack/openstacksdk/+/801872 | |
| 15:42:57 | gmann | gtema: ^^ | |
| 16:01:57 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Extend has_version function to accept version parameter https://review.opendev.org/c/openstack/openstacksdk/+/801900 | |
| 16:03:59 | gtema | gmann, gibi - my change extends version check to properly (hopefully) apply skipTest if v2 is not available | |
| 16:05:03 | gtema | locally for me it did the trick (check worked properly), but I couldn't verify func test on devstack since VirtualBox crashed | |
| 16:07:44 | gibi | gtema: I abandoned my patch in favor of gman's. for me both his and yours looks ok | |
| 16:09:36 | gtema | extending version check for me looks more future oriented (next deprecation) rather then maintaining blacklists, but let's see whether this work like I expect | |
| 16:10:25 | gibi | yeah, your reasoning make sense. | |
| 16:10:35 | gibi | I have to drop now for today | |
| 16:10:44 | gibi | gmann, gtema: thanks for jumping on this issue | |
| 16:10:48 | gibi | o/ | |
| 17:38:32 | gtema | anybody want to leave a comment on https://review.opendev.org/c/openstack/openstacksdk/+/801900 to unblock the gate? | |
| 18:20:31 | gmann | gtema: sure, make sense. abandon the blacklist way. | |
| 20:40:35 | opendevreview | Merged openstack/openstacksdk master: Extend has_version function to accept version parameter https://review.opendev.org/c/openstack/openstacksdk/+/801900 | |
| #openstack-sdks - 2021-07-23 | |||
| 11:51:42 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Handle removal of block-storage v2 API https://review.opendev.org/c/openstack/python-openstackclient/+/802012 | |
| 11:51:57 | stephenfin | gtema: Need to wait for CI, but that should unblock the OSC gate too ^ | |
| 11:52:07 | gtema | yupp, looking. thks | |
| 11:52:17 | stephenfin | thanks | |
| 11:54:25 | gtema | honestly this invocation is new to me ;-) but seems to work | |
| 11:54:49 | stephenfin | me too. I did "openstack --help | grep version" and spotted that :P | |
| 11:55:01 | stephenfin | but it works. Seems to query the service catalog | |
| 11:55:04 | gtema | new day new discovery | |
| 11:55:13 | stephenfin | and on a Friday, no less | |
| 11:55:23 | gtema | yes, I tend to usually use `openstack XXX --debug` | |
| 11:55:28 | stephenfin | #ProductiveFridaysCanBeAThing | |
| 11:55:32 | gtema | :) | |
| 11:56:00 | gtema | better than "Thursday's gate is blocked" | |
| 17:17:21 | opendevreview | Merged openstack/python-openstackclient master: tests: Handle removal of block-storage v2 API https://review.opendev.org/c/openstack/python-openstackclient/+/802012 | |
| 19:24:56 | opendevreview | cenne proposed openstack/openstacksdk master: Add support for reading node's "boot_mode" and "secure_boot" fields https://review.opendev.org/c/openstack/openstacksdk/+/802101 | |
| 21:40:36 | gmann | gtema: can you check this quick review https://review.opendev.org/c/openstack/osc-lib/+/800049 | |
| #openstack-sdks - 2021-07-24 | |||
| 06:12:00 | opendevreview | Merged openstack/osc-lib master: Moving IRC network reference to OFTC https://review.opendev.org/c/openstack/osc-lib/+/800049 | |
| #openstack-sdks - 2021-07-26 | |||
| 18:11:34 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: WIP release job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802379 | |
| #openstack-sdks - 2021-07-27 | |||
| 02:21:11 | opendevreview | melanie witt proposed openstack/python-openstackclient master: Fix TestListMigrationV223 test class MIGRATION_COLUMNS https://review.opendev.org/c/openstack/python-openstackclient/+/802421 | |
| 09:42:21 | opendevreview | Merged openstack/python-openstackclient master: Fix TestListMigrationV223 test class MIGRATION_COLUMNS https://review.opendev.org/c/openstack/python-openstackclient/+/802421 | |
| 11:46:15 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: volume: Add missing 'volume list --offset' parameter https://review.opendev.org/c/openstack/python-openstackclient/+/802480 | |
| 11:46:15 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Add pagination helpers https://review.opendev.org/c/openstack/python-openstackclient/+/802481 | |
| 11:46:16 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: network: Add '--marker', '--limit' to most list commands https://review.opendev.org/c/openstack/python-openstackclient/+/802482 | |
| 12:13:41 | opendevreview | Merged openstack/python-openstackclient master: compute: Note that '--password' is deployment-specific https://review.opendev.org/c/openstack/python-openstackclient/+/796696 | |
| 12:31:02 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Add pagination helpers https://review.opendev.org/c/openstack/python-openstackclient/+/802481 | |
| 12:31:03 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: network: Add '--marker', '--limit' to most list commands https://review.opendev.org/c/openstack/python-openstackclient/+/802482 | |
| 13:53:44 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: WIP release job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802379 | |
| 14:17:42 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: WIP release job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802379 | |
| 14:34:29 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: WIP release job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802379 | |
| 14:57:09 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: WIP release job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802379 | |
| 15:15:02 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: WIP release job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802379 | |
| 15:38:43 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Add pagination helpers https://review.opendev.org/c/openstack/python-openstackclient/+/802481 | |
| 15:38:44 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: network: Add '--marker', '--limit' to most list commands https://review.opendev.org/c/openstack/python-openstackclient/+/802482 | |
| 15:47:28 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: WIP release job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802379 | |
| 16:19:55 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add automatic release job triggered by tag https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802379 | |
| 16:21:06 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add automatic release job triggered by tag https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802379 | |
| 16:44:25 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Temporarily disable nodepool job https://review.opendev.org/c/openstack/openstacksdk/+/802556 | |
| 20:09:36 | opendevreview | Merged openstack/openstacksdk master: Temporarily disable nodepool job https://review.opendev.org/c/openstack/openstacksdk/+/802556 | |
| #openstack-sdks - 2021-07-28 | |||
| 03:35:59 | opendevreview | melanie witt proposed openstack/python-openstackclient master: Correct REST API response fields for /os-migrations API https://review.opendev.org/c/openstack/python-openstackclient/+/802650 | |
| 10:46:02 | opendevreview | Pavlo Shchelokovskyy proposed openstack/keystoneauth master: Enforce scope mutual exclusion for system https://review.opendev.org/c/openstack/keystoneauth/+/802683 | |
| 12:11:18 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add automatic release job triggered by tag https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802379 | |