| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-02-10 | |||
| 10:17:33 | frickler | tweining__: to me that would sound like bugs, can you share an example? | |
| 10:17:52 | tweining__ | frickler: https://storyboard.openstack.org/#!/story/2009808 | |
| 10:20:07 | opendevreview | Alex Hussey proposed openstack/ansible-collections-openstack master: Adds use_name variable https://review.opendev.org/c/openstack/ansible-collections-openstack/+/817150 | |
| 10:22:39 | opendevreview | Alex Hussey proposed openstack/ansible-collections-openstack master: Adds use_name variable https://review.opendev.org/c/openstack/ansible-collections-openstack/+/817150 | |
| 10:28:03 | frickler | tweining__: hmm, in that case it looks like a documentation error indeed, according to what the tests do, you would have to use find_load_balancer() if you have a name | |
| 10:28:43 | tweining__ | thanks, I'm just updating the docstrings. | |
| 10:31:50 | opendevreview | Alex Hussey proposed openstack/ansible-collections-openstack master: Adds use_name variable https://review.opendev.org/c/openstack/ansible-collections-openstack/+/817150 | |
| 10:43:58 | opendevreview | Tom Weininger proposed openstack/openstacksdk master: Correct documentation about Load Balancer API https://review.opendev.org/c/openstack/openstacksdk/+/828689 | |
| 10:44:52 | opendevreview | Alex Hussey proposed openstack/ansible-collections-openstack master: Adds use_name variable https://review.opendev.org/c/openstack/ansible-collections-openstack/+/817150 | |
| 11:40:54 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update compute service info module to use proxy layer https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828698 | |
| 11:46:48 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Move compute flavor info to use proxy layer https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828108 | |
| 13:12:02 | opendevreview | Dmitriy Rabotyagov proposed openstack/python-openstackclient master: Implement server migrate commands https://review.opendev.org/c/openstack/python-openstackclient/+/828708 | |
| 13:12:50 | opendevreview | Dmitriy Rabotyagov proposed openstack/python-openstackclient master: Implement server and host evacuate commands https://review.opendev.org/c/openstack/python-openstackclient/+/828708 | |
| 13:40:30 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test flavor changes https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828290 | |
| 13:51:37 | 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 | |
| 14:00:30 | 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 | |
| 14:56:31 | Adri2000 | hello, has anyone tried to resize (extend) an *attached* Cinder volume using the SDK? | |
| 14:57:33 | Adri2000 | according to https://docs.openstack.org/api-ref/block-storage/v3/?expanded=extend-a-volume-size-detail#extend-a-volume-size it requires API microversion 3.42 which doesn't seem to be specified: https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/block_storage/v3/volume.py#L112-L115 | |
| 14:59:05 | Adri2000 | I can file a bug for this, and would be happy to make an attempt at a patch, but am not familiar with the code. would it be easy from the code to check the status of the volume? e.g. if (volume is attached) then utils.require_microversion(session, "3.42") | |
| 16:44:04 | opendevreview | Merged openstack/ansible-collections-openstack master: Remove project properties tests and support https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828682 | |
| 16:48:22 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer identity_user https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828742 | |
| 16:52:24 | 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 | |
| 16:58:55 | 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 | |
| 17:02:20 | 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 | |
| 17:50:41 | 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 | |
| 18:28:16 | 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 | |
| 18:33:26 | 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 | |
| 18:36:33 | headphoneJames | gtema: if you have a chance, please let me know if I'm missing any other parts for: https://review.opendev.org/c/openstack/openstacksdk/+/826193 | |
| 18:36:40 | headphoneJames | Thanks! | |
| 18:41:29 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Allow retrieval of migration by UUID https://review.opendev.org/c/openstack/python-openstackclient/+/828785 | |
| 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 | |