Posted	Nick	Remark
#openstack-nova - 2017-05-01
17:47:49	cdent	but could very well have add more
17:48:53	openstackgerrit	Kaitlin Farr proposed openstack/nova master: Remove deprecated keymgr code https://review.openstack.org/439855
18:05:00	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-assisted-volume-snapshots endpoint https://review.openstack.org/461470
18:05:34	mriedem	cdent: yup, one tiny issue
18:05:43	cdent	yay!
18:06:12	cdent	nice catch
18:08:56	openstackgerrit	Chris Dent proposed openstack/nova master: devref and reno for nova-{api,metadata}-wsgi scripts https://review.openstack.org/461289
18:16:55	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add uuid to service.update notification payload https://review.openstack.org/461473
18:24:17	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-availability-zone endpoint instead of stevedore https://review.openstack.org/461475
18:29:05	openstackgerrit	Dan Smith proposed openstack/nova master: WIP: De-complicate some of the instance delete path https://review.openstack.org/458636
18:29:06	openstackgerrit	Dan Smith proposed openstack/nova master: Move to proper target_cell calling convention https://review.openstack.org/458634
18:29:06	openstackgerrit	Dan Smith proposed openstack/nova master: Make target_cell() yield a new context https://review.openstack.org/458635
18:29:07	openstackgerrit	Dan Smith proposed openstack/nova master: DNM: Canary test against multi-cell devstack https://review.openstack.org/458537
18:29:33	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add uuid to service.update notification payload https://review.openstack.org/461473
18:36:44	openstackgerrit	Mathieu Gagné proposed openstack/nova master: Add ability to signal and perform online volume size change https://review.openstack.org/454322
18:38:46	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-fixed-ips endpoint instead of stevedore https://review.openstack.org/461479
18:53:21	jaypipes	I think dansmith is going to add some vomit to his snot when he sees the SQL in my next patch.
18:56:33	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-baremetal-nodes endpoint instead of stevedore https://review.openstack.org/461480
18:59:37	cdent	vomitty snot! \o/
19:01:37	edleafe	that reminds me... lunchtime!
19:04:54	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-availability-zone endpoint instead of stevedore https://review.openstack.org/461475
19:05:14	openstackgerrit	Eric Fried proposed openstack/nova-specs master: Updates for [glance]api_servers and ksa https://review.openstack.org/461481
19:05:18	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-fixed-ips endpoint instead of stevedore https://review.openstack.org/461479
19:05:26	efried	mriedem mordred jaypipes ^^
19:05:39	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-baremetal-nodes endpoint instead of stevedore https://review.openstack.org/461480
19:19:19	dtroyer	I've got a question on the interaction between aggregates and cells… I have a Mitaka DevStack configuration that has n-cell enabled; the hosts from "nova host-list" look like cell!region@vxd-mitaka, which aggregate-add-host does not like. Is this not a valid thing? Or am I missing something here?
19:19:43	jaypipes	efried: gonna be while...
19:20:03	mriedem	dtroyer: cells v1 doesn't support aggregates
19:20:05	efried	jaypipes Don't think there's a hurry. mriedem asked me to update based on [glance]api_servers decision.
19:20:25	dtroyer	mriedem: cool, that's what I needed to know. is there a way from the API to know if I have cells v1 or v2?
19:21:09	mriedem	dtroyer: well, one thing i guess is GET /os-cells would return something for cells v1 if enabled https://developer.openstack.org/api-ref/compute/#cells-os-cells-capacities
19:21:17	mriedem	that's admin-only by default
19:21:19	mriedem	the api i mean
19:21:36	mriedem	but cells v1 runs the separate n-cell service, that's how devstack checks it
19:21:46	mriedem	dtroyer: cells v2 is baked in, not a separate service or API
19:21:55	dtroyer	mriedem: thanks. I'm trying to do graceful failures in OSC in the aggregate commands so admin-only is OK
19:21:58	mriedem	dtroyer: so starting in ocata, you're running cells v2 by design
19:22:00	dtroyer	ah, gotcha
19:22:10	mriedem	even if you're running cells v1, you have cells v2 :)
19:22:14	mriedem	consider your mind blown
19:22:45	mriedem	dtroyer: the other thing would probably be the /os-services API
19:23:08	mriedem	https://github.com/openstack/nova/blob/master/nova/cmd/cells.py#L41
19:23:08	dtroyer	mriedem: one more question… I have a note that nova-net required cells v1, that doesn't seem right to me…
19:23:23	mriedem	if you're running cells, there should be a nova-cells service binary in the GET /os-services response
19:23:35	mriedem	i think anyway
19:23:43	mriedem	dtroyer: other way around
19:23:47	mriedem	well,
19:24:01	mriedem	cells v1 support in-tree requires nova-net,
19:24:17	dtroyer	ok, that explains a couple of things. thanks
19:24:29	mriedem	dtroyer: in ocata we made nova-network only start under a cells v1 config
19:24:32	mriedem	that's probably the confusion
19:24:48	dtroyer	I bet so
19:25:07	mriedem	we basically just continued to disable nova-net in ocata,
19:25:15	mriedem	and switched all jobs to use neutron except for the cells v1 job
19:31:13	dansmith	jaypipes: I don't hate your sql, I admire it.
19:31:18	dansmith	jaypipes: I just note that I shan't be debugging it if something goes wrong
19:31:34	cdent	tshirt
19:39:21	openstackgerrit	Eric Fried proposed openstack/nova-specs master: Updates for [glance]api_servers and ksa https://review.openstack.org/461481
19:53:09	openstackgerrit	Ed Leafe proposed openstack/nova master: Remove the can_host column https://review.openstack.org/461494
19:56:21	melwitt	jaypipes: fyi I pushed an update to the quotas series attempting to address your comments https://review.openstack.org/#/c/446239
19:56:29	jaypipes	melwitt: yup, reviewing it now :)
19:56:37	jaypipes	while waiting for functional tests to complete..
19:57:01	melwitt	jaypipes: cool, thanks :)
20:02:24	ildikov	jaypipes: around for a meeting on #openstack-meeting-3? :)
20:03:03	jaypipes	ildikov: I'm sorry, Ildiko, I'm right in the middle of debugging a nasty little feature and really need to focus on it.
20:03:06	jaypipes	sorry :(
20:03:16	jaypipes	ildikov: will read back from channel later.
20:03:29	ildikov	jaypipes: ok, let's catch up later then
20:04:52	jaypipes	thx ildikov
21:10:04	openstackgerrit	Eric Fried proposed openstack/nova master: nova.utils.get_service_url() https://review.openstack.org/458257
21:16:54	mriedem	oh boy, so my daughter got home and there is a note from the teacher in her backpack about inappropriate behavior,
21:17:09	mriedem	e.g. a worksheet of sight words to fille out, every entry has the word "butt" in it
21:17:34	mriedem	she at least spelled it correctly...
21:23:48	cdent	\o/
21:25:39	dansmith	mriedem: sight words?
21:26:08	mriedem	dansmith: yeah, like simple single-syllable words that they should recognize immediately
21:26:12	openstackgerrit	Dan Smith proposed openstack/nova master: WIP: Debug race in delete-while-booting test https://review.openstack.org/461513
21:26:14	mriedem	they, here, were, are, them
21:26:22	mriedem	butt
21:26:25	dansmith	mriedem: and butt is inappropriate?
21:26:37	dansmith	maybe the teacher should read "everybody poops"
21:26:39	imacdonn	I'd recognise a butt anywhere!
21:26:40	mriedem	well, not only that, but the picture of a butt pooping
21:26:59	dansmith	mriedem: I dunno, I think I'd go argue with the teacher
21:28:33	cdent	tbachman++
21:28:59	dansmith	tbachman: that would be inappropriate (apparently)
21:30:39	edleafe	mriedem: the apple doesn't fall far...
21:32:28	melwitt	wat is this "UnicodeDecodeError \'ascii\' codec can\'t decode byte 0xc3 in position 3575: ordinal not in range(128)" coming from the cells v1 tempest job
21:32:29	mriedem	edleafe: it's true, it reminds me of things i got in trouble for in school
21:32:47	mriedem	melwitt: likely means we aren't using six.text_type somewhere
21:33:04	mriedem	melwitt: link and stacktrace?
21:33:22	mriedem	like, LOG.warning('something %s' % unicode) will do that
21:34:51	melwitt	mriedem: http://logs.openstack.org/39/446239/5/check/gate-tempest-dsvm-cells-ubuntu-xenial/7061c58/logs/screen-n-cell-region.txt.gz?#_May_01_20_22_15_324264
21:35:37	mriedem	hmm https://github.com/openstack/nova/blob/master/nova/cells/messaging.py#L1011
21:35:47	cdent	edleafe: I guess you didn't run all the functional tests before pushing that can_host change. I can no longer trust you!
21:36:00	mriedem	melwitt: i've seen something like that in n-cpu where we log an instance with plugging vifs
21:36:11	mriedem	it dumps the entire instance and there must be something in there that has unicode in it
21:36:23	melwitt	hm, okay. thanks
21:36:26	cdent	melwitt: saw some melwitt I've just started seeing that too
21:36:31	mriedem	melwitt: https://bugs.launchpad.net/nova/+bug/1580728

Earlier   Later