Earlier  
Posted Nick Remark
#openstack-nova - 2017-06-22
14:47:54 Kevin_Zheng maybe, I saw some coffee shop in the campus :)
14:49:43 alex_xu mriedem: I think shenzhen should be have many food, a donuts is to cheap for Kevin_Zheng :)
14:50:23 Kevin_Zheng :P
14:50:38 mriedem alex_xu: what would you suggest?
14:51:54 alex_xu mriedem: I'm not sure, but I'm sure Kevin_Zheng and his team members should have some plan, just remember to follow them :)
14:52:15 Kevin_Zheng the test passed
14:52:22 alex_xu \o/
14:53:30 openstackgerrit Balazs Gibizer proposed openstack/nova master: Use enum value instead of string service name https://review.openstack.org/476538
14:54:21 dansmith mriedem: working through meeting brain, but I think that paste makes sense
14:54:42 Kevin_Zheng mriedem: last time we had some coconut hotpot, tastes good :)
14:55:30 mriedem alex_xu: that's my plan
14:56:02 mriedem although now i'm finding out that dims and i might be in different hotels - the hotel email is all in chinese so it's hard to tell just where i'm actually staying
14:57:07 openstackgerrit Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622
14:57:10 openstackgerrit Artom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387
14:57:18 openstackgerrit Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082
14:57:22 Kevin_Zheng amber plaza and amber xxx ?
14:57:28 openstackgerrit Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941
14:57:29 openstackgerrit Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021
14:58:08 alex_xu mriedem: put the chinese in the google maps, you may get the english name.
14:58:14 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593
14:58:52 openstackgerrit Alex Szarka proposed openstack/nova master: [WIP]Transform instance-live_migration_abort notification https://review.openstack.org/420453
15:01:12 alex_xu mriedem: it will be great to prepare some notes for hotel's chinese address. the taxi driver doesn't know English totally. that is the way to ensure jaypipes arrive China and back to US safely :)
15:01:48 gibi mriedem: hi! I cannot joint to the nova meeting today. There was no real notification subteam meeting this week as I was alone.
15:03:13 mriedem alex_xu: yeah i printed out the email for the hotel already
15:03:20 mriedem gibi: ok
15:04:05 alex_xu mriedem: cool :)
15:05:35 cfriesen_ when detaching a volume, does nova/libvirt wait for an ack from the guest the way we do for PCI devices? (If so, is it possible for the guest to do something to cause nova to wait forever?)
15:09:49 openstackgerrit Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622
15:09:52 openstackgerrit Artom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387
15:10:02 openstackgerrit Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082
15:10:52 openstackgerrit Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021
15:10:52 openstackgerrit Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941
15:17:30 cfriesen_ mriedem: you've been reviewing the cinder stuff recently...is there a way to tell nova to just rudely detach a cinder volume without worrying about corrupting it (since it's going to be deleted right away, for example)
15:17:54 cfriesen_ mriedem: we're hitting a case where a heat stack delete is hanging because the guest is heavily using a cinder volume and so nova can't detach it
15:19:05 mriedem cfriesen_: not from the nova side
15:19:13 mriedem there is a force detach api in the cinder side i think
15:19:24 mriedem which is just to change the state on the volume from in-use to available so you can delete it
15:19:28 mriedem but,
15:19:52 mriedem you'll likely have issues deleting the instance then because nova will get volume not found - which may be ignored with a warning, not sure
15:20:50 cfriesen_ mriedem: so basically we need to quiesce the guest before detaching a cinder volume?
15:23:56 openstackgerrit Alex Szarka proposed openstack/nova master: Transform instance-live_migration_abort notification https://review.openstack.org/420453
15:24:29 mriedem cfriesen_: i think the only time we quiece the guest when a volume is attached is when taking a snapshot
15:25:11 cfriesen_ mriedem: I mean that more generally, like having the end user shut things down in the guest.
15:26:08 mriedem oh
15:26:15 mriedem does nova fail or just hang?
15:26:18 mriedem nova/os-brick
15:27:05 cfriesen_ mriedem: nova tries 7 times to detach the device and then gives up
15:27:54 cfriesen_ mriedem: not sure if nova returns an error or if heat ignores it
15:28:44 mriedem oh, well, it's a cast
15:28:49 mriedem so it's all async from a heat pov
15:29:00 mriedem so heat would have to timeout waiting for the volume to be detached
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

Earlier   Later