| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-10-05 | |||
| 13:25:24 | tikitavi | I really need help, for more than a month the review https://review.openstack.org/#/c/500091/ didn't pass because one or more instances cannot start because of the errors in n-cond | |
| 13:27:13 | openstackgerrit | Kevin Lefevre proposed openstack/nova stable/pike: Support qemu >= 2.10 https://review.openstack.org/509774 | |
| 13:41:05 | ArchiFleKs | Hi, do you know if this (https://review.openstack.org/#/c/505673/) can be backported to pike on other releases that run on Xenial, I have the issue on Pike with OpenStack Ansible, were the compute node have been update to qemu 2.10 | |
| 13:42:23 | openstackgerrit | Eric Berglund proposed openstack/nova master: WIP(5): PowerVM driver: ovs vif https://review.openstack.org/422512 | |
| 13:43:52 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081 | |
| 13:45:21 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081 | |
| 13:47:42 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | |
| 13:48:10 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | |
| 13:57:34 | ronlund | nova meeting in 3 minutes | |
| 14:11:57 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform aggregate.update_prop notification https://review.openstack.org/462576 | |
| 14:18:08 | jaypipes | cdent: re: moving all the module-level DB access functions to the bottom of the resource_provider.py file, I'd prefer to do that all in a single commit later on. | |
| 14:18:27 | cdent | that’s cool, just wanted to get it on the record | |
| 14:18:31 | dansmith | jaypipes: yeah so I can -1 that one | |
| 14:18:50 | dansmith | cdent: I dunno how you got that we organize files per my preference | |
| 14:19:08 | dansmith | I mean, it is my preference that we order them definition first then use, but that wasn't my doing | |
| 14:19:25 | cdent | I wasn’t saying you are the enforcer, I just happen to know your preference | |
| 14:19:35 | cdent | so wanted to mention it so that you knew I wasn’t blithely ignoring it | |
| 14:19:47 | cdent | I’m graciously taking it into account and choosing to consider it wrong ;) | |
| 14:19:58 | dansmith | mkay, made it sound like all our files are like this because if not I'd throw a hissy | |
| 14:20:05 | cdent | no sir | |
| 14:20:07 | jaypipes | dansmith: that's fine. :) | |
| 14:21:33 | jaypipes | dansmith, cdent: is it just me or is the ubuntu xenial full tempest job just hosed? | |
| 14:21:41 | openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Improve the performance of filtering instances by IP. https://review.openstack.org/509326 | |
| 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. | |