| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-01 | |||
| 10:06:30 | openstackgerrit | Claudiu Belu proposed openstack/nova master: API: Adds instance live-resize https://review.openstack.org/184584 | |
| 10:06:43 | kashyap | stephenfin: Near as I see, it a reasonably straightforward change of modifying the `ovs-vsctl` invocation to ignore if a (vhost-user) port already exists on a bridge device? | |
| 10:06:59 | kashyap | s/it a/isn't it a/ | |
| 10:07:06 | stephenfin | kashyap: That's what I'm thinking hence the question :) | |
| 10:07:08 | stephenfin | It seems odd | |
| 10:10:31 | openstackgerrit | Claudiu Belu proposed openstack/nova-specs master: Adds spec for instance live resize https://review.openstack.org/141219 | |
| 10:11:50 | bauzas | stephenfin: kashyap: yeah my concern is that I wonder if reusing a port is good or not | |
| 10:11:58 | bauzas | deleting and recreating a port is one thinh | |
| 10:12:07 | bauzas | reusing an existing one is a totally different story | |
| 10:12:23 | stephenfin | bauzas, kashyap, sean-k-mooney: Jackpot https://github.com/openstack/nova/commit/33cc64fb817 | |
| 10:12:27 | bauzas | for a reboot, it's a reasonable assumption | |
| 10:12:37 | bauzas | for any other case, I don't think so | |
| 10:12:59 | bauzas | hah ! | |
| 10:21:15 | cdent | stephenfin: you just +Ad the child of https://review.openstack.org/#/c/543262/ . Do you have thoughts on how/if/when/what needs to change on the parent (which is a much bigger change). Presumably unparent the child is the quickest option? | |
| 10:21:15 | kashyap | stephenfin: So upon every restart of Compute node the ports are deleted and re-created — know why is that the case? | |
| 10:21:52 | openstackgerrit | Chris Dent proposed openstack/nova-specs master: Update placement aggregates spec to clarify generation handling https://review.openstack.org/548237 | |
| 10:22:02 | stephenfin | cdent: I'd just unparent it, aye. I avoided the parent as it was marked WIP | |
| 10:22:32 | cdent | stephenfin: cool, but at some point it would be good to work out how to approach the issues that the parent solves | |
| 10:22:43 | stephenfin | kashyap: Based on the commit message, if you don't then this happens https://bugs.launchpad.net/nova/+bug/1270973 | |
| 10:22:45 | openstack | Launchpad bug 1270973 in OpenStack Compute (nova) "Remove and recreate interface if already exists" [Medium,Fix released] - Assigned to Aaron Rosen (arosen) | |
| 10:23:00 | stephenfin | cdent: I can take a look | |
| 10:23:23 | cdent | stephenfin: I don't think there's any rush but the changes point out some interesting things about the "way we do stuff" | |
| 10:23:26 | melwitt | mriedem: do you know if there's a way in launchpad to see the date when something was approved? I'm keeping a spreadsheet for rocky | |
| 10:29:33 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: Return Generation from Resource Provider Creation https://review.openstack.org/548903 | |
| 10:29:39 | efried | jaypipes, cdent: ^ | |
| 10:29:59 | Spazmotic | melwitt: does the full activity log not show the date for the status change? | |
| 10:30:16 | Spazmotic | n/m misread | |
| 10:30:20 | Spazmotic | you're talking about bluerprings :D | |
| 10:34:25 | jaypipes | efried, done | |
| 10:37:19 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: Return Generation from Resource Provider Creation https://review.openstack.org/548903 | |
| 10:37:29 | efried | jaypipes: ^ Dammit, how does that *happen* ?? | |
| 10:38:26 | Spazmotic | Trailing Whitespace will uhh... find a way | |
| 10:40:11 | mriedem | melwitt: nope | |
| 10:40:15 | mriedem | melwitt: LP doesn't | |
| 10:40:21 | mriedem | only when it's completed, | |
| 10:40:45 | mriedem | so if it has a spec, you can see when the spec was approved; otherwise if i approve a bp, i leave a comment in the whiteboard saying when i approved it | |
| 10:41:46 | melwitt | yeah, okay. I noticed that, so I'll be sure to add whiteboard comment for that | |
| 10:46:35 | jaypipes | Spazmotic, done | |
| 10:50:07 | openstackgerrit | Dinesh Bhor proposed openstack/nova-specs master: Strict isolation of group of hosts for images https://review.openstack.org/381912 | |
| 10:54:05 | openstackgerrit | Dinesh Bhor proposed openstack/nova-specs master: Strict isolation of group of hosts for images https://review.openstack.org/381912 | |
| 10:58:04 | Spazmotic | Thanks Jay :) | |
| 10:58:30 | openstackgerrit | Jianle He proposed openstack/nova-specs master: Back up volume-backed-instance https://review.openstack.org/530214 | |
| 11:00:15 | hrw | when/where nova group photo will be? | |
| 11:02:41 | takashin | hrw: "11:10 - 11:20 AM: nova team photo. Meet at registration at 11:10 sharp." in https://etherpad.openstack.org/p/nova-ptg-rocky | |
| 11:04:15 | hrw | thx | |
| 11:05:24 | openstackgerrit | Chris Dent proposed openstack/nova-specs master: Express forbidden traits in placement API https://review.openstack.org/548915 | |
| 11:09:47 | openstackgerrit | Chris Dent proposed openstack/nova master: Move db MAX constants to own file https://review.openstack.org/543469 | |
| 11:15:26 | openstackgerrit | Brooks Kaminski proposed openstack/nova master: XenAPI: XCP2.1+ Swallow VDI_NOT_IN_MAP Exception https://review.openstack.org/538415 | |
| 11:16:49 | Spazmotic | Thanks for taking the time Jay :) | |
| 11:23:27 | Spazmotic | Thanks for the review jaypipes, and for taking the time during the PTG | |
| 11:27:30 | hrw | are there nova stickers? | |
| 11:32:03 | jaypipes | hrw, you might ask one of the foundation folks. they may have extras somewhere... maybe diablo_rojo (Kendall) | |
| 11:32:14 | hrw | ok | |
| 11:32:40 | melwitt | I'm to pick up stickers on friday | |
| 11:33:06 | jianghuaw_ | Spazmotic: jfyi. I've put both of your patches in this priorities tracking page: https://etherpad.openstack.org/p/rocky-nova-priorities-tracking | |
| 11:34:24 | jianghuaw_ | It's recommended from the PTG to use that page to tracking priority things. | |
| 11:34:31 | Spazmotic | Ah I see. Thanks jianghuaw_ | |
| 11:34:48 | jianghuaw_ | np | |
| 11:35:51 | Spazmotic | I like this, will make for nicer organization | |
| 11:36:23 | melwitt | yep, thanks jianghuaw_ | |
| 11:38:21 | hrw | jianghuaw_: thx. added entry ;D | |
| 11:43:21 | jianghuaw_ | Spazmotic and hrw: you are welcome. Actually I like the priority page. Very useful for tracking ready-for-review patches. Thanks melwitt for bring it back. | |
| 11:43:43 | Spazmotic | Yeah same here I really like it. Even for those not on the subteam it allows for me for instance to quickly see if any XenAPI fixes need at tention | |
| 11:43:47 | Spazmotic | well done melwitt | |
| 11:45:43 | melwitt | Spazmotic: glad you like it but it's not originally my idea :) we used to use the priorities etherpad in pike, just trying to go back to that and keep it top-of-mind for people | |
| 11:46:43 | efried | Whatever, melwitt, you get full credit. Deal. | |
| 11:46:51 | melwitt | heh | |
| 12:06:04 | openstackgerrit | Jianle He proposed openstack/nova-specs master: Back up volume-backed-instance https://review.openstack.org/530214 | |
| 12:15:04 | openstackgerrit | Chris Dent proposed openstack/nova-specs master: Express forbidden traits in placement API https://review.openstack.org/548915 | |
| 12:17:52 | danpawlik | Hello, I have small question about nova placement configuration. I see that in puppet openstack integration, nova placement DB connection goes to nova_placement DB (new DB created for placement), but in some manuals they write to use nova_api. What is the correct DB for Ocata release? | |
| 12:19:02 | danpawlik | I see that since pike (https://docs.openstack.org/nova/pike/user/placement.html) there is a plan to add speciall DB for placement. So it looks like there is a bug in puppet-openstack-integration for Ocata branch | |
| 12:19:04 | danpawlik | or? | |
| 12:22:12 | cdent | danpawlik: can you point me to the puppet code? | |
| 12:22:37 | cdent | danpawlik: the hope is that eventually there would be a different database, but at the moment it is usually the nova api_db | |
| 12:27:13 | openstackgerrit | Chris Dent proposed openstack/nova-specs master: Express forbidden traits in placement API https://review.openstack.org/548915 | |
| 12:31:28 | openstackgerrit | Eric Fried proposed openstack/nova master: placement: Return new provider from POST /rps https://review.openstack.org/548934 | |
| 12:31:55 | efried | jaypipes: cdent ^ | |
| 12:55:33 | openstackgerrit | Merged openstack/nova master: Remove unused LOG variables https://review.openstack.org/546097 | |
| 12:55:42 | openstackgerrit | Merged openstack/nova master: Check for leaked server resource allocations in post_test_hook https://review.openstack.org/538510 | |
| 12:55:49 | openstackgerrit | Merged openstack/nova master: Add check for redundant import aliases https://review.openstack.org/543995 | |
| 12:56:00 | openstackgerrit | Merged openstack/nova master: Address nits in change I7e01f95d7173d9217f76e838b3ea71555151ef56 https://review.openstack.org/532898 | |
| 12:56:12 | openstackgerrit | Merged openstack/nova master: Remove warning in feature support matrix page https://review.openstack.org/544017 | |
| 12:56:23 | openstackgerrit | Merged openstack/nova master: [placement] api-ref: Fix a missing response code https://review.openstack.org/546494 | |
| 13:09:35 | openstackgerrit | Merged openstack/nova master: Enable native mode for ScaleIO volumes https://review.openstack.org/545304 | |
| 13:15:20 | danpawlik | cdent: sure, https://github.com/openstack/puppet-openstack-integration/blob/stable/ocata/manifests/nova.pp#L111 | |
| 13:43:14 | stephenfin | melwitt: I'm trying to organize accommodation for tomorrow night. Will be back in he room as soon as that's sorted | |
| 13:43:44 | stephenfin | melwitt: If you're starting on the neutron session, maybe do the other stuff first and I'll be back for the NUMA-aware vSwitch section, please :) | |
| 13:46:58 | melwitt | stephenfin: ack | |
| 13:48:14 | melwitt | stephenfin: we've been told the croke park hotel has rooms available that are not released to the public. did you ask there first? | |
| 14:02:28 | danpawlik | cdent: probably you didn't receive the url | |
| 14:02:45 | danpawlik | cdent: https://github.com/openstack/puppet-openstack-integration/blob/stable/ocata/manifests/nova.pp#L111 | |
| 14:02:51 | cdent | danpawlik: yeah, sorry, left the network for lunch | |
| 14:02:58 | cdent | and now I'm on some horrible network | |
| 14:03:19 | danpawlik | xD | |
| 14:05:05 | cdent | interesting, I'm going to have to do some grepping | |
| 14:05:15 | cdent | which will mean some waiting on the network | |
| 14:09:10 | danpawlik | cdent: I can send you url for nova module if you want | |
| 14:09:40 | cdent | danpawlik: grepping done. looks like EmilienM did that change around about the time when we were recommending that a separate database was the way to go, but then we didn't quite make it there | |
| 14:10:00 | cdent | which probably means that though the puppet variable is set it may not be actually used | |
| 14:10:03 | danpawlik | cdent: so I can revert that change for Octa? | |