Posted	Nick	Remark
#openstack-nova - 2018-06-21
23:58:45	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in test_compute_api.py (4) https://review.openstack.org/568462
23:59:25	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153
23:59:40	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in nova/tests/unit/virt/xenapi/stubs.py https://review.openstack.org/568412
#openstack-nova - 2018-06-22
01:54:14	openstackgerrit	Tetsuro Nakamura proposed openstack/nova master: Add microversion for nested allocation candidate https://review.openstack.org/565487
01:54:15	openstackgerrit	Tetsuro Nakamura proposed openstack/nova master: Fix nits in placement-return-all-resources series https://review.openstack.org/573604
02:02:52	openstackgerrit	Merged openstack/nova master: Clarify placement DB schema migration https://review.openstack.org/576265
02:05:08	openstackgerrit	Tetsuro Nakamura proposed openstack/nova master: Add microversion for nested allocation candidate https://review.openstack.org/565487
02:05:09	openstackgerrit	Tetsuro Nakamura proposed openstack/nova master: Fix nits in placement-return-all-resources series https://review.openstack.org/573604
02:52:34	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameters about trusted certificate IDs https://review.openstack.org/576046
02:52:52	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Use nova.test.TestingException https://review.openstack.org/575012
02:53:32	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Update admin/flavors document https://review.openstack.org/573063
02:54:03	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc https://review.openstack.org/529520
03:08:48	openstackgerrit	Merged openstack/nova master: Remove network info stubbing in functional test https://review.openstack.org/570018
03:08:54	openstackgerrit	Merged openstack/nova master: Cleanup nits in placement database changes https://review.openstack.org/577107
03:10:28	openstackgerrit	Chen proposed openstack/python-novaclient master: Fix help text in server-group-create https://review.openstack.org/577341
03:37:34	CBR09	Hi everyone
03:37:38	CBR09	Is there anybody here
03:37:57	CBR09	I want to ask something about upgrading nova
03:39:37	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Merge server create schema for BDM v2 extension https://review.openstack.org/577185
04:05:31	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in test_conductor.py (1) https://review.openstack.org/564658
04:05:48	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in test_conductor.py (2) https://review.openstack.org/564659
04:12:22	openstackgerrit	jiang wei proposed openstack/nova master: Add action initiator attribute to the update payload https://review.openstack.org/536243
04:30:49	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (5) https://review.openstack.org/570842
04:31:09	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (6) https://review.openstack.org/571330
04:31:25	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7) https://review.openstack.org/571992
04:31:43	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8) https://review.openstack.org/571993
05:07:53	openstackgerrit	Merged openstack/nova master: Fix nits from change I676291ec0faa1dea0bd5050ef8e3426d171de4c6 https://review.openstack.org/576985
05:08:00	openstackgerrit	Merged openstack/nova master: Implement discard for file backed memory https://review.openstack.org/576994
06:14:30	openstackgerrit	Lei Zhang proposed openstack/nova master: Add method to get cpu traits https://review.openstack.org/560317
06:23:30	openstackgerrit	Dinesh Bhor proposed openstack/nova master: PCPU: Define numa dedicated CPU resource class https://review.openstack.org/561770
06:23:31	openstackgerrit	Dinesh Bhor proposed openstack/nova master: NUMACell, InstanceNUMACell: Adopt 'PCPU' changes https://review.openstack.org/576021
06:23:43	openstackgerrit	Merged openstack/nova master: objects: Remove NUMATopologyLimits.obj_from_db_obj https://review.openstack.org/537412
06:42:34	vpc	where can i get ubuntu-dekstop image
07:02:27	openstackgerrit	Gergely Csatari proposed openstack/nova master: Adding oslo.service to test-requirements https://review.openstack.org/577095
07:34:36	openstackgerrit	Lenny Verkhovsky proposed openstack/nova stable/queens: WIP workaround for RHEL7.5 qemu 2.0 https://review.openstack.org/577361
07:36:30	openstackgerrit	Lenny Verkhovsky proposed openstack/nova stable/pike: WIP workaround for RHEL7.5 qemu 2.0 https://review.openstack.org/577362
07:37:31	openstackgerrit	Yang Youseok proposed openstack/nova master: Add data section parameters for mkfs.xfs command https://review.openstack.org/577363
07:38:34	giblet	happy Friday
08:02:19	openstackgerrit	Lei Zhang proposed openstack/nova master: Add method to get cpu traits https://review.openstack.org/560317
08:25:40	openstackgerrit	Lee Yarwood proposed openstack/nova master: libvirt: Log breadcrumb for known encryption bug https://review.openstack.org/577164
08:30:50	openstackgerrit	Dinesh Bhor proposed openstack/nova master: Report PCPU to placement https://review.openstack.org/577038
08:36:20	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Update launch-from-volume doc to latest version. https://review.openstack.org/572241
08:55:25	openstackgerrit	Dinesh Bhor proposed openstack/nova master: Report PCPU to placement https://review.openstack.org/577038
08:55:26	openstackgerrit	Dinesh Bhor proposed openstack/nova master: Virt driver: Build guest xml https://review.openstack.org/577372
09:13:18	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Extract create_allocation_list https://review.openstack.org/577200
09:13:19	openstackgerrit	Chris Dent proposed openstack/nova master: WIP: [placement] Add /reshaper handler for POST https://review.openstack.org/576927
09:15:11	deepak_mourya	@cdent hi, in this bug https://bugs.launchpad.net/nova/+bug/1770636
09:15:12	openstack	Launchpad bug 1770636 in OpenStack Compute (nova) "placement API not translating CannotDeleteParentResourceProvider to 409 Conflict" [Low,Triaged] - Assigned to Deepak Mourya (mourya007)
09:15:51	deepak_mourya	one doubt is that where i can place our new test for this because i couldn't find file related here https://github.com/openstack/nova/tree/master/nova/tests/unit/api/openstack/placement/handlers.
09:16:02	cdent	morning deepak_mourya, yeah, saw your latest comment there, but haven't had a chance to respond. Instead of a unit test, I'd say you want a gabbi test, they are found in nova/tests/functional/api/openstack/placement/gabbits
09:16:29	cdent	if you have some questions on how that works, after taking a look, find me in #openstack-placement
09:16:51	cdent	or if you like, we could work on it together: after you've done what you think is the fix I could add the gabbi test to the same commit
09:18:32	deepak_mourya	ok , let me take a try first i guess for gabbit one thenafter we can
09:18:34	deepak_mourya	:)
09:19:28	cdent	cool, I should be around for the next few hours
09:25:30	openstackgerrit	Lee Yarwood proposed openstack/nova master: Add encrypted volume support to feature matrix docs https://review.openstack.org/570255
09:38:16	openstackgerrit	Lee Yarwood proposed openstack/nova stable/queens: Be graceful about vif plugging in early ironic driver startup https://review.openstack.org/576948
09:39:29	cdent	dansmith, mriedem, efried_pto: wanted to draw your attention to this bug if you hadn't already seen it: https://bugs.launchpad.net/nova/+bug/1777591
09:39:30	openstack	Launchpad bug 1777591 in OpenStack Compute (nova) "‘limit’ in allocation_candidates where sometimes make fore_hosts invalid" [Undecided,In progress] - Assigned to xulei (605423512-j)
09:39:43	cdent	it's a bit unclear but I think there may be some interesting issues to extract from it
09:44:09	openstackgerrit	Dinesh Bhor proposed openstack/nova master: Virt driver: Build guest xml https://review.openstack.org/577372
09:54:01	openstackgerrit	Yang Youseok proposed openstack/nova master: Add data section parameters for mkfs.xfs command https://review.openstack.org/577363
10:17:24	openstackgerrit	Kashyap Chamarthy proposed openstack/nova master: conf: libvirt: Update the outdated `rng_dev_path` docs https://review.openstack.org/577385
10:18:45	openstackgerrit	Kashyap Chamarthy proposed openstack/nova master: conf: libvirt: Update the outdated `rng_dev_path` docs https://review.openstack.org/577385
10:30:13	openstackgerrit	Lee Yarwood proposed openstack/nova master: Not reraise DiskNotFound if instance is resized https://review.openstack.org/571410
10:30:45	lyarwood	mdbooth: just added you to ^ re https://bugs.launchpad.net/nova/+bug/1774249
10:30:46	openstack	Launchpad bug 1774249 in OpenStack Compute (nova) "update_available_resource will raise DiskNotFound after resize but before confirm" [Medium,In progress] - Assigned to Lee Yarwood (lyarwood)
10:34:20	openstackgerrit	yanpuqing proposed openstack/nova master: Rename auth_uri to www_authenticate_uri https://review.openstack.org/576820
10:35:11	mdbooth	lyarwood: ack.
11:41:29	openstackgerrit	Chason Chan proposed openstack/nova master: Fix the duplicated config options of api_database and placement_database https://review.openstack.org/577023
12:05:27	openstackgerrit	Jay Pipes proposed openstack/nova master: Fix bug to api-ref https://review.openstack.org/571375
12:06:52	openstackgerrit	Jay Pipes proposed openstack/nova master: Fix bug to filter_scheduler https://review.openstack.org/571998
12:07:27	jaypipes	tianhui: ^ rebased your patches. they were based on an old commit which was why they were not merging.
12:09:56	openstackgerrit	Surya Seetharaman proposed openstack/nova master: Make nova list and migration-list ignore down cells https://review.openstack.org/575734
13:09:15	cdent	melwitt, or anyone else: are we happy for people to start putting specs in a stein directory yet?
13:27:18	openstackgerrit	Matt Riedemann proposed openstack/nova master: Heal allocations with incomplete consumer information https://review.openstack.org/574488
13:30:56	fried_rice	cdent: Looked at the bug. Seems pretty clear to me. Should we avoid sending limit down when force_hosts is set?
13:31:40	cdent	that seems wrong to me
13:31:58	cdent	or rather: wasteful
13:32:16	cdent	we know which hosts we want in the response...
13:32:26	fried_rice	Alternative would be implementing a "limit allocation candidates to this list of (root?) providers"
13:32:34	fried_rice	Actually...
13:32:53	fried_rice	If we know the specific host, and nested isn't in play, we could construct an allocation request without ever talking to allocation_candidates.
13:33:07	fried_rice	With nested and/or shared, it's a bit harder, cause now we're duplicating logic.
13:33:20	fried_rice	And btw, I'm saying we don't bother checking capacity; if the request bounces, it bounces.
13:35:11	cdent	the point of talking to placement at all in this context, is to make sure there is capacity, so we shouldn't skip that part
13:35:20	cdent	sorry there should be a "?|" on the end of that
13:35:31	cdent	first day with the new keyboard
13:43:31	fried_rice	cdent: The primary goal of talking to placement in this context is to get an allocation. If we already know which host we want to land on and what resources we want, we can just ask for that allocation. We don't need "candidates". If there's not capacity, we'll bounce (hopefully with an error code that lets us know that we bounced due to exceeded capacity as opposed to some other error).
13:44:01	cdent	Hmm. I would have assumed we would want to bounce "sooner"
13:45:05	fried_rice	cdent: I haven't looked lately, but isn't the scheduler flow very tightly
13:45:05	fried_rice	GET /a_c
13:45:05	fried_rice	pick a candidate
13:45:05	fried_rice	PUT /allocations/{u}
13:45:05	fried_rice	?
13:46:29	fried_rice	I'm saying if we know the host, we could in simple cases skip those first two steps. Because we don't need to *ask* about candidates - we can figure it out ourselves.
13:46:44	cdent	fried_rice: no, getting null candidats back _if_ we had a way to say "this host only"
13:47:52	fried_rice	Right, I get that, I'm brainstorming an alternative to a new microversion with yet another query param.
13:47:53	cdent	right, and I'm saying, if (big if) we felt like adding the capability, we could avoid making the spawn call, which I assume is expensive
13:48:01	cdent	microversions are cheap

Earlier   Later