Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-09-25
16:38:01 openstackgerrit Merged openstack/nova master: Remove allocations when unshelve fails on host https://review.openstack.org/506458
16:38:21 openstackgerrit Merged openstack/nova master: Add _wait_for_action_fail_completion to InstanceHelperMixin https://review.openstack.org/506660
16:40:31 yufei I remember that, when user use a token to build keystone client, keystone is unable to reauth for 401 error, but nova service use username and password to build keystone session client to call neutron or neutron, keystone will help to reauth in this way
16:41:54 mriedem maybe edmondsw can reply in the patch, he knows more about ksa than i do
16:42:40 edleafe mriedem: do you have a sucke^H^H^H^H volunteer for API-SIG liaison for Nova?
16:43:06 mriedem edleafe: it's either you or cdent
16:43:18 yufei for live-migration or some other actions, user send a request to nova service with token is context, if token expired, action failed.
16:43:29 edleafe mriedem: it can't be
16:43:34 openstackgerrit Matthew Booth proposed openstack/nova master: libvirt: Don't VIR_MIGRATE_NON_SHARED_INC without migrate_disks https://review.openstack.org/507202
16:43:36 yufei thanks, mriedem
16:43:53 edleafe the liaisons are the ones who review changes *after* the cores have approved them
16:43:55 mriedem edleafe: then we just remove that entry, since gilliard isn't working on openstack
16:44:39 edleafe mriedem: ok, I can remove it now, and maybe you can find someone at the next Nova meeting
16:44:47 openstackgerrit Balazs Gibizer proposed openstack/nova master: Moving more utils to ServerResourceAllocationTestBase https://review.openstack.org/499539
16:44:48 openstackgerrit Balazs Gibizer proposed openstack/nova master: factor out compute service start in ServerMovingTest https://review.openstack.org/503037
16:44:48 openstackgerrit Balazs Gibizer proposed openstack/nova master: Test resource allocation during soft delete https://review.openstack.org/495159
16:45:26 yufei @edmondsw could you please help take a look at this patch when you are free? this is a patch about nova call ironic with keystone session, https://review.openstack.org/#/c/502382/
16:47:20 openstackgerrit Balazs Gibizer proposed openstack/nova master: Moving more utils to ServerResourceAllocationTestBase https://review.openstack.org/499539
16:47:21 openstackgerrit Balazs Gibizer proposed openstack/nova master: factor out compute service start in ServerMovingTest https://review.openstack.org/503037
16:47:21 openstackgerrit Balazs Gibizer proposed openstack/nova master: Test resource allocation during soft delete https://review.openstack.org/495159
16:49:01 openstackgerrit Masanori Itoh proposed openstack/nova master: Fix IoOpsFilter test case class name. https://review.openstack.org/507205
16:51:06 openstackgerrit Balazs Gibizer proposed openstack/nova master: use context mgr in instance.delete https://review.openstack.org/443764
16:51:06 openstackgerrit Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297
16:51:07 openstackgerrit Balazs Gibizer proposed openstack/nova master: Send soft_delete from context manager https://review.openstack.org/476459
17:00:01 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add functional for live migrate delete https://review.openstack.org/499583
17:01:58 gibi mriedem: Can I fast approve https://review.openstack.org/#/c/499583? It was already on the gate but got merge conflicts. So I resolved a trivial merge conflict and pushed it back
17:04:32 mriedem yeah
17:06:29 gibi mriedem: thanks. This was my first fast approve so I wanted to make sure :)
17:13:41 jianghuaw jaypipes, the default value 1 for display heads is requested by bauzas to cover the case where the head-counts don't return.
17:13:57 jaypipes jianghuaw: understood, but that's wrong. :)
17:14:38 jaypipes jianghuaw: if the virt driver cannot determine head counts, it simply should not write an inventory record for the VGPU_DISPLAY_HEAD resource class
17:14:44 jianghuaw it's possible if the vGPU doesn't support multiple-heads. By default at least it should support one head?
17:15:30 jianghuaw Then does it mean these vGPU can't be scheduled if it's requesting for one display head?
17:15:43 openstackgerrit Merged openstack/nova master: Copy some tests to a cellsv1 mixin https://review.openstack.org/505442
17:16:19 jaypipes jianghuaw: no, the opposite. by not having an inventory record for VGPU_DISPLAY_HEAD, that means all the pGPUs (or GPU groups) would be schedulable.
17:16:45 cdent jaypipes: in a choice between another process (just for placement-periodics) and cron, I’d definitely choose cron
17:16:59 jaypipes jianghuaw: if the hypervisor cannot determine display heads, then the flavor should not request VGPU_DISPLAY_HEAD resources
17:17:31 cdent mriedem, edleafe, dansmith: your thoughts on jay’s comments on https://review.openstack.org/#/c/504540/ encouraged
17:17:56 jaypipes jianghuaw: the problem of having an inventory of VGPU_DISPLAY_HEAD=1 would mean only a single instance requesting VGPU_DISPLAY_HEAD:1 would be possible on the host.
17:18:32 jianghuaw jaypipes, ah, I see.
17:18:47 jianghuaw yes, you're correct.
17:19:07 jianghuaw jaypipes, thanks for pointing it out:-)
17:20:40 mriedem gibi: oops something failed in the rebase here https://review.openstack.org/#/c/499539/9
17:21:06 gibi gibi: looking
17:21:08 gibi mriedem: looking
17:21:10 mriedem https://review.openstack.org/#/c/499539/9
17:21:11 mriedem oops
17:21:16 mriedem TypeError: _delete_and_check_allocations() takes exactly 2 arguments (4 given)
17:21:57 gibi OK I can fix that up quickly
17:24:08 openstackgerrit Balazs Gibizer proposed openstack/nova master: Moving more utils to ServerResourceAllocationTestBase https://review.openstack.org/499539
17:24:09 openstackgerrit Balazs Gibizer proposed openstack/nova master: factor out compute service start in ServerMovingTest https://review.openstack.org/503037
17:24:09 openstackgerrit Balazs Gibizer proposed openstack/nova master: Test resource allocation during soft delete https://review.openstack.org/495159
17:24:59 gibi mriedem: it should be better now
17:25:13 mriedem ok
17:31:21 openstackgerrit Jianghua Wang proposed openstack/nova-specs master: Support virtual GPU resources https://review.openstack.org/450122
17:32:06 jianghuaw jaypipes, see the revised spec ^
17:32:07 jianghuaw thanks.
17:35:33 jianghuaw bauzas, I did some minor change basing on jaypipes' comments. Need your help to review it again. thanks. https://review.openstack.org/#/c/450122
17:53:44 openstackgerrit Mathieu Gagné proposed openstack/nova master: Regenerate and pass configdrive when rebuild Ironic nodes https://review.openstack.org/503088
17:53:53 jaypipes bauzas: k, I'm +2 on jianghuaw's spec
17:55:13 edmondsw mriedem yufei replied in https://review.openstack.org/#/c/502382
17:55:37 jianghuaw jaypipes, thanks very much:-)
17:56:07 edmondsw mriedem yufei I mean that I replied...
18:06:45 openstackgerrit Chris Dent proposed openstack/nova master: Move project_id and user_id to Allocation object https://review.openstack.org/500410
18:06:46 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Allow _set_allocations to delete allocations https://review.openstack.org/501051
18:06:46 openstackgerrit Chris Dent proposed openstack/nova master: WIP: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073
18:06:47 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Limit number of attempts to delete allocations https://review.openstack.org/507224
18:09:44 mriedem py35 unit test job seems to be jacked all of a sudden http://logs.openstack.org/47/498947/6/check/gate-nova-python35/15aee2f/console.html#_2017-09-25_16_45_38_542562
18:13:49 dansmith mriedem: that's odd
18:16:24 mriedem i've seen that in 2 jobs now
18:16:28 mriedem 2 changes i mean
18:17:33 openstackgerrit Merged openstack/nova-specs master: Support virtual GPU resources https://review.openstack.org/450122
18:18:05 dansmith is there something we can do about the quadrupling of the output on py35 due to warnings?
18:18:25 mriedem sdague was working on slimming some of those down already
18:18:32 dansmith okay
18:18:39 sdague I never got over to the py35 side
18:18:44 sdague I was trying to trim on the other side
18:18:44 dansmith mriedem: so one of my patches failed with that, but it does show a test fail in the testr results
18:18:54 sdague I could take a whack at the py35 one though
18:19:09 mriedem dansmith: yeah i was looking at your migration one
18:19:12 mriedem that's the link
18:19:24 mriedem Failed: 0
18:19:26 dansmith ah okay
18:19:33 mriedem {0} nova.tests.unit.test_rpc.TestRPC.test_cleanup_legacy_notifier_null [] ... inprogress
18:19:34 mriedem timeout?
18:19:39 mriedem i think that's a known one
18:19:40 sdague mriedem: yeh
18:19:40 mriedem yeah
18:19:51 sdague that's the issue that mtreinish is going to have to look at
18:20:00 sdague because that's something about the worker not returning
18:20:30 sdague it could be subunit parsing
18:20:49 dansmith does it actually take a while?
18:21:01 dansmith maybe that's the bug I'm seeing locally where some test worker takes a long time
18:21:45 sdague dansmith: I don't know, I've only seen it in the gate
18:21:56 dansmith sdague: I only see it on my 24T boxes
18:21:58 mriedem i've seen that TestRPC class timeout in lots of unit test jobs randomly
18:22:13 mriedem probably because there is global state involved in those
18:22:31 dansmith although in this gate run, it shows one worker as N/A which is not what I see
18:22:48 dansmith is it getting killed?
18:22:57 mriedem i just got this locally too

Earlier   Later