| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-10 | |||
| 15:46:49 | mriedem | then git review to push it up | |
| 15:46:56 | ioni | mriedem, cool. i'll keep in mind | |
| 15:47:43 | leakypipes | cdent: oh, that's -w because I still need to submit a mini-spec or blueprint. | |
| 15:48:22 | cdent | leakypipes: ended up leaving it off this week's update | |
| 15:48:33 | sfinucan | johnthetubaguy, mriedem: noddy stuff, but I'd like to fix this in stable/ocata before you make the point release next week https://review.openstack.org/#/c/443086/ | |
| 15:48:54 | sfinucan | (AJaeger asked me to do so for openstack-manuals) | |
| 15:48:57 | artom | mriedem, I don't remember talking about it, but OK, your explanation makes sense | |
| 15:49:56 | mriedem | sfinucan: can you show me where those are broken today? | |
| 15:50:01 | mriedem | a bug would have been nice | |
| 15:50:10 | mriedem | if we're backporting | |
| 15:50:13 | leakypipes | cdent: totally fine :) mostly was just pushing it off my lappie so I could pull to my workstation ;) | |
| 15:50:15 | mriedem | plus links to see what the bug is would be nice | |
| 15:50:51 | cdent | leakypipes: I know how that can be. I left a comment on it as a sort of note to self to remember my confusion and sadness | |
| 15:51:04 | leakypipes | :) | |
| 15:51:28 | cdent | and with that I'm going for a walk, biab | |
| 15:51:34 | sfinucan | mriedem: https://docs.openstack.org/newton/config-reference/compute/config-options.html look for 'flat_network_bridge' | |
| 15:51:48 | sfinucan | or any of the giant text with grey bars on the left | |
| 15:52:25 | sfinucan | I didn't think formatting issues like that warranted a bug but I'll draw up one shortly | |
| 15:52:25 | mriedem | ok i see | |
| 15:52:39 | mriedem | sfinucan: well i just didn't see any examples in the commit message | |
| 15:52:45 | mriedem | also, this doesn't seem like a huge issue | |
| 15:53:01 | mriedem | and it's been like this for awhile already, like all of ocata | |
| 15:53:42 | sfinucan | mriedem: I can't include images in commit messages, and the issues won't exist once they're fixed | |
| 15:54:17 | sfinucan | openstack-manuals hasn't released the ocata docs yet, hence why I'm eager to fix it now | |
| 15:54:22 | mriedem | ok i guess i just won't ask for any context or examples | |
| 15:54:31 | mriedem | anyway, +2 | |
| 15:55:40 | sfinucan | mriedem: ta | |
| 15:59:53 | mriedem | ioni: https://review.openstack.org/#/c/444374/ | |
| 16:01:27 | ioni | mriedem, do i have to do something? :D | |
| 16:02:01 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Always set MTU when plugging devices https://review.openstack.org/408781 | |
| 16:03:52 | mriedem | ioni: nope | |
| 16:07:25 | openstackgerrit | Stephen Finucane proposed openstack/nova master: hardware: Flatten functions https://review.openstack.org/367470 | |
| 16:07:26 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Standardize '_get_XXX_constraints' functions https://review.openstack.org/385071 | |
| 16:07:27 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Rename '_numa_get_constraints_XXX' functions https://review.openstack.org/385072 | |
| 16:07:27 | openstackgerrit | Stephen Finucane proposed openstack/nova master: De-duplicate _numa_get_flavor_XXX_map_list https://review.openstack.org/385074 | |
| 16:08:04 | leakypipes | rfolco: morning! | |
| 16:08:25 | rfolco | leakypipes, afternoon! | |
| 16:08:25 | leakypipes | rfolco: so, I can support cdent's review suggestions on your patch. Go with KeyError raise. | |
| 16:10:10 | rfolco | leakypipes, nice. I can do something like: http_except = webob.exc.status_map.get(req) .... if not None, then raise http_except | |
| 16:10:23 | rfolco | instead of status_map[req] | |
| 16:10:54 | rfolco | this way I avoid keyerror and get rid of dict | |
| 16:11:41 | leakypipes | rfolco: well, I think cdent actually prefers raising a KeyError there, since this is a code path that is dev-only -- i.e. users cannot influence this code path and we want to bomb out early if a developer adds an call to raise_http_code_if_microversion() is passed an improper status code. | |
| 16:13:39 | rfolco | hmm makes sense. If keyerror is fine, easier for me :) | |
| 16:13:39 | rfolco | leakypipes, thx++ | |
| 16:13:40 | leakypipes | rfolco: go for it :) | |
| 16:22:32 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: PowerVM Driver: SSP emphemeral disk support https://review.openstack.org/443189 | |
| 16:30:45 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: introduce basic framework for security proxying https://review.openstack.org/345396 | |
| 16:30:46 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: introduce framework for RFB authentication https://review.openstack.org/345397 | |
| 16:30:47 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: provide an RFB security proxy implementation https://review.openstack.org/345399 | |
| 16:30:47 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: introduce the VeNCrypt RFB authentication scheme https://review.openstack.org/345398 | |
| 17:02:14 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: [WIP] Read NIC features in libvirt https://review.openstack.org/444415 | |
| 17:06:56 | mriedem1 | dansmith: so no obfuscation for https://review.openstack.org/#/c/438023/25/nova/context.py ? | |
| 17:08:09 | melwitt | mriedem: it's not storing the url in the context anymore | |
| 17:09:30 | mriedem | oh cool | |
| 17:09:45 | mriedem | i'm going to have to rebase my quotas fix now.. :( | |
| 17:11:38 | leakypipes | dansmith: your once over on gibi's patch here pls: https://review.openstack.org/#/c/440347 | |
| 17:14:59 | melwitt | leakypipes: not sure I understand the comment on that one, that instance.flavor isn't connected to data in the db. it's connected to instance_extra and can be lazy-loaded on an instance that was queried without joining with instance_extra earlier | |
| 17:15:47 | mriedem | melwitt: leakypipes: bauwser: also need to get these in https://review.openstack.org/#/q/topic:bug/1670627 - that's a major regression in ocata and need to get that backported for a 13.0.1 next week | |
| 17:16:01 | mriedem | fwiw dan was +2 on the bottom change yesterday, and had skimmed the rest of the series already | |
| 17:16:18 | leakypipes | melwitt: yeah, it's weird. basically, what we're attempting to prevent is the lazy-loading of flavor *projects* -- i.e. the second-stage lazy-loading of Flavor attributes. | |
| 17:16:36 | mriedem | i'm going to be basically MIA the rest of the afternoon, it's my last day here so lunch, exit interview and then they kick my ass out | |
| 17:16:40 | mriedem | and my new laptop hasn't shown up yet | |
| 17:17:03 | leakypipes | melwitt: so the instance.Flavor is populated when expected_attributes=['flavor'] and that's fine. But by orphaning the Flavor in instance.flavor, we prevent the instance.flavor.projects attribute from being lazy-loaded. | |
| 17:17:05 | melwitt | leakypipes: so like a cascading lazy-load | |
| 17:17:07 | leakypipes | melwitt: weird, I know :) | |
| 17:17:12 | leakypipes | melwitt: yeah. | |
| 17:17:26 | melwitt | ah, I see | |
| 17:17:39 | melwitt | mriedem: ack, will review those today | |
| 17:18:22 | leakypipes | dansmith: time for you to enter the phone booth and emerge as superdan. | |
| 17:21:33 | artom | Is Phone Booth is mdbooth's rapper name? | |
| 17:22:59 | melwitt | heh | |
| 17:44:01 | andrewbogott | Is 'Host xxx has more disk space than database expected' ever anything important? That message shows up in several bug reports but best I can tell the message is a red herring in all cases. | |
| 17:44:16 | andrewbogott | (I have 4 out of 13 compute nodes that throw that warning regularly.) | |
| 17:47:19 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: SSP emphemeral disk support https://review.openstack.org/443189 | |
| 17:47:59 | efried | thorst_afk adreznec esberglu_ ^^ live tested, works, ready for subteam review. | |
| 17:59:47 | openstackgerrit | Dan Smith proposed openstack/nova master: Get instance availability_zone without hitting the api db https://review.openstack.org/439754 | |
| 17:59:47 | openstackgerrit | Dan Smith proposed openstack/nova master: Make os-availability-zones know about cells https://review.openstack.org/442163 | |
| 17:59:49 | openstackgerrit | Dan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info https://review.openstack.org/439891 | |
| 17:59:50 | openstackgerrit | Dan Smith proposed openstack/nova master: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025 | |
| 17:59:50 | openstackgerrit | Dan Smith proposed openstack/nova master: Add identity helper property to CellMapping https://review.openstack.org/444038 | |
| 19:04:29 | cfriesen | someone pointed out to me that InstanceActionEvent.pack_action_event_finish() uses a "message" field, but the InstanceActionEvent model in the DB uses "details". Is this a bug? | |
| 19:07:53 | mriedem_afk | andrewbogott: this is the comment by that warning | |
| 19:07:54 | mriedem_afk | # can occur when an instance in database is not on host | |
| 19:08:16 | mriedem | andrewbogott: which is similar to the _sync_power_states periodic task in the compute manager, but that might not be a warning | |
| 19:08:18 | mriedem | would have to check | |
| 19:08:35 | andrewbogott | mriedem: thanks. I did a side-by-side comparison on one of the hosts already but maybe I missed a discrepancy. | |
| 19:08:58 | andrewbogott | At least… you'd expect 'nova list —host <hostname>' to show the db contents and not the disk contents, right? | |
| 19:10:10 | openstackgerrit | John Griffith proposed openstack/nova master: Add Cinder API version detection https://review.openstack.org/444465 | |
| 19:10:30 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/443357 | |
| 19:11:49 | jgriffith | ildikov mriedem ^^ don't know if that's more along the lines of what you'd like to see or not | |
| 19:12:23 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/395347 | |
| 19:18:24 | openstackgerrit | Jay Pipes proposed openstack/nova master: virt: implement get_inventory() for Ironic https://review.openstack.org/441544 | |
| 19:19:42 | leakypipes | cdent: hi. I'm getting a single unit test failure for nova.tests.unit.test_fixtures.TestPlacementFixture.test_responds_to_version locally on ^^ patch. it looks like this: http://paste.openstack.org/show/602305/ | |
| 19:20:09 | leakypipes | cdent: nothing in the change to that patch should affect the placement API as far as I can tell. have you seen that error before? | |
| 19:20:28 | cdent | i've not, one sec | |
| 19:23:24 | cdent | leakypipes: are the functional tests running okay? | |
| 19:23:32 | leakypipes | cdent: yes. | |
| 19:26:12 | cdent | imma need to try this stuff myself leakypipes | |
| 19:26:31 | leakypipes | cdent: np | |
| 19:36:44 | leakypipes | JayF: thx for the reviews on the ironic patch. responded to your query on the latest revision. | |