Posted	Nick	Remark
#openstack-sdks - 2020-05-11
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
14:21:12	dtantsur	okay, these ones are a bit alike, but not identical (and 1 cannot be a variable name)
14:48:15	gtema	oh, after moving temporarily to another machine I do not get notifications from IRC.
14:53:00	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add template for generation of artibtrary module https://review.opendev.org/727216
15:02:21	noonedeadpunk	hey folks. has a question - was there any recent issue related to nova client and instance filtering by project name?
15:03:35	mordred	gtema: "awesome"
15:04:40	noonedeadpunk	just have a horizon (and openstack) installation that while filtering instances by project name returns all available isntances.
15:05:20	noonedeadpunk	I was told that most likely that panel don't do any filtering itsef but relies on client
15:05:41	noonedeadpunk	so decided to ask if you probably heard about smth like this
15:20:19	openstackgerrit	Monty Taylor proposed openstack/ansible-collections-openstack master: Remove all of the os_ prefixes https://review.opendev.org/713056
15:20:20	openstackgerrit	Monty Taylor proposed openstack/ansible-collections-openstack master: Rename all of the modules https://review.opendev.org/726538
15:20:21	openstackgerrit	Monty Taylor proposed openstack/ansible-collections-openstack master: Remove incorrect os_ prefix in documentation https://review.opendev.org/727226
15:20:55	openstackgerrit	Monty Taylor proposed openstack/ansible-collections-openstack master: Rename all of the modules https://review.opendev.org/726538
15:25:37	mordred	noonedeadpunk: I haven't - but I try to stay as far away from python-*client as I can, so I'm not much help
15:32:07	noonedeadpunk	mordred: ok, I see:) I didn't found anything there as well actualy:( so maybe it's in horizon...
15:35:15	mordred	noonedeadpunk: we should add to our already-way-too-large-todo-list updating horizon to use sdk :)
15:36:03	noonedeadpunk	that would be great... but tough:)
15:49:42	mordred	noonedeadpunk: yeah. tell me about it

Earlier   Later