Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-03-08
22:26:17 mriedem in pci_device_get_by_addr
22:26:38 openstackgerrit Dan Smith proposed openstack/nova master: Change MQ targeting to honor only what is in the context https://review.openstack.org/438023
22:26:38 openstackgerrit Dan Smith proposed openstack/nova master: Teach os-migrations about cells https://review.openstack.org/442685
22:26:39 openstackgerrit Dan Smith proposed openstack/nova master: Add cell field to Destination object https://review.openstack.org/438024
22:26:39 openstackgerrit Dan Smith proposed openstack/nova master: Teach simple_tenant_usage about cells https://review.openstack.org/441388
22:26:40 openstackgerrit Dan Smith proposed openstack/nova master: Teach os-aggregates about cells https://review.openstack.org/442200
22:26:40 openstackgerrit Dan Smith proposed openstack/nova master: Refactor cell loading in compute/api https://review.openstack.org/443337
22:26:44 mriedem if we don't want to expose the pcidevice.id in the rest api, and get them by that id, and we don't have a uuid, we need something else
22:27:16 mriedem unless we add a uuid to the pcidevices table/object
22:27:31 dansmith *shudder*
22:27:45 mriedem yeah the only unique constraints are the id (primary key) and node_id/address
22:27:52 mriedem so, we'd have to add a uuid
22:28:14 mriedem heh, or we just deprecate that dumbass api and say you can't use it with multiple cells :)
22:28:22 dansmith heh
22:28:31 mriedem i feel like that was a half-baked api
22:28:42 dansmith s/api//
22:29:06 mriedem and if you want to know what pci devices are on a given compute node, you get them via the compute node resource provider
22:29:12 mriedem traits on the resource provider
22:29:23 mriedem jaypipes: will pcidevices be a trait on a compute node resource provider?
22:29:34 mriedem will/can
22:31:56 jaypipes mriedem: depends on what kind of PCI device.
22:32:06 mriedem dansmith: shart, services table doesn't have uuid either
22:32:57 jaypipes mriedem: SRIOV physical functions will be either child providers of a compute node *or* child providers of a NUMA socket resource provider, depending on whether the SRIOV NIC is affined to a particular NUMA socket.
22:33:29 efried mikal - Any chance we can start getting some reviews on change sets under https://review.openstack.org/#/q/topic:bp/powervm-nova-compute-driver ? The ones with +1s from nova-powervm cores (e.g. thorst adreznec esberglu) are ready for your look. Conveniently numbered, too ;-)
22:33:48 jaypipes mriedem: but pcidevices will definitely not be a "trait". Rather, things like the physical network tag would be a trait on the SRIOV physical function resource provider.
22:33:59 mriedem jaypipes: ack
22:34:24 mriedem i was looking for a way to weasle out of the compute rest api needing to show pcidevice per node information
22:34:54 openstackgerrit Sarafraj Singh proposed openstack/nova-specs master: WIP: Replace live migration steps and delays with memory iteration logic https://review.openstack.org/443339
22:34:57 mriedem it's all information in os-pci, like all pcidevices on a node, and all pcidevices for a server
22:35:08 mriedem *informational
22:35:11 jaypipes mriedem: right.
22:35:24 mriedem and if i could say, get that information elsewhere...
22:35:26 mriedem my life would be easier
22:35:54 mriedem seems to fall into the 'nova does not want to be a metrics gatherer' anymore stadium
22:36:02 jaypipes mriedem: eventually the placement API will be able to represent the tree of providers on a compute node, yes. But things like PCI address information will not be in the placement API.
22:36:14 mriedem sure
22:36:24 mriedem ok, well i'll just spec this up with the way things are today then
22:36:33 jaypipes kk
22:36:45 jaypipes mriedem: I'm still yet to get to that spec though...
22:36:53 jaypipes the pcidevices one you linked to earlier.
22:36:56 jaypipes have it on the list.
22:37:11 jaypipes currently working on a review of moshele's SRIOV port pair allocation one.
22:37:18 jaypipes https://review.openstack.org/#/c/182242/33/specs/pike/approved/user-controlled-sriov-ports-allocation.rst
22:37:30 mriedem jaypipes: yeah this is different actually
22:37:33 mriedem related to cells v2
22:37:37 mriedem shenanigans
22:37:37 jaypipes ah, ok.
22:37:39 jaypipes sorry
22:37:42 mriedem don't be
22:38:01 mriedem i just use you as my encyclopedia of nova horrors
22:44:38 openstackgerrit Sujitha proposed openstack/nova master: Reduce calls to load_tags() to 0 https://review.openstack.org/435146
22:47:08 mriedem jaypipes: dansmith: i'm going to head home, but wrote up the abstract for the spec i'm going to work on later https://blueprints.launchpad.net/nova/+spec/service-hyper-pci-uuid-in-api
22:47:15 mriedem let me know if i screwed anything up in there
22:47:22 jaypipes mriedem: will do. thx.
22:53:04 openstackgerrit Dan Peschman proposed openstack/nova master: No API cell up-call to delete consoleauth tokens https://review.openstack.org/415922
22:53:35 openstackgerrit Eric Fried proposed openstack/nova master: WIP: PowerVM Driver: SSP emphemeral disk support https://review.openstack.org/443189
22:54:07 openstackgerrit Eric Fried proposed openstack/nova master: WIP: PowerVM Driver: SSP emphemeral disk support https://review.openstack.org/443189
23:13:41 openstackgerrit Eric Fried proposed openstack/nova master: WIP: PowerVM Driver: SSP emphemeral disk support https://review.openstack.org/443189
23:28:32 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/443357
23:33:07 openstackgerrit Jay Pipes proposed openstack/nova master: virt: implement get_inventory() for Ironic https://review.openstack.org/441544
23:33:12 jaypipes dansmith: ^^
23:34:56 dansmith jaypipes: done thanks
23:35:03 jaypipes dansmith: danke
#openstack-nova - 2017-03-09
00:03:09 alex_xu mriedem: will check that patch in the morning
00:03:47 alex_xu jaypipes: something looking for your feedback https://review.openstack.org/441829, cdent didn't want the cache :)
00:04:23 jaypipes alex_xu: yeah, it's next on my list. currently reviewing https://review.openstack.org/#/c/435954/6/specs/pike/approved/enable-sriov-nic-features.rst
00:05:00 alex_xu jaypipes: thanks :)
00:09:59 dansmith what kind of monster doesn't want a cache?
00:21:02 dansmith alex_xu: can you drop your -1 from this now? https://review.openstack.org/#/c/441388/9
00:25:30 mriedem dansmith: just setup amazon household for prime, finally
00:25:34 mriedem oh so sweet
00:25:50 mriedem now i don't have to secretly log into laura's account to order things
00:25:52 dansmith you mean you just got prime, or just got setup so laura can order stuff without you?
00:25:55 dansmith oh heh
00:25:59 mriedem she's had prime for years
00:26:01 dansmith welcome to 2009!
00:26:03 mriedem but has always been my proxy
00:26:06 mriedem \o/
00:26:10 jaypipes mriedem: can you record the Grand Tour and let me stream from your house? :)
00:27:05 dansmith jaypipes: I know you're old, but... amazon prime video is streaming and generally "recording" it is hard to do
00:27:56 jaypipes dansmith: I was expecting mriedem to set up a camcorder, tape it, upload the tape to his personal server and let me stream it from there.
00:28:10 jaypipes rofl
00:29:01 jaypipes alex_xu: ok, sorry, I goofed. you were actually third in the queue, right behind my current review on https://review.openstack.org/#/c/364468/11/specs/pike/approved/reserve-numa-with-pci.rst
00:43:19 openstackgerrit Dan Smith proposed openstack/nova master: Change MQ targeting to honor only what is in the context https://review.openstack.org/438023
00:43:19 openstackgerrit Dan Smith proposed openstack/nova master: Teach os-migrations about cells https://review.openstack.org/442685
00:43:20 openstackgerrit Dan Smith proposed openstack/nova master: Add cell field to Destination object https://review.openstack.org/438024
00:43:21 openstackgerrit Dan Smith proposed openstack/nova master: Teach simple_tenant_usage about cells https://review.openstack.org/441388
00:43:21 openstackgerrit Dan Smith proposed openstack/nova master: Teach os-aggregates about cells https://review.openstack.org/442200
00:44:18 mriedem who wants to volunteer to run the meeting in the morning?
00:57:53 openstackgerrit Dan Smith proposed openstack/nova master: Change MQ targeting to honor only what is in the context https://review.openstack.org/438023
00:57:54 openstackgerrit Dan Smith proposed openstack/nova master: Teach os-migrations about cells https://review.openstack.org/442685
00:57:54 openstackgerrit Dan Smith proposed openstack/nova master: Add cell field to Destination object https://review.openstack.org/438024
00:57:55 openstackgerrit Dan Smith proposed openstack/nova master: Teach simple_tenant_usage about cells https://review.openstack.org/441388
00:57:55 openstackgerrit Dan Smith proposed openstack/nova master: Teach os-aggregates about cells https://review.openstack.org/442200
01:14:15 dansmith gfdi
01:14:28 dansmith jaypipes: your requests have thrown off the whole cosmic balance of my patches
01:14:44 jaypipes dansmith: :(
01:14:48 jaypipes sorry duder.
01:15:39 melwitt jaypipes: I've got the transactional safety change in my working dir. just finishing up some changes to the quota count patch
01:16:14 jaypipes melwitt: coolio.

Earlier   Later