| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-30 | |||
| 14:36:02 | bauzas | jaypipes: doesn't that impact your work on cleaning that up ? | |
| 14:37:12 | jaypipes | bauzas: see my comments on that review from earlier. basically, I'm fine adding those NIC features as a temporary effort until we get traits done (and have the flavor able to communicate required and preferred traits) | |
| 14:37:25 | jaypipes | bauzas: more of a compromise than anything else, really. | |
| 14:37:31 | ralonsoh | bauzas: once os-traits and resource providers is working, we will migrate it | |
| 14:37:49 | bauzas | jaypipes: okay, just wanted to make sure you balanced that | |
| 14:37:57 | bauzas | - which I was pretty sure of tho - :) | |
| 14:38:55 | bauzas | jaypipes: FWIW, just reviewing https://review.openstack.org/#/c/448283/ as a side one :) | |
| 14:39:10 | sfinucan | jaypipes, bauzas, ralonsoh: ^ if that point's not in the spec, it should be | |
| 14:39:29 | sfinucan | the "this is a temporary solution until placement can take over" bit, that is | |
| 14:39:42 | bauzas | sfinucan: that's just a gerrit comment for the moment | |
| 14:40:10 | bauzas | sfinucan: I'm not violently opposed on merging that that way | |
| 14:40:23 | bauzas | sfinucan: given the gerrit comment is visible | |
| 14:40:41 | ralonsoh | sfinucan: https://review.openstack.org/#/c/435954/11/specs/pike/approved/enable-sriov-nic-features.rst, L222 | |
| 14:40:41 | bauzas | only if we were having a new PS, I'd ask for that being in the spec | |
| 14:40:43 | sfinucan | bauzas: I should hope you're not "violently opposed" to anything in OpenStack :D | |
| 14:41:13 | sfinucan | ralonsoh: Ah, good stuff :) | |
| 14:41:26 | bauzas | sfinucan: I'm violently opposed to bureaucracy, which is hopefully not a trait of OpenStack :) | |
| 14:41:34 | ralonsoh | sfinucan: :P | |
| 14:42:21 | bauzas | ralonsoh: I'm missing how you express the NIC offload capabilities when booting in the spec | |
| 14:42:29 | bauzas | ralonsoh: same old story I guess with flavors ? | |
| 14:42:48 | bauzas | nevermind L127 | |
| 14:42:56 | ralonsoh | bauzas: flavors is now out of spec | |
| 14:43:05 | ralonsoh | bauzas: yes, using a neutron port | |
| 14:43:41 | sfinucan | edleafe: Aye, death to the fanatics | |
| 14:44:11 | bauzas | edleafe: I can be extremist for food | |
| 14:44:37 | sfinucan | jaypipes: If you're looking at nova-specs, this is a little improvement on an existing spec https://review.openstack.org/#/c/447401/ | |
| 14:44:50 | sfinucan | *one of your existing specs | |
| 14:48:20 | bauzas | ralonsoh: just to understand correctly, we need to define a neutron port (with mentioning its binding profile) before doing nova boot with the flavor expressing the asked capabilities ? | |
| 14:54:33 | jaypipes | sfinucan: reviewed. | |
| 14:58:19 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: SSP emphemeral disk support https://review.openstack.org/443189 | |
| 14:59:24 | openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for GET RP inventories https://review.openstack.org/449765 | |
| 14:59:25 | openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for POST RP inventories https://review.openstack.org/451855 | |
| 15:02:14 | openstackgerrit | Lee Yarwood proposed openstack/nova master: encryptors: Switch to os-brick encryptor classes https://review.openstack.org/391597 | |
| 15:04:11 | openstackgerrit | Eric Fried proposed openstack/nova master: Glance download: only fsync files https://review.openstack.org/451094 | |
| 15:04:47 | efried | mdbooth Good call on those square brackets. ^^ | |
| 15:06:46 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Detach volumes on _rebuild_default_impl failure https://review.openstack.org/442105 | |
| 15:08:37 | ralonsoh | bauzas: sorry, I was afk. That's almost correct: 1) creation of neutron port, 2) define pci filter (with these new parameters) and 3) boot the machine | |
| 15:08:52 | ralonsoh | bauzas: you don't need to modify the flavor | |
| 15:11:25 | ralonsoh | bauzas: when I say "define pci filter" I mean to enable it | |
| 15:15:46 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: config drive https://review.openstack.org/409404 | |
| 15:15:47 | mriedem | apparently it doesn't take much time to get to or through SJC and here i sit with all of this idle time | |
| 15:16:46 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: PCI NUMA Policies https://review.openstack.org/361140 | |
| 15:18:10 | sdague | mriedem: do you want to adopt any of this language to your quota doc - https://review.openstack.org/#/c/450244/ ? I had mostly meant it to be an example, but it looks like the other doc thing merged | |
| 15:18:50 | bauzas | ralonsoh: so, I +W'd it but I have a question https://review.openstack.org/#/c/435954/11/specs/pike/approved/enable-sriov-nic-features.rst@236 | |
| 15:19:23 | jaypipes | ralonsoh: spec is merged. you can un-WIP your patch series :) | |
| 15:19:39 | ralonsoh | bauzas: yes, I'm on it. I know this is the trickiest point | |
| 15:19:46 | ralonsoh | jaypipes: thanks! | |
| 15:20:17 | openstackgerrit | Anusha Unnam proposed openstack/nova master: Add keypairs field to InstancePayload https://review.openstack.org/419730 | |
| 15:20:21 | mriedem | ralonsoh: what is the current state of intel 3rd party nfv ci since wznoinsk left? | |
| 15:20:33 | mriedem | before we talk about adding more new nfv things and new ci | |
| 15:20:52 | mriedem | sdague: maybe? i was going to try and spend some time reading your limits spec | |
| 15:21:04 | ralonsoh | mriedem: this is still in progress. Now I can't tell you (I don't know) which is the current status | |
| 15:21:12 | sdague | mriedem: yeh, the limits one is more important | |
| 15:21:28 | vladikr | jaypipes, bauzas, hey could you please take a look at the xen vgpu spec, when you have a moment. https://review.openstack.org/#/c/450122 - I'm trying to convince the author to make the spec more generic, so we could implement the libvirt side of the story under the same spec | |
| 15:21:37 | ralonsoh | mriedem, bauzas: of course, intel 3rd party CI is the right place to put those tests | |
| 15:21:42 | bauzas | vladikr: sure thing | |
| 15:21:55 | vladikr | bauzas, thanks | |
| 15:22:12 | bauzas | ralonsoh: mriedem: I didn't want to block that spec, but I do wonder why we couldn't have functional in-tree tests for that | |
| 15:22:13 | jaypipes | vladikr: yup, it's in my queue. btw, jianghuaw__ is the author :) | |
| 15:22:37 | vladikr | jaypipes, thanks :) | |
| 15:22:38 | mriedem | bauzas: if we can, then great | |
| 15:22:49 | bauzas | ralonsoh: mriedem: I know it's a huge implementation, but we could use some fixtures for that | |
| 15:22:56 | mriedem | assuming the functional tests don't incorrectly mock out things | |
| 15:23:23 | bauzas | mriedem: for example, some NFVLibvirt fixture providing the capabilities | |
| 15:23:28 | ralonsoh | bauzas, mriedem: that could be the easiest and fastest option | |
| 15:24:04 | bauzas | plus a NFVNeutronPCIportFixture :p | |
| 15:24:42 | ralonsoh | bauzas, mriedem: I need to take a look at SRIOVServersTest | |
| 15:43:02 | openstackgerrit | Jiri Suchomel proposed openstack/nova master: Retry few times when creating the initial connection to libvirtd. https://review.openstack.org/416928 | |
| 15:50:05 | openstackgerrit | Roman Podoliaka proposed openstack/osc-placement master: WIP: implement API version 1.0 https://review.openstack.org/451877 | |
| 16:01:23 | kaisers__ | mdbooth: ping? | |
| 16:08:46 | esberglu | I am upgrading my multinode environment from newton to ocata. Currently I have to deploy the control, then the compute, then go back to the control and run | |
| 16:08:48 | esberglu | "nova-manage cell_v2 simple_cell_setup" | |
| 16:08:52 | esberglu | Otherwise I get an error about no host to cell mapping and can't deploy any instances | |
| 16:08:56 | esberglu | Not sure if there is some sort of devstack configuration that I am missing or if this the correct way to go about doing it | |
| 16:10:09 | dansmith | esberglu: you should be able to run simple_cell_setup prior to doing anything with the computes | |
| 16:10:42 | dansmith | esberglu: this bit of upgrading devstack is actually in grenade though.. are you talking about a real setup, or literally upgrading your devstack? | |
| 16:11:48 | esberglu | Not sure I understand your question | |
| 16:19:55 | mriedem | sdague: done https://review.openstack.org/#/c/440815 | |
| 16:20:12 | mriedem | sdague: i think we'll need to talk about the rollout / upgrade plan at the forum | |
| 16:20:17 | mriedem | but i'm on board with the general direction | |
| 16:20:21 | mriedem | good story telling old man | |
| 16:20:34 | mriedem | there are some wording cleanups that could happen in a follow up for clarification, but otherwise lgtm | |
| 16:40:14 | openstackgerrit | Samantha Blanco proposed openstack/nova master: Make flavor-rxtx policy more granular https://review.openstack.org/444036 | |
| 16:49:58 | openstackgerrit | Dan Smith proposed openstack/nova master: Teach HostAPI about cells https://review.openstack.org/442162 | |
| 16:54:32 | alextricity25 | has anyone seen this before: | |
| 16:54:43 | alextricity25 | /openstack/log/605056-infra02_nova_api_os_compute_container-a15d94c3/nova/nova-api-os-compute.log:2017-03-29 19:01:07.237 10449 DEBUG nova.api.openstack.wsgi [req-4219ada0-490c-46c5-870f-27823c4d6e2c e1eeaf7a6cec4756aa2880efda64448f fdf3fe3df3f5428a8b30daa70736bc6a - default default] Returning 400 to user: Block Device Mapping is Invalid: failed to get volume c82eb821-ec3a-4ce6-8abc-12466d1c7fe0. __call__ /opens | |
| 17:08:05 | bauzas | dansmith: heh, fun tweet | |
| 17:08:18 | bauzas | dansmith: but honestly, the scheduler is good :) | |
| 17:08:39 | bauzas | dansmith: the real problem is not about the scheduler, rather about how the resources are asked :) | |
| 17:08:45 | dansmith | I seriously want to burn it all right now | |
| 17:08:49 | dansmith | and I mean ALL of it | |
| 17:08:59 | bauzas | why ? because of my comments ? :p | |
| 17:09:36 | dansmith | no, because it's terrible | |
| 17:10:03 | bauzas | well, it took me 1 year to understand all the scheduler modules :p | |
| 17:10:16 | bauzas | and then, I discovered the RT :p | |
| 17:11:03 | bauzas | tbh, I'd prefer to kill the RT *before* killing the scheduler :p | |
| 17:11:11 | bauzas | I mean, filters are nice | |
| 17:11:26 | bauzas | of course, we have problems with the HostManager :) | |
| 17:23:50 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Move detach logic from manager into driver BDM https://review.openstack.org/439520 | |
| 17:29:55 | sajeesh | ping jaypipes | |