| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2021-07-12 | |||
| 14:47:11 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient stable/train: Fix reverted osc-lib interface change https://review.opendev.org/c/openstack/python-openstackclient/+/799662 | |
| 14:47:11 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient stable/train: [stable-only] Cap bandit to 1.6.2, bump lower-constraints https://review.opendev.org/c/openstack/python-openstackclient/+/799690 | |
| 14:54:39 | 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 | |
| 20:02:38 | opendevreview | Uemit Seren proposed openstack/ansible-collections-openstack master: Add subnet pool module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/800463 | |
| 21:24:15 | opendevreview | Uemit Seren proposed openstack/ansible-collections-openstack master: Add subnet pool module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/800463 | |
| #openstack-sdks - 2021-07-13 | |||
| 02:23:16 | opendevreview | Zane Bitter proposed openstack/cliff master: Colourise and automatically page help output https://review.opendev.org/c/openstack/cliff/+/799344 | |
| 02:23:17 | opendevreview | Zane Bitter proposed openstack/cliff master: Automatically page interactive root help output https://review.opendev.org/c/openstack/cliff/+/799345 | |
| 08:14:42 | opendevreview | Uemit Seren proposed openstack/ansible-collections-openstack master: Add subnet pool module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/800463 | |
| 09:37:42 | opendevreview | Uemit Seren proposed openstack/ansible-collections-openstack master: Add subnet pool module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/800463 | |
| 10:10:53 | stephenfin | diablo_rojo_phone: I totally forgot to send you stuff for students to work on. Sorry! /o\ I assume we're past that? | |
| 10:20:04 | stephenfin | gtema: Can you look at https://review.opendev.org/c/openstack/python-openstackclient/+/799662 and its predecessor (which fixes stable/train for OSC)? | |
| 10:20:14 | gtema | yupp | |
| 10:20:38 | stephenfin | gtema: Also, if you want another stable core on OSC/SDK then I'm happy to help out. I'm stable on oslo so I know the rules | |
| 10:20:51 | gtema | :) | |
| 10:20:56 | gtema | cool | |
| 10:21:23 | gtema | actually you should be, since currently all cores are maintainers if I remember correctly | |
| 10:21:34 | stephenfin | it doesn't appear so | |
| 10:21:41 | gtema | ok, will check | |
| 10:56:56 | diablo_rojo_phone | stephenfin: actually not. Thursday is the day of the event. | |
| 10:57:26 | stephenfin | diablo_rojo_phone: \o/ I can come up with some stuff after lunch if you still need it? | |
| 11:06:23 | diablo_rojo_phone | stephenfin: yes please! Works for me. | |
| 11:20:21 | opendevreview | Uemit Seren proposed openstack/ansible-collections-openstack master: Add subnet pool module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/800463 | |
| 12:38:43 | opendevreview | Merged openstack/python-openstackclient stable/train: [stable-only] Cap bandit to 1.6.2, bump lower-constraints https://review.opendev.org/c/openstack/python-openstackclient/+/799690 | |
| 12:38:47 | opendevreview | Merged openstack/python-openstackclient stable/train: Fix reverted osc-lib interface change https://review.opendev.org/c/openstack/python-openstackclient/+/799662 | |
| 13:25:32 | opendevreview | Uemit Seren proposed openstack/ansible-collections-openstack master: Add subnet pool module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/800463 | |
| 14:07:02 | opendevreview | Uemit Seren proposed openstack/ansible-collections-openstack master: Add subnet pool module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/800463 | |
| 19:08:41 | stephenfin | diablo_rojo_phone: https://storyboard.openstack.org/#!/story/2009054 | |
| 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 | 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? | |
| 16:42:34 | mchlumsky | Hi! :) | |
| 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 | |