| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-12-01 | |||
| 13:44:06 | fried_rice | I had thought you were making local changes to it, but looking back it seemed you had only rebased it. | |
| 13:44:13 | leakypipes | fried_rice: yeah, I'll work on that this mroning. | |
| 13:44:17 | fried_rice | Cool beans. | |
| 13:45:30 | fried_rice | finucannot FYI apparently we're "waiting for an infra-root to speak some healing words" (per AJaeger) | |
| 13:45:48 | fried_rice | Oh, you're in -infra too. I'll quit relaying. | |
| 13:48:46 | AJaeger | fried_rice: I'm not here in general, continue relaying, please | |
| 13:48:52 | mriedem | gmann_afk: no we don't need to add query param schema for the cells APIs | |
| 13:48:56 | mriedem | those are likely gone in rocky | |
| 13:49:13 | fried_rice | AJaeger But finucannot is in -infra, so he can follow the conversation there. | |
| 13:49:20 | AJaeger | ;) | |
| 14:04:28 | openstackgerrit | Eric Fried proposed openstack/nova stable/pike: Proper error handling by _ensure_resource_provider https://review.openstack.org/524618 | |
| 14:04:55 | fried_rice | mriedem ^ pike cherry-pick, with conflicts. Think I'll wait for zuul before doing the ocata one. | |
| 14:06:15 | mriedem | ok | |
| 14:07:19 | finucannot | lyarwood: I assume you could review this, seeing as you only cherry-picked it (vs. writing it)? https://review.openstack.org/#/c/524157/ | |
| 14:09:05 | mriedem | finucannot: not really how backport reviews work | |
| 14:09:43 | finucannot | mriedem: ok, I was curious about that :) Can you review because you wrote it? Can he review because he backported? | |
| 14:09:44 | mriedem | i must say, however, that whoever did write the original must be very handsome indeed | |
| 14:10:39 | mriedem | done | |
| 14:14:53 | Guest78_ | I'm having issues unshelving an instance that has PCI passthrough aliases associated with it......the wrong hypervisor host is selected. Is there a setting that I'm missing to include PciPassthroughFilter when scheduling for unshelve? | |
| 14:16:56 | mriedem | unshelve should go through the same scheduling as the initial create | |
| 14:17:07 | mriedem | so if you have the filters setup for the initial create properly, unshelve should work the same | |
| 14:17:26 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Deprecate file injection https://review.openstack.org/522027 | |
| 14:18:05 | leakypipes | Guest78_: note that this channel is for development questions, so generally that would be a question for the openstack@ mailing list. However, since it's Friday... if you just want a quick and dirty fix, you can probably muck with the data in the instance_extra table for that shelved instance and change the flavor's PCI requirements manually. | |
| 14:18:06 | Guest78_ | That's what I thought but that doesn't seem to be the case: 2017-12-01 13:55:52.843 2450 WARNING nova.scheduler.host_manager [req-b7876409-1149-4f8a-837d-46f0023fed10 6136e80cc67e4759843a5f95f7a9a98b 7618d5a893f04349b9feb2ca541a6fce - default default] Selected host: overcloud-novacompute-0.localdomain failed to consume from instance. Error: PCI device request [InstancePCIRequest(alias_name='GeForce10\ | |
| 14:18:06 | Guest78_ | 80',count=1,is_new=<?>,request_id=<?>,spec=[{dev_type='type-PCI',product_id='1b06',vendor_id='10de'}])] failed: PciDeviceRequestFailed: PCI device request [InstancePCIRequest(alias_name='GeForce1080',count=1,is_new=<?>,request_id=<?>,spec=[{dev_type='type-PCI',product_id='1b06',vendor_id='10de'}])] failed | |
| 14:19:11 | ildikov | mriedem: hi | |
| 14:19:33 | ildikov | mriedem: rebased the new attach patch and ran your test on it too, it looks good so far | |
| 14:19:57 | leakypipes | giblet: ++ | |
| 14:20:04 | leakypipes | giblet: on both the beer and the learning. | |
| 14:20:25 | coffee_cat | giblet: I think it's late enough in the afternoon to get a beer now :) | |
| 14:20:43 | cdent | there’s no beer in the hosue | |
| 14:24:50 | giblet | ahh I'm still in the tea phase for this afternoon | |
| 14:25:49 | mriedem | superdan: just came across this in triage https://bugs.launchpad.net/nova/+bug/1735719 - seems the delete_host command should mark the compute node as mapped=0 so it can be discovered again | |
| 14:25:49 | openstack | Launchpad bug 1735719 in OpenStack Compute (nova) "Cell_v2 discover_hosts Not Working, after deleted a host from a cell " [Medium,Triaged] | |
| 14:25:57 | mriedem | since we don't have a way to directly delete compute node records | |
| 14:26:14 | mriedem | coffee_cat: ack | |
| 14:26:17 | superdan | hrm, will look | |
| 14:27:27 | denture | meh | |
| 14:28:23 | superdan | cdent: cdenture maybe? | |
| 14:28:25 | mriedem | cdent: did you see that patch i started for some policy rules/file stuff for placement? how much do you remember of all that when laski was doing it a few releases ago? | |
| 14:28:34 | mriedem | https://review.openstack.org/#/c/524425/ | |
| 14:28:53 | mriedem | turns out that now that people are starting to roll up to using placement, they actually care about configurable policy for the service :) | |
| 14:30:33 | cdent | mriedem: I saw it briefly this morning before I rolled out but didn’t have a chance to look at it for real yet. I had thought that we were planning to put it off until we actually had some reason to have policy other than “you must be this high to ride any of this ride” to align with the “minimal config” thing? | |
| 14:31:16 | leakypipes | mriedem, superdan, giblet: melwitt did some good sleuthing on the bug report associated with https://review.openstack.org/#/c/518022. I concur with her findings and think the patch is good to go. | |
| 14:31:17 | cdent | superdan: cdenture could be workable, but it’s not quite right enough for me to be all in | |
| 14:31:49 | mriedem | cdent: given it's all admin-only apis i think that's also why we put off configurable policy, | |
| 14:31:55 | mriedem | the issue that someone brought up last night was the admin role, | |
| 14:31:59 | mriedem | they don't have an "admin" role i guess | |
| 14:32:09 | mriedem | so they are hacking around it with a config option | |
| 14:32:11 | mriedem | mgagne: ^ | |
| 14:32:22 | mriedem | this https://gist.github.com/mgagne/b43c1e085c1f1d50bebc054a7d387688 | |
| 14:32:51 | mriedem | anyway, not sure it's high priority, just something to have on the radar | |
| 14:32:55 | cdent | ah | |
| 14:32:56 | cdent | hmmm | |
| 14:32:57 | mriedem | since i totally forgot about this | |
| 14:32:59 | superdan | cdent: hah, okay | |
| 14:35:12 | fried_rice | cdent Or "indenture" :) | |
| 14:35:27 | cdent | progress | |
| 14:36:04 | superdan | oof | |
| 14:36:06 | cdent | of greater concern than picking a good nick is the zomg, effort, of having to remember to change it on fridays. _So_ exhausting. | |
| 14:36:09 | superdan | friday nicks are supposed to be fun :) | |
| 14:36:15 | fried_rice | "decdent" would be a little morbid | |
| 14:39:16 | mriedem | superdan: melwitt: tssurya: i'll start throwing cellsv2 bugs in the meeting agenda for lack of a better place https://wiki.openstack.org/wiki/Meetings/NovaCellsv2#Agenda | |
| 14:39:56 | figleaf | cdent -> decent | |
| 14:40:32 | fried_rice | Mm, that's a good one. | |
| 14:40:39 | fried_rice | or dcent | |
| 14:40:40 | figleaf | or, more appropriately, descent | |
| 14:40:44 | fried_rice | heh | |
| 14:41:56 | cdent | this is reminding me of my server naming scheme from late 90s: depressing words that start with ‘d’ | |
| 14:42:18 | superdan | mriedem: okay | |
| 14:42:36 | cdent | Has anything placement-related happened in the last 12 hours that I ought to be aware of for the update? | |
| 14:43:29 | mriedem | just the policy thing | |
| 14:43:40 | cdent | thanks | |
| 14:45:37 | mriedem | mikal: https://bugs.launchpad.net/nova/+bug/1735663 | |
| 14:45:39 | openstack | Launchpad bug 1735663 in OpenStack Compute (nova) "privsep.daemon takes 9 seconds during taking volume attaching operation" [Undecided,New] | |
| 14:46:09 | fried_rice | cdent You got the _ensure bug on your list? | |
| 14:46:18 | cdent | yessir | |
| 14:46:27 | fried_rice | coo | |
| 14:46:52 | fried_rice | cdent And this series: https://review.openstack.org/#/c/523932/ | |
| 14:47:11 | openstackgerrit | Jay Pipes proposed openstack/nova master: handle traits with sharing providers https://review.openstack.org/517119 | |
| 14:47:12 | openstackgerrit | Jay Pipes proposed openstack/nova master: Test alloc_cands with non overlapping sharing RPs https://review.openstack.org/519380 | |
| 14:47:12 | openstackgerrit | Jay Pipes proposed openstack/nova master: Test alloc_cands with one RP shared between two RPs https://review.openstack.org/519617 | |
| 14:47:13 | openstackgerrit | Jay Pipes proposed openstack/nova master: Test allocation candidates: multiple aggregates https://review.openstack.org/518633 | |
| 14:47:13 | openstackgerrit | Jay Pipes proposed openstack/nova master: Test helper: validate provider summaries https://review.openstack.org/518982 | |
| 14:47:14 | openstackgerrit | Jay Pipes proposed openstack/nova master: Fix accumulated nits in refactor series https://review.openstack.org/521189 | |
| 14:47:14 | openstackgerrit | Jay Pipes proposed openstack/nova master: Add aggregates check in allocation canditates https://review.openstack.org/522407 | |
| 14:47:15 | openstackgerrit | Jay Pipes proposed openstack/nova master: Add functional test for get_all_with_shared() https://review.openstack.org/522408 | |
| 14:47:15 | openstackgerrit | Jay Pipes proposed openstack/nova master: Fix missing rps in allocation candidates https://review.openstack.org/522409 | |
| 14:47:16 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: func tests for multiple shared RPs https://review.openstack.org/498737 | |
| 14:47:54 | cdent | fried_rice: that’ll get covered with the n-r-p topic | |
| 14:48:00 | fried_rice | ight | |
| 14:48:18 | leakypipes | fried_rice, giblet, superdan, mriedem: ok, series starting at https://review.openstack.org/#/c/517119/ is rebased and ready for reviews (mostly new func tests for alloc candidate permutations) | |
| 14:48:28 | fried_rice | leakypipes ack | |
| 14:49:05 | mriedem | i have to spend quality time with https://review.openstack.org/#/c/330285/ this morning | |
| 14:50:53 | leakypipes | hehe | |
| 14:51:19 | leakypipes | coffee_cat: why? it's only on revision 165... that's not even a middle-aged patch. | |
| 14:51:41 | coffee_cat | leakypipes: if you happen to have time while Zuul is chewing on all of those above you could chime in too :) | |
| 14:51:50 | mriedem | guh, MOS bugs from mitaka https://bugs.launchpad.net/nova/+bug/1735407 | |
| 14:51:51 | openstack | Launchpad bug 1735407 in Mirantis OpenStack 9.x "[Nova] Evacuation doesn't respect anti-affinity rules" [High,New] - Assigned to Oleksiy Molchanov (omolchanov) | |
| 14:51:54 | leakypipes | coffee_cat: yeah, I have it open :) | |