| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-17 | |||
| 13:24:45 | edmondsw | :) | |
| 13:30:25 | ericyoung | if core reviewer wants to look over some code, I could use a second + on https://review.openstack.org/#/c/452538/ | |
| 13:30:40 | dansmith | sdague: nope, i think it's all on that doc, including the step they missed | |
| 13:30:57 | dansmith | sdague: dunno about the ubuntu docs, but the first one linked which we own has got the stuff | |
| 13:33:49 | bhagyashris | johnthetubaguy: Hi, I have addressed the review comment. Could you please take some insight on https://review.openstack.org/#/c/409644/? Thank you. | |
| 13:35:34 | sdague | dansmith: ok, you want to close out the issue as invalid? | |
| 13:35:46 | dansmith | yeah I will write something and do that | |
| 14:14:55 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: power_on/off and reboot https://review.openstack.org/427380 | |
| 14:27:49 | openstackgerrit | Chris Friesen proposed openstack/nova master: remove unused parameter in rpc call https://review.openstack.org/447055 | |
| 14:28:11 | cfriesen | ^ that should sort out the unit test change | |
| 14:32:37 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #3: TaskFlow https://review.openstack.org/438729 | |
| 14:46:46 | gcb | mriedem, please help review https://review.openstack.org/#/c/457188/ which fixes py35 gate failure when oslo.config 4.0 is released | |
| 15:00:45 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #4: full flavor https://review.openstack.org/391288 | |
| 15:01:09 | alex_xu | jaypipes: we still need this spec https://review.openstack.org/#/c/351063/ for setting a set of required traits and preferred traits in flavor, right? | |
| 15:02:07 | jaypipes | alex_xu: yes | |
| 15:02:16 | alex_xu | jaypipes: ok, got it | |
| 15:02:28 | jaypipes | alex_xu: you want to get that in shape? | |
| 15:03:08 | mriedem | jaypipes: alex_xu: is that for pike though? | |
| 15:03:11 | alex_xu | jaypipes: yes | |
| 15:03:12 | mriedem | required vs preferred? | |
| 15:03:21 | mriedem | i'm pretty sure that's queens at this point | |
| 15:03:28 | alex_xu | i guess no for pike | |
| 15:03:46 | mriedem | we needed traits to mark a compute resource provider, as well, compute - so we could do the shared storage aggregate stuff i thought? | |
| 15:03:58 | jaypipes | mriedem: probably wouldn't hurt to get going on that in Pike. | |
| 15:04:08 | jaypipes | mriedem: but understand it's lower priority. | |
| 15:04:13 | mriedem | jaypipes: it would hurt if it means pushing out 3 other things we already said we'd do | |
| 15:04:20 | jaypipes | ack | |
| 15:04:29 | mriedem | like, doing that means no nested resource providers i'm sure | |
| 15:04:32 | mriedem | etc | |
| 15:04:47 | jaypipes | mriedem: ya. | |
| 15:05:02 | alex_xu | yea, I probably just writing some PoC for that, low priority | |
| 15:05:05 | mriedem | jaypipes: do you know if mirantis still has a perf benchmark lab? | |
| 15:05:11 | mriedem | for dina's team? | |
| 15:06:00 | jaypipes | mriedem: we do. | |
| 15:06:05 | mriedem | going back to the claims in scheduler vs conductor discussion, it's hard to make that decision without some actual data from both scenarios with a large number of computes | |
| 15:06:18 | jaypipes | mriedem: though it's much much smaller than it used to be... | |
| 15:06:44 | mriedem | i wonder if we could fake things out with a functional test that mocks out 300 computes or something | |
| 15:07:05 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: console https://review.openstack.org/409402 | |
| 15:07:20 | mriedem | and then run both options against that - you'd have to do something to introduce random collisions though to force the retry | |
| 15:07:39 | mriedem | didn't yingxin from intel mock something up like that | |
| 15:08:17 | jaypipes | mriedem: kinda I think? | |
| 15:11:24 | mriedem | jaypipes: was thinking of this https://docs.google.com/presentation/d/1UG1HkEWyxPVMXseLwJ44ZDm-ek_MPc4M65H8EiwZnWs/edit?ts=571fcdd5#slide=id.g12d2cf15cd_2_90 | |
| 15:11:27 | mriedem | from the austin summit | |
| 15:11:48 | mriedem | alex_xu: is yingxin still on your team? | |
| 15:12:24 | alex_xu | mriedem: yingxin is in the team based on shanghai, he is moving to ceph stuff | |
| 15:12:51 | alex_xu | mriedem: but I remember yingxin put his tool in the github | |
| 15:14:31 | mriedem | https://docs.google.com/presentation/d/1UG1HkEWyxPVMXseLwJ44ZDm-ek_MPc4M65H8EiwZnWs/edit?ts=571fcdd5#slide=id.g12d2cf15cd_2_143 | |
| 15:14:36 | mriedem | 1.079 query/secondIn 1000 node simulation | |
| 15:14:52 | mriedem | 99% of the time spent in the scheduler was get_all_host_state | |
| 15:15:19 | mriedem | so we should have dropped that with using placement now for the simple filters | |
| 15:15:38 | mriedem | but it would be interesting to know how much, i.e. 75% now? 50%? | |
| 15:16:26 | edleafe | mriedem: really hard to extrapolate. It would be so nice to have a large test bed to get real numbers from | |
| 15:16:42 | mriedem | edleafe: yeah, if only there was some kind of openstack innovation center... | |
| 15:16:54 | edleafe | R.I.P. | |
| 15:16:58 | mriedem | :( | |
| 15:22:08 | dansmith | mriedem: fwiw, I have a replacement ready for that revert, but will wait until it clears the gate so I don't rebase and kick it out | |
| 15:22:43 | mriedem | ok | |
| 15:22:53 | mriedem | i'm working on a functional test for a regression with filtering servers by tags | |
| 15:23:04 | mriedem | works fine on py27 but fails on py35 of course | |
| 15:34:31 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add regression test for server filtering by tags bug 1682693 https://review.openstack.org/457103 | |
| 15:34:31 | mriedem | dansmith: this ^ | |
| 15:34:35 | openstack | bug 1682693 in OpenStack Compute (nova) "tags and not-tags cannot work properly" [High,In progress] https://launchpad.net/bugs/1682693 - Assigned to Zhenyu Zheng (zhengzhenyu) | |
| 15:35:11 | dansmith | ack | |
| 16:00:14 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: SSP emphemeral disk support https://review.openstack.org/443189 | |
| 16:05:02 | clarkb | mriedem: edleafe can you not mock it up with a bunch of computes that are just pretending to be scheduled to? I think nova even includes a fake virt driver for this :) | |
| 16:05:15 | mriedem | clarkb: yes | |
| 16:05:32 | edleafe | clarkb: yes, that's what it looks like we will have to do | |
| 16:05:34 | mriedem | clarkb: that was my original idea, do it as a nova-functional test or something | |
| 16:06:05 | edleafe | the main chunk will be the db query and return of matching hosts | |
| 16:07:00 | edleafe | so setting up a reasonable scenario with 1000 hosts and various inventory/allocations should be close enough | |
| 16:25:49 | mriedem | does someone want to put the +W on this novaclient 2.42 change? https://review.openstack.org/#/c/429512/ that's blocking some other pike microversions from getting into novaclient - artom, you might want to review that one too | |
| 16:27:43 | edleafe | Is this doc still correct for cells in ocata? https://docs.openstack.org/ocata/config-reference/compute/cells.html | |
| 16:28:05 | mriedem | edleafe: that's all cells v1 | |
| 16:28:23 | edleafe | that's what I thought | |
| 16:28:25 | mriedem | i think i opened a bug for that at one point, sec | |
| 16:28:44 | mriedem | oh nope, similar but different page https://bugs.launchpad.net/openstack-manuals/+bug/1673616 | |
| 16:28:44 | openstack | Launchpad bug 1673616 in openstack-manuals "Scaling in Operations Guide - cells section needs to be updated" [Medium,Triaged] - Assigned to Zhenyu Zheng (zhengzhenyu) | |
| 16:29:12 | edleafe | is there a doc for the HostMappingNotFound error? | |
| 16:29:33 | mriedem | edleafe: this? https://review.openstack.org/#/c/456923/ | |
| 16:30:09 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | |
| 16:30:10 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215 | |
| 16:30:11 | openstackgerrit | Jay Pipes proposed openstack/nova master: pci: add uuid field to PciDevice object https://review.openstack.org/414768 | |
| 16:30:11 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: integrate ProviderTree to report client https://review.openstack.org/415921 | |
| 16:30:12 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: SRIOV PF devices as child providers https://review.openstack.org/414769 | |
| 16:30:13 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds ProviderTree for nested resources https://review.openstack.org/415920 | |
| 16:30:13 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | |
| 16:30:14 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: add nested resource providers https://review.openstack.org/377138 | |
| 16:30:45 | edleafe | mriedem: ok, I'll pass that along | |
| 16:30:55 | mriedem | edleafe: to whom? that's in the ML by the way | |
| 16:30:57 | mriedem | from friday | |
| 16:31:19 | mriedem | edleafe: http://lists.openstack.org/pipermail/openstack-dev/2017-April/115486.html | |
| 16:33:30 | mriedem | here is a bug for the cells config docs page https://bugs.launchpad.net/openstack-manuals/+bug/1683416 | |
| 16:33:30 | openstack | Launchpad bug 1683416 in openstack-manuals "Cells in Configuration Reference - warning" [Undecided,New] | |
| 16:33:35 | mriedem | we should expand and clarify that warning | |
| 16:33:39 | edleafe | to someone on my team running the ironic 3rd party CI: http://lists.openstack.org/pipermail/openstack-dev/2017-April/115487.html | |
| 16:34:37 | mriedem | yeah looks like the same problem | |
| 16:47:14 | openstackgerrit | Chris Dent proposed openstack/nova master: [WIP] Register osapi_compute when nova-api is wsgi https://review.openstack.org/457283 | |
| 16:47:28 | cdent | sdague (and others): ^ | |
| 16:47:41 | cdent | has an important flaw (noted in the commit message) | |
| 16:54:38 | sdague | cdent: so, besides test fixups, what are your concerns there? | |