Posted	Nick	Remark
#openstack-sdks - 2021-01-13
15:00:49	gtema	we passed Ussuri (where Nova dropped support for it)
15:00:56	dtantsur	ahh
15:01:24	dtantsur	so it affects Train and older (potentially)?
15:01:38	gtema	seems to be so
15:01:53	gtema	I am not sure somebody was recently using it in cloud layer at all
15:02:01	gtema	"These will fail with a 404 starting from microversion 2.36. They were removed in the 21.0.0 (Ussuri) release."
15:04:33	gtema	mordred, not sure whether you are here, if yes - ^^
15:06:10	mordred	I'm pretty sure we still use nova-network calls for rackspace
15:06:29	mordred	I mean - it's not actually calls ... but the server network info dict is the nova version iirc
15:06:29	gtema	really? Oh that's sad
15:07:09	mordred	but yeah - I'm not sure who out there might still be running an older cloud, but the theory at least was that things should still work
15:07:11	mordred	THAT SAID
15:07:34	gtema	that's around 1500 loc, that nobody else is really using
15:08:07	mordred	yeah - there's a bunch in here that's essentially in support of rackspace that I kept wanting to come up with a plan for dealing with better
15:08:35	gtema	perhaps we can move it similarly to what I do for my cloud with profile?
15:08:45	mordred	it made more sense when there were two clouds and one of them was rackspace
15:08:53	slaweq	gtema: I'm ok with that, I think nova already removed all of that
15:08:59	mordred	yeah - that's sort of what I was thinking
15:09:08	mordred	make a rackspace plugin essentially
15:09:30	mordred	except - doing the work for that would be tricky - especially because of the cloud-layer fallback logic
15:09:39	gtema	mordred: https://github.com/opentelekomcloud/python-otcextensions/blob/master/otcextensions/sdk/__init__.py that's kind of initialization hack that I do
15:10:03	gtema	when the package is present and in clouds.yaml there is profile: otc - it get's injected and override some things
15:10:05	mordred	it might be more work to engiineer a good plugin layer that can handle the current use cases than to just maintian the existing code
15:10:44	mordred	yeah. trickiest part is not breaking the opendev nodepool while doing the work :)
15:12:02	gtema	at least according to https://graphite.opendev.org/ nova/os-floating-ip seem to be the only used stuff
15:12:17	gtema	networking comes from network
15:12:32	gtema	(whether it all reliable source of truth)
15:12:47	mordred	oh - you're mostly talking about the actual network calls, like list networks, list routers, etc?
15:13:10	gtema	pretty much everything in cloud
15:13:35	mordred	the nova-network related paths in cloud layer that are important are the things in meta - like get_server_external_ipv4 and the things its calls
15:13:51	gtema	disturbing thing is that normally we would invoke network.find_subnet, but currently we mostly go through complete list
15:13:56	mordred	I don't think any of that actually calls nova net endpoints
15:14:47	mordred	yah - we do full lists so that the caching works better and it's not a thundering herd in nodepool
15:15:21	mordred	one thread calling list subnets once and caching it and 1000 threads doing a lookup in the list rather than 1000 threads doing find_subnet
15:16:09	gtema	ugh
15:16:36	mordred	which for some of the thigns (liek servers) it could totally work to just move into the nodepool openstack driver - but for the stuff in meta it's called behind the scenes - so it's hard to defer the batching logic to nodepool
15:16:49	mordred	yeah. sorry - this is the area where the most complex use cases lurk
15:17:10	gtema	which meta exactly you mean?
15:17:18	mordred	basically anything that touches openstack.cloud.meta is a potential nightmare to change
15:17:28	gtema	ah, yes
15:18:36	gtema	okay. Then it is clear that we can't drop it without leaving backup for Rack (nodepool)
15:18:43	mordred	that said - it almost certainly _should_ get reworked - it's carrying too much complexity on its shoulders. but - it's carrying too much complexity on its shoulders so reworking it without major breakages will be "fun"
15:18:45	gtema	makes my night complex
15:19:19	gtema	yeah, I liked US beers
15:23:13	mordred	remember when we could sit down an have a beer in the same room?
15:23:38	gtema	puh, when can this happen again
15:25:41	mordred	you can visit new orleans and we can have beers outside - but then you might not be allowed back in germany
15:25:54	gtema	yupp ;-)
15:26:40	gtema	listing functions in cloud layer already return empty lists for nova case
15:49:38	gtema	I doubt "filters" in calls like get_subnet where name_or_id is always present are making any sense
15:50:14	gtema	am I missing something?
15:51:46	openstackgerrit	Lee Yarwood proposed openstack/openstacksdk stable/train: Fix aggregate functional test for id restriction https://review.opendev.org/c/openstack/openstacksdk/+/767526
15:52:07	mordred	filters can additionally filter - for cases where the only thing that's passed to name_or_id is name?
15:52:21	mordred	(subnet names are not guaranteed to be unique iirc)
15:52:31	gtema	aaah, right
15:53:34	dtantsur	beers with people.. I seem to be vaguely remembering..
15:58:04	openstackgerrit	Stephen Finucane proposed openstack/os-client-config stable/train: Uncap jsonschema https://review.opendev.org/c/openstack/os-client-config/+/770598
17:50:15	stephenfin	dtantsur: If you're still about, could you push this through? gtema approved earlier but there were more issues https://review.opendev.org/c/openstack/os-client-config/+/770598
17:50:25	stephenfin	CI is green now
18:59:18	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: Switch networking function in cloud layer to proxy https://review.opendev.org/c/openstack/openstacksdk/+/766783
19:43:45	openstackgerrit	Merged openstack/os-client-config stable/train: Uncap jsonschema https://review.opendev.org/c/openstack/os-client-config/+/770598
23:38:57	rm_work	anyone here familiar with gophercloud + openstack? would love to snipe someone for just a minute to ask a couple questions about passing mTLS data with connections
23:38:57	rm_work	anyone here familiar with gophercloud + openstack? would love to snipe someone for just a minute to ask a couple questions about passing mTLS data with connections
23:55:25	mordred	rm_work: I know that it exists and people use it
23:56:08	rm_work	:P
#openstack-sdks - 2021-01-14
00:24:49	openstackgerrit	Merged openstack/openstacksdk master: Add query parameters to listing heat stacks https://review.opendev.org/c/openstack/openstacksdk/+/769484
00:24:55	openstackgerrit	Merged openstack/python-openstackclient master: Remove retired Karbor support https://review.opendev.org/c/openstack/python-openstackclient/+/769980
07:44:15	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: Complete compute.hypervisor functions https://review.opendev.org/c/openstack/openstacksdk/+/763202
08:54:52	openstackgerrit	Irina Pereiaslavskaia proposed openstack/ansible-collections-openstack master: New keypairs_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/770760
09:19:21	openstackgerrit	Stephen Finucane proposed openstack/os-client-config stable/stein: Uncap jsonschema https://review.opendev.org/c/openstack/os-client-config/+/770599
09:30:50	openstackgerrit	Irina Pereiaslavskaia proposed openstack/ansible-collections-openstack master: New keypairs_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/770760
10:39:26	openstackgerrit	Irina Pereiaslavskaia proposed openstack/ansible-collections-openstack master: New keypairs_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/770760
10:43:46	openstackgerrit	Irina Pereiaslavskaia proposed openstack/ansible-collections-openstack master: New keypairs_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/770760
12:00:36	openstackgerrit	Stephen Finucane proposed openstack/os-client-config stable/stein: Uncap jsonschema https://review.opendev.org/c/openstack/os-client-config/+/770599
14:13:35	openstackgerrit	Stephen Finucane proposed openstack/os-client-config stable/stein: Uncap jsonschema https://review.opendev.org/c/openstack/os-client-config/+/770599
14:47:20	openstackgerrit	Slawek Kaplonski proposed openstack/openstacksdk master: Allow unknown attributes to be included in request body https://review.opendev.org/c/openstack/openstacksdk/+/768208
17:44:04	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: Switch networking function in cloud layer to proxy https://review.opendev.org/c/openstack/openstacksdk/+/766783
17:45:29	openstackgerrit	Merged openstack/os-client-config stable/stein: Uncap jsonschema https://review.opendev.org/c/openstack/os-client-config/+/770599
17:53:25	openstackgerrit	Anuradha proposed openstack/openstacksdk master: Support Deploy Templates for Ironic API https://review.opendev.org/c/openstack/openstacksdk/+/768612
18:06:36	openstackgerrit	Tosin Farai proposed openstack/ansible-collections-openstack master: Add support for setting node boot device https://review.opendev.org/c/openstack/ansible-collections-openstack/+/770845
20:32:18	openstackgerrit	Tosin Farai proposed openstack/ansible-collections-openstack master: Add support for setting node boot device https://review.opendev.org/c/openstack/ansible-collections-openstack/+/770845
#openstack-sdks - 2021-01-15
04:10:02	openstackgerrit	Merged openstack/python-openstackclient master: compute: Add 'server volume update' command https://review.opendev.org/c/openstack/python-openstackclient/+/765366
08:56:16	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: Complete compute.hypervisor functions https://review.opendev.org/c/openstack/openstacksdk/+/763202
09:29:37	openstackgerrit	wushiming proposed openstack/python-openstackclient master: Drop lower-constraints job https://review.opendev.org/c/openstack/python-openstackclient/+/770915
09:42:00	openstackgerrit	Artem Goncharov proposed openstack/ansible-collections-openstack master: Add stack_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/769516
10:52:37	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: Switch networking function in cloud layer to proxy https://review.opendev.org/c/openstack/openstacksdk/+/766783
11:50:11	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: Apply urllib.parse.quote in unittests to get_mock_url https://review.opendev.org/c/openstack/openstacksdk/+/770929
12:26:12	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: Apply urllib.parse.quote in unittests to get_mock_url https://review.opendev.org/c/openstack/openstacksdk/+/770929
12:32:31	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: Switch networking function in cloud layer to proxy https://review.opendev.org/c/openstack/openstacksdk/+/766783
14:41:26	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: Switch cloud.networking.qos* operations to rely on proxy layer https://review.opendev.org/c/openstack/openstacksdk/+/770945
14:49:24	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: Switch networking function in cloud layer to proxy https://review.opendev.org/c/openstack/openstacksdk/+/766783
15:38:34	openstackgerrit	Shnaidman Sagi proposed openstack/ansible-collections-openstack master: Add stack_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/769516
15:40:40	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: Stop sending tenant_id to Neutron https://review.opendev.org/c/openstack/openstacksdk/+/770952
16:46:16	openstackgerrit	Merged openstack/ansible-collections-openstack master: Fix volume_info result for SDK < 0.19 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/770267
17:35:49	openstackgerrit	Shnaidman Sagi proposed openstack/ansible-collections-openstack master: Remove protocols choice in security rules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/771055
17:44:15	openstackgerrit	Shnaidman Sagi proposed openstack/ansible-collections-openstack master: Add stack_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/769516
20:59:13	openstackgerrit	Shnaidman Sagi proposed openstack/ansible-collections-openstack master: Remove protocols choice in security rules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/771055
21:03:04	openstackgerrit	Shnaidman Sagi proposed openstack/ansible-collections-openstack master: Add stack_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/769516
21:34:09	openstackgerrit	Merged openstack/python-openstackclient master: compute: Add missing options for 'hypervisor list' https://review.opendev.org/c/openstack/python-openstackclient/+/765368
21:34:31	openstackgerrit	Merged openstack/python-openstackclient master: compute: Add '--force' option to 'server delete' https://review.opendev.org/c/openstack/python-openstackclient/+/765369

Earlier   Later