| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-12-01 | |||
| 11:10:47 | openstackgerrit | Merged openstack/nova stable/pike: Fix race in delete allocation in ServerMovingTests https://review.openstack.org/508872 | |
| 12:08:54 | efried | nsingh I have noticed that too - In Nova we go through all this trouble to create a session from conf parameters, and then we create the client for the service which creates its own session. | |
| 12:09:18 | efried | nsingh Eventually the vision (per mordred) is to get rid of the clients altogether and just use the ksa adapter for the API communication. | |
| 12:32:51 | nsingh | efried: thank you for the info. :) | |
| 12:38:12 | ildikov | johnthetubaguy: hi, daily pretty please: https://review.openstack.org/#/c/330285/ | |
| 12:43:52 | openstackgerrit | Merged openstack/python-novaclient master: inject file: add description of injecting multiple files. https://review.openstack.org/515655 | |
| 13:22:22 | fried_rice | Friday! | |
| 13:24:16 | openstackgerrit | Chen Hanxiao proposed openstack/python-novaclient master: inject file: add method of showing quota value of injecting files for 'rebuild' command https://review.openstack.org/524588 | |
| 13:26:08 | fried_rice | stephenfin You around? | |
| 13:27:34 | fried_rice | or bauzas or gibi or alex_xu ? Easy review, which will merge another patch for a backportable bug: https://review.openstack.org/#/c/524245/ | |
| 13:40:46 | stephenfin | fried_rice: I am. Looking now | |
| 13:40:53 | fried_rice | stephenfin Thanks! | |
| 13:42:14 | finucannot | fried_rice: Are you finding Gerrit slow today too? | |
| 13:42:28 | fried_rice | finucannot Yeah, totally. I asked in -infra, but nobody bit yet. | |
| 13:42:43 | finucannot | So not my network. Good to know. | |
| 13:42:52 | fried_rice | That was an hour ago - more people are awake now. I'll ask again... | |
| 13:43:05 | jaypipes | fried_rice: mornin | |
| 13:43:10 | fried_rice | jaypipes Howdy! | |
| 13:43:19 | cdent | I’ve been out of a bed for a long time, but I make no claims to being awake. | |
| 13:43:28 | leakypipes | fried_rice: what was the thing with alex from earlier? | |
| 13:43:46 | fried_rice | leakypipes He was asking about https://review.openstack.org/#/c/517119/ | |
| 13:44:04 | leakypipes | fried_rice: ah, k | |
| 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_ | 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: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: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 | 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: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: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 | |