Posted	Nick	Remark
#openstack-sdks - 2020-06-26
16:16:19	mordred	I'm going to fix that just with a special case ... but it seems like we should perhaps invent a way to say "this is a read-only attribute never attempt to send it" - but also why would we have sent a Header as part of the body? (I'm assuming the uknown attributes code for image is involved here)
16:16:48	gtema_	ugh, ogh
16:17:08	gtema_	tough question at the end of the work week
16:17:46	gtema_	unknown_attributes should have not affected headers, those are only for body
16:18:19	gtema_	this __image__ stuff
16:18:39	gtema_	readonly attributes make totally sense for me
16:24:46	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Upload image via interop import if needed https://review.opendev.org/737608
16:25:03	mordred	gtema_: there is a different approach from listing this particular thing as a Resource attribute
16:25:35	gtema_	ok, might be
16:25:50	mordred	I'll try to think of some sort of readonly something - as well as try to figure out why the header attribute got added to properties
16:27:58	gtema_	good, thks
22:08:34	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Upload image via interop import if needed https://review.opendev.org/737608
22:08:34	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Add support for multiple image stores https://review.opendev.org/737845
22:08:35	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Add use of flake8-bugbear https://review.opendev.org/738041
22:08:35	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Don't use random.SystemRandom in tests https://review.opendev.org/738285
22:08:36	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Reformat with black https://review.opendev.org/738286
22:08:36	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Sort imports with isort https://review.opendev.org/738287
#openstack-sdks - 2020-06-27
10:30:25	openstackgerrit	Lewis Denny proposed openstack/python-openstackclient master: Add API check for server_groups.create https://review.opendev.org/736104
15:07:12	openstackgerrit	Merged openstack/openstacksdk master: Don't use random.SystemRandom in tests https://review.opendev.org/738285
15:08:16	openstackgerrit	Merged openstack/openstacksdk master: Upload image via interop import if needed https://review.opendev.org/737608
15:11:02	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Add support for multiple image stores https://review.opendev.org/737845
15:11:02	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Add use of flake8-bugbear https://review.opendev.org/738041
15:11:03	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Reformat with black https://review.opendev.org/738286
15:11:03	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Sort imports with isort https://review.opendev.org/738287
15:11:04	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Remove enforcer https://review.opendev.org/738316
#openstack-sdks - 2020-06-28
14:46:34	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add volume_info module https://review.opendev.org/735949
22:08:23	openstackgerrit	Lewis Denny proposed openstack/python-openstackclient master: Add API check for server_groups.create https://review.opendev.org/736104
#openstack-sdks - 2020-06-29
08:05:44	dtantsur	mordred: hey, I'd like to request an SDK release with https://review.opendev.org/#/c/735601/
08:05:48	dtantsur	if there are no objections
09:40:17	openstackgerrit	Merged openstack/openstacksdk master: Add support for multiple image stores https://review.opendev.org/737845
09:40:19	openstackgerrit	Merged openstack/openstacksdk master: Remove enforcer https://review.opendev.org/738316
09:47:12	dtantsur	I've rechecked a few more things that should have merged already
09:47:40	dtantsur	will request a release once/if/ they merge
12:33:37	openstackgerrit	Merged openstack/openstacksdk master: Wait for the project cleanup to complete https://review.opendev.org/734016
12:44:34	nightmare_unreal	hello can someone look at this : https://storyboard.openstack.org/#!/story/2007440
12:45:41	nightmare_unreal	I can reproduce the issue but looking at the source , it doesn't look like it's causing issue
12:46:38	nightmare_unreal	https://github.com/openstack/python-openstackclient/blob/master/openstackclient/compute/v2/server.py#L741
13:42:38	openstackgerrit	Merged openstack/openstacksdk master: Fix deleting stacks by id when waiting for result https://review.opendev.org/717423
14:48:59	mordred	dtantsur: wfm!
14:52:47	dtantsur	mordred: do we count dropping 3.5 support as breaking?
14:55:27	mordred	dtantsur: no, I don't think we do
14:55:39	mordred	dtantsur: we've got the python-requires in place properly for the ussuri release
14:55:46	dtantsur	k cool
14:55:49	mordred	so folks on 3.5 should just not get the new release
18:44:33	openstackgerrit	Monty Taylor proposed openstack/python-openstackclient master: Expose flag for forcing use of import for images https://review.opendev.org/737625
19:23:18	sshnaidm|afk	mordred, hi, what can be done to move on with https://review.opendev.org/#/c/730360/ ? Anything I can add there?
20:45:26	dmsimard	would love a review on https://review.opendev.org/#/c/712678/ for fixing "openstack quota list" ignoring the --project argument
21:04:45	mordred	sshnaidm|afk: I'm in and out - but will review them soon - sorry for the delay
21:06:55	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add volume_info module https://review.opendev.org/735949
21:15:33	dmsimard	mordred: <3 thanks
22:22:26	openstackgerrit	Merged openstack/python-openstackclient master: Allow os quota list query to filter by project https://review.opendev.org/712678
#openstack-sdks - 2020-06-30
05:38:32	frickler	mordred: any reason you reverted my updates on 737625? except being distracted by houses? ;)
11:54:10	openstackgerrit	Vishakha Agarwal proposed openstack/openstacksdk master: Add access rule CRUD support https://review.opendev.org/734549
12:01:10	mordred	frickler: nope! total distraction with houses
12:01:18	mordred	frickler: I was trying to add the sdk minimum
12:02:27	openstackgerrit	Monty Taylor proposed openstack/python-openstackclient master: Expose flag for forcing use of import for images https://review.opendev.org/737625
12:02:30	mordred	frickler: &&
12:05:00	frickler	mordred: thx. if you want more joy while waking up, you may want to look into either ze01 things or gitea things
12:17:21	mordred	frickler: I meant ^^ ;) (still coffeeing)
12:18:52	frickler	mordred: yeah, I could parse that, even though those keys are nowhere next to each other on my german keyboard ;)
12:41:25	openstackgerrit	Marc-Antoine Bourgeot proposed openstack/ansible-collections-openstack master: Add openstack logger and Ansible display utility https://review.opendev.org/738595
12:49:52	openstackgerrit	Marc-Antoine Bourgeot proposed openstack/ansible-collections-openstack master: Add openstack logger and Ansible display utility https://review.opendev.org/738595
12:51:19	openstackgerrit	Marc-Antoine Bourgeot proposed openstack/ansible-collections-openstack master: Add openstack logger and Ansible display utility https://review.opendev.org/738595
13:28:16	openstackgerrit	Marc-Antoine Bourgeot proposed openstack/ansible-collections-openstack master: Add openstack logger and Ansible display utility https://review.opendev.org/738595
13:31:20	openstackgerrit	Marc-Antoine Bourgeot proposed openstack/ansible-collections-openstack master: Add openstack logger and Ansible display utility https://review.opendev.org/738595
13:56:17	openstackgerrit	Marc-Antoine Bourgeot proposed openstack/ansible-collections-openstack master: Add openstack logger and Ansible display utility https://review.opendev.org/738595
14:03:18	openstackgerrit	Marc-Antoine Bourgeot proposed openstack/ansible-collections-openstack master: Add openstack logger and Ansible display utility https://review.opendev.org/738595
#openstack-sdks - 2020-07-01
20:52:52	openstackgerrit	Merged openstack/keystoneauth stable/train: Inject /v3 in token path for v3 plugins https://review.opendev.org/731520
#openstack-sdks - 2020-07-02
11:11:24	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Cache auth token in keyring https://review.opendev.org/735352
11:28:21	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: Add support for filters into the project cleanup https://review.opendev.org/735336
11:47:42	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Cache auth token in keyring https://review.opendev.org/735352
15:10:33	openstackgerrit	Vishakha Agarwal proposed openstack/openstacksdk master: Add access rule CRUD support https://review.opendev.org/734549
16:07:49	openstackgerrit	Gonéri Le Bouder proposed openstack/ansible-collections-openstack master: security_group_rule: use OpenStackModule https://review.opendev.org/739086
16:34:26	openstackgerrit	Gonéri Le Bouder proposed openstack/ansible-collections-openstack master: security_group_rule: use OpenStackModule class https://review.opendev.org/739095
16:34:27	openstackgerrit	Gonéri Le Bouder proposed openstack/ansible-collections-openstack master: keypair: make use of OpenStackModule class https://review.opendev.org/739096
18:00:26	openstackgerrit	Gonéri Le Bouder proposed openstack/ansible-collections-openstack master: keypair: make use of OpenStackModule class https://review.opendev.org/739096
18:00:27	openstackgerrit	Gonéri Le Bouder proposed openstack/ansible-collections-openstack master: security_group_rule: use OpenStackModule class https://review.opendev.org/739095
18:52:52	openstackgerrit	Marc-Antoine Bourgeot proposed openstack/ansible-collections-openstack master: Add openstack logger and Ansible display utility https://review.opendev.org/738595
19:23:16	openstackgerrit	Gonéri Le Bouder proposed openstack/ansible-collections-openstack master: security_group_rule: use OpenStackModule class https://review.opendev.org/739095
#openstack-sdks - 2020-07-03
02:22:45	openstackgerrit	Ian Wienand proposed openstack/openstacksdk master: Ignore IPv6 addresses if force_ipv4 is set https://review.opendev.org/739159
06:20:54	frickler	mordred: I have a question on the use of force_ipv4 in ^^
06:23:07	ianw	frickler: what do you mean by the nit for Nit: IPv4/6 ?
06:23:25	openstackgerrit	Marc-Antoine Bourgeot proposed openstack/ansible-collections-openstack master: Add openstack logger and Ansible display utility https://review.opendev.org/738595
06:27:02	frickler	ianw: uppercase the "IP"
06:27:34	ianw	ok, cool
06:28:36	ianw	it looks like it can be set globally in the client section, or per cloud
06:30:08	frickler	ianw: if that is the case, then I'd prefer to make the example per cloud, but I can do that in a followup if you want
06:32:48	frickler	another unrelated note: doc/source/index.html includes README.rst, which links to https://docs.openstack.org/openstacksdk/latest/user/config/configuration.html , which being an absolute link a) looks weird inside the docs and b) breaks the ability to follow to the changed doc in the docs preview
06:34:06	openstackgerrit	Ian Wienand proposed openstack/openstacksdk master: Ignore IPv6 addresses if force_ipv4 is set https://review.opendev.org/739159
06:34:43	ianw	frickler: ^ that documents both -- if it's wrong, happy for -1's :) but we use it per-cloud in system-config so ...
09:03:29	nightmare_unreal	mordred: hey , any update regarding the microversion support in cli , last we discussed this was during PTG meeting.
09:03:29	nightmare_unreal	Thanks
09:04:30	openstackgerrit	Marc-Antoine Bourgeot proposed openstack/ansible-collections-openstack master: Add openstack logger and Ansible display utility https://review.opendev.org/738595
09:26:28	frickler	do we need to make sure that tests fail instead of being skipped when an API disappears like image v1? otherwise how can we claim to still support it? (c.f. https://review.opendev.org/698808)
09:27:28	frickler	similar situation for cinder and swift. do we want jobs where we test master sdk/osc against old stable branches?
12:39:17	mordred	frickler: we do want jobs against old stable - when they're possible
12:39:59	mordred	frickler: we've got a mechanism in the functional tests where we set variables in the job based on what we expect to be present, and a test that will fail if our tests will skip things we don't want them to skip
12:40:54	mordred	frickler: with image v1 we've got pretty good requests-mock unit tests too, and that's currently as good as we've got available to us (there are no older working stable branches with v1)
12:41:47	mordred	this is, incidentally, the reason I think requests-mock unittests are better than the ones that mock out resource objects ... it lets us use real http traces against real clouds to make the test cases
14:05:09	openstackgerrit	Merged openstack/openstacksdk master: Ignore IPv6 addresses if force_ipv4 is set https://review.opendev.org/739159

Earlier   Later