Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-07-30
21:04:30 dansmith sean-k-mooney: it's internal and more complicated to let them choose
21:04:43 dansmith right now it can't change unless we change it or do things
21:05:00 dansmith I can't really see a compelling reason to let them change it
21:06:48 sean-k-mooney im not saying it a good idea. its a great way to shoot yourself in the foot and mess up our resouce tracking. but it would make s10 usecase doable without a lot of db hacking
21:07:42 s10 Or can I just change compute_driver to Fake on reserved node, execute evacuate to this node and then just put disks from old host, and than change config back and restart nova-compute...
21:08:47 s10 compute_driver=fake.FakeDriver. and than back to libvirt.LibvirtDriver.
21:12:42 sean-k-mooney s10: that will likely take longer then pulling the os driver if your goal is to minimise guest downtime
21:17:54 s10 sean-k-mooney: nova host-evacuate --target-host --force can be executed immediately after host failure. Same time, while instances are being "evacuated" to the fake host, support will pull disks from failed server and put them in the reserved, right after that we stop nova-compute, mount disks from old host, change nova config, start it again, and finally nova reboot for all instances. This will take much less time than moving system m.2 disk
21:19:17 s10 Only physical operation that should be performed is moving disks. Other way requires opening the lid on the top of the server with dismounting it out of the rack.
21:19:36 mriedem i highly suggest not using the --force option
21:19:51 mriedem since it bypasses the scheduler and can mess things up if the target host is not actually a proper fit for the instance
21:20:05 dansmith also,
21:20:17 dansmith the fake driver will not have the right inventory in placement
21:20:31 dansmith so your scheduler or conductor-based allocations would be all fubar
21:20:33 sean-k-mooney mriedem: in this case s10 is forcing a migrate to a fake libvirt node so not sure it applies
21:20:41 dansmith it definitely applies
21:21:11 sean-k-mooney dansmith: oh because it skip the schduler and never update placement
21:21:12 mriedem i'm not sure why using the fake virt driver in any of this is a good idea
21:21:15 s10 yep. that is not the right way because of the inventory in placement. if we can't change it after.
21:21:47 dansmith sean-k-mooney: not just that, but the inventory in placement that will come from the fake driver won't look anything like the inventory that the system will provide after you change it to libvirt
21:21:53 openstackgerrit Merged openstack/nova master: Updated AggregateImagePropertiesIsolation filter illustration https://review.openstack.org/586317
21:22:08 sean-k-mooney s10: dumb question. why can you not put the os dirve in one of the hot swap bays so its just as easy to move as the other disks
21:22:29 dansmith sean-k-mooney: m2 drives on the motherboard are getting common for system drives today
21:22:35 dansmith but of course, he could do that :)
21:22:47 dansmith or buy systems with enough hot-swap bays
21:22:53 dansmith or tell his vendor he doesn't like their placement of the m2
21:23:01 dansmith or use a system with a rear-mounted hot-swap bay for system drives
21:23:03 dansmith lots of options
21:23:32 s10 sean-k-mooney: because we have 4 slots for disk in 1U server. we use it all for storage disks or have to use one of them for system.
21:25:41 sean-k-mooney s10: ok but i have racked a lot of sytems in the intel dev lab. it really does not take that much time to pull an internal drive. for an m2 it 1 screw most of the time and you often dont need to even unrack the server.
21:26:48 sean-k-mooney s10: if you really do have this requirement then fair enough but it seams like premature optimization unless you expect to have to do this often
21:28:13 s10 sean-k-mooney: you are right, this is a premature optimisation.
21:45:15 openstackgerrit Eric Fried proposed openstack/nova master: [placement] Add /reshaper handler for POST https://review.openstack.org/576927
21:45:16 openstackgerrit Eric Fried proposed openstack/nova master: reshaper: Look up provider if not in inventories https://review.openstack.org/585033
21:45:17 openstackgerrit Eric Fried proposed openstack/nova master: Make get_allocations_for_resource_provider sane https://review.openstack.org/584598
21:45:18 openstackgerrit Eric Fried proposed openstack/nova master: Report client: Real get_allocs_for_consumer https://review.openstack.org/584599
21:45:19 openstackgerrit Eric Fried proposed openstack/nova master: Report client: get_allocations_for_provider_tree https://review.openstack.org/584648
21:45:20 openstackgerrit Eric Fried proposed openstack/nova master: Report client: _reshape helper, placement min bump https://review.openstack.org/585034
21:45:21 openstackgerrit Eric Fried proposed openstack/nova master: Report client: update_from_provider_tree w/reshape https://review.openstack.org/585049
21:45:22 openstackgerrit Eric Fried proposed openstack/nova master: WIP: Compute: Handle reshaped provider trees https://review.openstack.org/576236
22:41:27 openstackgerrit Eric Fried proposed openstack/nova master: WIP: Compute: Handle reshaped provider trees https://review.openstack.org/576236
23:07:33 mriedem cfriesen__: where is the code for the nova api proxy thing for starlingx?
23:07:37 mriedem is that in the nova staging repo?
23:10:48 mriedem i only see https://github.com/openstack/stx-nfv/tree/master/nova-api-proxy
23:14:40 openstackgerrit Artom Lifshitz proposed openstack/nova stable/pike: Refactor _build_device_metadata https://review.openstack.org/587248
23:14:41 openstackgerrit Artom Lifshitz proposed openstack/nova stable/pike: Consider hostdev devices when building metadata https://review.openstack.org/587249
23:27:50 cfriesen__ mriedem: one sec
23:32:43 cfriesen__ mriedem: You've got the right code. The proxy passes most things directly through to the "real" nova-api, while a subset of calls (see https://github.com/openstack/stx-nfv/blob/master/nova-api-proxy/nova-api-proxy/nova_api_proxy/apps/acceptor.py#L114) get forwarded to what we call the "VIM", which I think is now at "https://github.com/openstack/stx-nfv/tree/master/nfv/nfv-vim/nfv_vim"
23:33:48 cfriesen__ mriedem: the reason for this is so that the vim can remain up-to-date about the state of the instance even in the face of races
23:34:44 cfriesen__ I'm not very experienced in that area of the code...mostly I can just ignore it.
23:37:34 cfriesen__ mriedem: the vim handles things like automated recovery if a qemu process crashes, or a compute node goes down.
23:38:25 cfriesen__ mriedem: also things like pushing all instances off a node if we want to take it down for maintenance
#openstack-nova - 2018-07-31
01:03:59 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in test_compute_api.py (4) https://review.openstack.org/568462
01:04:26 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (5) https://review.openstack.org/570842
01:04:41 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (6) https://review.openstack.org/571330
01:05:03 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7) https://review.openstack.org/571992
01:05:20 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8) https://review.openstack.org/571993
01:20:02 openstackgerrit Merged openstack/nova master: Retry decorator fix for instances which go into ERROR state during bulk delete https://review.openstack.org/563418
02:01:18 openstackgerrit Merged openstack/nova master: Set default of oslo.privsep.daemon logging to INFO level https://review.openstack.org/586643
02:06:07 alex_xu mriedem: really don't know whether we need Chinese in metadata, I thought the Chinses will be used in the UI, not sure using Chinese in the metadata is good idea. It really a production question, Kevin_Zheng is good answer that I guess.
02:08:12 Kevin_Zheng alex_xu: people are using api too, I think the patch is about metadata key?
02:09:23 alex_xu Kevin_Zheng: the bug reported for key and value
02:10:01 Kevin_Zheng I will check, I remember we already support
02:10:06 Kevin_Zheng In value
02:10:39 alex_xu Kevin_Zheng: do you have cusomter really want to use Chinese in key?
02:11:40 Kevin_Zheng I will ask, seems easystack has :)
02:11:49 alex_xu hah
02:17:15 Kevin_Zheng they are checking
04:20:46 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Use jsonutils of oslo.serialization https://review.openstack.org/587278
05:02:22 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Enable nested allocation candidates in scheduler https://review.openstack.org/585672
05:02:23 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Functional test for booting with nested resources https://review.openstack.org/527728
05:24:28 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add the guideline to write API reference https://review.openstack.org/569058
05:38:47 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] api-ref: add traits parameter https://review.openstack.org/578048
06:27:55 openstackgerrit Chen proposed openstack/nova master: Fix bad links for admin-guide https://review.openstack.org/580259
07:10:52 openstackgerrit huanhongda proposed openstack/nova master: NIT: Delete duplicated word in comments https://review.openstack.org/587302
07:39:05 kashyap alex_xu: Morning, do you have time to look at this: https://review.openstack.org/#/c/567258/ (libvirt: Remove usage of migrateToURI{2} APIs)
07:39:20 kashyap alex_xu: It has went through several iterations and fixed various things
07:39:32 alex_xu kashyap: yea, let me check
07:39:38 kashyap alex_xu: Just thought I'd ping you to see if you have time, as I'd be off from tomorrow until 17th August
07:40:04 kashyap alex_xu: Thank you!
07:45:10 alex_xu kashyap: ok, got it, so put it in my todo list first :)
07:47:45 kashyap alex_xu: Sure. I think it will just have to wait for merge until I come back on 17th I guess :-(
08:11:49 gibi tetsuro: thanks for the work on putting the 1.28, 1.29 and functional tests in the same series. I happy to see that the functional test passes at the end.
08:12:25 gibi tetsuro: I will review the changes and also try to make a step forward with 1.28 today
08:14:50 openstackgerrit jichenjc proposed openstack/nova master: remove ec2 object definitions https://review.openstack.org/557150
08:14:51 openstackgerrit jichenjc proposed openstack/nova master: remove ec2 db functions https://review.openstack.org/557572
08:28:38 openstackgerrit jichenjc proposed openstack/nova master: add zvm into support matrix https://review.openstack.org/532720
08:28:39 openstackgerrit jichenjc proposed openstack/nova master: Add zvm admin intro and hypervisor information https://review.openstack.org/533125
08:28:40 openstackgerrit jichenjc proposed openstack/nova master: Add zvm CI information https://review.openstack.org/533512
08:59:26 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Functional test for booting with nested resources https://review.openstack.org/527728
08:59:27 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Functional test for moving with nested resources https://review.openstack.org/587350
09:02:40 tetsuro gibi: ^ I was about to push another PS so I addressed your points
09:03:10 tetsuro Now I leave this to you. Feel free to amend.
09:04:00 gibi tetsuro: cool, thanks!
09:04:19 tetsuro Thanks a lot, too.
09:04:22 gibi tetsuro: I will let you know what I was able to do at the end of my day
09:06:32 tetsuro Ack. Will check it tomorrow morning.
09:57:49 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Retry allocation writes server side https://review.openstack.org/586048
10:31:25 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Remove redundant join in _anchors_for_sharing_providers https://review.openstack.org/587377
10:32:42 openstackgerrit Merged openstack/nova master: Fix being able to hard reboot a pausing instance https://review.openstack.org/566143

Earlier   Later