Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-06-22
15:57:46 cfriesen_ andreykurilin: did you want me to copy it to the legacy tests so it gets run in both places?
15:58:40 andreykurilin cfriesen_: you can create a base class in the legacy module and inherit from it in your new TestCase
16:00:03 cfriesen_ andreykurilin: you're thinking a new base class with just the one test?
16:00:11 andreykurilin yup
16:00:23 cfriesen_ andreykurilin: okay, can try that
16:01:01 sfinucan gibi, Kevin_Zheng: I assume [1] needs to be rebased to work with [2] now? [1] https://review.openstack.org/#/c/459493/ [2] https://review.openstack.org/#/c/463001/
16:01:39 sfinucan mkucia: I should know that. Lemme dig out notes
16:03:14 mkucia sfinucan: That would be great, thanks!
16:06:44 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: [WIP] Add abstract OVSDB API https://review.openstack.org/476612
16:06:57 sfinucan mkucia: Looks to be something to do with performance
16:07:02 sfinucan https://github.com/openstack/nova/blob/master/nova/pci/stats.py#L39-L44
16:07:23 sfinucan bauzas, jaypipes or sean-k-mooney might be able to offer more info, but that would be my guess
16:07:55 sfinucan What's that now, ralonsoh? ;)
16:08:06 jaypipes mkucia: pci device pools are stupid, IMHO. they need to be killed off.
16:08:14 mkucia I see. Thanks for the tip on the people.
16:09:11 mkucia jaypipes: Agree, there are a frew things in nova PCI handling that seems a bit off.
16:09:25 sfinucan mkucia: Careful - you'll set him off ;)
16:09:35 mkucia :O
16:09:50 sfinucan (jaypipes is a not a fan of the PCI or NUMA code in nova, heh)
16:11:04 jaypipes bauzas, dansmith, mriedem: speaking of PCI devices... this would be great to get in. https://review.openstack.org/#/c/469147/
16:12:16 ralonsoh sfinucan: hey
16:12:38 ralonsoh sfinucan: you mean https://review.openstack.org/476612?
16:15:06 sfinucan ralonsoh: Indeed
16:15:13 sfinucan Good to see. I'll keep an eye on that
16:15:56 ralonsoh sfinucan: perfect! This is a WIP. Then I'll need to: 1) add unit tests, 2) add native implementation and 3) probably add functional tests
16:16:02 sfinucan I was trying to something kind of similar with the linux-bridge stuff, but got annoyed by how "porcelain'y" pyroute2 is (https://review.openstack.org/#/c/451433/)
16:16:21 sfinucan ralonsoh: Sounds like a plan to me. I assume we can kill the non-native version then?
16:16:52 ralonsoh sfinucan: we'll keep both (for Windows) and configurable via conf file
16:16:58 ralonsoh sfinucan: like in Neutron
16:17:18 sfinucan Ah, Windows. Gotcha
16:17:26 ralonsoh sfinucan: I'll take a look at https://review.openstack.org/#/c/451433/
16:17:31 sfinucan That'll do nicely. Ping you when it's ready for eyes :)
16:17:39 sfinucan ralonsoh: Don't bother. It's not even close to complete
16:17:42 ralonsoh sfinucan: ok!
16:17:45 sfinucan More of a hobby project, really
16:21:29 sfinucan jaypipes: Ahem https://docs.openstack.org/contributor-guide/writing-style/openstack-components.html https://review.openstack.org/#/c/475810/1/doc/source/scheduling.rst
16:21:44 sfinucan But I can go with 'Nova' if you _reallllly_ want it
16:22:14 jaypipes sfinucan: I stand corrected.
16:23:04 sfinucan jaypipes: Thoughts on that diagram tool in general? Seems like a lower barrier to entry than Inkscape etc., even if the output is less pretty?
16:23:21 jaypipes sfinucan: yup, I like it a lot.
16:23:27 sfinucan sweet
16:23:46 sfinucan with that, /me is off to tag (rugby)
16:47:54 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Rehome unit tests to ``tests\unit`` folder https://review.openstack.org/476623
16:52:34 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Handle uuids in os-hypervisors API https://review.openstack.org/471899
16:52:34 openstackgerrit Matt Riedemann proposed openstack/nova master: Use uuid for id in os-services API https://review.openstack.org/464280
16:52:35 openstackgerrit Matt Riedemann proposed openstack/nova master: Support paging over compute nodes with a uuid marker https://review.openstack.org/476625
16:52:36 mriedem dansmith: page-o-rama ^
16:52:39 mriedem dtp: ^
16:55:59 openstackgerrit Chris Friesen proposed openstack/python-novaclient master: 2.47: Show flavor info in server details https://review.openstack.org/435141
17:08:34 openstackgerrit Ed Leafe proposed openstack/nova master: Extract custom resource classes from flavors https://review.openstack.org/473627
17:08:50 openstackgerrit Jay Pipes proposed openstack/nova master: placement: support GET /allocation_candidates https://review.openstack.org/475448
17:08:50 openstackgerrit Jay Pipes proposed openstack/nova master: move resources_from_request_spec() to utils https://review.openstack.org/476629
17:08:51 openstackgerrit Jay Pipes proposed openstack/nova master: Add uuid to scheduler's HostState object https://review.openstack.org/476630
17:08:51 openstackgerrit Jay Pipes proposed openstack/nova master: placement: scheduler uses allocation candidates https://review.openstack.org/476631
17:08:52 openstackgerrit Jay Pipes proposed openstack/nova master: WIP - placement: claim resources in scheduler https://review.openstack.org/476632
17:12:51 openstackgerrit Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622
17:12:53 openstackgerrit Artom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387
17:13:02 openstackgerrit Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082
17:13:13 openstackgerrit Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941
17:13:14 openstackgerrit Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021
17:13:48 melwitt mriedem: yes, those are ready for review. however I wanted to run it by jaypipes because, I'm interested in the "right way" to solve the BFV 0 disk problem in placement but I have no idea what to do or if it's possible right now
17:14:21 dtp mriedem want me to look those over?
17:19:58 mriedem dtp: i want everyone and their dog to look it over
17:25:12 dtp Bianca and Noodles are looking now
17:25:38 openstackgerrit melanie witt proposed openstack/nova master: Detach volume after deleting instance with no host https://review.openstack.org/340614
17:26:09 mriedem heh
17:28:24 openstackgerrit Merged openstack/nova master: Update api-ref to indicate swap param https://review.openstack.org/474517
17:29:59 openstackgerrit Alex Szarka proposed openstack/nova master: Simplify a condition https://review.openstack.org/476637
17:38:05 openstackgerrit Maciej Kucia proposed openstack/nova master: SR-IOV passthrough: Check PF only if VF is enabled https://review.openstack.org/476642
18:00:05 openstackgerrit Alex Szarka proposed openstack/nova master: Reduce code complexity - libvirt/config.py https://review.openstack.org/359879
18:00:12 openstackgerrit Alex Szarka proposed openstack/nova master: ransform instance-live_migration_abort notification https://review.openstack.org/420453
18:07:07 melwitt jaypipes: FYI, here's the root_gb=0 patches again, I rebased them yesterday. I'm interested in the placement solution but didn't know if it's possible to start on that right now or if other pieces need to be put in place first https://review.openstack.org/#/c/428481/
18:14:22 edleafe melwitt: You might want to take a look at https://review.openstack.org/#/c/473627/ - it includes the ability to zero-out standard resource class amounts in scheduler, while still having them present in the underlying flavor
18:14:58 melwitt edleafe: a-ha, cool. will look at that then. thanks
18:16:59 openstackgerrit Alex Szarka proposed openstack/nova master: Reduce code complexity - linux_net.py https://review.openstack.org/359871
18:19:46 openstackgerrit Alex Szarka proposed openstack/nova master: Changed maximum code complexity to 30 https://review.openstack.org/351176
18:23:27 openstackgerrit Alex Szarka proposed openstack/nova master: Refactor init_instance:resume_guests_state https://review.openstack.org/349582
18:32:03 openstackgerrit Alex Szarka proposed openstack/nova master: Split exception cases in manager.py https://review.openstack.org/347275
18:34:36 mriedem sdague: you know how we're going to fold the GET /os-hypervisors/{hypervisor_hostname_pattern}/search and GET /os-hypervisors/{hypervisor_hostname_pattern}/servers APIs into GET /os-hypervisors (index) and GET /os-hypervisors/detail (detail)?
18:34:59 mriedem we also have this weird GET /os-hypervisors/{hypervisor_id}/uptime which is like GET /os-hypervisors/{hypervisor_id} but it includes this uptime field
18:35:11 mriedem it seems like uptime should just be a query parameter on GET /os-hypervisors/{hypervisor_id}
18:35:21 sdague yeh, or just be there
18:35:25 sdague is it expensive to get?
18:35:53 mriedem it's an rpc call to the compute
18:36:00 mriedem it's an uptime call on the hypervisors
18:36:36 mriedem maybe ok to leave that separate then, idk
18:36:57 mriedem it's also weird that you can get servers in the response when listing hypervisors but not when showing just hypervisor details
18:37:04 mriedem so i'm probably going to have to add that to show also
18:38:08 edleafe mriedem: I won't be at my computer for the Nova meeting. Maybe jaypipes or you could fill in the scheduler subteam status?
18:38:37 mriedem ok
18:39:35 sdague mriedem: so... why don't we put uptime in the compute checkin?
18:39:56 mriedem shrug
18:40:03 mriedem because it was probably a bolt on after that
18:40:09 sdague sure
18:40:24 mriedem i'm already bending over backward on rewriting these apis so i'm not going to go so far as to change all of that lower level plumbing
18:40:36 mriedem just going to leave some todos in this patch for now
18:40:40 sdague ok
18:41:29 mriedem i was just updating the api ref and got thinking, it's really weird that i can list computes w/o details, but with servers on each host, but i can't show host details and have that include the servers in it
18:41:37 mriedem just a really weirdly bolted together set of apis
18:41:41 mriedem client side people must hate it

Earlier   Later