Posted	Nick	Remark
#openstack-sdks - 2021-02-12
16:03:02	openstackgerrit	Merged openstack/openstacksdk master: Drop .json suffix from networking URLs https://review.opendev.org/c/openstack/openstacksdk/+/775200
16:03:33	openstackgerrit	Merged openstack/openstacksdk master: Support Deploy Templates for Ironic API https://review.opendev.org/c/openstack/openstacksdk/+/768612
16:05:01	openstackgerrit	Merged openstack/openstacksdk master: Remove unnecessary string formatting https://review.opendev.org/c/openstack/openstacksdk/+/775377
#openstack-sdks - 2021-02-13
07:40:25	openstackgerrit	Tosin Farai proposed openstack/ansible-collections-openstack master: Add support for node VIFs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/774635
#openstack-sdks - 2021-02-14
02:21:11	openstackgerrit	Manuel Rodriguez proposed openstack/openstacksdk master: Expose description parameter when creating a server https://review.opendev.org/c/openstack/openstacksdk/+/775513
02:27:27	openstackgerrit	Manuel Rodriguez proposed openstack/ansible-collections-openstack master: Allow description field to be set with os_server https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775514
#openstack-sdks - 2021-02-15
10:33:56	openstackgerrit	Tosin Farai proposed openstack/ansible-collections-openstack master: Add functionality to change node provision state https://review.opendev.org/c/openstack/ansible-collections-openstack/+/771430
12:02:50	openstackgerrit	Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: WIP fix CI ansible-test errors https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775604
12:07:45	openstackgerrit	Tosin Farai proposed openstack/ansible-collections-openstack master: Add support for node VIFs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/774635
12:13:32	openstackgerrit	Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: WIP fix CI ansible-test errors https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775604
12:14:17	openstackgerrit	Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: WIP fix CI ansible-test errors https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775604
14:32:34	openstackgerrit	Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Mark 2.9 ansible-test sanity as non-voting https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775604
15:54:46	openstackgerrit	Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Mark 2.9 ansible-test sanity as non-voting https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775604
16:13:24	openstackgerrit	Manuel Rodriguez proposed openstack/openstacksdk master: Expose description parameter when creating a server https://review.opendev.org/c/openstack/openstacksdk/+/775513
18:50:40	openstackgerrit	Sebastian Haderecker proposed openstack/ansible-collections-openstack master: Fix some typos in readme https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775696
19:10:54	openstackgerrit	Sebastian Haderecker proposed openstack/ansible-collections-openstack master: Guidelines: Fix links and formatting https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775700
19:31:57	openstackgerrit	Manuel Rodriguez proposed openstack/ansible-collections-openstack master: Allow description field to be set with os_server https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775514
19:41:40	openstackgerrit	Merged openstack/ansible-collections-openstack master: Mark 2.9 ansible-test sanity as non-voting https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775604
19:47:38	ade_lee	mordred, fungi hi - could I get some +2s on https://review.opendev.org/c/openstack/openstacksdk/+/767411 please
20:17:35	fungi	ade_lee: i went ahead and approved it, stephenfin has +2'd earlier
20:17:54	fungi	i think we've discussed the situation at great length by now, in multiple venues
20:18:10	ade_lee	fungi, great thanks!
20:18:43	fungi	i'm not a usual sdk reviewer, but i was fairly confident in my review of that change
20:19:44	fungi	if others disagree, it's easy enough to revert
20:21:06	fungi	ade_lee: were you going to propose the followup code comment change stephenfin suggested?
20:21:23	ade_lee	fungi, yup - I can do that
20:21:26	fungi	just so we don't forget to rip it out once we only support 3.9 and later
20:21:40	ade_lee	fungi, will submit shortly
20:21:42	fungi	which is years away, so likely easy to forget in that time
20:49:21	openstackgerrit	Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Migrating image_info module from AnsibleModule to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/773687
20:49:39	openstackgerrit	Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Guidelines: Fix links and formatting https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775700
21:09:56	openstackgerrit	Ashley Rodriguez proposed openstack/openstacksdk master: Add resource to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/775707
21:53:29	mordred	ade_lee, fungi also looks good to me - thanks. I'm glad a solution was reached on this one - the original "md5 bad mmkay" was tiring :)
22:21:02	openstackgerrit	Merged openstack/openstacksdk master: encapsulate md5 calls for fips https://review.opendev.org/c/openstack/openstacksdk/+/767411
22:23:27	openstackgerrit	Merged openstack/ansible-collections-openstack master: Fix some typos in readme https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775696
23:20:06	openstackgerrit	Merged openstack/ansible-collections-openstack master: ironic: deprecate sub-options of driver_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/770015
#openstack-sdks - 2021-02-16
00:43:52	openstackgerrit	Merged openstack/ansible-collections-openstack master: Migrating image_info module from AnsibleModule to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/773687
09:41:05	openstackgerrit	Merged openstack/ansible-collections-openstack master: Guidelines: Fix links and formatting https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775700
10:43:54	openstackgerrit	Merged openstack/ansible-collections-openstack master: Allow description field to be set with os_server https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775514
11:47:41	openstackgerrit	Merged openstack/ansible-collections-openstack master: ironic: stop putting meaningless values to properties https://review.opendev.org/c/openstack/ansible-collections-openstack/+/770036
14:08:45	uzumaki	I'm here, dtantsur
14:09:01	dtantsur	uzumaki: do you call anything to configure logging?
14:11:15	uzumaki	openstack.enable_logging(debug=True)
14:11:28	uzumaki	is this for STDOUT logs or file logs? I suppose by default it will target STDOUT
14:11:57	dtantsur	I think so. So yeah, you enabled the logs yourselves, maybe remove it? Or use debug=False?
14:12:10	uzumaki	will try, thanks a lot =) Guten Tag!
14:13:40	dtantsur	:)
16:56:16	openstackgerrit	Tosin Farai proposed openstack/ansible-collections-openstack master: Add functionality to change node provision state https://review.opendev.org/c/openstack/ansible-collections-openstack/+/771430
17:06:11	openstackgerrit	Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: WIP install Octavia plugin for testing LB https://review.opendev.org/c/openstack/ansible-collections-openstack/+/772613
17:22:22	openstackgerrit	Anuradha proposed openstack/openstacksdk master: Support Introspection rules for ironic API https://review.opendev.org/c/openstack/openstacksdk/+/775878
17:26:20	openstackgerrit	Tosin Farai proposed openstack/ansible-collections-openstack master: Add support for node VIFs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/774635
17:34:11	openstackgerrit	Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Release version 1.3.0 of Openstack Collection https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775882
18:04:00	openstackgerrit	Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add Octavia job for testing LB https://review.opendev.org/c/openstack/ansible-collections-openstack/+/772613
19:42:06	openstackgerrit	Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Bump dev version in galaxy.yml https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775920
20:17:12	openstackgerrit	Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add Octavia job for testing LB https://review.opendev.org/c/openstack/ansible-collections-openstack/+/772613
20:45:35	openstackgerrit	Merged openstack/ansible-collections-openstack master: Release version 1.3.0 of Openstack Collection https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775882
21:55:42	openstackgerrit	Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add Octavia job for testing LB https://review.opendev.org/c/openstack/ansible-collections-openstack/+/772613
#openstack-sdks - 2021-02-17
01:51:54	openstackgerrit	Ashley Rodriguez proposed openstack/openstacksdk master: Add resource to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/775707
04:53:03	openstackgerrit	Merged openstack/ansible-collections-openstack master: Bump dev version in galaxy.yml https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775920
04:54:11	openstackgerrit	Adit Sarfaty proposed openstack/python-openstackclient stable/victoria: Remove None valued network quota entries https://review.opendev.org/c/openstack/python-openstackclient/+/775905
04:55:14	openstackgerrit	Adit Sarfaty proposed openstack/python-openstackclient stable/ussuri: Remove None valued network quota entries https://review.opendev.org/c/openstack/python-openstackclient/+/775906
04:55:44	openstackgerrit	Adit Sarfaty proposed openstack/python-openstackclient stable/train: Remove None valued network quota entries https://review.opendev.org/c/openstack/python-openstackclient/+/775907
11:33:44	openstackgerrit	Tosin Farai proposed openstack/ansible-collections-openstack master: Add support for node VIFs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/774635
12:00:01	openstackgerrit	Tosin Farai proposed openstack/ansible-collections-openstack master: Add support for node VIFs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/774635
12:50:17	openstackgerrit	Anuradha proposed openstack/openstacksdk master: Improve Ironic API on OpenStack SDK https://review.opendev.org/c/openstack/openstacksdk/+/766531
13:11:02	tobiash	hi, did someone in the past had the issue that openstacksdk ran onto an infinite paging loop with the same marker when requesting the server list?
13:11:28	gtema	nope
13:11:47	gtema	I think this can happen if server responds wrongly
13:11:52	tobiash	we just lost one nodepool region because of that and resolved it for now by upping the max page side on nova
13:12:32	gtema	do you have some API logs (from SDK side)?
13:14:24	tobiash	not anymore, but I've injected request logging which looked like this: http://paste.openstack.org/show/802733/
13:14:43	tobiash	I guess the links section in the nova response kept a wrong next link then
13:15:07	gtema	most likely. Any updates (SDK, Nova, etc) happened?
13:16:08	tobiash	no, just were the first time >1000 nodes in one tenant
13:16:26	tobiash	I have to say it's a pretty old nova
13:16:29	gtema	okay. Will try to simulate locally
13:16:52	gtema	I know we have lot's of cases with completely corrupted pagination implementation
13:17:07	gtema	I will try to implement some safeguarding for that
13:17:25	tobiash	in a different cloud using the same version it works normally
13:18:03	tobiash	I would guess maybe towards some inconsistent data due to earlier failures (had some rabbitmq issues in the past)
13:18:41	gtema	tja, looks really more a nova issue, than SDK
13:19:52	tobiash	I guess so, thanks for confirming
13:20:17	gtema	wlcm, will anyway have a look whether we can protect from that
13:31:24	tobiash	I think we might have been hit by https://bugs.launchpad.net/nova/+bug/1721791
13:31:26	openstack	Launchpad bug 1721791 in OpenStack Compute (nova) "Pagination of instances works incorrect" [High,Fix released] - Assigned to Dan Smith (danms)
13:32:23	gtema	ok. Any chance to get fix done on the cloud or we really need to have local fix for that?
13:33:07	gtema	at least we can prevent ourselves querying with the same marker and aborting
13:34:20	tobiash	we have a local workaround by config fix and my cloud guys are already in contact with our vendor to check if we can get a backport of the fix (we're still on 16.1.x)
13:34:45	tobiash	so I think we don't need to workaround that in openstacksdk
13:35:24	tobiash	however it was hard to find that issue since we got no exception anywhere, just stuck services
13:35:58	tobiash	so I think an abort with an error when we get repeatedly the same marker might be useful for others to spot such issues easier
13:36:33	gtema	sure
14:12:48	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: Prevent the endless loop in resource listing https://review.opendev.org/c/openstack/openstacksdk/+/776203
14:25:53	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: Set resource URI properties in listing method https://review.opendev.org/c/openstack/openstacksdk/+/776205
15:00:12	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
15:48:28	openstackgerrit	Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add Octavia job for testing LB https://review.opendev.org/c/openstack/ansible-collections-openstack/+/772613
16:51:43	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: Set resource URI properties in listing method https://review.opendev.org/c/openstack/openstacksdk/+/776205
16:52:04	openstackgerrit	Tosin Farai proposed openstack/ansible-collections-openstack master: Add support for node VIFs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/774635
17:08:14	openstackgerrit	Yadnesh Kulkarni proposed openstack/python-openstackclient master: Make instance reset-state an interactive operation https://review.opendev.org/c/openstack/python-openstackclient/+/776242
17:59:13	openstackgerrit	Merged openstack/openstacksdk master: Add ALPN support to load balancer pools https://review.opendev.org/c/openstack/openstacksdk/+/752097

Earlier   Later