| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-11-07 | |||
| 00:14:37 | openstackgerrit | Merged openstack/nova stable/ocata: Pass requested_destination in filter_properties https://review.openstack.org/491001 | |
| 00:31:26 | openstackgerrit | Chris Dent proposed openstack/nova-specs master: Add spec for symmetric GET and PUT of allocations https://review.openstack.org/508164 | |
| 00:37:48 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035 | |
| 01:06:53 | openstackgerrit | gengchc2 proposed openstack/nova master: Improved the judgment of the service status in memcache https://review.openstack.org/423754 | |
| 01:21:23 | openstackgerrit | Merged openstack/nova stable/pike: Avoid deleting allocations for instances being built https://review.openstack.org/517879 | |
| 01:47:40 | moshele | stephenfin: hi | |
| 02:19:00 | openstackgerrit | yangweiwei proposed openstack/nova stable/pike: Fix a bug in libvirt driver https://review.openstack.org/518198 | |
| 03:09:59 | openstackgerrit | Merged openstack/nova stable/pike: Clean up allocations if instance deleted during build https://review.openstack.org/517876 | |
| 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. | |