| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2021-07-22 | |||
| 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: Add pagination helpers https://review.opendev.org/c/openstack/python-openstackclient/+/802481 | |
| 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: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 | |
| 12:35:17 | opendevreview | Uemit Seren proposed openstack/ansible-collections-openstack master: Add subnet pool module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/800463 | |
| 14:14:02 | 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 | |
| 14:16:58 | opendevreview | Merged openstack/ansible-collections-openstack master: Add automatic release job triggered by tag https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802379 | |
| 15:17:41 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Change the pipeline of releasing to tag pipeline https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802741 | |
| 15:28:26 | 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 | |
| 16:55:57 | opendevreview | Merged openstack/ansible-collections-openstack master: Wait for pool to be active and online https://review.opendev.org/c/openstack/ansible-collections-openstack/+/801286 | |
| 18:10:44 | opendevreview | Merged openstack/python-openstackclient master: Correct REST API response fields for /os-migrations API https://review.opendev.org/c/openstack/python-openstackclient/+/802650 | |
| 20:44:00 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Change the pipeline of releasing to tag pipeline https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802741 | |
| 23:14:41 | opendevreview | Merged openstack/ansible-collections-openstack master: Change the pipeline of releasing to tag pipeline https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802741 | |
| #openstack-sdks - 2021-07-29 | |||
| 07:19:04 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Allow to attach multiple floating ips to a server https://review.opendev.org/c/openstack/ansible-collections-openstack/+/770120 | |
| 07:21:13 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Remove semver check from release job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802831 | |
| 09:32:53 | opendevreview | Merged openstack/ansible-collections-openstack master: Remove semver check from release job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802831 | |
| 10:32:56 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add mandatory requires_ansible version to metadata https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802859 | |
| 11:52:49 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add mandatory requires_ansible version to metadata https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802859 | |
| 12:12:12 | valery_tschopp | Keystone group question: how can I list all members of a group with the openstacksdk ? | |
| 12:12:47 | valery_tschopp | like GET /groups/{group_id}/users ? | |
| 12:33:15 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add mandatory requires_ansible version to metadata https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802859 | |
| 12:49:12 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add mandatory requires_ansible version to metadata https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802859 | |
| 13:25:57 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add mandatory requires_ansible version to metadata https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802859 | |
| 13:45:43 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add mandatory requires_ansible version to metadata https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802859 | |
| 13:54:42 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add mandatory requires_ansible version to metadata https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802859 | |
| 14:05:20 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add mandatory requires_ansible version to metadata https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802859 | |
| 17:21:25 | gouthamr | diablo_rojo_phone stephenfin dtantsur: o/ we need gtema's gate-blocker fix from the master branch applied to the feature/r1 branch as well - https://review.opendev.org/c/openstack/openstacksdk/+/802930/ - could you please review? | |
| 18:19:32 | gouthamr | gtema: you're supposed to be on vacation | |
| 18:19:39 | gouthamr | :P thanks anyway! | |
| 22:18:06 | opendevreview | Merged openstack/ansible-collections-openstack master: Add mandatory requires_ansible version to metadata https://review.opendev.org/c/openstack/ansible-collections-openstack/+/802859 | |
| #openstack-sdks - 2021-07-30 | |||
| 00:15:54 | opendevreview | Merged openstack/osc-lib master: Correct the tox option for skipping sdist generation https://review.opendev.org/c/openstack/osc-lib/+/796910 | |
| 08:49:58 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Only add or remove router interfaces when needed https://review.opendev.org/c/openstack/ansible-collections-openstack/+/770125 | |
| 14:36:42 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Allow to attach multiple floating ips to a server https://review.opendev.org/c/openstack/ansible-collections-openstack/+/770120 | |
| 14:37:31 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Allow to attach multiple floating ips to a server https://review.opendev.org/c/openstack/ansible-collections-openstack/+/770120 | |
| 16:06:45 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Reenable ansible 2.11 linters job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/803057 | |
| 19:55:42 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Allow to attach multiple floating ips to a server https://review.opendev.org/c/openstack/ansible-collections-openstack/+/770120 | |
| #openstack-sdks - 2021-07-31 | |||
| 07:02:47 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Run asnible devel sanity tests on py38 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/803102 | |
| 07:44:13 | opendevreview | Merged openstack/ansible-collections-openstack master: Reenable ansible 2.11 linters job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/803057 | |