Posted	Nick	Remark
#openstack-sdks - 2020-05-10
23:05:43	rm_work	And worked on the new one until the output was identical
#openstack-sdks - 2020-05-11
00:54:58	rm_work	the way that owner_id/owner patch is done confuses the heck out of me, lol
00:55:21	rm_work	owner alias owner_id + owner_id alias owner
04:50:17	rm_work	mordred: also, again, still waiting on https://review.opendev.org/#/c/725427/ to merge -- are you good with that?
04:52:35	rm_work	and what are merge guidelines on openstacksdk? we need two +2 reviews to workflow stuff?
09:49:13	nightmare_unreal	how does OSC CLI determine which api version to use ? for e.g. use 2.15 or something or 2.60 etc etc.
13:01:30	openstackgerrit	Merged openstack/ansible-collections-openstack master: Ensure modules work with queens devstack https://review.opendev.org/726421
13:27:39	mordred	nightmare_unreal: it currently depends on the user setting an api version
13:28:10	nightmare_unreal	like on openrc/localrc file ?
13:28:12	mordred	nightmare_unreal: we have discussions about changing it to do discovery - but that's probably at least a cycle if not more away
13:28:28	nightmare_unreal	ah okay cool :) Thanks for clarification
13:28:29	mordred	nightmare_unreal: like OS_COMPUTE_API_VERSION= or --os-compute-api-version
13:28:45	nightmare_unreal	understood.
13:38:57	mordred	rm_work: nod. so - looking again, I think there is code in osc to map owner from sdk to owner_id in osc - we still need to bring back owner in sdk - so we might need an osc patch too
13:39:28	mordred	rm_work: my biggest concern is that owner_id isn't a thing in glance, and if it's been showing up in properties that means it is likely it hasn't actually been set properly
14:44:55	frickler	the behaviour of "openstack network agent add/remove router $agent $router" is weird, it is silently a no-op unless one adds the "--l3" option
14:45:22	openstackgerrit	Monty Taylor proposed openstack/ansible-collections-openstack master: Remove all of the os_ prefixes https://review.opendev.org/713056
14:45:23	openstackgerrit	Monty Taylor proposed openstack/ansible-collections-openstack master: Rename all of the modules https://review.opendev.org/726538
15:44:02	openstackgerrit	Monty Taylor proposed openstack/ansible-collections-openstack master: Rename all of the modules https://review.opendev.org/726538
16:02:31	openstackgerrit	Radosław Piliszek proposed openstack/js-openstack-lib master: CI: New jobs https://review.opendev.org/702132
16:02:31	openstackgerrit	Radosław Piliszek proposed openstack/js-openstack-lib master: CI: New jobs https://review.opendev.org/702132
17:27:02	openstackgerrit	Monty Taylor proposed openstack/ansible-collections-openstack master: Rename all of the modules https://review.opendev.org/726538
18:16:16	openstackgerrit	Merged openstack/python-openstackclient master: Correct image lookup during server rebuild https://review.opendev.org/725427
18:16:21	openstackgerrit	Merged openstack/openstacksdk master: Strip self from incoming glance properties https://review.opendev.org/726626
18:17:46	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Rename all of the modules https://review.opendev.org/726538
18:18:42	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Remove all of the os_ prefixes https://review.opendev.org/713056
18:23:52	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Rename all of the modules https://review.opendev.org/726538
19:40:51	donnyd	I am hoping someone can help me understand how application credentials work
19:40:51	donnyd	when I try to use them with openstackclient - I get this error back
19:40:51	donnyd	```Error authenticating with application credential: Application credentials cannot request a scope```
20:29:28	openstackgerrit	Merged openstack/openstacksdk master: Don't error if clouds.yaml is not readable https://review.opendev.org/726416
20:38:50	rm_work	mordred: right that's what my patch did, that I abandoned already
20:39:15	rm_work	Could bring it back, or could not
20:39:37	rm_work	Or I wonder if it's just something that happened in my cloud because of something we set manually?
20:39:54	rm_work	Maybe it's just "fine"
20:47:37	rm_work	https://review.opendev.org/#/c/726562/ <-- this one
20:49:17	rm_work	but, i think it's possible you're correct that it's not actually supposed to be set? like, I actually see it show up in the raw curl response from glance, but that could be due to a bug in some upload script on our side setting that additional property
20:55:58	openstackgerrit	Monty Taylor proposed openstack/ansible-collections-openstack master: Rename all of the modules https://review.opendev.org/726538
20:57:11	mordred	rm_work: yeah - I think you might have a script that thinks it's setting owner in glance and instead is setting a k/v property in the image called owner_id which doesn't do anything
20:57:49	mordred	rm_work: which ... this is clearly a mess - but I want to make it out of the other side making sure we don't have anybody thinking setting owner_id in the image json is going to do anything - because that could, you know, matter
20:58:07	rm_work	ok, so i'm fine with your patch i guess
20:58:18	rm_work	if you think that's looking correct on your side
21:11:08	openstackgerrit	Monty Taylor proposed openstack/ansible-collections-openstack master: Remove all of the os_ prefixes https://review.opendev.org/713056
21:11:09	openstackgerrit	Monty Taylor proposed openstack/ansible-collections-openstack master: Rename all of the modules https://review.opendev.org/726538
21:11:10	openstackgerrit	Monty Taylor proposed openstack/ansible-collections-openstack master: Run linters in a temp dir instead of an install https://review.opendev.org/726952
21:18:24	openstackgerrit	Monty Taylor proposed openstack/ansible-collections-openstack master: Run linters in a temp dir instead of an install https://review.opendev.org/726952
21:18:25	openstackgerrit	Monty Taylor proposed openstack/ansible-collections-openstack master: Remove all of the os_ prefixes https://review.opendev.org/713056
21:18:26	openstackgerrit	Monty Taylor proposed openstack/ansible-collections-openstack master: Rename all of the modules https://review.opendev.org/726538
21:54:10	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Redesign OpenstackModule class https://review.opendev.org/722283
22:05:46	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add OpenstackModule to os_server_action https://review.opendev.org/721746
22:11:13	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Redesign OpenstackModule class https://review.opendev.org/722283
22:21:28	openstackgerrit	Emilien Macchi proposed openstack/openstacksdk stable/ussuri: Don't error if clouds.yaml is not readable https://review.opendev.org/726962
22:21:40	openstackgerrit	Emilien Macchi proposed openstack/openstacksdk stable/train: Don't error if clouds.yaml is not readable https://review.opendev.org/726963
22:27:03	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add OpenstackModule to os_server_action https://review.opendev.org/721746
22:34:23	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Run linters in a temp dir instead of an install https://review.opendev.org/726952
22:51:13	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Run linters in a temp dir instead of an install https://review.opendev.org/726952
23:13:10	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Remove all of the os_ prefixes https://review.opendev.org/713056
23:13:16	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Rename all of the modules https://review.opendev.org/726538
#openstack-sdks - 2020-05-12
07:27:11	frickler	donnyd: I didn't have any issue when I tested app creds some time ago, can you show your command in detail? do you use a clouds.yaml config or environment variables? in the latter case, make sure you don't have too many of them
08:06:29	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: Extend statistics reporting https://review.opendev.org/727097
09:00:38	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: Extend statistics reporting https://review.opendev.org/727097
09:06:50	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Run linters in a temp dir instead of an install https://review.opendev.org/726952
10:13:58	gtema	what has happened to pep? Who made gods angry?
10:46:44	AJaeger	gtema: see #openstack-qa, new release of flake8
10:53:34	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: Extend statistics reporting https://review.opendev.org/727097
12:59:35	donnyd	frickler: turns out I had some env variables set. It was a layer 8 problem
13:18:08	rm_work	ah damn, sdk got hit by the flake8 thing
13:25:16	openstackgerrit	Monty Taylor proposed openstack/ansible-collections-openstack master: Remove all of the os_ prefixes https://review.opendev.org/713056
13:25:21	openstackgerrit	Monty Taylor proposed openstack/ansible-collections-openstack master: Rename all of the modules https://review.opendev.org/726538
13:55:42	openstackgerrit	Monty Taylor proposed openstack/ansible-collections-openstack master: Remove all of the os_ prefixes https://review.opendev.org/713056
13:55:43	openstackgerrit	Monty Taylor proposed openstack/ansible-collections-openstack master: Rename all of the modules https://review.opendev.org/726538
13:55:45	openstackgerrit	Monty Taylor proposed openstack/ansible-collections-openstack master: WIP Use stub modules for deprecated aliases https://review.opendev.org/727197
14:09:53	mordred	gtema, AJaeger : patch on the way to fix the new flake8 issues
14:10:48	AJaeger	mordred: there's a plan to release a new hacking version to cap flake8 - but fixing them is better ;) Thanks!
14:13:21	mordred	AJaeger: yeah - these are all good errors
14:13:50	mordred	like - this is literally the sort of thing flake8 should be checking
14:14:06	AJaeger	then let's fix them ;)
14:15:15	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Fix issues found by latest flake8 https://review.opendev.org/727204
14:15:35	mordred	gtema, mnaser, dtantsur: if you have a second - it's a quick patch but will unbreak the gate here: &&
14:15:41	mordred	I meant ^^ :)
14:15:52	dtantsur	mordred: FYI we're working on releasing a new hacking that caps flake8
14:16:02	mordred	dtantsur: totally - but the errors are all good
14:16:05	mordred	so we should fix them
14:16:23	dtantsur	mordred: well, the nit-picking around "l" variables is so bad that we disabled that check in ironic
14:16:29	mordred	I thnik it's great :)
14:16:59	dtantsur	*shrug* telling people how to name variables in list/dist comprehensions
14:17:06	mordred	yeah - sure
14:17:41	mordred	I mean - I'm not going to argue the case broadly - just saying here it was 2 comprehensions in the hole code base, and I'd be happy if we never used single-letter variables for anything in sdk
14:17:56	mordred	so I don't think it's an issue - but I can understand it being frustrating in other paces
14:18:11	mordred	I care more about the unused format stuff - it actually caught an actual bug
14:18:17	dtantsur	yep, the format check is awesome
14:18:31	dtantsur	there will be a hacking release uncapping it again, so don't worry
14:18:44	mordred	(we were passing an argument to the format that should have been an argument to the surrounding method)
14:18:53	dtantsur	and note that it's not about single-letter variables, it's about "l is easy to confuse with I"
14:19:02	dtantsur	aka "we don't know how to set up fonts" :)
14:19:22	mordred	:)
14:20:00	dtantsur	actually, I'm using the default Fedora fonts and l looks nothing alike I
14:20:02	dtantsur	but dunno
14:20:39	mordred	l and 1 are teh ones I've seen be more confusing on bad fonts
14:20:59	mordred	I use the hack font - so it is not a problem for me

Earlier   Later