| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-09-30 | |||
| 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: Initialize tests of real clouds https://review.opendev.org/c/openstack/openstacksdk/+/859026 | |
| 13:59:48 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Rework network functional tests https://review.opendev.org/c/openstack/openstacksdk/+/859114 | |
| 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 | |
| 12:24:13 | jm1 | gtema: it might return the one the server is connected to, but it could also return the other one | |
| 12:24:42 | gtema | because historically it searches for this "NAT network" which should be a private net server is connected to and connected through router to public net | |
| 12:25:06 | gtema | do you have logs (or how to reproduce? | |
| 12:25:15 | jm1 | gtema: https://6aef0912e1e7e067f94c-3957ec525977d9a3771da3eb17c87a1d.ssl.cf5.rackcdn.com/852119/18/check/ansible-collections-openstack-functional-devstack/593fd7f/job-output.txt | |
| 12:26:03 | jm1 | gtema: here is the culprit: https://opendev.org/openstack/openstacksdk/src/commit/093e71e5a6050e6f7d47a03b5f03a62b8f60ca60/openstack/cloud/_network_common.py#L180 | |
| 12:26:34 | jm1 | gtema: this will look though all subnets which will be returned even though the server might not be attached to that | |
| 12:26:47 | gtema | which which change it is (so that I reconstruct conditions)? | |
| 12:27:12 | jm1 | gtema: https://review.opendev.org/c/openstack/ansible-collections-openstack/+/852119/18 | |
| 12:27:59 | jm1 | gtema: my approach would be to return all possible nat destinations and let the caller choose what the best is | |
| 12:28:34 | gtema | actually there is clouds.yaml parameter for manual name of such net | |
| 12:29:24 | jm1 | gtema: but _nat_destination_port() can decide what nat destination to use without the user having to help the sdk | |
| 12:30:05 | gtema | I told you this is now all quite historical and not too much scalable, haven't I? | |
| 12:30:05 | jm1 | gtema: _nat_destination_port gets a server and has a list of ports. it just has to check which port is actually on a nat_destination network | |
| 12:30:13 | jm1 | gtema: hmm | |
| 12:30:16 | jm1 | gtema: so ignore? | |
| 12:30:37 | gtema | nope, I want to have a functest for that directly in SDK | |
| 12:31:27 | jm1 | gtema: the use case is pretty straight forward though: one has a server and wants it to have a floating ip. this should not fail just because the user has two networks with a gateway.. | |
| 12:31:30 | gtema | https://review.opendev.org/q/topic:better_tests is a chain where this would need to go | |
| 12:31:56 | gtema | this is clear, jm1. But I would need to create a proper functest for that | |
| 12:32:36 | jm1 | gtema: sure. question is, should i write a patch for this _nat_destination_port or is this going away anyway? | |
| 12:33:14 | gtema | no, I would first want to have a test that reproduces error and with that start reworking of the whole madness | |
| 12:34:55 | jm1 | gtema: ok sooo.. should i write a test? | |
| 12:35:13 | gtema | just describe to me detailed conditions | |
| 12:36:03 | jm1 | gtema: here or in a bug report? | |
| 12:36:29 | gtema | doesn't matter - even an email to me. Important is that I do not loose it | |
| 12:36:47 | gtema | and bugreport is exactly the place to loose everything once you close the tab | |
| 12:44:20 | jm1 | gtema: ok. devstack default setup with public network. 2 private networks with a subnet each. all three networks attached to the router. server attached to one private network. call to add_ips_to_server(server, auto_ip=True, reuse=False). sometimes fails with "No port on server... was found matching your NAT destination network" because _nat_destination_port() returns the other private network, not the one the server has been attached to. | |
| 12:44:29 | jm1 | gtema: plus links above. enough? | |
| 12:46:23 | gtema | oh, so actually all private nets attached to one router? that is a great fact to know | |
| 12:49:56 | jm1 | gtema: yes. although this would not change the bug because it lists all subnets anyway | |
| 12:51:02 | jm1 | gtema: have more than one network with a gateway? boom | |
| 12:51:21 | jm1 | gtema: or rather "potential boom" | |
| 12:52:03 | gtema | where is the connectivity between pub net and router? don't see it in the change | |
| 12:53:10 | jm1 | gtema: https://review.opendev.org/c/openstack/ansible-collections-openstack/+/852119/18/ci/roles/server/tasks/main.yml#37 | |
| 12:53:24 | gtema | ah, overseen | |
| 12:53:47 | gtema | ok, will sketch the test | |
| 12:56:07 | jm1 | gtema: while you are at it.. https://storyboard.openstack.org/#!/story/2010153 ;) | |
| 12:56:40 | jm1 | gtema: he also submitted a patch | |
| 12:57:26 | gtema | ugh, I don't know how to deal with patches into something that is so fragile | |
| 12:59:07 | jm1 | gtema: this patch actually looks good. it does not remove or change functionality, it just adds a missing parameter | |
| 12:59:41 | gtema | which works "magically" | |
| 13:00:06 | gtema | there are no really reasonable tests for this whole "chapter" | |
| 13:00:19 | jm1 | gtema: so you cannot break something with merging this | |
| 13:00:34 | gtema | lol - I can break you even further | |
| 13:02:09 | jm1 | gtema: not with this patch though ;) | |
| 13:02:28 | gtema | sure? | |
| 13:03:11 | jm1 | gtema: not more than what is broken already | |
| 13:03:13 | gtema | it doesn't look bad, I am just too skeptic since there are no real tests here | |
| 13:03:35 | gtema | okay, then I can go with merging it and restructure with it being added | |