| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-10-05 | |||
| 14:21:51 | dansmith | just you? | |
| 14:22:00 | dansmith | we've been merging things | |
| 14:22:29 | jaypipes | maybe just needs a recheck.,.. | |
| 14:22:41 | dansmith | neither I just now, so maybe something is up | |
| 14:22:48 | cdent | logs.o.o has been behaving poorly for me of late | |
| 14:26:26 | efried | cdent I rechecked yours, looked like the same thing as last time. | |
| 14:27:16 | cdent | thanks | |
| 14:29:24 | efried | jaypipes One qq before I start slinging that spec: How are generic resources specified in the flavor? And what stage of reality is that stuff in (spec, proposed, merged, etc.)? | |
| 14:33:15 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add support for Windows network commands https://review.openstack.org/487405 | |
| 14:35:14 | efried | edleafe Where's that agenda? https://wiki.openstack.org/wiki/Meetings/Scheduler just says "See mailing-list" | |
| 14:35:39 | mriedem | andreas_s_: done https://review.openstack.org/#/admin/groups/511,members | |
| 14:35:55 | edleafe | https://wiki.openstack.org/wiki/Meetings/NovaScheduler | |
| 14:36:00 | edleafe | efried: ^^ | |
| 14:36:29 | efried | edleafe Oh. Is that other one dead? | |
| 14:37:04 | edleafe | efried: didn' | |
| 14:37:13 | edleafe | didn't even know there was another one | |
| 14:37:23 | edleafe | so yes | |
| 14:37:23 | jaypipes | efried: by "generic resources" are you referring to things like vcpu, memory_mb, etc? | |
| 14:38:15 | efried | jaypipes Ultimately those would be included, yes, but I understand they're treated specially today because they're (mandatory) separate fields in a flavor. | |
| 14:38:47 | efried | jaypipes But for purposes of the immediate discussion, let's say it's something totally generic that isn't handled by RT at all. | |
| 14:39:12 | efried | For example, where does an Ironic flavor specify the ironic node inventory it wants? | |
| 14:39:17 | efried | (Or is that another special case?) | |
| 14:39:44 | andreas_s_ | mriedem, Thanks! | |
| 14:41:20 | cdent | efried: that’s kind of a special case and was sort of the genesis of custom resource classes | |
| 14:41:58 | cdent | because the idea was that an ironic node is a single consumable thing | |
| 14:42:32 | efried | edleafe I updated https://wiki.openstack.org/wiki/Meetings/Scheduler to point to https://wiki.openstack.org/wiki/Meetings/NovaScheduler | |
| 14:42:57 | efried | cdent jaypipes So does that mean we actually don't have a way to specify generic resources in flavors yet? | |
| 14:43:27 | jaypipes | efried: in the flavor's extra_specs. there's a special key pattern called "resources:$RESOURCE_CLASS" and the value is the amount of that resource class the flavor wants. | |
| 14:43:52 | efried | jaypipes Ah, perfect. And that's been implemented already? | |
| 14:43:57 | jaypipes | efried: those special extra_specs resources: things are called resource overrides. | |
| 14:44:03 | jaypipes | efried: yes. | |
| 14:44:16 | jaypipes | efried: edleafe and alex_xu implemented that IIRC | |
| 14:44:44 | efried | jaypipes Cool; and (as the name "overrides" implies) I can specify "standard" resources there too, like VCPU, DISK_GB, etc., and their corresponding hardcoded fields will be ignored? | |
| 14:47:09 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.rebuild_scheduled notification https://review.openstack.org/473929 | |
| 14:47:33 | efried | jaypipes So in a nutshell, this new thing will take resources$X:$RESOURCE_CLASS=$N and traits$X:$TRAIT=required and translate them into GET /allocation_candidates?resources$X=$N...&required$X=$TRAIT | |
| 14:47:45 | efried | jaypipes Does that align with your thinking? | |
| 14:48:10 | jaypipes | efried: precisely. | |
| 14:48:14 | efried | beaut | |
| 14:48:40 | mriedem | gibi: thanks for https://review.openstack.org/#/c/509759/ i was just looking at fixing that | |
| 14:48:44 | mriedem | gibi: comment within | |
| 14:48:51 | openstackgerrit | Dan Smith proposed openstack/nova master: Fix minor input items from previous patches https://review.openstack.org/506416 | |
| 14:48:51 | openstackgerrit | Dan Smith proposed openstack/nova master: Merge build requests into the sortmaster 9000 https://review.openstack.org/508595 | |
| 14:48:52 | openstackgerrit | Dan Smith proposed openstack/nova master: Add CellMapping.get_by_project_id() query method https://review.openstack.org/509002 | |
| 14:48:52 | openstackgerrit | Dan Smith proposed openstack/nova master: Make get_instance_objects_sorted() be smart about cells https://review.openstack.org/509003 | |
| 14:49:04 | dansmith | mriedem: rebased now that your metadata cleanup has merged ^ | |
| 14:49:17 | mriedem | ok | |
| 14:49:56 | mriedem | says the bottom change is in merge conflict... ? | |
| 14:50:19 | dansmith | um | |
| 14:50:20 | mriedem | but it's queued up so idk | |
| 14:50:23 | dansmith | Iliterally just pushed | |
| 14:50:27 | mriedem | maybe a zuulv3 thing | |
| 14:52:20 | gibi | mriedem: I will push a new ps soon with your suggestion implemented | |
| 14:52:46 | mriedem | dansmith: i see we're running singleconductor mode in grenade between pike->queens, couldn't we be running superconductor? | |
| 14:53:08 | mriedem | maybe that would require making grenade aware of that kind of mode? | |
| 14:53:36 | dansmith | yeah I think we just punted on any sort of telling grenade about that since we couldn't upgrade from one layout to the other | |
| 14:53:47 | dansmith | I dunno how much it really needs to know about, but I haven't looked | |
| 14:55:05 | openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Improve the performance of filtering instances by IP. https://review.openstack.org/509326 | |
| 14:55:06 | mriedem | https://github.com/openstack-dev/grenade/blob/03de9e0fc7f4fc50a00db5d547413e26cf0780dd/grenade.sh#L222 | |
| 14:55:29 | mriedem | might just be worth removing that from master grenade and see how it goes | |
| 14:55:59 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform aggregate.update_prop notification https://review.openstack.org/462576 | |
| 14:56:32 | openstackgerrit | Dan Smith proposed openstack/nova master: Pre-create migration object https://review.openstack.org/498950 | |
| 14:56:33 | openstackgerrit | Dan Smith proposed openstack/nova master: Revert allocations by migration uuid https://review.openstack.org/498949 | |
| 14:56:33 | openstackgerrit | Dan Smith proposed openstack/nova master: Refactor resource tracker to account for migration allocations https://review.openstack.org/506419 | |
| 14:56:34 | openstackgerrit | Dan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420 | |
| 14:56:57 | dansmith | jaypipes: cdent: I didn't see that failure in ^ locally, so this is a rebase and we'll see how it goes this time | |
| 14:57:04 | jaypipes | kk | |
| 15:00:43 | jaypipes | dansmith, mriedem: I'm ready on this one after a number of back and forths with johnthetubaguy. https://review.openstack.org/#/c/507052/ | |
| 15:01:01 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.live_migration_rollback_dest notification https://review.openstack.org/480214 | |
| 15:01:06 | johnthetubaguy | jaypipes: thanks for your time on that, glad to move that forward | |
| 15:02:51 | dansmith | edleafe: is this fail related to your change? http://logs.openstack.org/15/486215/14/check/gate-nova-tox-functional-py35-ubuntu-xenial/f7d2653/testr_results.html.gz | |
| 15:04:16 | mriedem | jaypipes: i'll put it in the tab qeuue | |
| 15:04:19 | mriedem | *queue | |
| 15:04:51 | mriedem | dansmith: i'll go through https://review.openstack.org/498950 again | |
| 15:05:11 | dansmith | mriedem: thanks, I don't think you had any feedback on that did you? | |
| 15:05:24 | mriedem | yes... | |
| 15:05:54 | dansmith | oh the fail cleanup, right | |
| 15:09:29 | tikitavi | look to the nova-conductor logs, please (http://logs.openstack.org/91/500091/8/check/gate-functional-neutron-dsvm-ec2api-ubuntu-xenial/206b664/). We have problem with starting instances in stable/ocata and stable/pike. | |
| 15:10:35 | dansmith | tikitavi: you mean the traceback? | |
| 15:10:44 | tikitavi | yes! | |
| 15:10:44 | dansmith | 2017-10-05 10:48:16.014 30844 ERROR oslo_messaging.rpc.server IOError: [Errno 2] No such file or directory: '/usr/local/lib/python2.7/dist-packages/six-1.10.0.dist-info/METADATA' | |
| 15:10:47 | openstackgerrit | Jay Pipes proposed openstack/nova master: rp: de-ORM ResourceProvider.get_by_uuid() https://review.openstack.org/509025 | |
| 15:10:48 | openstackgerrit | Jay Pipes proposed openstack/nova master: rp: Move RP._get|set_aggregates() to module scope https://review.openstack.org/509026 | |
| 15:10:48 | openstackgerrit | Jay Pipes proposed openstack/nova master: rp: Remove RP.get_traits() method https://review.openstack.org/509027 | |
| 15:10:49 | openstackgerrit | Jay Pipes proposed openstack/nova master: rp: move RP._set_traits() to module scope https://review.openstack.org/509028 | |
| 15:10:49 | openstackgerrit | Jay Pipes proposed openstack/nova master: rp: remove CRUD operations on Inventory class https://review.openstack.org/509029 | |
| 15:10:50 | openstackgerrit | Jay Pipes proposed openstack/nova master: rp: streamline InventoryList.get_all_by_rp_uuid() https://review.openstack.org/509030 | |
| 15:10:50 | openstackgerrit | Jay Pipes proposed openstack/nova master: rp: remove dead code in Allocation._create_in_db() https://review.openstack.org/509031 | |
| 15:10:51 | openstackgerrit | Jay Pipes proposed openstack/nova master: rp: remove ability to delete 1 allocation record https://review.openstack.org/509032 | |
| 15:10:51 | openstackgerrit | Jay Pipes proposed openstack/nova master: rp: fix up AllocList.get_by_resource_provider_uuid https://review.openstack.org/509033 | |
| 15:10:52 | openstackgerrit | Jay Pipes proposed openstack/nova master: rp: rework AllocList.get_all_by_consumer_id() https://review.openstack.org/509035 | |
| 15:10:52 | dansmith | tikitavi: seems clearly like some installation problem | |
| 15:10:52 | openstackgerrit | Jay Pipes proposed openstack/nova master: rp: remove _HasAResourceProvider mixin https://review.openstack.org/509036 | |
| 15:11:40 | tikitavi | dansmith: it is repeatable. we can't pass this review for more than a month | |
| 15:12:03 | dansmith | tikitavi: I understand, but still, it's a package problem, nothing to do with any nova ode | |
| 15:12:04 | dansmith | *code | |
| 15:13:00 | edleafe | dansmith: I ran those tests locally w/o a fail, so I don't think so. | |
| 15:13:18 | dansmith | edleafe: yea, I rechecked and noted that the next patch didn't hit the same failure | |
| 15:16:08 | mriedem | sdague: did your "wait for host mapping" change in devstack get backported to stable/pike? i'm seeing http://logs.openstack.org/56/509656/1/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/6851396/logs/old/screen-n-cond.txt.gz#_2017-10-04_23_25_13_456 in stable/pike grenade runs only | |
| 15:17:09 | mriedem | oh hm https://review.openstack.org/#/c/490635 | |
| 15:17:38 | sdague | right, that will need to land there. I can push it in | |