| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-01 | |||
| 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? | |
| 14:10:23 | danpawlik | ack | |
| 14:10:26 | cdent | danpawlik: I can't remember what you said before: is it breaking something, or did you just notice that the puppet was weird? | |
| 14:11:02 | danpawlik | cdent: I was wondering what DB should I use for [placement_database] connection param | |
| 14:11:41 | cdent | in ocata there's no support in placement itself for using a database connection other than the nova db api | |
| 14:11:44 | danpawlik | cdent: so to summarry what you told me: I can use nova_api DB for https://github.com/openstack/puppet-openstack-integration/blob/stable/ocata/manifests/nova.pp#L111 but normally it will be ignored in Ocata ye? | |
| 14:11:56 | cdent | yes | |
| 14:11:57 | danpawlik | ack | |
| 14:12:24 | cdent | danpawlik: I'll see EmilienM a bit later in the day and we'll chat about it | |
| 14:12:33 | danpawlik | cdent: thats what I was looking for | |
| 14:12:52 | danpawlik | I try to catch him on irc later :) | |
| 14:13:00 | danpawlik | cdent: thanks! | |
| 14:26:49 | mriedem | jroll: fyi https://docs.openstack.org/neutron/latest/admin/config-routed-networks.html | |
| 14:28:38 | mriedem | linked some summit videos in the etherpad as well | |
| 14:31:43 | jroll | mriedem: thanks, I was getting conflicting info earlier this week. making a todo to make sure nova docs are up to date with that | |
| 14:31:59 | jroll | or maybe it was pike neutron docs I had | |
| 14:39:25 | openstackgerrit | Merged openstack/nova stable/pike: Clean up ports and volumes when deleting ERROR instance https://review.openstack.org/546221 | |
| 14:45:29 | openstackgerrit | Merged openstack/nova stable/pike: Add functional test for deleting BFV server with old attach flow https://review.openstack.org/546812 | |
| 14:50:38 | openstackgerrit | Eric Fried proposed openstack/nova master: placement: Return new provider from POST /rps https://review.openstack.org/548934 | |
| 15:31:06 | Spazmotic | Work computer what did you do | |
| 15:36:44 | mriedem | jroll: could have been tied up in the stuff where we were going to move port creation to conductor so we could do network-aware scheduling | |
| 15:37:11 | mriedem | i'm still not really sure in how all of the neutron stuff works today | |
| 15:39:02 | mriedem | dansmith: if i don't care about reading deleted records from the cell db, or filtering those records based on project_id/user_id, then i don't need to use the model_query() in the DB API right? | |
| 15:39:31 | openstackgerrit | Merged openstack/nova stable/pike: Ensure attachment_id always exists for block device mapping https://review.openstack.org/546275 | |
| 15:39:49 | dansmith | mriedem: you'll get deleted records if you don't use model_query().. if that's what you don't care about, then yeah | |
| 15:40:04 | mriedem | i don't want deleted records....damn | |
| 15:40:05 | mriedem | ok | |
| 15:40:11 | mriedem | well, i can just filter on deleted=0 | |
| 15:40:18 | melwitt | yes, that | |