| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-09-29 | |||
| 13:08:41 | gtema | pls have a look whether it can be used in this form from ansible to unify handling there | |
| 13:10:32 | jm1 | gtema: thank you, will have a look :) | |
| 14:00:49 | opendevreview | Taehyun Park proposed openstack/openstacksdk master: Fix get_import_info function in image v2 proxy https://review.opendev.org/c/openstack/openstacksdk/+/858360 | |
| 15:11:23 | opendevreview | Merged openstack/openstacksdk master: Allow to attach a floating ip to a specific fixed address https://review.opendev.org/c/openstack/openstacksdk/+/859672 | |
| 17:13:01 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Lowered maximum OpenStack SDK version to 0.98.999 in inventory plugin https://review.opendev.org/c/openstack/ansible-collections-openstack/+/859884 | |
| 18:44:50 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Ensure openstacksdk compatibility in inventory plugin https://review.opendev.org/c/openstack/ansible-collections-openstack/+/859887 | |
| 18:47:14 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Ensure openstacksdk compatibility in inventory plugin https://review.opendev.org/c/openstack/ansible-collections-openstack/+/859887 | |
| 19:26:33 | opendevreview | Merged openstack/ansible-collections-openstack stable/1.0.0: Lowered maximum OpenStack SDK version to 0.98.999 in inventory plugin https://review.opendev.org/c/openstack/ansible-collections-openstack/+/859884 | |
| 21:08:04 | opendevreview | Rafael Castillo proposed openstack/openstacksdk master: Use /servers/detail endpoint in find_server proxy method https://review.opendev.org/c/openstack/openstacksdk/+/857987 | |
| 23:16:22 | opendevreview | Merged openstack/ansible-collections-openstack stable/1.0.0: Ensure openstacksdk compatibility in inventory plugin https://review.opendev.org/c/openstack/ansible-collections-openstack/+/859887 | |
| #openstack-sdks - 2022-09-30 | |||
| 06:31:15 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Dropped minimum OpenStack SDK version in inventory plugin https://review.opendev.org/c/openstack/ansible-collections-openstack/+/859934 | |
| 06:34:29 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Ensure openstacksdk compatibility in inventory plugin https://review.opendev.org/c/openstack/ansible-collections-openstack/+/859936 | |
| 07:21:29 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Replaced expensive get_server() and fixed issues in server module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/852119 | |
| 07:50:18 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Initialize tests of real clouds https://review.opendev.org/c/openstack/openstacksdk/+/859026 | |
| 08:46:10 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Add SDK logging option for openstack ansible collections. https://review.opendev.org/c/openstack/ansible-collections-openstack/+/859945 | |
| 09:58:03 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: doc: Update cinder command mapping https://review.opendev.org/c/openstack/python-openstackclient/+/859952 | |
| 09:58:04 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: quota: Add support for detailed volume quotas https://review.opendev.org/c/openstack/python-openstackclient/+/859954 | |
| 09:58:04 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: quota: Add 'quota delete' command https://review.opendev.org/c/openstack/python-openstackclient/+/859953 | |
| 09:58:05 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: quota: Add 'quota show --usage' option https://review.opendev.org/c/openstack/python-openstackclient/+/859956 | |
| 09:58:05 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: quota: Simplify logic used to list, show quotas https://review.opendev.org/c/openstack/python-openstackclient/+/859955 | |
| 09:58:06 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: quota: Deprecate 'quota show --class', 'quota set --class' arguments https://review.opendev.org/c/openstack/python-openstackclient/+/859958 | |
| 09:58:06 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: quota: Allow showing project-specific quotas https://review.opendev.org/c/openstack/python-openstackclient/+/859957 | |
| 09:58:07 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: quota: Trivial style fixups https://review.opendev.org/c/openstack/python-openstackclient/+/859959 | |
| 09:58:08 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Remove duplicate FakeImagev1Client https://review.opendev.org/c/openstack/python-openstackclient/+/859961 | |
| 09:58:08 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Move fake clients to top of file https://review.opendev.org/c/openstack/python-openstackclient/+/859960 | |
| 09:58:10 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: WIP: quota: Add 'quota set --default' option https://review.opendev.org/c/openstack/python-openstackclient/+/859963 | |
| 09:58:10 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Remove unnecessary nesting of volume resources https://review.opendev.org/c/openstack/python-openstackclient/+/859962 | |
| 10:02:09 | stephenfin | gtema: You probably need to look at https://review.opendev.org/c/openstack/openstacksdk/+/858086 since you added that code originally | |
| 10:03:45 | gtema | okay, will have a look today | |
| 10:10:10 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Fix pre-commit issues https://review.opendev.org/c/openstack/openstacksdk/+/859964 | |
| 10:10:24 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: compute: Fix '*volume_attachment' proxy methods https://review.opendev.org/c/openstack/openstacksdk/+/821733 | |
| 10:10:25 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: compute: Add functional tests for volume attachments https://review.opendev.org/c/openstack/openstacksdk/+/822314 | |
| 10:12:51 | opendevreview | PB proposed openstack/ansible-collections-openstack master: updated exit message and usage of protocol var https://review.opendev.org/c/openstack/ansible-collections-openstack/+/859965 | |
| 10:32:29 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient stable/yoga: compute: Require image when rebuilding a volume-backed server https://review.opendev.org/c/openstack/python-openstackclient/+/859969 | |
| 10:33:06 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient stable/xena: compute: Require image when rebuilding a volume-backed server https://review.opendev.org/c/openstack/python-openstackclient/+/859970 | |
| 10:34:06 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient stable/wallaby: compute: Require image when rebuilding a volume-backed server https://review.opendev.org/c/openstack/python-openstackclient/+/859971 | |
| 10:53:11 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Speed up standard flavor list command https://review.opendev.org/c/openstack/python-openstackclient/+/852254 | |
| 11:02:13 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Add nova v2.92 support for import keypair mandatory https://review.opendev.org/c/openstack/python-openstackclient/+/851244 | |
| 11:04:45 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Fix server evacuate command https://review.opendev.org/c/openstack/python-openstackclient/+/849990 | |
| 11:09:57 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Drop second argument passed to emit_duplicated_warning() https://review.opendev.org/c/openstack/python-openstackclient/+/850896 | |
| 11:41:32 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: quota: Simplify logic used to list, show quotas https://review.opendev.org/c/openstack/python-openstackclient/+/859955 | |
| 11:41:33 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: quota: Allow showing project-specific quotas https://review.opendev.org/c/openstack/python-openstackclient/+/859957 | |
| 11:41:33 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: quota: Add 'quota show --usage' option https://review.opendev.org/c/openstack/python-openstackclient/+/859956 | |
| 11:41:34 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: quota: Trivial style fixups https://review.opendev.org/c/openstack/python-openstackclient/+/859959 | |
| 11:41:34 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: quota: Deprecate 'quota show --class', 'quota set --class' arguments https://review.opendev.org/c/openstack/python-openstackclient/+/859958 | |
| 11:41:43 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: quota: Deprecate "force" behavior for network quotas https://review.opendev.org/c/openstack/python-openstackclient/+/819627 | |
| 11:41:49 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Move fake clients to top of file https://review.opendev.org/c/openstack/python-openstackclient/+/859960 | |
| 11:41:58 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Remove duplicate FakeImagev1Client https://review.opendev.org/c/openstack/python-openstackclient/+/859961 | |
| 11:42:08 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Remove unnecessary nesting of volume resources https://review.opendev.org/c/openstack/python-openstackclient/+/859962 | |
| 11:42:14 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: WIP: quota: Add 'quota set --default' option https://review.opendev.org/c/openstack/python-openstackclient/+/859963 | |
| 11:59:07 | opendevreview | Merged openstack/openstacksdk master: Drop query parameter 'id' from identity mapping https://review.opendev.org/c/openstack/openstacksdk/+/858522 | |
| 11:59:10 | opendevreview | Merged openstack/python-openstackclient stable/zed: compute: Require image when rebuilding a volume-backed server https://review.opendev.org/c/openstack/python-openstackclient/+/859768 | |
| 11:59:17 | opendevreview | Merged openstack/python-openstackclient stable/zed: compute: Add support for microversion 2.93 https://review.opendev.org/c/openstack/python-openstackclient/+/859769 | |
| 11:59:19 | opendevreview | Merged openstack/python-openstackclient master: image: Add 'image task show' commands https://review.opendev.org/c/openstack/python-openstackclient/+/813436 | |
| 11:59:21 | opendevreview | Merged openstack/python-openstackclient master: image: Add 'image task list' command https://review.opendev.org/c/openstack/python-openstackclient/+/813554 | |
| 11:59:24 | opendevreview | Merged openstack/openstacksdk master: network: Remove duplicate module, update references https://review.opendev.org/c/openstack/openstacksdk/+/858086 | |
| 12:01:17 | opendevreview | Merged openstack/openstacksdk master: network: Rename ikepolicy module https://review.opendev.org/c/openstack/openstacksdk/+/858148 | |
| 12:01:20 | opendevreview | Merged openstack/python-openstackclient master: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/c/openstack/python-openstackclient/+/853842 | |
| 12:01:22 | opendevreview | Merged openstack/python-openstackclient master: Fix missing closing brackets in metavar https://review.opendev.org/c/openstack/python-openstackclient/+/854462 | |
| 12:01:25 | opendevreview | Merged openstack/python-openstackclient master: remove unicode prefix from code https://review.opendev.org/c/openstack/python-openstackclient/+/853114 | |
| 12:10:30 | opendevreview | Merged openstack/python-openstackclient master: network: Add tenant project filter for RBAC list https://review.opendev.org/c/openstack/python-openstackclient/+/834443 | |
| 12:10:37 | opendevreview | Merged openstack/python-openstackclient master: config: Also mask non-prefix config https://review.opendev.org/c/openstack/python-openstackclient/+/811005 | |
| 12:23:15 | opendevreview | Merged openstack/python-openstackclient master: Add trustor and trustee filtering to trusts list https://review.opendev.org/c/openstack/python-openstackclient/+/824721 | |
| 12:23:18 | opendevreview | Merged openstack/python-openstackclient master: Improve help text for network create --external https://review.opendev.org/c/openstack/python-openstackclient/+/858708 | |
| 12:23:20 | opendevreview | Merged openstack/python-openstackclient master: Fix wrong assertion methods https://review.opendev.org/c/openstack/python-openstackclient/+/857046 | |
| 12:31:51 | opendevreview | Merged openstack/python-openstackclient master: Add authorization_ttl for identity providers https://review.opendev.org/c/openstack/python-openstackclient/+/849669 | |
| 12:31:53 | opendevreview | Merged openstack/python-openstackclient master: compute: Only retrieve necessary images https://review.opendev.org/c/openstack/python-openstackclient/+/837616 | |
| 12:31:55 | opendevreview | Merged openstack/python-openstackclient master: docs: Add missing command mappings https://review.opendev.org/c/openstack/python-openstackclient/+/848955 | |
| 13:12:00 | opendevreview | Merged openstack/python-openstackclient master: Speed up standard flavor list command https://review.opendev.org/c/openstack/python-openstackclient/+/852254 | |
| 13:12:07 | opendevreview | Merged openstack/python-openstackclient master: Fix server evacuate command https://review.opendev.org/c/openstack/python-openstackclient/+/849990 | |
| 13:59:48 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Rework network functional tests https://review.opendev.org/c/openstack/openstacksdk/+/859114 | |
| 13:59:48 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Initialize tests of real clouds https://review.opendev.org/c/openstack/openstacksdk/+/859026 | |
| 14:15:32 | opendevreview | Merged openstack/ansible-collections-openstack stable/1.0.0: Add SDK logging option for openstack ansible collections. https://review.opendev.org/c/openstack/ansible-collections-openstack/+/859945 | |
| 17:35:22 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Improve swift headers handling https://review.opendev.org/c/openstack/openstacksdk/+/860007 | |
| 17:50:01 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Whitelist cloud functional tests in acceptance https://review.opendev.org/c/openstack/openstacksdk/+/860009 | |
| #openstack-sdks - 2022-10-01 | |||
| 03:22:10 | opendevreview | Eunyoung Kim proposed openstack/openstacksdk master: image: Add metadef schema resource to v2 api https://review.opendev.org/c/openstack/openstacksdk/+/857200 | |
| 11:39:21 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Improve swift headers handling https://review.opendev.org/c/openstack/openstacksdk/+/860007 | |
| 11:52:42 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Rework network functional tests https://review.opendev.org/c/openstack/openstacksdk/+/859114 | |
| 11:52:43 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Whitelist cloud functional tests in acceptance https://review.opendev.org/c/openstack/openstacksdk/+/860009 | |
| #openstack-sdks - 2022-10-03 | |||
| 09:55:17 | opendevreview | PB proposed openstack/ansible-collections-openstack master: refactored expanding prefixes into a method https://review.opendev.org/c/openstack/ansible-collections-openstack/+/860073 | |
| 10:45:30 | opendevreview | Daniel Garcia Moreno proposed openstack/os-api-ref master: Remove deprecated sphinx-testing dependency https://review.opendev.org/c/openstack/os-api-ref/+/860080 | |
| 12:37:24 | opendevreview | Eunyoung Kim proposed openstack/openstacksdk master: image: Add metadef schema resource to v2 api https://review.opendev.org/c/openstack/openstacksdk/+/857200 | |
| 13:23:53 | opendevreview | Stephen Finucane proposed openstack/cliff master: columns: Useful __str__, __repr__ implementation https://review.opendev.org/c/openstack/cliff/+/858550 | |
| 15:17:50 | opendevreview | Merged openstack/openstacksdk master: image: Add metadef schema resource to v2 api https://review.opendev.org/c/openstack/openstacksdk/+/857200 | |
| 18:17:24 | opendevreview | Merged openstack/openstacksdk master: Fix pre-commit issues https://review.opendev.org/c/openstack/openstacksdk/+/859964 | |
| 18:20:56 | opendevreview | Merged openstack/openstacksdk master: tests: Remove unnecessary service check https://review.opendev.org/c/openstack/openstacksdk/+/856269 | |
| #openstack-sdks - 2022-10-04 | |||
| 09:07:39 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Release 1.10.0 version https://review.opendev.org/c/openstack/ansible-collections-openstack/+/860223 | |
| 09:11:55 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored floating_ip and floating_ip_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828613 | |
| 09:57:49 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Replaced expensive get_server() and fixed issues in server module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/852119 | |
| 12:20:19 | jm1 | gtema: o/ running into yet another issue with add_ips_to_server and i need your input on how to solve this. | |
| 12:20:33 | gtema | ok, I hear | |
| 12:20:42 | jm1 | gtema: suppose you have two private networks, a server which is attached to one of these networks and now you call add_ips_to_server(server) without any other arguments, hence auto_ip=True | |
| 12:21:22 | jm1 | gtema: you also have a router and a public network and neutron | |
| 12:21:44 | jm1 | gtema: i would assume that add_ips_to_server would attach a floating ip to the server | |
| 12:22:40 | jm1 | gtema: but actually it will fail here https://opendev.org/openstack/openstacksdk/src/commit/093e71e5a6050e6f7d47a03b5f03a62b8f60ca60/openstack/cloud/_floating_ip.py#L1129 | |
| 12:22:47 | gtema | jm1 - generally in any way this can work only: the net server is attached to is connected through router with public network | |
| 12:23:19 | jm1 | gtema: sure, the router is attached to public network and the private networks | |
| 12:23:27 | gtema | the place you mention is exactly one of the black holes | |
| 12:23:31 | jm1 | gtema: we also have subnets | |
| 12:23:56 | jm1 | gtema: the issue is, that this line here might return the "wrong" network https://opendev.org/openstack/openstacksdk/src/commit/093e71e5a6050e6f7d47a03b5f03a62b8f60ca60/openstack/cloud/_floating_ip.py#L1109 | |