| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-12-14 | |||
| 18:00:56 | mriedem | melwitt: ok | |
| 18:02:10 | mriedem | smcginnis: which means other drivers could blow up | |
| 18:02:22 | mriedem | smcginnis: i'm thinking the volume manager's attachment_delete shouldn't call the driver if there is no connector | |
| 18:03:17 | vivsoni__ | mriedem, smcginnis: many 3dr paty CI has skipped those TCs | |
| 18:03:38 | mriedem | ok so i'll push a fix quick | |
| 18:03:55 | mriedem | vivsoni__: do you have a bug number? | |
| 18:04:28 | vivsoni__ | mriedem: No... not sure whether there is a bug raised | |
| 18:04:39 | mriedem | vivsoni__: ok, can you open a cinder bug then? | |
| 18:04:45 | vivsoni__ | sure | |
| 18:05:21 | vivsoni__ | please give a min.. will raised and let you know in a min or two | |
| 18:05:40 | smcginnis | We had discussed this in Cinder several months back. If there are any drivers that are not handling not having a connector, a bug should be filed against those drivers. | |
| 18:05:59 | smcginnis | But most (maybe some) should be handling it correctly. | |
| 18:07:19 | melwitt | I notice a lot of tracing of ComputeHostNotFound in gate n-cpu logs too | |
| 18:09:34 | mriedem | melwitt: that's been around for a long time, on startup | |
| 18:09:39 | mriedem | since ocata | |
| 18:09:51 | mriedem | happens when n-cpu starts before the host is discovered i think | |
| 18:10:37 | melwitt | oh. I thought it used to be a warning but not a backtrace | |
| 18:10:39 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Make request_spec.spec MediumText https://review.openstack.org/528012 | |
| 18:11:01 | mriedem | it's a warning + traceback | |
| 18:11:06 | mriedem | because of an oslo.log bug | |
| 18:11:19 | mriedem | see my failed attempts https://review.openstack.org/#/c/517102/ | |
| 18:14:12 | melwitt | looking at the oslo.log patch and trying to figure out, is the fix obvious or not -_- | |
| 18:15:22 | vivsoni__ | mriedem: https://bugs.launchpad.net/cinder/+bug/1738254 | |
| 18:15:24 | openstack | Launchpad bug 1738254 in Cinder "TEMPEST test case failed "tempest.api.compute.volumes.test_attach_volume.AttachVolumeShelveTestJSON.test_detach_volume_shelved_or_offload_server"" [Undecided,New] | |
| 18:15:47 | vivsoni__ | mriedem: i have raised a cinder bug against that tempest failure | |
| 18:17:22 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add PCI NUMA policies https://review.openstack.org/527472 | |
| 18:17:23 | openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Modify signature of _filter_non_requested_pfs https://review.openstack.org/527473 | |
| 18:17:23 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Document PCI NUMA affinity policy https://review.openstack.org/528011 | |
| 18:17:58 | melwitt | I think I can propose something | |
| 18:18:30 | stephenfin | bauzas: For when you're back online, that image-flavor conflict exception gets bubbled all the way back to the user's terminal via a HTTP 400. I think we're good | |
| 18:29:35 | stephenfin | melwitt: I'm not about tomorrow but if you have a chance to review [1] before Monday, I'd be eternally grateful [1] https://review.openstack.org/#/q/topic:bp/websocket-proxy-to-host-security+status:open | |
| 18:30:21 | stephenfin | melwitt: I think jianghuaw_'s comments are valid but something that can be addressed in a follow-up (I don't know how much work 3.8 -> 3.3 conversion would be) | |
| 18:30:45 | melwitt | stephenfin: I've been looking at it and it looks overall good to me, but I saw recently someone -1ed it saying it doesn't work with xen? okay | |
| 18:47:32 | pprokop | 1 | |
| 18:55:44 | melwitt | mriedem: let's see how this goes https://review.openstack.org/#/c/528036 | |
| 18:56:24 | mriedem | cool | |
| 19:05:15 | cfriesen | when dealing with quotas, why do we distinguish between "in use" and "reserved"? | |
| 19:06:33 | melwitt | because the old system had an initial "reserve" step that happened before switching it from "reserved" -> "in use" | |
| 19:07:14 | melwitt | reserve() reserved it and commit() changed it from reserved to in use | |
| 19:08:53 | openstackgerrit | Ed Leafe proposed openstack/nova master: Make conductor pass and use host_lists https://review.openstack.org/511358 | |
| 19:08:53 | openstackgerrit | Ed Leafe proposed openstack/nova master: Change compute RPC to use alternates for resize https://review.openstack.org/526436 | |
| 19:09:15 | cfriesen | melwitt: thanks...so in pike would we ever see a nonzero reserved value? | |
| 19:11:25 | melwitt | cfriesen: right, no more reserve step so reserved always zero | |
| 19:12:46 | cfriesen | melwitt: great, thanks | |
| 20:45:05 | openstackgerrit | melanie witt proposed openstack/nova master: live-mig: keep disk device address same https://review.openstack.org/518022 | |
| 21:27:15 | mriedem | can someone hit this? https://review.openstack.org/#/c/527878/ - unblocks the 2.58 api change and is needed for the common server create api samples after 2.57 where 'personality' is not an allowed request parameter | |
| 21:27:34 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Implement set_and_clear_allocations in report client https://review.openstack.org/528089 | |
| 21:28:17 | cdent | dansmith. mriedem that ^ is a start on POST to /allocations for migration stuff. It's just a spike at this point but I figured I'd push it up for early "is this what we were thinking style review". Also I needed to do something besides talk. | |
| 21:28:39 | mriedem | heh, cool. thanks. | |
| 21:28:45 | mriedem | that was on my list as well | |
| 21:30:01 | dansmith | sweet | |
| 21:30:20 | dansmith | I was literally lying in bed last night thinking I needed to ask if you were interested in doing that | |
| 21:30:21 | dansmith | and then forgot | |
| 21:30:26 | cdent | it appears to "work". the test_servers functional tests pass, and I assume it exercised by that | |
| 21:31:12 | tonyb | mriedem: Thanks for the update. I'll try again to look at thos backports. | |
| 21:31:12 | cdent | actually, dansmith, I was standing in your room last night, watching you sleep | |
| 21:31:41 | cdent | heh | |
| 21:31:54 | openstackgerrit | Merged openstack/nova master: Fix some typos in nova doc https://review.openstack.org/527856 | |
| 21:35:57 | mriedem | the live migrate task calls this same code in the migrate module right? | |
| 21:36:16 | mriedem | yup | |
| 21:36:17 | mriedem | ok | |
| 21:36:23 | mriedem | looks like a sane start to me | |
| 21:36:37 | mriedem | not sure we need to handle the 1.13 not being there but it doesn't really hurt either | |
| 21:37:06 | cdent | mriedem: yeah, I responded on that within. I couldn't remember whether we had nailed the 1.14 requirment as for realz or not | |
| 21:37:31 | mriedem | it is | |
| 21:37:36 | mriedem | create resource provider doesn't fallback if it's not there | |
| 21:37:48 | mriedem | https://review.openstack.org/#/c/526505/ | |
| 21:37:52 | cdent | then I suppose may as well remove that fallback | |
| 21:38:12 | cdent | so does that mean we can remove all the fallback code in report client? | |
| 21:38:16 | cdent | as there's some others too | |
| 21:40:40 | mriedem | maybe | |
| 21:40:45 | mriedem | i think we can remove the safe_connect decorator | |
| 21:41:07 | mriedem | that was for placement being optional in newton | |
| 21:42:44 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: libvirt: properly decode error message from qemu guest agent https://review.openstack.org/517727 | |
| 21:45:08 | cdent | it does other things | |
| 21:45:24 | cdent | including dealing with the fact that placement is not currently up, but might be in a little while | |
| 21:45:34 | cdent | which helps make it resilient across reboots | |
| 21:45:52 | cdent | or during installs where things get out of order | |
| 21:46:22 | mriedem | oh i suppose | |
| 21:47:54 | openstackgerrit | Merged openstack/nova master: XenAPI: Don't use nicira-iface-id for XenServer VIF https://review.openstack.org/451641 | |
| 21:47:59 | openstackgerrit | Merged openstack/nova master: Retry _trait_sync on deadlock https://review.openstack.org/527836 | |
| 21:51:58 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Retry _trait_sync on deadlock https://review.openstack.org/528094 | |
| 22:02:23 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_certs object https://review.openstack.org/489408 | |
| 22:02:23 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | |
| 22:04:11 | mriedem | who wants to delete some stuff? | |
| 22:04:21 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Delete the TypeAffinityFilter https://review.openstack.org/528099 | |
| 22:04:33 | mriedem | jogo: i believe this was your baby ^ | |
| 22:07:17 | mriedem | i suppose i should run pep8 | |
| 22:10:12 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Delete the TypeAffinityFilter https://review.openstack.org/528099 | |
| 22:14:15 | cfriesen | mriedem: has anyone ever talked about not overwriting the UEFI variables on every boot? | |
| 22:15:06 | cfriesen | mriedem: I don't see any related nova bug, but we've got a customer saying that instances should preserve UEFI variables over stop/start, for example. | |
| 22:15:43 | mriedem | cfriesen: no one has ever talked about uefi period | |
| 22:15:46 | openstackgerrit | melanie witt proposed openstack/nova master: Enable test_volume_backed_live_migration in tempest https://review.openstack.org/528104 | |
| 22:15:50 | cfriesen | mriedem: heh | |
| 22:20:18 | openstackgerrit | Jackie Truong proposed openstack/python-novaclient master: Microversion 2.58 - Add trusted_image_certificates https://review.openstack.org/500396 | |
| 22:26:54 | openstackgerrit | Merged openstack/nova stable/ocata: Fix 'force' parameter in os-quota-sets PUT schema https://review.openstack.org/526426 | |
| 22:40:26 | mriedem | cfriesen: are you going to be in dublin? | |
| 22:40:40 | mriedem | did you see my query about that old bp of yours about specific numa scheduling? | |
| 22:41:16 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_certs object https://review.openstack.org/489408 | |
| 22:52:02 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix the formatting for 2.56 in the compute REST API history doc https://review.openstack.org/528114 | |