| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-22 | |||
| 21:05:21 | cfriesen | mriedem: correct, no live migration with sriov ports attached. | |
| 21:06:05 | cfriesen | are we wanting to go anywhere with cold migration support for LVM? | |
| 21:11:37 | mriedem | efried: i think you should have alex_xu review that | |
| 21:11:45 | mriedem | given he wrote the traits support in the allocation candidates API | |
| 21:11:51 | mriedem | he should be back to the work grind now | |
| 21:12:16 | efried | mriedem: Sure, sounds good. | |
| 21:12:17 | mriedem | cfriesen: doesn't matter to me | |
| 21:13:25 | dansmith | mriedem: looks fairly straightforward to me | |
| 21:14:37 | dansmith | the biggest question I'd have is over the sql stuff, but jay specifically made his mark on that bit | |
| 21:17:51 | mriedem | nit in the release note, but could be cleaned up on top | |
| 21:18:19 | mriedem | +1 | |
| 21:22:28 | dansmith | efried: if you wanna update that nit I'll throw my +2 on there and we can try to get alex_xu to look during his day | |
| 21:23:08 | efried | dansmith: Roger that. Picking nit... | |
| 21:23:21 | dansmith | but it looks pretty good to me | |
| 21:24:29 | openstackgerrit | Eric Fried proposed openstack/nova master: rp: GET /resource_providers?required= |
|
| 21:24:35 | efried | dansmith, mriedem: Hecho ^ | |
| 21:25:45 | dansmith | establadajo ^ | |
| 21:26:43 | efried | dansmith: Thanks. | |
| 21:26:46 | efried | I mean, you're welcome. | |
| 21:26:50 | dansmith | heh | |
| 21:28:59 | efried | Spanish? That's Esperanto. | |
| 21:30:26 | efried | Oh, shit, it actually is | |
| 21:31:00 | dansmith | what? can't be, I just made it up | |
| 21:31:08 | efried | https://translate.google.com/#eo/en/establadajo | |
| 21:31:23 | dansmith | damn, I must be a sevant or something | |
| 21:31:32 | efried | Note that Esperanto was also "just made up" by people. | |
| 21:31:38 | dansmith | *savant | |
| 21:31:42 | dansmith | haha | |
| 21:31:52 | efried | It didn't come about like any other normal languages. | |
| 21:32:42 | efried | Was supposed to be a simple language that anyone could learn, and therefore become the universal lingua franca of the world. | |
| 21:32:51 | efried | But the creators effed it up | |
| 21:33:09 | efried | Like, the first thing you should do is make sure there aren't any goofy exceptions in spelling and grammar and stuff. | |
| 21:33:35 | efried | But they totally brought that kind of garbage in from their native languages, which I think were Portuguese and English. | |
| 21:33:48 | efried | For that reason and/or others, it never caught on. | |
| 21:33:53 | efried | That's my understanding anyway. | |
| 21:35:30 | efried | This article was clearly written by an advocate/evangelist, not an impartial researcher. | |
| 21:37:33 | dansmith | heh | |
| 21:56:30 | efried | Okay kids, I'm out til Dublin. See some of y'all there! | |
| 22:00:10 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add vif info to migration object https://review.openstack.org/515423 | |
| 22:00:11 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP stub out changes for multi port bindings https://review.openstack.org/522537 | |
| 22:00:15 | mriedem | sean-k-mooney: finally got this series all lined up and rebased, and i think the conductor changes are mostly done ^ | |
| 22:00:32 | mriedem | likely need to split that into virt drivers > compute manager changes > conductor at the end | |
| 22:00:36 | mriedem | since conductor turns it all on | |
| 22:00:37 | mriedem | for the new flow | |
| 22:03:12 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP stub out changes for multi port bindings https://review.openstack.org/522537 | |
| 22:05:55 | dansmith | vif info on migration object huh? | |
| 22:06:43 | mriedem | migratedata | |
| 22:11:10 | dansmith | ah | |
| 22:11:16 | dansmith | subject says migration | |
| 22:15:20 | mriedem | dansmith: if you're looking for a good time https://review.openstack.org/#/q/topic:bug/1750666+status:open+branch:stable/pike | |
| 22:15:23 | mriedem | all need a final +2 | |
| 22:15:36 | dansmith | gaahd | |
| 22:15:55 | mriedem | well sylvain ran off and never reviewed those | |
| 22:15:59 | mriedem | so tag you're it | |
| 22:25:34 | dansmith | hmm, cinder failure on the cinder-related patch... | |
| 22:25:42 | dansmith | you checked it to confirm it's unrelated? | |
| 22:29:12 | openstackgerrit | Merged openstack/nova-specs master: Re-propose use Neutron's new port binding API https://review.openstack.org/543578 | |
| 22:33:56 | mriedem | dansmith: yeah it's a cinder api consistency group snapshot test which is nowhere related to nova http://logs.openstack.org/75/546275/4/check/tempest-full/a6efea1/job-output.txt.gz#_2018-02-22_20_19_14_919584 | |
| 22:35:39 | dansmith | yeah, alright | |
| 23:40:28 | openstackgerrit | Matthew Edmonds proposed openstack/nova master: make PowerVM capabilities explicit https://review.openstack.org/547169 | |
| 23:53:31 | openstackgerrit | Matthew Edmonds proposed openstack/nova master: make PowerVM capabilities explicit https://review.openstack.org/547169 | |
| #openstack-nova - 2018-02-23 | |||
| 00:35:07 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Support shared and dedicated VMs in one host https://review.openstack.org/543805 | |
| 00:55:53 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/queens: [placement] Add sending global request ID in put (3) https://review.openstack.org/543113 | |
| 02:20:50 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/queens: [placement] Add sending global request ID in get https://review.openstack.org/543116 | |
| 02:34:53 | openstackgerrit | Merged openstack/nova stable/pike: Drop extra loop which modifies Cinder volume status https://review.openstack.org/546218 | |
| 03:36:33 | mnaser | i'm seeing more and more cinder snapshot failures in stable/pike | |
| 04:43:31 | alex_xu | mriedem, dansmith, working on eric patch now | |
| 05:21:56 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Support shared/dedicated vCPUs in one instance https://review.openstack.org/545734 | |
| 05:26:26 | openstackgerrit | Nakanishi Tomotaka proposed openstack/nova master: Test Compute API in multiple cells https://review.openstack.org/547273 | |
| 05:34:00 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Support shared/dedicated vCPUs in one instance https://review.openstack.org/545734 | |
| 06:11:18 | Spazmotic | Morning Novaers | |
| 06:16:54 | tetsuro | is cfriesen around? | |
| 06:17:05 | itlinux | hello nova team.. i have a question will it be better to set the hypervisor to kvm or just leave them to qemu? | |
| 06:17:14 | itlinux | thanks for the advice.. | |
| 06:19:39 | cfriesen | tetsuro: I'm here. | |
| 06:20:02 | tetsuro | itlinux: KVM has more functionality. Let me quote from the document https://docs.openstack.org/ocata/config-reference/compute/hypervisors.html “QEMU - Quick EMUlator, generally only used for development purposes.” | |
| 06:20:25 | itlinux | ok. | |
| 06:20:30 | tetsuro | cfriesen thanks for reply | |
| 06:20:35 | cfriesen | tetsuro: and I just added a bunch of comments to the review | |
| 06:20:40 | itlinux | thanks.. | |
| 06:20:51 | tetsuro | Yup, I didn’t understand your comment… | |
| 06:21:03 | tetsuro | so what is it from the placement view? | |
| 06:21:40 | tetsuro | alway 9 VCPUs in the inventory, right? | |
| 06:21:53 | cfriesen | I'm not an expert on placement, but I think so | |
| 06:23:27 | cfriesen | I'm not sure how it tracks allocated cpus though, whether it factors in cpu_allocation_ratio or not | |
| 06:23:58 | tetsuro | okay, if you want (8*1+1*(1/2))=9, who will exclude the host if you claim the third shared vcpu ? | |
| 06:24:26 | tetsuro | I mean (1/2)*2+(1/2)>1 | |
| 06:25:09 | tetsuro | where is the logic of “(1/2)*2+(1/2)>1”? | |
| 06:26:50 | cfriesen | in our internal patch we allocate "dedicated" pCPUs dynamically, so we would start with 9, allocate 8 dedicated ones (reporting a usage of 8), then allocate a "shared" one (and report a usage of 8.5) | |
| 06:26:51 | tetsuro | Maybe corefilter is extended and he is aware of shared cpus and shared dedicated cpus? | |
| 06:26:54 | tetsuro | oops | |
| 06:27:35 | cfriesen | then at that point CoreFilter would prevent you from allocating another dedicated pCPU since that would put usage at 9.5 which is larger than 9 | |
| 06:28:36 | cfriesen | but you could still allocate another shared pCPU since that would put usage right at 9 | |
| 06:29:54 | cfriesen | If we statically determine which pCPUs are shared/dedicated, then I think CoreFilter or the placement equivalent would need to be modified. | |
| 06:31:21 | tetsuro | Ah, so you mean your approach doesn’t need to pre-set something like “cpu_dedicate_set” ? | |
| 06:32:26 | itlinux | ok thanks tetsuro: my nova.conf says virt_type = kvm but my openstack hypervisor list show qemu type.. confused | |
| 06:32:30 | cfriesen | yes. anything in NUMACell.cpuset that is not in NUMACell.pinned_cpus is available to run "shared" vCPUs | |
| 06:32:46 | tetsuro | You are taking dynamically allocation approach that I put into alternative. | |
| 06:32:52 | tetsuro | Okay. | |
| 06:33:00 | cfriesen | yes, that's why I mentioned it originally | |