Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-11-07
04:48:46 openstackgerrit Chris Dent proposed openstack/nova master: [placement] set accept to application/json if accept not set https://review.openstack.org/518223
04:49:36 cdent edleafe: if you have thoughts on the microversion issue on that, that would be good to know ^
05:27:40 openstackgerrit Chen proposed openstack/nova-specs master: This spec is to add support for subnet assignment when creating servers. https://review.openstack.org/518227
05:41:05 openstackgerrit Chen proposed openstack/nova-specs master: Add support for subnet assignment when creating servers https://review.openstack.org/518227
06:18:54 openstackgerrit Vu Cong Tuan proposed openstack/nova master: Do not use “-y” for package install https://review.openstack.org/518238
06:48:04 openstackgerrit Yikun Jiang proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406
06:51:11 openstackgerrit yangweiwei proposed openstack/nova master: Fix a bug in libvirt driver https://review.openstack.org/518250
07:58:11 openstackgerrit Merged openstack/nova master: check query param for used_limits function https://review.openstack.org/499091
08:09:07 openstackgerrit Balazs Gibizer proposed openstack/nova master: reno for notification-transformation-pike https://review.openstack.org/518018
09:02:52 openstackgerrit Matt Riedemann proposed openstack/nova stable/ocata: Set group_members when converting to legacy request spec https://review.openstack.org/517860
09:50:09 openstackgerrit Merged openstack/nova master: VMware: add support for graceful shutdown of instances https://review.openstack.org/494169
10:16:52 openstackgerrit Yikun Jiang proposed openstack/nova master: WIP: Add cross cell sort support for get_migrations https://review.openstack.org/517273
11:05:18 openstackgerrit Jianghua Wang proposed openstack/nova master: rt: Make resource tracker always invoking get_inventory() https://review.openstack.org/518294
11:06:08 jianghuaw bauzas, ^
11:07:05 jianghuaw bauzas, Please help to check if the change is expected. Thanks.
12:54:53 openstackgerrit Eric Fried proposed openstack/nova master: Refactor test_allocation_candidates https://review.openstack.org/518124
12:55:14 openstackgerrit Yikun Jiang proposed openstack/nova master: WIP: Add cross cell sort support for get_migrations https://review.openstack.org/517273
13:46:42 openstackgerrit Eric Fried proposed openstack/nova master: Reformat _get_all_with_shared https://review.openstack.org/512799
14:04:33 openstackgerrit Mike Lowe proposed openstack/nova master: Take no chance an assigned disk device address will be overwritten during migration https://review.openstack.org/518022
14:28:50 openstackgerrit Balazs Gibizer proposed openstack/nova master: reno for notification-transformation-queens https://review.openstack.org/518018
14:44:32 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM Driver: config drive https://review.openstack.org/409404
14:55:28 jaypipes efried: not sure what Eric Berglund's IRC nick is, could you clue me in?
14:55:35 efried esberglu
14:55:39 efried jaypipes ^
14:55:54 jaypipes aha
14:55:56 esberglu jaypipes: What's up?
14:56:13 jaypipes esberglu: hey, if you make that one correction on the config drive patch, I'll quick +2 it.
14:56:27 jaypipes esberglu: noticed something on last review that I missed on an earlier revision.
14:58:33 efried jaypipes Nice catch. I believe we're doing that same thing in *all* of our OOT tests.
14:58:51 jaypipes efried: OOT?
14:58:59 efried (Out-of-tree)
14:59:05 jaypipes efried: ah :)
14:59:10 jaypipes you and your TLAs.
14:59:38 jaypipes efried: changing to NoDBTestCase provides a nice little performance boost, FYI.
14:59:47 efried Good deal.
15:00:50 efried esberglu When you get a chance, wanna do a sweep of the nova-powervm test suites and swap all of them over to NoDBTestCase?
15:01:16 esberglu efried: Will do
15:01:19 efried thx
15:01:59 jaypipes efried: one other nice thing about NoDBTestCase is that if any of the test code *does* touch the DB, there is a poison fixture that will blow up, telling the developer that they are stating the test code doesn't touch the DB, but in fact, does touch the DB.
15:02:29 efried jaypipes I really *hope* our virt driver code doesn't touch the DB directly.
15:02:47 jaypipes efried: every once in a while, that poison fixture has been a friend to me identifying places where I had no idea we were touching the DB ;)
15:02:48 efried Though actually I think I saw a case of it the other day...
15:04:04 efried jaypipes https://review.openstack.org/#/c/422512/25/nova/virt/powervm/tasks/network.py@155 <= that's a DB touch, nah?
15:04:18 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM Driver: config drive https://review.openstack.org/409404
15:04:24 esberglu jaypipes: Done. tx
15:04:32 jaypipes efried: yes, it is. :)
15:05:22 jaypipes esberglu: cool, thanks.
15:10:29 jaypipes alex_xu: you around?
15:10:35 snasir Hi, was hoping if anyone could kindly help me understand how does nova differentiate between a baremetal instance vs a VM when launched with nova boot ? Is it the scheduler filters that determine that or is it the scheduler_host_manager ? Thanks
15:11:05 jaypipes alex_xu: I'm sorry, but I'm not understanding which comment you are referring to in your last review response on https://review.openstack.org/#/c/516778/. Could you elaborate please? Thanks!
15:25:21 jaypipes snasir: good question. there is a property on the image called hypervisor_type='baremetal' that will trigger the ironic host manager to pick the Ironic baremetal node to deploy to.
15:27:45 snasir javpipes: Oh so does that mean, that I can deploy baremetal nodes even if i dont set scheduler_host_manager=ironic_host_manager ? Is the regular host_manager smart to determine that
15:28:08 snasir jaypipes: Oh so does that mean, that I can deploy baremetal nodes even if i dont set scheduler_host_manager=ironic_host_manager ? Is the regular host_manager smart to determine that
15:28:37 jaypipes snasir: no, you still need to do the ironic_host_manager thing unfortunately. it's a giant mess.
15:29:42 jaypipes snasir: I don't know why it's still like that :(
15:30:01 jaypipes snasir: we really should just remove the ironic_host_manager entirely, IMHO
15:30:30 snasir jaypipes: Ok Basically I'm trying to figure out if I can share nova services to spawn both BM nodes and VMs. I know I have to have two nova-computes running (with their respective drivers), and looks like I'll need two nova-schedulers running too with their respective host_managers... Am i correct in saying that ?
15:34:29 jaypipes snasir: I believe so, yes.
15:34:58 jaypipes snasir: though I would point you to Ironic operators to get a definitive answer on that.
15:35:05 jaypipes snasir: of course, most are in Sydney :)
15:35:25 jaypipes snasir: so probably a good idea to post to the openstack@ or openstack-operators@ mailing list.
15:36:05 jaypipes snasir: I'm particularly thinking of James Penick as a possible resource for you. Usually he's on IRC as "penick", but he's in Sydney this week.
15:38:06 openstackgerrit Surya Seetharaman proposed openstack/nova master: instance_mapping/request_spec records exist even after the instances are archived https://review.openstack.org/515034
15:38:09 snasir jaypipes: I tried asking on the Ironic IRC, but couldn't get a definite answer....Thanks I'll post on @openstack.. Its too early in Sydney atm :)
15:38:21 jaypipes snasir: too late :)
15:38:43 snasir jaypipes: lol, yeah too late i meant, haha
15:47:31 efried jaypipes FYI, repeat performance at the dentist. Back in 60-90m
16:32:42 openstackgerrit C Leavett-Brown proposed openstack/nova master: Modify nova routing table to process null requests https://review.openstack.org/518119
16:34:58 jmlowe jaypipes: you around?
16:35:10 jaypipes jmlowe: I am indeed. :)
16:36:03 jmlowe My knee jerk reaction is to just add a second test to the test_update_volume_xml unit test but I suspect that isn't the right thing to do
16:36:32 jaypipes jmlowe: I think your knee-jerk reaction is good. :)
16:37:09 jaypipes jmlowe: just make sure you put a little code comment in the test saying basically "make sure we don't change the disk device address if we're live migrating"...
16:38:10 jmlowe ok, I think I have all the stuff from a live broken one, will cover virtio-scsi and rbd not just virtio-block and iscsi once I'm done
16:39:27 jmlowe not sure my reply to your comment actually made it in there, changing a device address during cold migration is bad but doesn't cause a fatal error for the operator just the user of the vm
16:40:41 jaypipes gotcha
17:18:00 jmlowe here goes, my confidence in the unit test is a little shaky
17:18:05 openstackgerrit Mike Lowe proposed openstack/nova master: live-mig: keep disk device address same https://review.openstack.org/518022
17:46:20 efried jaypipes Here's an interesting bug. I haven't fully nailed it down yet, but I think it works like this:
17:47:05 efried My non-sharing RP has one resource class. My sharing RP has three, including the same one as the non-sharing RP. I ask for all three. The candidates I get back are all effed up.
17:48:15 efried I get back three candidates. One of them is the common RC plus *one* of the others; and one of them is similar but with the *other* non-common RC (that is, two of the candidates come back with only two out of the three requested RCs)
17:48:46 efried The third candidate is the one you would expect: all three resources from the RP that has all three.
17:49:29 efried Now, I know this represents a couple things we don't support. But it seems to point to a flaw in the logic somewhere that should *never* be allowing a candidate that doesn't have all the requested resources.
17:58:41 jaypipes efried: correct. an allocation request should always contain all of the requested resources, provided by one or more resource providers.
17:59:16 efried jaypipes I'll write up a specific test case. Though it'll be pretty weird from a non-sharing-RP-is-always-the-compute-node perspective.
18:00:12 openstackgerrit Eric Fried proposed openstack/nova master: Test alloc candidates with same RC in cn & shared https://review.openstack.org/513149
18:00:46 jaypipes efried: l
18:00:49 jaypipes efried: k
18:03:42 efried jaypipes Actually, I think I was in a situation where both of the RPs were marked as sharing. (And there would be no non-sharing RPs anywhere in the db)
18:05:26 jaypipes efried: in that case, if there were no non-sharing resource providers, there should be allocation request results.
18:05:51 efried jaypipes *no* allocation request results?
18:06:39 jaypipes efried: correct. allocation requests must contain at least one non-sharing provider.
18:07:00 jaypipes efried: because you can't send a boot request to a shared storage provider...
18:07:01 efried jaypipes So I suspect some of the stuff you did in the refactor will "fix" this. I'll write it up and confirm.
18:07:14 jaypipes k
18:07:26 efried jaypipes Well, again, that's assuming we're coming out of GET /allocation_candidates wanting to do a boot :)
18:07:49 jaypipes efried: yes, that is a basic assumption of GET /allocation_candidates right now.
18:07:54 efried ack
18:08:22 openstackgerrit Eric Fried proposed openstack/nova master: Test alloc candidates with same RC in cn & shared https://review.openstack.org/513149
18:19:06 efried jaypipes Interesting. So before the refactor stack, the results are as I stated above - we have three candidates where two of them don't even have all the resources.

Earlier   Later