Posted	Nick	Remark
#openstack-nova - 2017-12-04
21:46:13	mikal	Yep, https://bugs.launchpad.net/nova/+bug/1735663
21:46:15	openstack	Launchpad bug 1735663 in OpenStack Compute (nova) "privsep.daemon takes 9 seconds during taking volume attaching operation" [Undecided,New]
21:47:44	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add regression test for rebuilding a volume-backed server https://review.openstack.org/521200
21:47:44	openstackgerrit	Matt Riedemann proposed openstack/nova master: Get original image_id from volume for volume-backed instance rebuild https://review.openstack.org/521391
21:47:45	openstackgerrit	Matt Riedemann proposed openstack/nova master: Fail fast if changing image on a volume-backed server rebuild https://review.openstack.org/520660
21:48:04	mriedem	dansmith: think i addressed your comments in ^
21:48:33	dansmith	slright
22:07:33	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Add aggregate link note in API ref https://review.openstack.org/521502
22:07:57	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for traits API https://review.openstack.org/524094
22:08:20	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Add 'Location' parameters in API ref https://review.openstack.org/521541
22:08:38	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [cellv2] Improve getting BDMs in multiple cells https://review.openstack.org/521400
22:08:57	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Add a warning in 'nova-manage cell_v2 delete_cell' https://review.openstack.org/513771
22:09:39	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for resource class API https://review.openstack.org/524506
22:10:37	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Fix foreign key constraint error https://review.openstack.org/516880
22:11:09	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Fix getting placement request ID https://review.openstack.org/523606
22:11:26	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Fix missing marker functions https://review.openstack.org/514579
22:12:56	openstackgerrit	Hongbin Lu proposed openstack/nova master: [WIP] Handle RPC timeout on get_console_output https://review.openstack.org/525335
22:21:35	efried	jaypipes Been staring at this for a bit, can't figure out where 1.10 is coming from: http://logs.openstack.org/93/385693/83/check/legacy-grenade-dsvm-neutron/b513223/logs/screen-n-cpu.txt.gz#_Dec_04_15_47_11_988730
22:31:31	gryf	efried, looks like you run pike version of placement against latest source
22:31:50	efried	gryf Wow, that would be... bad. How can you tell?
22:32:12	gryf	1.10 was the max microversion in pike
22:33:22	efried	cdent If you're still around -- how does legacy-grenade-dsvm-neutron spin up the placement API? Is there some chance we could be using the pike version in latest master?
22:33:33	efried	cdent And wouldn't lots of other stuff have broken by now if that were the case?
22:33:57	cdent	efried: I assume you’re talking about the thing that’s claiming 1.10 is the max version?
22:34:05	efried	cdent yeah
22:34:12	cdent	was just reading that in my email
22:35:27	cdent	it could be that a new compute is talking to an older placement, which I think is something we _may_ do intentionally in some tests
22:35:48	efried	oy vay, that could be... complicated to fix.
22:36:03	cdent	and for the most part we’ve tried to make the report client be smart about that
22:37:53	cdent	efried: hmmm. yeah, that code doesn’t have a fall back, but I’m not sure we want that or not. jaypipes, mriedem or dansmith will be able to say more on whether we need to be able to work with new compute/old placement
22:38:13	efried	cdent Okay, I gotcha.
22:38:16	cdent	If we don’t need to support that, the other chance is simply that the new placement hasn’t manage to get restarted yet
22:38:19	dansmith	old compute needs to work with new placement
22:38:32	cdent	dansmith: yes, but vice versa?
22:39:28	dansmith	not vice versa
22:39:47	efried	cdent "hasn't managed to get restarted yet" - so actually a recheck may clear it?
22:40:41	cdent	efried: probably, but it may be that the process mgt in grenade is insufficiently robust and could be robustified
22:40:56	efried	BTW, this was just exposed because of https://review.openstack.org/#/c/524263/ -- I think otherwise we would have blown right by this error and not noticed. So it may be a long-standing latent bug.
22:41:23	cdent	yup, I dig
22:41:55	cdent	efried: if you haven’t got time to think about grenade right now (or ever), I’m happy to go digging instead, but wouldn’t be until tomorrow
22:42:01	efried	Well, I guess I'll try a recheck, but it seems like something that ought to be fixed *somewhere*
22:42:10	efried	cdent I've never heard of grenade.
22:42:21	efried	And it's jaypipes' patch anyway :P
22:43:01	cdent	code ownership is, like, _so_ 1990, but yeah, my curiosity is piqued
22:43:16	cdent	grenade is the sort of framework used to manage upgrade testing
22:43:32	cdent	a devstack in the previous version is started, verified a bit
22:44:18	cdent	then new code is installed, but old configruations kept
22:50:07	openstackgerrit	Ed Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver https://review.openstack.org/495854
22:50:07	openstackgerrit	Ed Leafe proposed openstack/nova master: Move the to_dict() method to the Selection object https://review.openstack.org/523492
22:50:07	openstackgerrit	Ed Leafe proposed openstack/nova master: Modify select_destinations() to return objects and alts https://review.openstack.org/510159
22:50:43	edleafe	mriedem: ^^ addressed your comments
22:51:02	edleafe	mriedem: will start on https://review.openstack.org/#/c/516707/ tomorrow
22:52:36	openstackgerrit	Eric Fried proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215
22:52:36	openstackgerrit	Eric Fried proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807
22:52:37	openstackgerrit	Eric Fried proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693
22:52:59	efried	jaypipes ^ this is just a rebase on top of _Provider-private so I can start rebasing the other stuff in that series (e.g. _set_inventory doing nested stuff)
22:59:08	jaypipes	efried: roger
23:24:06	openstackgerrit	Eric Fried proposed openstack/nova master: Scheduler set_inventory_for_provider does nested https://review.openstack.org/520643
23:24:14	efried	jaypipes Like that ^
23:24:37	efried	jaypipes That _validate_provider thing really comes into its own now.
23:24:43	efried	in the tests
23:28:21	efried	jaypipes cdent et al: FYI I'm out tomorrow; working W, Th, and first half of Fr; then (officially) out til Jan. I'll probably be able to check in periodically to shepherd simple rebases/reviews, but not a lot of big stuff.
23:29:11	cdent	efried: roger that. I’m fairly around throughout most of december, happy to mangle your stuff as required during that time
23:29:52	efried	cdent I really hope to get things in a state where that would be remotely possible before I bugger off. Right now most of my patches are kind of a mess.
23:30:07	efried	cdent And thank you for the offer.
23:52:45	openstackgerrit	Hongbin Lu proposed openstack/nova master: Add support for listing hosts in cellv2 https://review.openstack.org/524755
#openstack-nova - 2017-12-05
00:43:00	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Modify _poll_shelved_instances periodic task call _shelve_offload_instance() https://review.openstack.org/524047
00:43:58	Kevin_Zheng	mriedem Hi, could you check https://review.openstack.org/#/c/515631/ again when you got time?
00:47:04	gmann	stephenfin: zuulv3- i did not find any other user for 'tempest-dsvm-neutron-src-oslo.versionedobject', we should move this to nova then, any specific reason for not moving it
00:53:09	mriedem	gmann: not nova,
00:53:13	mriedem	the oslo.versionedobject repo
00:53:35	gmann	mriedem: but they do not run this. its only needed by nova as of now
00:54:12	mriedem	it should be run by oslo.versionedobjects
00:54:16	gmann	but yea as its their lib testing then if they want to own
00:57:00	gmann	mriedem: yea, as of now they do not run, they run only 'legacy-tempest-dsvm-neutron-src'
00:59:53	mriedem	isn't it the same thing? run the dsvm neutron job with src oslo.versionedobjects for the change in question?
00:59:57	mriedem	if the job is a duplicate, just drop it
01:01:41	openstackgerrit	Matt Riedemann proposed openstack/nova master: Mention API behavior change when over quota limit https://review.openstack.org/515631
01:01:47	mriedem	Kevin_Zheng: done
01:02:15	Kevin_Zheng	mriedem thanks, updating
01:02:16	mriedem	alex_xu: ^ want to get that?
01:02:21	mriedem	Kevin_Zheng: i already updated it
01:02:47	Kevin_Zheng	Ah, ok
01:06:42	gmann	mriedem: yea. as oslo run neutron job, it make sure their master src does not break anything. that is enough i think.
01:17:43	openstackgerrit	Merged openstack/nova-specs master: Remove the py27 tox target https://review.openstack.org/520680
01:28:41	mriedem	interesting fact,
01:28:52	mriedem	my kid sings the star spangled banner when in the bathroom now,
01:28:54	mriedem	like all the time
01:37:00	mriedem	edleafe: https://review.openstack.org/#/c/495854/ for your morning
01:37:04	mriedem	missed something in my initial review
01:37:29	mriedem	on the plus side, +2 on https://review.openstack.org/#/c/510159/
01:47:04	openstackgerrit	Matt Riedemann proposed openstack/nova master: Modify _poll_shelved_instances periodic task call _shelve_offload_instance() https://review.openstack.org/524047
01:55:47	openstackgerrit	Matt Riedemann proposed openstack/nova master: Transform instance.resize_revert notification https://review.openstack.org/396811
01:55:55	mriedem	Kevin_Zheng: some more comments in https://review.openstack.org/524047
01:56:37	Kevin_Zheng	mriedem: thx
02:12:16	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Fix an error message in API validation https://review.openstack.org/524088
02:24:35	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Fix wrong argument order in functional test https://review.openstack.org/525363
02:36:31	alex_xu	Kevin_Zheng: should we doc that in the api-ref also?
02:37:00	Kevin_Zheng	alex_xu: which one?
02:37:15	alex_xu	Kevin_Zheng: https://review.openstack.org/#/c/515631/5

Earlier   Later