Posted	Nick	Remark
#openstack-sdks - 2019-01-18
18:13:00	Shrews	so creepy_owlet was dtantsur|afk ??
18:13:13	Shrews	names are hard
18:14:45	mordred	Shrews: yah. it's casual nick friday
21:43:42	openstackgerrit	Merged openstack/openstacksdk master: Add baremetal Node fields from versions 1.47 - 1.49 https://review.openstack.org/631841
#openstack-sdks - 2019-01-19
13:17:16	openstackgerrit	Merged openstack/openstacksdk master: Return retries on HTTP CONFLICT to baremetal.attach_vif_to_node https://review.openstack.org/631836
#openstack-sdks - 2019-01-21
09:45:07	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: WIP Add DNS support https://review.openstack.org/623009
#openstack-sdks - 2019-01-22
01:57:27	openstackgerrit	caoyuan proposed openstack/os-client-config master: Update hacking version to latest https://review.openstack.org/628597
07:31:19	openstackgerrit	zhouxinyong proposed openstack/api-sig master: Change openstack-dev to openstack-discuss in setup.cfg https://review.openstack.org/632255
08:26:07	openstackgerrit	zhouxinyong proposed openstack/cliff master: Change openstack-dev to openstack-discuss in setup.cfg https://review.openstack.org/632295
09:00:14	openstackgerrit	Dmitry Tantsur proposed openstack/openstacksdk master: Fixed incorrect exception raising in configdrive generation https://review.openstack.org/632326
09:20:01	openstackgerrit	Dmitry Tantsur proposed openstack/openstacksdk master: Fixed incorrect exception raising in configdrive generation https://review.openstack.org/632326
11:16:06	openstackgerrit	Jens Harbott (frickler) proposed openstack/openstacksdk master: Update cirros version for functional tests https://review.openstack.org/632415
11:16:59	frickler	mordred: Shrews: ^^ we need to match devstack again
11:27:01	dtantsur	ugh
11:35:27	openstackgerrit	zhouxinyong proposed openstack/oaktree master: Change openstack-dev to openstack-discuss in setup.cfg https://review.openstack.org/632424
11:38:38	openstackgerrit	zhouxinyong proposed openstack/oaktreemodel master: Change openstack-dev to openstack-discuss in setup.cfg https://review.openstack.org/632426
11:52:56	openstackgerrit	zhouxinyong proposed openstack/openstackclient master: Change openstack-dev to openstack-discuss in setup.cfg https://review.openstack.org/632439
12:16:04	openstackgerrit	zhouxinyong proposed openstack/osc-lib master: Fix author-email in setup.cfg https://review.openstack.org/632450
12:40:10	mordred	frickler: so I have an idea
12:40:44	mordred	frickler: what if, in devstack, we uploaded the cirros image as "cirros" and not "cirros-0.4.0-x86_64-disk"
12:48:15	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Set DEFAULT_IMAGE_NAME in localrc https://review.openstack.org/632456
12:48:34	mordred	frickler, Shrews, dtantsur: ^^ maybe like that (I +3d the image name patch)
12:48:41	dtantsur	mordred: cirros-x86_64-disk and cirros-x86_64-uec (there's at least two image types)
12:49:01	frickler	mordred: that doesn't sound too far fetched. might break some other stuff, but I like the idea
12:49:14	dtantsur	although for this one job it does not matter
12:49:21	dtantsur	(I thought you're going to change devstack itself)
12:49:35	frickler	ah, that's what I was assuming, too
12:50:06	dtantsur	mordred, frickler, please watch out for the ironic job if it inherits from this job
12:51:22	mordred	dtantsur, frickler: yeah - I think updating devstack would be lovely - but figured that might take a little bit of time :)
12:51:33	mordred	and good to know re: disk and uec
12:56:36	dtantsur	I suspect only ironic cares about uec..
12:57:24	mordred	ZOMG. I have now read the devstack code related to this
12:59:05	mordred	dtantsur: -uec seems to only be added to IMAGE_URLS if the virt driver is xenserver
12:59:33	dtantsur	mordred: ironic has a separate devstack plugin, I guess we add it there
12:59:43	mordred	nod
12:59:48	mordred	well - my patch above isn't going to work
12:59:58	dtantsur	mordred: oh, fun https://github.com/openstack/ironic/blob/39bf7fa86f3c1d5cd53e27e42d616b40aa739023/devstack/common_settings#L36-L46
13:00:19	mordred	DEFAULT_IMAGE_NAME is not used to set the image name for glance
13:00:37	mordred	the image name in glance is derived from the filename of the image from its url
13:00:51	mordred	the DEFAULT_IMAGE_NAME just tells other things what the image name is
13:05:44	dtantsur	mordred: well, there's always this: https://github.com/openstack/metalsmith/blob/master/playbooks/integration/cirros-image.yaml#L8 :D
13:07:16	mordred	dtantsur: yeah :)
14:12:51	openstackgerrit	Merged openstack/openstacksdk master: Update cirros version for functional tests https://review.openstack.org/632415
16:08:57	openstackgerrit	Dmitry Tantsur proposed openstack/openstacksdk master: Add a non-voting job with metalsmith https://review.openstack.org/632496
16:08:58	openstackgerrit	Dmitry Tantsur proposed openstack/openstacksdk master: [DNM] Testing metalsmith CI https://review.openstack.org/632497
16:10:43	openstackgerrit	Dmitry Tantsur proposed openstack/openstacksdk master: [DNM] Testing metalsmith CI https://review.openstack.org/632497
16:10:43	openstackgerrit	Dmitry Tantsur proposed openstack/openstacksdk master: Add a non-voting job with metalsmith https://review.openstack.org/632496
18:02:25	openstackgerrit	Merged openstack/os-service-types master: Updated from OpenStack Service Type Authority https://review.openstack.org/631850
21:32:12	slaweq	mordred: hi
21:32:48	slaweq	mordred: do You remember, some time ago I reported bug that using osprofiler with openstackclient is not possible for neutron
21:33:08	slaweq	mordred: it's in fact bug in openstacksdk IMO, I reported it in https://storyboard.openstack.org/#!/story/2004618 - please check my comment there
21:33:24	slaweq	mordred: maybe You would have any idea how to fix that :)
21:35:12	mordred	slaweq: I shall investigate!
21:35:19	slaweq	mordred: thx
21:36:13	mordred	slaweq: ok - so - there is a set of patches that will make that TaskManager code go away
21:36:45	mordred	slaweq: it'll still probably be a couple of weeks - we need to land a patch in keystoneauth first and kmalloc has been rudely "taking vacation"
21:36:51	mordred	slaweq: but - I hope to get that finished up soon
21:37:00	mordred	and at that point, I expect everything will just magically work :)
21:37:11	slaweq	mordred: that would be great :)
21:37:17	slaweq	mordred: I can wait for it
21:37:26	mordred	let me get you links to the changes ...
21:38:10	mordred	slaweq: https://review.openstack.org/#/c/605043/ is the ksa and https://review.openstack.org/#/c/604926/ is the main sdk patch
21:39:20	slaweq	mordred: thx, I will take a look and try it on my local devstack vm when I will have few minutes :)
21:41:10	mordred	slaweq: all of that free time we have right? :)
21:41:44	slaweq	mordred: yep :)
21:48:09	kmalloc	mordred: hah
21:48:13	kmalloc	mordred: vacation... :P
21:48:29	kmalloc	mordred: do we have a ksa patch or we need to create one?
21:57:54	mordred	kmalloc: https://review.openstack.org/#/c/605043/
21:58:14	mordred	kmalloc: I think we're still wanting a functional test something something
21:59:54	kmalloc	ah yeah
21:59:57	kmalloc	now i remember
22:00:08	kmalloc	it's the whole i need to take vacation and check out for a month
22:00:14	kmalloc	things are slowly coming back to me
22:00:28	kmalloc	it's like i haven't written code or even looked at it for quite a while :P
22:03:05	mordred	kmalloc: this is the correct life decision
22:03:47	kmalloc	i actually have internet at home now, yay old house that makes running things like ethernet unfun
22:04:31	kmalloc	now if only my equalizer didn't pickup NPR (i mean, i don't mind listening to radio, but i don't really want to listen to it when i don't want to listen to it)
22:36:35	mordred	kmalloc: bwhahaha
22:55:18	kmalloc	mordred: aha found it ... bad XLR cable must have had failing sheilding
22:55:32	kmalloc	i mean it's still there if i crank the gain to massive, but way better with a diff. xlr
23:47:28	openstackgerrit	Merged openstack/openstacksdk master: Fixed incorrect exception raising in configdrive generation https://review.openstack.org/632326
#openstack-sdks - 2019-01-23
14:45:44	openstackgerrit	Johannes Kulik proposed openstack/openstacksdk master: Support dict of links in pagination detection https://review.openstack.org/632736
15:28:59	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: use overriden base_path in remaining CRUD operations https://review.openstack.org/632757
15:33:40	mordred	kmalloc: https://review.openstack.org/#/c/632736 is to fix a keystone related thing - could you ack?
17:07:08	kmalloc	checking
17:08:39	openstackgerrit	Morgan Fainberg proposed openstack/openstacksdk master: Support dict of links in pagination detection https://review.openstack.org/632736
17:10:36	mordred	kmalloc: thanks!
17:13:31	kmalloc	ooh, thats kinda icky
17:13:50	kmalloc	fixed the pep8 btw.
17:15:05	kmalloc	links is so useless sometimes.
17:15:13	kmalloc	most of the time* especially in keystone
19:29:57	jabadia	Can someone +2 this trivial bug fix ? https://review.openstack.org/#/c/627734/1
19:30:04	jabadia	( Please :) )
19:35:02	tosky	jabadia: I can't grant the +2 permission, but I think that you should replace the line
19:35:03	tosky	Fix for bug: https://bugs.launchpad.net/python-openstackclient/+bug/1674213
19:35:06	openstack	Launchpad bug 1674213 in python-openstackclient "openstack output as json end with shell line" [Wishlist,Triaged] - Assigned to Yossi Ovadia (jabadia)
19:35:07	tosky	with a simpler
19:35:13	tosky	Closes-Bug: 1674213
19:35:28	tosky	see https://wiki.openstack.org/wiki/GitCommitMessages

Earlier   Later