| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-06-27 | |||
| 06:17:04 | frickler | gtema: yes, although that even is with neutron master still. not sure if we could hard drop tenant_id from the API response if project_id is present. otherwise disabling that deprecation might be the right way to do, yes | |
| 06:19:31 | gtema | There's even change open for dropping warnings | |
| 07:02:13 | frickler | maybe make that specific to tenant_id. I'm not sure there is another attribute that is deprecated but still in widespread use. the other option would be to check whether the resource is coming from user input or from the API, but I guess that distinction would be difficult to make | |
| 07:03:02 | gtema | Well, we can add additional (suppress_warning) attribute ;-) | |
| 07:11:56 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Make publish_collection more universal https://review.opendev.org/c/openstack/ansible-collections-openstack/+/847659 | |
| 07:15:39 | frickler | but then everyone will set that and in the end we could have just dropped the warnings right away | |
| 07:15:51 | frickler | gtema: just found that this one was wrong, too https://review.opendev.org/c/openstack/python-openstackclient/+/824746/2/openstackclient/network/v2/security_group.py#b38 | |
| 07:15:55 | gtema | Right | |
| 07:16:12 | frickler | makes tenant_id show up in the rules. I'll do a revert for that line | |
| 07:16:38 | gtema | Yeah, this is what I meant - something explicitly touches attr | |
| 07:21:45 | frickler | oh, wait, it's not only that single line that was wrong, need to revert all the column_map changes in that patch, too. /me sighs | |
| 08:02:21 | frickler | this is really a can of worms, why do floating IPs have a subnet attr? can't find anything about that in the api-ref. maybe it was a nova-network thing? | |
| 08:03:07 | frickler | and I'm also not convinced that copying the floating_ip_address as "name" attribute is a good thing | |
| 08:03:44 | gtema | Right, networking is a pretty dangerous thing | |
| 08:53:46 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Don't show tenant_id in network objects https://review.opendev.org/c/openstack/python-openstackclient/+/847751 | |
| 08:53:47 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: WIP: Test for tenant_id and project_id columns https://review.opendev.org/c/openstack/python-openstackclient/+/847752 | |
| 08:54:40 | opendevreview | ribaudr proposed openstack/python-openstackclient master: Microversion 2.91: Support specifying destination host to unshelve https://review.opendev.org/c/openstack/python-openstackclient/+/831902 | |
| 08:58:43 | gtema | frickler - https://review.opendev.org/c/openstack/openstacksdk/+/816790 was to disable deprecation warning | |
| 10:25:37 | frickler | gtema: yes, I found that, that's what I was referring to earlier. don't disable all warnings, only if attr="tenant_id", then I think we should merge that | |
| 10:27:12 | gtema | I fear this is really more widespread than only tenant_id, but ok, will update the change | |
| 10:39:48 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Disable deprecation warning for tenant_id https://review.opendev.org/c/openstack/openstacksdk/+/816790 | |
| 10:42:33 | gtema | frickler: ^^ | |
| 11:12:54 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: WIP: Test for tenant_id and project_id columns https://review.opendev.org/c/openstack/python-openstackclient/+/847752 | |
| 11:34:27 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored endpoint module and explained region attribute https://review.opendev.org/c/openstack/ansible-collections-openstack/+/847293 | |
| 12:45:35 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: WIP: Test for tenant_id and project_id columns https://review.opendev.org/c/openstack/python-openstackclient/+/847752 | |
| 12:48:51 | opendevreview | Merged openstack/ansible-collections-openstack stable/1.0.0: Make publish_collection more universal https://review.opendev.org/c/openstack/ansible-collections-openstack/+/847659 | |
| 13:11:34 | frickler | it seems that https://review.opendev.org/c/openstack/python-openstackclient/+/828106 is responsible for much of the functional test instability we are seeing | |
| 13:12:10 | frickler | it sets the network quota to 1, leading to failures if any other test runs in parallel and tries to create a network | |
| 13:13:14 | frickler | test_quota_network_set_with_force() in particular. not sure what ralonsoh was thinking there. I'll make a patch to skip that test | |
| 13:25:23 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Skip test_quota_network_set_with_force func test https://review.opendev.org/c/openstack/python-openstackclient/+/847786 | |
| 13:29:23 | frickler | is there any chance we could revert to using launchpad instead of storyboard? trying to find out whether a story already exists for an issue seems impossible to me | |
| 13:33:40 | gtema | I feel also terribly about storyboard. Honestly I want to see the outcome of the general 3rd party projects discussion, cause using gitea issue tracker is at least mentioned there. If we get a chance to get this working I would rather go this way and skip fallback to launchpad now (it has also not really an awesome UX) | |
| 13:56:43 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Moves security_group_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/840660 | |
| 14:05:21 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Moves security_group_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/840660 | |
| 14:11:09 | opendevreview | Jan Hartkopf proposed openstack/python-openstackclient master: add support for microversion 2.91 https://review.opendev.org/c/openstack/python-openstackclient/+/847792 | |
| 14:28:53 | opendevreview | Merged openstack/osc-lib master: Do not output log on bash autocomplete generation https://review.opendev.org/c/openstack/osc-lib/+/841307 | |
| 14:44:36 | slaweq | gtema hi, I have a quick question to You about this neutron-vpnaas support in OSC/SDK - is there any story or something created already so if I would like to help with it, I will not e.g. duplicate work done by someone else already? | |
| 14:45:16 | gtema | we do not have a story, since nobody here is happy about using Storyboard | |
| 14:46:19 | slaweq | so there is no any tracker for that now? | |
| 14:46:26 | gtema | nope | |
| 14:46:35 | slaweq | ok | |
| 14:46:48 | slaweq | I will ping You simply if/when I will have some time to work on this | |
| 14:46:59 | gtema | sure, great | |
| 15:49:40 | opendevreview | Merged openstack/openstacksdk master: Warn when no statsd library available https://review.opendev.org/c/openstack/openstacksdk/+/843855 | |
| 15:54:13 | opendevreview | Merged openstack/openstacksdk master: test: Remove duplicated tests https://review.opendev.org/c/openstack/openstacksdk/+/847577 | |
| 15:54:16 | opendevreview | Merged openstack/openstacksdk master: tests: Add pointers to docs on running tests https://review.opendev.org/c/openstack/openstacksdk/+/847578 | |
| 15:54:18 | opendevreview | Merged openstack/python-openstackclient master: Don't show tenant_id in network objects https://review.opendev.org/c/openstack/python-openstackclient/+/847751 | |
| 15:54:22 | opendevreview | Merged openstack/python-openstackclient master: volume: Correct output of 'volume attachment create' https://review.opendev.org/c/openstack/python-openstackclient/+/841894 | |
| 21:59:40 | opendevreview | Merged openstack/ansible-collections-openstack master: Update project_info module to new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/837276 | |
| #openstack-sdks - 2022-06-28 | |||
| 04:56:39 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Skip test_quota_network_set_with_force func test https://review.opendev.org/c/openstack/python-openstackclient/+/847786 | |
| 08:26:57 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Update image for new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/845304 | |
| 09:51:11 | opendevreview | Merged openstack/python-openstackclient master: Skip test_quota_network_set_with_force func test https://review.opendev.org/c/openstack/python-openstackclient/+/847786 | |
| 11:31:08 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Move keypair module to proxy layer https://review.opendev.org/c/openstack/ansible-collections-openstack/+/847960 | |
| 13:04:24 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Update image for new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/845304 | |
| 13:13:37 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update image for new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/845304 | |
| 13:33:04 | opendevreview | Merged openstack/openstacksdk master: Disable deprecation warning for tenant_id https://review.opendev.org/c/openstack/openstacksdk/+/816790 | |
| 13:39:19 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update image for new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/845304 | |
| 13:59:00 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Update image for new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/845304 | |
| 15:02:18 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update image for new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/845304 | |
| 16:23:11 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update image for new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/845304 | |
| 16:27:52 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update image for new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/845304 | |
| 16:57:10 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update image for new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/845304 | |
| 17:04:06 | 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 | |
| 17:06:16 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update image for new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/845304 | |
| 17:40:43 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update image for new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/845304 | |
| 17:55:39 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update image for new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/845304 | |
| 19:47:42 | opendevreview | Merged openstack/ansible-collections-openstack master: Update image for new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/845304 | |
| #openstack-sdks - 2022-06-29 | |||
| 07:09:18 | opendevreview | Yusuke Niimi proposed openstack/keystoneauth master: Add doc of OAuth2.0 Client Credentials Grant Flow https://review.opendev.org/c/openstack/keystoneauth/+/838104 | |
| 08:10:03 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Update subnet module to be compatible with new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/845183 | |
| 10:28:22 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Move keypair module to proxy layer https://review.opendev.org/c/openstack/ansible-collections-openstack/+/847960 | |
| 10:31:07 | opendevreview | Danila Balagansky proposed openstack/openstacksdk master: WIP: Cron Triggers proxy https://review.opendev.org/c/openstack/openstacksdk/+/848095 | |
| 12:38:10 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Update subnet module to be compatible with new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/845183 | |
| 13:00:28 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Update subnet module to be compatible with new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/845183 | |
| 14:54:11 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: Add qos rule type filtering https://review.opendev.org/c/openstack/python-openstackclient/+/827684 | |
| 15:27:56 | opendevreview | Merged openstack/ansible-collections-openstack master: Update subnet module to be compatible with new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/845183 | |
| 17:28:42 | opendevreview | mbu proposed openstack/python-openstackclient master: Wait for volume being available to set bootable or readonly https://review.opendev.org/c/openstack/python-openstackclient/+/845991 | |
| #openstack-sdks - 2022-06-30 | |||
| 09:03:33 | opendevreview | mbu proposed openstack/python-openstackclient master: Moving osc-functional-devstack and osc-functional-devstack-tips as non voting https://review.opendev.org/c/openstack/python-openstackclient/+/848233 | |
| 09:05:03 | opendevreview | mbu proposed openstack/python-openstackclient master: Moving osc-functional-devstack and osc-functional-devstack-tips as non voting https://review.opendev.org/c/openstack/python-openstackclient/+/848233 | |
| 09:11:12 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Fix: create image from volume command https://review.opendev.org/c/openstack/python-openstackclient/+/844268 | |
| 09:19:41 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Stop enabling network segments https://review.opendev.org/c/openstack/python-openstackclient/+/848237 | |
| 09:36:38 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Use different segments for different tests https://review.opendev.org/c/openstack/python-openstackclient/+/848242 | |
| 10:55:44 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Fix: create image from volume command https://review.opendev.org/c/openstack/python-openstackclient/+/844268 | |
| 10:55:50 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: image: Trivial style changes https://review.opendev.org/c/openstack/python-openstackclient/+/848252 | |
| 10:55:51 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: image: Split image creation depending on service https://review.opendev.org/c/openstack/python-openstackclient/+/848254 | |
| 10:55:51 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: image: Make better use of argparse https://review.opendev.org/c/openstack/python-openstackclient/+/848253 | |
| 11:13:04 | frickler | gtema: are you around? https://review.opendev.org/c/openstack/python-openstackclient/+/848242 looks like it could fix a lot of our unstable func tests | |
| 11:16:14 | gtema | Going on train in few hours, will have a look. Sounds great if that really helps | |
| 12:07:54 | opendevreview | Lajos Katona proposed openstack/python-openstackclient master: Add support for CRUD operations for QoS minimum packet rate rule https://review.opendev.org/c/openstack/python-openstackclient/+/810559 | |
| 12:19:53 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: block storage: Add support for the Group resource https://review.opendev.org/c/openstack/openstacksdk/+/782013 | |
| 12:19:53 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: block storage: Add support for group type specs https://review.opendev.org/c/openstack/openstacksdk/+/847579 | |
| 12:19:54 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: block storage: Add support for the GroupSnapshot resource https://review.opendev.org/c/openstack/openstacksdk/+/781163 | |
| 12:20:01 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: WIP: block storage: Add update_volume proxy method https://review.opendev.org/c/openstack/openstacksdk/+/848264 | |
| 12:20:01 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: docs: Add missing docs for block storage v3 proxy APIs https://review.opendev.org/c/openstack/openstacksdk/+/848263 | |
| 12:22:31 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: docs: Add missing docs for block storage v3 proxy APIs https://review.opendev.org/c/openstack/openstacksdk/+/848263 | |
| 12:22:32 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: block storage: Add support for the Group resource https://review.opendev.org/c/openstack/openstacksdk/+/782013 | |
| 12:22:32 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: block storage: Add support for group type specs https://review.opendev.org/c/openstack/openstacksdk/+/847579 | |
| 12:22:33 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: block storage: Add support for the GroupSnapshot resource https://review.opendev.org/c/openstack/openstacksdk/+/781163 | |
| 12:22:39 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: block storage: Add update_volume proxy method https://review.opendev.org/c/openstack/openstacksdk/+/848264 | |
| 12:34:18 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Moves stack module from cloud to proxy https://review.opendev.org/c/openstack/ansible-collections-openstack/+/848266 | |
| 13:24:53 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Updates security group rule for latest sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/846830 | |