| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-14 | |||
| 01:09:07 | Kevin_Zheng | mriedem Hi, saw you guys discussing about the quota thing, any conclusion? | |
| 01:26:43 | openstackgerrit | Artom Lifshitz proposed openstack/nova-specs master: Live migration with CPU pinning https://review.openstack.org/552722 | |
| 02:11:21 | jianghuaw_ | jichen, is this what you're looking for? https://etherpad.openstack.org/p/rocky-nova-priorities-tracking | |
| 02:12:16 | Spaz-Home | lol indeed artom , Night shift, so I work Asia hours :p | |
| 02:14:04 | artom | Spaz-Home, ah! I did night shift in a previous life, working support for a web host. It was... interesting | |
| 02:14:43 | Spaz-Home | That it is.. i've been doing it for about 10 years.. It can be extremely unhealthy, Just in the paste 2 years or so i've started to prioritize my health in the situation. | |
| 02:14:56 | Spaz-Home | But also.. I get to bother jianghuaw_ all night, which is probably the best benefit :P | |
| 02:15:15 | jianghuaw_ | Spaz-Home, good morning:-) | |
| 02:15:31 | Spaz-Home | Welcome home, sir. | |
| 02:15:35 | artom | I only did it for a few months, it definitely messed with me, I can't imagine doing it for years. We're not cats. | |
| 02:15:47 | jichen | jianghuaw_: thanks, I missed line 98 in the etherpad... :) | |
| 02:16:04 | Spaz-Home | Hehe yes sir.. I've honestly seem people's minds go a little crazy .. some people just are not made for that transition. | |
| 02:16:08 | jianghuaw_ | jichen, np:-) | |
| 02:18:32 | jianghuaw_ | Spaz-Home, thanks. Good job on https://review.openstack.org/#/c/538415/ :-) | |
| 02:18:39 | Spaz-Home | Hey jianghuaw_ before I lose ya, Dan is asking for a Xen Subteam member to attend Nova Meeting this week for https://blueprints.launchpad.net/nova/+spec/live-migration-in-xapi-pool | |
| 02:18:45 | Spaz-Home | Haha thanks sir, made me happy | |
| 02:19:31 | jianghuaw_ | Spaz-Home, yes. Naichuan or myself will attend this week's meeting. | |
| 02:19:43 | jianghuaw_ | Hope will get that BP be approved. | |
| 02:19:47 | Spaz-Home | Sounds good. I'd also like to discuss that with you a bit today or tomorrow if you have the time | |
| 02:19:54 | Spaz-Home | I do thin kthey plan to approve | |
| 02:20:08 | jianghuaw_ | Yes, we had some discussion in the PTG. | |
| 02:20:52 | jianghuaw_ | Spaz-Home, so what do you want to to chat on that topic? | |
| 02:21:20 | Spaz-Home | Well I understand the reason for the patch and it's basically just moving away from aggregates into the more refined pools, so I cannot speak anythign bad about the patch itself as it's the same situation but a different method | |
| 02:21:31 | Spaz-Home | However I still dislike that we're forcing an optional feature into the xenapi drives for LM to work | |
| 02:22:20 | jianghuaw_ | LM? | |
| 02:22:27 | Spaz-Home | My apologies, Live Migration | |
| 02:23:09 | Spaz-Home | I guess the best example would be my own environment we work in, We do not use Aggregates or Pooling, and just allow VMs to migrate around within the same Cell | |
| 02:23:39 | Spaz-Home | Since Networks are assigned through the cell and inventory is similar.. My thought however is how common that swetup would be rather than using pooling or aggregates. | |
| 02:23:50 | jianghuaw_ | Yes, the original plan for this feature is to support XAPI pool which is the direction which we think is better than the aggregation pool. | |
| 02:23:51 | openstackgerrit | Merged openstack/nova master: Reparent placement objects to oslo_versionedobjects https://review.openstack.org/551529 | |
| 02:24:04 | Spaz-Home | I can certainly agree on that point | |
| 02:24:46 | jianghuaw_ | But later we noticed the existing aggregation pool is broken when introducing cells. | |
| 02:26:11 | Spaz-Home | Hrm. | |
| 02:27:25 | jianghuaw_ | I guess the use case you mentioned should be still supported. | |
| 02:27:47 | Spaz-Home | Well, as of right now it's definately not since it will fail pulling the aggregate, and I think it will now as well trying to pull a pool master. | |
| 02:28:00 | jianghuaw_ | As you're not using pool or aggregation, so ML should still work. | |
| 02:30:21 | Spaz-Home | yeah I don't think it will work looking at the new is_same_pool_host | |
| 02:30:29 | Spaz-Home | IT's going to look for that dest in the pool and fail just like the aggregation check does. | |
| 02:30:49 | Spaz-Home | I'll need to fire up a host to confirm that, but should throw an error | |
| 02:31:59 | Spaz-Home | I guess in the end I had two thoughts. If (when we make this final change to change the checks themselves) we could check if the pool is empty AND if CONF [cells]->Enable=True to figure out if a destination is acceptable | |
| 02:32:38 | Spaz-Home | Or if you guys do decide to jsut rely on pools, I would like to submit a followup patch to increase parity with block-migration. Right now block-migration is very much not aligned, as this is how we currently live migrate to avoid the aggregation check. | |
| 02:33:08 | Spaz-Home | That way things remain even and clean. We could use that opportunity to always run the assert_will_migrate and ensure that block-migration checks for the pool st atus as well. | |
| 02:34:40 | jianghuaw_ | Spaz-Home, https://git.openstack.org/cgit/openstack/nova/tree/nova/virt/xenapi/vmops.py#n2322 | |
| 02:34:49 | jianghuaw_ | I think it should go to this branch. | |
| 02:35:14 | jianghuaw_ | It's not able to use shared SR to speed-up live-migration. | |
| 02:36:15 | jianghuaw_ | If want to use shared SR, surely must add hosts into the same pool. | |
| 02:38:14 | jianghuaw_ | Spaz-Home, also please note we will have a follow-up patch to remove the this upcall - _ensure_host_in_aggregate. | |
| 02:38:22 | Spaz-Home | Yeah got ya | |
| 02:39:13 | Spaz-Home | It's wierd that we're using block_migration like that.. but I actually hadn't looked at this line before | |
| 02:39:33 | Spaz-Home | Sicne block_migration is intended purely for iscsi stuff.. it's itneresting that we're using it to reroute the code lol | |
| 02:41:01 | Spaz-Home | Ok.. with that line I am fine with this then sir | |
| 02:41:06 | jianghuaw_ | Indeed. I agree. At sometime we should refactor this part to make it more reasonable. | |
| 02:41:25 | Spaz-Home | I think we need to just refactor the process overall.. I was thinking in a month or so just getting a whiteboard and seeing what I can do and send you pictures lol | |
| 02:41:32 | Spaz-Home | But a project for another time | |
| 02:42:36 | jianghuaw_ | Great. yes. just ping me if you have any thoughts at any time. | |
| 02:42:52 | jianghuaw_ | I'm happy that you're interested at it. | |
| 02:42:55 | jianghuaw_ | :-) | |
| 02:43:20 | Spaz-Home | Absolutely :D I'll submit a commit here this week as well to fix that assert_can_migrate as well so evertything asserts, and keep bug diving :P | |
| 02:43:26 | Spaz-Home | Enjoy the rest of your shift sir, let me know if you need anything | |
| 02:43:59 | jianghuaw_ | Thanks. It's too late for you. | |
| 02:44:05 | jianghuaw_ | time for bed | |
| 02:50:15 | Spaz-Home | Hehe nah I stay on my night shift on my weekends. Will be up until well after you're asleep. | |
| 02:50:36 | Spaz-Home | With my wife in Korea, I have to stay on the same hours as you basically :p | |
| 02:53:53 | openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Allow abort live migrations in queued status https://review.openstack.org/536722 | |
| 02:55:44 | openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Allow abort live migrations in queued status https://review.openstack.org/536722 | |
| 02:56:39 | Spaz-Home | Moving my second commit back down to subteam review. I self -1'ed it for the moment. Will fix unit tests tomorrow. | |
| 05:46:41 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/552774 | |
| 05:55:14 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/533918 | |
| 06:17:36 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/548772 | |
| 06:34:25 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486 | |
| 08:14:31 | openstackgerrit | sahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/497457 | |
| 08:27:43 | openstackgerrit | Jie Li proposed openstack/nova-specs master: Support volume-backed server rebuild https://review.openstack.org/532407 | |
| 08:36:43 | openstackgerrit | sahid proposed openstack/nova-specs master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/539605 | |
| 09:05:42 | openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | |
| 09:05:43 | openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | |
| 09:05:43 | openstackgerrit | Chris Dent proposed openstack/nova master: Move placement exceptions into the placement package https://review.openstack.org/549862 | |
| 09:08:01 | kashyap | mdbooth: Saw the change last evening was letting Zuul go through its course. | |
| 09:20:34 | openstackgerrit | Chris Dent proposed openstack/nova master: Move placement exceptions into the placement package https://review.openstack.org/549862 | |
| 09:20:35 | openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | |
| 09:20:35 | openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | |
| 09:27:18 | bauzas | man, I forgot my manners | |
| 09:27:25 | bauzas | good morning Nova | |
| 09:35:52 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova-specs master: Complex (Anti)-Affinity Policies https://review.openstack.org/546925 | |
| 09:37:40 | pooja_jadhav | cdent: Hi | |
| 09:38:24 | cdent | good morning pooja_jadhav | |
| 09:39:14 | pooja_jadhav | cdent: actually i want discuss with you regarding shared resource provider thing. | |
| 09:39:27 | pooja_jadhav | cdent: very gm :) | |
| 09:40:44 | cdent | pooja_jadhav: sure, but you should be aware that share providers are still a bit of a work in progress. Most of the functionality is nearly there, but nothing really uses it yet. | |
| 09:41:32 | cdent | What's your plan? | |
| 09:42:19 | pooja_jadhav | cdent: planning to test correct disk usage or not on shared storage | |
| 09:42:46 | pooja_jadhav | cdent : cdent: i have configured nfs backend, and created shared resource provider, but when i booted the instance it failing at line [1] https://github.com/openstack/nova/blob/master/nova/scheduler/manager.py#L138 | |
| 09:43:41 | cdent | pooja_jadhav: is there an aggregate in place that associates the shared storage provider with one or more compute nodes that use it? | |
| 09:43:55 | pooja_jadhav | yes, aggregate is there | |
| 09:44:18 | pooja_jadhav | its not getting allocation request | |
| 09:44:52 | pooja_jadhav | when i create the instance, that instance going into error state | |
| 09:46:05 | pooja_jadhav | actually, where am i missing something, i am not getting :( | |
| 09:47:01 | cdent | tetsuro recently wrote an email with some updated information on the state of sharing providers, let me find that. It may be that what you're trying to do simply doesn't work in /allocation_candidates yet | |
| 09:47:36 | cdent | this message: http://lists.openstack.org/pipermail/openstack-dev/2018-March/128141.html | |
| 09:47:39 | pooja_jadhav | ohkk | |
| 09:48:47 | cdent | That first item he lists, about the resource class existing in both places, may be part of the problem? Is the compute node reporting disk inventory ? | |