Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-03-10
15:45:57 mriedem1 and that's why i omitted nic tags from the spec
15:46:11 mriedem that's lazy on my part, i just didn't want to sort out the interface APIs first
15:46:33 mriedem ioni: i do git commit -a --amend
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 mriedem ok i see
15:52:25 sfinucan I didn't think formatting issues like that warranted a bug but I'll draw up one shortly
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: De-duplicate _numa_get_flavor_XXX_map_list https://review.openstack.org/385074
16:07:27 openstackgerrit Stephen Finucane proposed openstack/nova master: Rename '_numa_get_constraints_XXX' functions https://review.openstack.org/385072
16:08:04 leakypipes rfolco: morning!
16:08:25 leakypipes rfolco: so, I can support cdent's review suggestions on your patch. Go with KeyError raise.
16:08:25 rfolco leakypipes, afternoon!
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 leakypipes, thx++
16:13:39 rfolco hmm makes sense. If keyerror is fine, easier for me :)
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: introduce the VeNCrypt RFB authentication scheme https://review.openstack.org/345398
16:30:47 openstackgerrit Stephen Finucane proposed openstack/nova master: console: provide an RFB security proxy implementation https://review.openstack.org/345399
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: Make os-availability-zones know about cells https://review.openstack.org/442163
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: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: Add identity helper property to CellMapping https://review.openstack.org/444038
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
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.

Earlier   Later