| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-11-21 | |||
| 11:02:49 | tetsuro | is alex or jay or efried here? | |
| 11:03:52 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Fix the usage of instance.snapshot notification sample https://review.openstack.org/520579 | |
| 11:07:42 | openstackgerrit | Naichuan Sun proposed openstack/nova master: (WIP)xenapi(N-R-P): Add API to support compute node resource provider update and create https://review.openstack.org/521041 | |
| 11:09:33 | openstackgerrit | Naichuan Sun proposed openstack/nova master: (WIP)xenapi(N-R-P): Add API to support compute node resource provider update and create https://review.openstack.org/521041 | |
| 11:14:16 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Symmetric GET and PUT /allocations/{consumer_uuid} https://review.openstack.org/510626 | |
| 11:14:17 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073 | |
| 11:15:57 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix missing marker functions https://review.openstack.org/514579 | |
| 11:29:40 | openstackgerrit | jichenjc proposed openstack/nova master: WIP: Remove quota reserve/commit/rollback https://review.openstack.org/521470 | |
| 12:00:51 | mdbooth | Anybody know why I'm getting logged out of gerrit every 5 minutes? | |
| 12:01:41 | cdent | mdbooth: because gerrit is trying to do you a favor? | |
| 12:02:29 | mdbooth | cdent: Heh. It would be the first time ever, though :) | |
| 12:05:23 | openstackgerrit | jichenjc proposed openstack/nova master: Remove command nova-manage quota refresh https://review.openstack.org/521829 | |
| 12:18:15 | openstackgerrit | jichenjc proposed openstack/nova master: remove nova-manage account and nova-manage project https://review.openstack.org/521833 | |
| 12:19:30 | openstackgerrit | jichenjc proposed openstack/nova master: Remove command nova-manage quota refresh https://review.openstack.org/521829 | |
| 12:23:08 | openstackgerrit | Merged openstack/nova master: [placement] Fix a wrong redirection in placement doc https://review.openstack.org/517163 | |
| 12:24:40 | openstackgerrit | jichenjc proposed openstack/nova master: remove nova-manage shell command https://review.openstack.org/521835 | |
| 12:26:45 | openstackgerrit | Merged openstack/nova master: Fix format in live-migration-usage.rst https://review.openstack.org/516416 | |
| 12:26:51 | openstackgerrit | Merged openstack/nova master: Using --option ARGUMENT https://review.openstack.org/516238 | |
| 12:27:04 | openstackgerrit | Merged openstack/nova master: Document the real behavior of notify_on_state_change https://review.openstack.org/516264 | |
| 12:27:11 | openstackgerrit | Merged openstack/nova master: Add a note about versioned notification samples being per-release https://review.openstack.org/520193 | |
| 12:33:25 | openstackgerrit | jichenjc proposed openstack/nova master: remove 'nova-manage host' and 'nova-manage agent' https://review.openstack.org/521837 | |
| 12:38:20 | openstackgerrit | Eric Fried proposed openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057 | |
| 12:39:52 | efried | tetsuro Sorry, just saw your ping. What's up? | |
| 12:44:38 | efried | dims Definitely an American-centric email, that. It being Thanksgiving, people eat turkey ("cousin of a yard bird") and watch football (for some reason mriedem_away is a Vikings fan). | |
| 12:45:02 | tetsuro | cool | |
| 12:46:04 | tetsuro | i was debugging and thought i solve it, but seems that it was my misunderstanding. | |
| 12:46:40 | tetsuro | Forgive me for messing up here... https://review.openstack.org/#/c/519081/ | |
| 12:49:31 | tetsuro | The day after tomorrow is a thanksgiving day in Japan. | |
| 12:51:15 | efried | tetsuro I don't see any harm done on https://review.openstack.org/#/c/519081/ -- what's the summary of this issue? | |
| 12:52:43 | tetsuro | I just thought I had solved it and wondered where to throw a patch so was about to ask you on which patch I should make it. | |
| 12:53:23 | tetsuro | But now I am still testing, so please forget my ping. | |
| 12:53:57 | efried | tetsuro Okay, no worries. Appreciate your efforts. | |
| 12:54:00 | tetsuro | now i should get back home from my office. | |
| 12:54:09 | efried | tetsuro Looking at that sort, I can't imagine why it's there or what it thinks it's doing. | |
| 12:55:01 | tetsuro | it is sorting not to do inner join at first, I think. Anyway, thanks for answering ! | |
| 12:55:29 | tetsuro | not to do outer join at first. | |
| 12:57:25 | tetsuro | I gotta go. see you tomorrow. | |
| 13:21:14 | sean-k-mooney | mriedem_away: at the ptg we discussed the fact that rodolfo had created a bluprint in os-vif for this but we dont use them there so you suggested just tracking this in nova since os-vif is a nova deliverable. | |
| 13:22:39 | sean-k-mooney | mriedem_away: so yes https://blueprints.launchpad.net/nova/+spec/vif-port-profile is a nova blueprint for work that is predominately done in os-vif | |
| 13:49:07 | efried | stephenfin gibi bauzas Anyone up for a couple of easy doc reviews? https://review.openstack.org/521216 https://review.openstack.org/521563 | |
| 13:49:40 | stephenfin | efried: Sure thing | |
| 13:49:47 | efried | Ta verramuch | |
| 14:02:16 | stephenfin | sean-k-mooney, ralonsoh: Say, can you get 82599ES (Niantic) NICs in embedded form? | |
| 14:02:27 | stephenfin | e.g. as part of the board, rather than a discrete board | |
| 14:05:26 | jianghuaw | gibi, hi. | |
| 14:08:04 | ralonsoh | stephenfin: I'll ask this question to drivers team. But I don't think so: https://ark.intel.com/products/codename/32659/Niantic | |
| 14:14:05 | gibi | efried: hi! I'm still going through my mail backlog, fit those patches are still open when I finish that I can check these | |
| 14:14:14 | gibi | jianghuaw: hi! | |
| 14:14:28 | efried | gibi stephenfin already pushed them. Thanks! | |
| 14:15:26 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Change 'InstancePCIRequest' spec field https://review.openstack.org/449257 | |
| 14:15:27 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Format NIC features using os-traits definitions https://review.openstack.org/466051 | |
| 14:15:27 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777 | |
| 14:15:40 | jianghuaw | gibi, are you going to help reviewing this patch? https://review.openstack.org/#/c/512580/ | |
| 14:15:49 | stephenfin | bauzas: Want to take a second look at https://review.openstack.org/#/c/466051/ | |
| 14:16:17 | jianghuaw | gibi, we've got +2 and serveral +1s on it. But need another core help to review it. | |
| 14:16:38 | jianghuaw | We mentioned this patches in last week. | |
| 14:18:35 | gibi | jianghuaw: I will try but | |
| 14:18:52 | gibi | jianghuaw: I have a long backlog for today | |
| 14:19:40 | jianghuaw | gibi, I see. It's appreciated if you can put it in the todo queue. | |
| 14:19:45 | mriedem_away | sean-k-mooney: ack thanks | |
| 14:20:41 | jaypipes | I can't read "TripleO IPSEC Integration" without thinking "Triple-sec integration" and then thinking about Cointreau. | |
| 14:24:51 | stephenfin | ralonsoh: Thought so, but I wasn't sure. Thanks :) | |
| 14:25:37 | stephenfin | efried, jaypipes: Am I correct in saying the Memory/DiskFilter is going away/gone thanks to placement? If so, what about the weighers? | |
| 14:25:59 | efried | stephenfin Weighers won't be affected by placement. | |
| 14:26:15 | efried | stephenfin Because placement doesn't do any weighing; everything's binary. | |
| 14:26:18 | jaypipes | stephenfin: yes. and what efried said. | |
| 14:26:21 | mriedem | jianghuaw: on https://review.openstack.org/#/c/512580/ - was there a specific reason to use MultiStrOpt over ListOpt? | |
| 14:26:25 | efried | stephenfin As for those filters going away... I would expect the answer is yes. | |
| 14:26:31 | efried | ...eventually. | |
| 14:26:37 | stephenfin | efried: That's what I was thinking. So does this mean something like this wouldn't be affected by placement? https://review.openstack.org/#/c/379525/ | |
| 14:26:49 | mriedem | stephenfin: the core/ram/disk filters are no longer part of the default list of enabled filters b/c of placement | |
| 14:26:50 | bauzas | stephenfin: ralonsoh: so the problem is that we don't have a os-traits tag, right? | |
| 14:26:55 | stephenfin | (not saying it's something we want, but placement is not the reason not to want it) | |
| 14:27:03 | efried | stephenfin Agree. | |
| 14:27:08 | stephenfin | bauzas: Correct. We need to wait for the next release | |
| 14:27:11 | bauzas | stephenfin: ralonsoh: honestly, we could just create a new release | |
| 14:27:12 | bauzas | mriedem: ^ | |
| 14:27:18 | stephenfin | Also, I wouldn't want to block that patch while we wait for it | |
| 14:27:24 | jaypipes | stephenfin: eventually the *inputs* to the weighers will be created from the returned results of the GET /allocation_candidates call's provider_summaries response, instead of a constructing from the ComputeNode object/cell DB table. | |
| 14:27:26 | stephenfin | That's also an option :D | |
| 14:27:32 | bauzas | mriedem: context is https://review.openstack.org/#/c/466051/12..15/nova/virt/libvirt/config.py | |
| 14:27:43 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | |
| 14:28:01 | stephenfin | jaypipes: But filters would still be a nova (vs placement thing), right? | |
| 14:28:05 | bauzas | mriedem: I could just provide a new release for os-traits, fine with you ? | |
| 14:28:12 | stephenfin | Sorry - weighters | |
| 14:28:16 | mriedem | bauzas: sure | |
| 14:28:23 | ralonsoh | bauzas: you are right, os-traits is not updated. But the patch can work as is | |
| 14:28:24 | jaypipes | stephenfin: weighers will continue to be in nova-scheduler, yes | |
| 14:28:30 | mriedem | bauzas: you'll be working this week if the gate explodes and the rest of us are on vacation? | |
| 14:28:37 | bauzas | stephenfin: ralonsoh: I'll just provide a new release change for os-traits | |
| 14:28:54 | bauzas | mriedem: sure, I can | |
| 14:29:06 | jianghuaw | mriedem, both should work. But I think bauzas suggested MultiStrOpt when reviewing the spec. | |
| 14:29:07 | stephenfin | jaypipes, efried: If either of you want to review that once I do, I'd appreciate it https://review.openstack.org/#/c/379525/ | |
| 14:29:43 | jaypipes | stephenfin: k | |
| 14:30:14 | stephenfin | Cheers :) Seems I'm on the hook for nested RP reviews this week too, you'll be happy to hear | |
| 14:30:19 | mriedem | stephenfin: https://review.openstack.org/#/c/379525/ isn't happening for queens | |
| 14:30:20 | efried | stephenfin This is for a 3-year-old blueprint that's not approved for Q ? | |
| 14:30:23 | stephenfin | /unhappy | |
| 14:30:28 | bauzas | jianghuaw: I'd love to see you reviewing my inventory patch for the VGPU in libvirt | |
| 14:30:51 | stephenfin | Ahh, hold on so. I didn't realize there was a BP attached | |