Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-06-22
15:30:40 cfriesen_ mriedem: it's kind of too bad we can't do an unclean teardown if we're going to be deleting everything anyway
15:32:16 openstackgerrit Alex Szarka proposed openstack/nova master: Transform instance.resize.error notifications https://review.openstack.org/417882
15:39:25 bauzas jaypipes: just a rebase for https://review.openstack.org/#/c/469037/ given you already +2d it
15:39:44 bauzas mriedem: dansmith: possibly some others could look at it ^
15:40:11 openstackgerrit Sylvain Bauza proposed openstack/nova master: Fix ChanceScheduler returning a wrong node https://review.openstack.org/466725
15:41:08 jaypipes bauzas: done
15:41:30 bauzas cools
15:45:23 openstackgerrit Ildiko Vancsa proposed openstack/nova master: WIP/POC Implement new attach Cinder flow https://review.openstack.org/330285
15:49:07 mkucia Hi! I wonder if someone know what is the story behind `PCI device pools` in nova. Were those introduced for performance reasons? Why not plain list of devices?
15:51:22 cfriesen_ andreykurilin: are you around?
15:51:54 andreykurilin cfriesen_: hi! a bit (will need to go in 10 minutes)
15:57:15 cfriesen_ andreykurilin: about https://review.openstack.org/#/c/435141/, if I move the test into functional/v2/legacy/test_servers,py, it seems like those would run with the 2.1 API version, not 2.47
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

Earlier   Later