Posted	Nick	Remark
#openstack-sdks - 2021-02-22
20:18:55	zigo	tosky: +2w
20:19:10	zigo	I'm not sure if mordred would review.
20:40:19	tosky	zigo, mordred: thank you both :)
20:40:34	tosky	now let's hope nothing broke since October, but in case I will fix it
#openstack-sdks - 2021-02-23
11:55:43	openstackgerrit	Tosin Farai proposed openstack/ansible-collections-openstack master: Add support for node traits https://review.opendev.org/c/openstack/ansible-collections-openstack/+/774089
13:01:04	openstackgerrit	Anuradha proposed openstack/openstacksdk master: Improve Ironic API on OpenStack SDK https://review.opendev.org/c/openstack/openstacksdk/+/766531
13:52:45	openstackgerrit	Andrew Karpow proposed openstack/python-openstackclient master: Use network service for Server AddFixedIP https://review.opendev.org/c/openstack/python-openstackclient/+/698749
17:08:46	openstackgerrit	Anuradha proposed openstack/openstacksdk master: Improve Ironic API on OpenStack SDK https://review.opendev.org/c/openstack/openstacksdk/+/766531
18:54:41	openstackgerrit	Anuradha proposed openstack/openstacksdk master: Support Introspection rules for ironic API https://review.opendev.org/c/openstack/openstacksdk/+/775878
22:09:35	mordred	dtantsur: you know lots of things - did we ever write a plugin for ksa or sdk to allow people to use some of the non-keystone auths?
22:10:14	mordred	(I remember it being a weird thing to think about but I feel like one we did to allow for deprecating old non-sdk/osc things)
22:10:16	mordred	corvus: ^^
22:11:55	corvus	(i'm looking at whether i can use a swift aio installation without keystone in an integration test with an app that uses openstacksdk)
22:17:47	mordred	as a followup - python-swiftclient has a keystoneauth plugin for v1auth
22:22:24	mordred	I kinda feel like we shoudl import that in to either sdk or ksa - I could understand why we wouldn't do it in ksa - but we put http basic in there already
22:23:04	mordred	it seems like supporting swauth would be friendly to our users who sometimes might need to interact with a swift that's not using keystone
22:23:17	mordred	but - you know - I'm here like an hour a week, so I'm not going to make too much of a fuss about it P)
22:23:18	mordred	:)
22:41:39	corvus	mordred: that worked, thanks!
22:43:26	mordred	\o/
#openstack-sdks - 2021-02-24
00:00:27	openstackgerrit	Brian Haley proposed openstack/python-openstackclient master: Add --name to port list https://review.opendev.org/c/openstack/python-openstackclient/+/777264
06:16:49	openstackgerrit	sue proposed openstack/openstacksdk master: add masakari enabled to segment https://review.opendev.org/c/openstack/openstacksdk/+/777299
06:26:22	openstackgerrit	Dylan Zapzalka proposed openstack/openstacksdk master: Added support for the Limits resource https://review.opendev.org/c/openstack/openstacksdk/+/777303
06:36:25	openstackgerrit	sue proposed openstack/openstacksdk master: add masakari enabled to segment https://review.opendev.org/c/openstack/openstacksdk/+/777299
07:19:26	openstackgerrit	Dylan Zapzalka proposed openstack/openstacksdk master: Added support for the Limits resource https://review.opendev.org/c/openstack/openstacksdk/+/776318
09:23:25	openstackgerrit	Dr. Jens Harbott proposed openstack/openstacksdk master: Keep creating keystone admin endpoint for heat https://review.opendev.org/c/openstack/openstacksdk/+/777343
10:07:50	dtantsur	mordred: we have http_basic
10:43:58	openstackgerrit	Radosław Piliszek proposed openstack/openstacksdk master: Fix a trivial error in one of the error messages https://review.opendev.org/c/openstack/openstacksdk/+/777354
10:43:58	openstackgerrit	Radosław Piliszek proposed openstack/openstacksdk master: Fix a trivial error in one of the error messages https://review.opendev.org/c/openstack/openstacksdk/+/777354
12:35:25	openstackgerrit	Tosin Farai proposed openstack/ansible-collections-openstack master: Add support for node traits https://review.opendev.org/c/openstack/ansible-collections-openstack/+/774089
12:45:23	openstackgerrit	Tosin Farai proposed openstack/ansible-collections-openstack master: Add support for node traits https://review.opendev.org/c/openstack/ansible-collections-openstack/+/774089
15:44:33	openstackgerrit	James Palmer proposed openstack/openstacksdk master: WIP Add support for Resource Filters https://review.opendev.org/c/openstack/openstacksdk/+/776271
15:50:34	twilit_dragon	stephenfin: Thanks for the code review comments you sent yesterday. I used autopep8 for formatting so hopefully that resolves any pep8 issues.
16:00:33	stephenfin	ack
16:57:01	iurygregory	stephenfin, hey o/ I'm trying to understand your comment https://review.opendev.org/c/openstack/openstacksdk/+/766531/9/openstack/baremetal/v1/conductor.py =)
16:57:25	iurygregory	or the only things that need to change are the docstrings in _proxy.py? =)
17:02:26	iurygregory	I've asked the intern to do the changes in _proxy.py only atm (she will push a new patchset)
17:06:30	openstackgerrit	Anuradha proposed openstack/openstacksdk master: Improve Ironic API on OpenStack SDK https://review.opendev.org/c/openstack/openstacksdk/+/766531
22:22:12	openstackgerrit	Merged openstack/openstacksdk master: Fix a trivial error in one of the error messages https://review.opendev.org/c/openstack/openstacksdk/+/777354
#openstack-sdks - 2021-02-25
02:37:02	openstackgerrit	Ashley Rodriguez proposed openstack/openstacksdk master: Add share resource to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/775707
07:21:51	openstackgerrit	Hudson Dahly proposed openstack/openstacksdk master: Added support for Messages resource https://review.opendev.org/c/openstack/openstacksdk/+/777500
07:37:31	openstackgerrit	Hudson Dahly proposed openstack/openstacksdk master: Added support for Messages resource https://review.opendev.org/c/openstack/openstacksdk/+/777502
07:47:11	openstackgerrit	Dylan Zapzalka proposed openstack/openstacksdk master: Added support for the Limits resource https://review.opendev.org/c/openstack/openstacksdk/+/776318
08:00:57	openstackgerrit	Alexander Wilcox proposed openstack/openstacksdk master: Attempting to pass Zuul tests https://review.opendev.org/c/openstack/openstacksdk/+/777503
11:00:01	openstackgerrit	Radosław Piliszek proposed openstack/openstacksdk stable/victoria: Fix a trivial error in one of the error messages https://review.opendev.org/c/openstack/openstacksdk/+/777540
11:00:01	openstackgerrit	Radosław Piliszek proposed openstack/openstacksdk stable/victoria: Fix a trivial error in one of the error messages https://review.opendev.org/c/openstack/openstacksdk/+/777540
11:52:21	iurygregory	stephenfin, if you have some time today can you re-review https://review.opendev.org/c/openstack/openstacksdk/+/766531 ? ty =)
13:25:24	iurygregory	stephenfin, tyvm!
14:47:32	openstackgerrit	Stephen Finucane proposed openstack/openstacksdk master: Add support for Messages resource https://review.opendev.org/c/openstack/openstacksdk/+/777502
14:50:02	stephenfin	iurygregory: np :)
15:08:11	stephenfin	gtema: Can you look at https://review.opendev.org/c/openstack/openstacksdk/+/776271/4/openstack/block_storage/v3/resource_filter.py#23 ?
15:08:25	gtema	yes
15:08:28	openstackgerrit	Dylan Zapzalka proposed openstack/openstacksdk master: Added support for the Limits resource https://review.opendev.org/c/openstack/openstacksdk/+/776318
15:08:29	stephenfin	I have a question and think you might know the answer :)
15:13:45	gtema	I was sure I missed one patch for reviews. This was the one
15:14:51	gtema	we might hint James to change his gerrit account to be full name or a nick.
15:15:59	stephenfin	good idea
15:16:14	gtema	I actually missed this change exactly because of that
16:00:26	openstackgerrit	Alexander Wilcox proposed openstack/openstacksdk master: Attempting to pass Zuul tests https://review.opendev.org/c/openstack/openstacksdk/+/777503
16:19:40	openstackgerrit	Alexander Wilcox proposed openstack/openstacksdk master: This is the Host Extension Resource, for capstone. https://review.opendev.org/c/openstack/openstacksdk/+/777617
16:30:33	openstackgerrit	Alexander Wilcox proposed openstack/openstacksdk master: This is the Host Extension Resource, for capstone. https://review.opendev.org/c/openstack/openstacksdk/+/776320
22:20:35	openstackgerrit	Merged openstack/openstacksdk master: Improve Ironic API on OpenStack SDK https://review.opendev.org/c/openstack/openstacksdk/+/766531
#openstack-sdks - 2021-02-26
03:47:58	openstackgerrit	Dylan Zapzalka proposed openstack/openstacksdk master: Added support for the Limits resource https://review.opendev.org/c/openstack/openstacksdk/+/776318
10:40:36	openstackgerrit	Simon Li proposed openstack/keystoneauth master: fix: miss attr _discovery_cache in ServiceTokenAuthWrapper https://review.opendev.org/c/openstack/keystoneauth/+/777739
15:13:14	openstackgerrit	Thiago Paiva Brito proposed openstack/openstacksdk master: Add set_readonly_volume to BlockStorageCloudMixin https://review.opendev.org/c/openstack/openstacksdk/+/776266
15:36:08	openstackgerrit	Thiago Paiva Brito proposed openstack/openstacksdk master: Adding retype_volume to BlockStorageCloudMixin https://review.opendev.org/c/openstack/openstacksdk/+/776272
15:43:13	openstackgerrit	Thiago Paiva Brito proposed openstack/openstacksdk master: Add set_readonly_volume to BlockStorageCloudMixin https://review.opendev.org/c/openstack/openstacksdk/+/776266
18:03:55	openstackgerrit	Thiago Paiva Brito proposed openstack/openstacksdk master: Implement metadata operation support for Volumes https://review.opendev.org/c/openstack/openstacksdk/+/777801
18:05:03	openstackgerrit	Thiago Paiva Brito proposed openstack/openstacksdk master: Implement metadata operation support for Volumes https://review.opendev.org/c/openstack/openstacksdk/+/777801
#openstack-sdks - 2021-02-27
23:19:11	openstackgerrit	James Palmer proposed openstack/openstacksdk master: Add support for Resource Filters https://review.opendev.org/c/openstack/openstacksdk/+/776271
#openstack-sdks - 2021-03-01
00:05:36	openstackgerrit	Dylan Zapzalka proposed openstack/openstacksdk master: WIP Added support for the Capabilities resource https://review.opendev.org/c/openstack/openstacksdk/+/777895
04:14:41	openstackgerrit	Dylan Zapzalka proposed openstack/openstacksdk master: WIP Added support for the Capabilities resource https://review.opendev.org/c/openstack/openstacksdk/+/777895
04:15:50	openstackgerrit	Dylan Zapzalka proposed openstack/openstacksdk master: WIP Added support for the Capabilities resource https://review.opendev.org/c/openstack/openstacksdk/+/777895
07:21:00	openstackgerrit	Dylan Zapzalka proposed openstack/openstacksdk master: WIP Added support for the Quota class set extension resource https://review.opendev.org/c/openstack/openstacksdk/+/777934
07:59:14	openstackgerrit	Maxim Babushkin proposed openstack/ansible-collections-openstack master: Add binding profile to port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/777936
08:03:33	openstackgerrit	Maxim Babushkin proposed openstack/ansible-collections-openstack master: Add binding profile to port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/777936
08:52:03	openstackgerrit	Maxim Babushkin proposed openstack/ansible-collections-openstack master: Add binding profile to port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/777936
12:17:03	openstackgerrit	Maxim Babushkin proposed openstack/ansible-collections-openstack master: Add binding profile to port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/777936
13:40:52	yoctozepto	morning
13:44:15	yoctozepto	how could I help/motivate the merging of https://review.opendev.org/c/openstack/openstacksdk/+/777299 ? :-)
13:44:38	yoctozepto	this is important for masakari this cycle
13:44:59	yoctozepto	I truly hope we did not miss any deadlines
15:01:49	frickler	yoctozepto: mentioning it here often enough will help, not sure whom best to ping about it in addition, possibly gtema as ptl
15:03:24	frickler	weird error regarding volumes, if I do "openstack --os-volume-api-version 3.42 volume set --size 3 volume_name", I get "No volume with a name or ID of 'volume_name' exists.". using the volume id works fine.
15:03:58	frickler	using the same command without the special api version also resolves the volume_name correctly, but obviously fails when the volume is in use
15:07:03	yoctozepto	frickler: thanks, will keep bugging then
15:07:51	gtema	yoctozepto, just having a look at this, you might want also to add _max_microversion on the resource to ensure microversion header is send (if it is required)
15:09:56	yoctozepto	gtema: thanks; how does it work though?
15:10:17	gtema	you specify resource attribute (i.e. https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/baremetal/v1/chassis.py#L23)
15:11:23	gtema	it is then used in the underlaying methods (i.e. https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/resource.py#L1218) to be send as a header
15:11:54	gtema	https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/resource.py#L1453
15:25:39	yoctozepto	gtema: ah, ok, ack, thank you; I think I have wrapped my head around it now
15:26:14	yoctozepto	gtema: I have a related question now: is it wrong that python-masakariclient sets the default microversion?
15:26:29	yoctozepto	as that seems to just override any other smartness at the sdk level
15:27:03	gtema	there is nothing right and nothing wrong. Each client is now behaving differently and thus we try to normalize OSC to use SDK to have less of differencies
15:27:24	gtema	if you rely on SDK - you should not do this at all, unless user want that
15:28:02	gtema	that is the "future" OSC behavior (based on current SDK): use highest possible unless user explicitly ask for the lower one
15:28:35	yoctozepto	gtema: ack, that makes sense
15:30:11	yoctozepto	gtema: I am asking from the position of a PTL that got this from the previous team that mysteriously disappeared and I have been involved in devops side more before (vide kolla)
15:30:25	gtema	:)

Earlier   Later