| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-02-10 | |||
| 20:50:34 | opendevreview | Florian Haas proposed openstack/openstacksdk master: Add get_*_by_id() methods for Magnum clusters and templates https://review.opendev.org/c/openstack/openstacksdk/+/828791 | |
| 20:54:48 | opendevreview | Florian Haas proposed openstack/openstacksdk master: Add get_*_by_id() methods for Magnum clusters and templates https://review.opendev.org/c/openstack/openstacksdk/+/828791 | |
| 21:41:47 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | |
| 22:43:13 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828742 | |
| #openstack-sdks - 2022-02-11 | |||
| 02:15:26 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | |
| 08:00:39 | gtema | ok headphoneJames, will have a look today | |
| 09:38:56 | opendevreview | Jakob Meng proposed openstack/osc-lib master: Respect 'interface' key from clouds.yaml https://review.opendev.org/c/openstack/osc-lib/+/784559 | |
| 09:42:28 | jm1 | hello :) who would i have to ask for a workflow+1 on this osc-lib patch? https://review.opendev.org/c/openstack/osc-lib/+/784559 | |
| 10:35:33 | opendevreview | Tom Weininger proposed openstack/openstacksdk master: Correct documentation about Load Balancer API https://review.opendev.org/c/openstack/openstacksdk/+/828689 | |
| 10:59:17 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Allow retrieval of migration by UUID https://review.opendev.org/c/openstack/python-openstackclient/+/828785 | |
| 10:59:18 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Move server migrations commands to their own file https://review.opendev.org/c/openstack/python-openstackclient/+/828855 | |
| 11:08:54 | frickler | jm1: I've put it into my review list, but won't get to it today. from a quick glance, I'm wondering whether it would help to have a test for that scenario that would avoid regressing again | |
| 12:46:47 | opendevreview | Florian Haas proposed openstack/openstacksdk master: Add get_*_by_id() methods for Magnum clusters and templates https://review.opendev.org/c/openstack/openstacksdk/+/828791 | |
| 13:17:50 | jm1 | frickler: thanks :) | |
| 13:18:49 | jm1 | frickler: regarding the tests.. not sure but i think the tests did not catch this flaw because it asserts 'public' which is exactly DEFAULT_INTERFACE https://opendev.org/openstack/osc-lib/src/branch/master/osc_lib/tests/test_shell.py#L356 | |
| 13:31:14 | opendevreview | Jakob Meng proposed openstack/osc-lib master: Fixed test for interface key https://review.opendev.org/c/openstack/osc-lib/+/828876 | |
| 13:41:06 | jm1 | frickler: submitted a patch which fixes the test. of course zuul now fails but this is to be expected as the other patch has not been merged yet. https://review.opendev.org/c/openstack/osc-lib/+/828876 | |
| 13:52:59 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | |
| 13:57:34 | Adri2000 | about the in-use/attached volume resize issue I mentioned here yesterday, here is a bug report I just filed: https://storyboard.openstack.org/#!/story/2009850 | |
| 15:03:30 | lajoskatona | Hi, a quick question, we have a feature for Neutron (actually huge parts in Nova/Placement also): QoS min pps rule | |
| 15:04:34 | lajoskatona | The client patch is here: https://review.opendev.org/c/openstack/python-openstackclient/+/810559 which needs the SDK one which is merged to featrue/r1 (https://review.opendev.org/c/openstack/openstacksdk/+/810364 ) | |
| 15:05:33 | lajoskatona | is there a chance that we will have a release of things hanging on feature/r1 in Yoga timeframe, or better to cherry-pick to master ? | |
| 15:21:01 | frickler | lajoskatona: that is already merged to master, see the "included in" link in gerrit. but I don't know when this will be released, gtema? | |
| 15:21:43 | gtema | well, master is now not in the state to be released | |
| 15:22:12 | gtema | I fear it may take another month or so to polish what r1 broke (and what other projects depend on) | |
| 15:22:36 | gtema | but there is still target to make r1 till mid spring | |
| 15:27:28 | lajoskatona | frickler, gtema: ok thanks, so it will not be Yoga-3 :-(, Anyway we keep an eye on it, and thanks for the hard work with it | |
| 15:27:59 | gtema | wlcm | |
| 15:38:19 | ralonsoh | frickler, https://3ef969af3b83bcedd744-0f0c0c25612519a3a781fc7a48e209af.ssl.cf1.rackcdn.com/828877/2/check/openstack-tox-bashate/b597d4a/job-output.txt do you know what is this error? | |
| 15:40:31 | frickler | ralonsoh: yes, split the "local var" and "var=..." into two lines. also that's for -qa and not -sdk ;) | |
| 15:40:39 | ralonsoh | frickler, ahhhhh | |
| 15:40:41 | ralonsoh | thanks a lot | |
| 18:28:23 | opendevreview | Merged openstack/osc-lib master: parseractions: Make key validation reusable https://review.opendev.org/c/openstack/osc-lib/+/784110 | |
| #openstack-sdks - 2022-02-13 | |||
| 12:09:58 | opendevreview | Merged openstack/ansible-collections-openstack master: Adds use_name variable https://review.opendev.org/c/openstack/ansible-collections-openstack/+/817150 | |
| #openstack-sdks - 2022-02-14 | |||
| 09:29:50 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: WIP fix collection for new SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | |
| 16:13:02 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add CentOS 9 tripleo job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827066 | |
| 16:13:03 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test sa 9 job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829079 | |
| 16:29:52 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add CentOS 9 tripleo job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827066 | |
| 16:30:14 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test sa 9 job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829079 | |
| 16:42:10 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add CentOS 9 tripleo job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827066 | |
| 16:43:26 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test sa 9 job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829079 | |
| #openstack-sdks - 2022-02-15 | |||
| 07:06:30 | opendevreview | Ales Musil proposed openstack/openstacksdk master: network: Fix update of network provider https://review.opendev.org/c/openstack/openstacksdk/+/761933 | |
| 08:28:06 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored and documented floating_ip module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828613 | |
| 09:13:53 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Bumped minimum required OpenStack SDK release to SDK 0.36.0 (Train) https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829198 | |
| 09:51:08 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add CentOS 9 wallaby and master https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829206 | |
| 10:56:44 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Release 1.7.0 version https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829217 | |
| 12:04:14 | opendevreview | Merged openstack/ansible-collections-openstack master: Add CentOS 9 tripleo job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827066 | |
| 12:22:06 | opendevreview | Merged openstack/ansible-collections-openstack master: Add CentOS 9 wallaby and master https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829206 | |
| 12:22:08 | opendevreview | Merged openstack/ansible-collections-openstack master: Release 1.7.0 version https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829217 | |
| 13:46:34 | thomasb06 | gtema: Hello. Would you have something small for me to do? | |
| 13:48:04 | gtema | hi thomasb06. I actually recently found some more files with not fixed docstrings (i guess openstacksdk/{proxy,resource}.py). You may have a look whether there are also other places where we can fix stuff. And now you can propose changes directly to master | |
| 13:49:27 | thomasb06 | gtema: sure, let me have a look. By the way, would you have the link of the docstring webpage after compilation? You gave it to me once but I can find it anymore, sorry | |
| 13:49:58 | gtema | https://docs.openstack.org/openstacksdk/latest/user/index.html#api-documentation | |
| 13:50:14 | thomasb06 | cheers, now it's bookmarked | |
| 14:25:34 | opendevreview | Piotr Parczewski proposed openstack/ansible-collections-openstack master: Fix os_quota docs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829273 | |
| 21:05:47 | fuhrmannb[m] | Hi, do you plan to support asyncio in openstacksdk? | |
| #openstack-sdks - 2022-02-16 | |||
| 01:02:56 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | |
| 01:02:57 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: DNM test identity_user_info changes https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829427 | |
| 07:02:57 | opendevreview | wushiming proposed openstack/openstacksdk master: Updating python testing classifier as per Yoga testing runtime https://review.opendev.org/c/openstack/openstacksdk/+/829483 | |
| 10:21:10 | opendevreview | Anton Sidelnikov proposed openstack/openstacksdk master: Now proxy methods for load balancer such as _find and _delete can use additional parameters. To change base_path, for example. https://review.opendev.org/c/openstack/openstacksdk/+/829514 | |
| 10:27:03 | opendevreview | Anton Sidelnikov proposed openstack/openstacksdk master: load_balancer proxy _find _delete now accept attrs https://review.opendev.org/c/openstack/openstacksdk/+/829514 | |
| 12:34:53 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Run jobs on stable branch https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829530 | |
| 12:47:21 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Fix os_quota docs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829273 | |
| 12:49:00 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Fix os_quota docs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829273 | |
| 13:17:20 | opendevreview | Anton Sidelnikov proposed openstack/openstacksdk master: load_balancer proxy _find _delete now accept attrs https://review.opendev.org/c/openstack/openstacksdk/+/829514 | |
| 13:17:21 | opendevreview | Anton Sidelnikov proposed openstack/openstacksdk master: load_balancer proxy _find _delete now accept attrs https://review.opendev.org/c/openstack/openstacksdk/+/829540 | |
| 14:47:52 | opendevreview | Anton Sidelnikov proposed openstack/openstacksdk master: Change-Id: I6d156bf56e7b5618a1e84724480062d188f12c89 https://review.opendev.org/c/openstack/openstacksdk/+/829551 | |
| 15:03:39 | opendevreview | Anton Sidelnikov proposed openstack/openstacksdk master: load_balancer proxy _find _delete now accept attrs https://review.opendev.org/c/openstack/openstacksdk/+/829514 | |
| 15:08:50 | opendevreview | Merged openstack/ansible-collections-openstack master: Fix os_quota docs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829273 | |
| 16:06:27 | stephenfin | fuhrmannb[m]: It's not something that anyone has scoped. It's doable, but we'd need people to work on it. Would be a nice effort | |
| 16:10:11 | fuhrmannb[m] | Ok, that can be a topic I'm interested to work. I never contribute that deep in OpenStack but I can start to have a look. | |
| 16:13:30 | gtema | I fear amount of work is not going to be small, since you need to enable it everywhere, and SDK already depend on at least keystoneauth (as basic requests handling layer) where it would also require work | |
| 16:56:18 | fuhrmannb[m] | Clearly, all network related dependencies, like keystoneauth1, will require to support async too | |
| #openstack-sdks - 2022-02-17 | |||
| 03:32:39 | headphoneJames | gtema: if anyone has time for a review, please have a look at: https://review.opendev.org/c/openstack/openstacksdk/+/826193 Thanks! | |
| 10:00:09 | opendevreview | Vadym Markov proposed openstack/openstacksdk master: Make "find resource" case insensitive by default https://review.opendev.org/c/openstack/openstacksdk/+/829666 | |
| 10:04:29 | opendevreview | Vadym Markov proposed openstack/openstacksdk master: Make "find resource" case sensitive by default https://review.opendev.org/c/openstack/openstacksdk/+/829666 | |
| 10:08:16 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Run jobs on stable branch https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829530 | |
| 10:08:35 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack stable/1.0.0: Run jobs on stable branch https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829638 | |
| 10:10:55 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack stable/1.0.0: DNM test branch jobs for collection https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829668 | |
| 11:46:24 | frickler | why does "openstack volume list" need a compute endpoint? if I deploy cinder before nova and try to verify it by creating a volume, I get an error because the compute endpoint is missing | |
| 13:05:48 | opendevreview | Merged openstack/openstacksdk master: network: Fix update of network provider https://review.opendev.org/c/openstack/openstacksdk/+/761933 | |
| 13:22:54 | opendevreview | Thobias Trevisan proposed openstack/python-openstackclient master: compute: Add 'Security Groups' for 'server list' https://review.opendev.org/c/openstack/python-openstackclient/+/819368 | |
| 14:54:51 | stephenfin | frickler: I suspect that's a hangover from the days when volumes were a nova thing | |
| 15:04:16 | frickler | stephenfin: actually no, I just looked a bit, it's trying to pre-cache the server list in order to be able to show attachments more efficient. the generation of the cache is wrapped in a try, but not the creation of the client, I think that's an easy patch | |
| 15:09:20 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: volume list: don't fail when there's no compute service https://review.opendev.org/c/openstack/python-openstackclient/+/829725 | |
| 15:21:37 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack stable/1.0.0: Run jobs on stable branch https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829638 | |
| 15:21:58 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack stable/1.0.0: DNM test branch jobs for collection https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829668 | |
| 15:22:13 | isabek_ | stephenfin: Hi! Can I refactor to sdk properties network_fakes in OSC (example [1]) . Is there any task for this? 1) https://github.com/openstack/python-openstackclient/blob/master/openstackclient/tests/unit/network/v2/fakes.py#L116 | |
| 15:43:47 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack stable/1.0.0: Run jobs on stable branch https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829638 | |
| 15:43:56 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack stable/1.0.0: DNM test branch jobs for collection https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829668 | |
| 17:41:57 | stephenfin | isabek_: Go for it | |
| 17:42:25 | stephenfin | though of course, if you update the fakes you need to update the code that the fakes is testing to use the SDK :) | |
| #openstack-sdks - 2022-02-18 | |||
| 08:45:38 | thomasb06 | gtema: Hello. The file openstack/proxy.py is ready. To push it I need to `checkout master`, `pull origin master`, and then copy my file and commit it? | |
| 08:46:08 | gtema | yes, should be | |
| 08:46:18 | thomasb06 | let's go | |
| 09:14:44 | opendevreview | Thomas Bucaioni proposed openstack/openstacksdk master: Indentation of the docstrings https://review.opendev.org/c/openstack/openstacksdk/+/829862 | |
| 09:15:01 | thomasb06 | ah, it worked... | |
| 09:15:28 | thomasb06 | now I check in the UI, sometimes I miss some details | |
| 09:18:48 | thomasb06 | gtema: the other file is ressources? | |
| 09:19:36 | thomasb06 | more precisely, openstack/ressource.py | |