Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-12-14
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
23:35:53 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: WIP: Add support for the 2.57 microversion https://review.openstack.org/528128
23:54:23 openstackgerrit Merged openstack/nova stable/newton: Only query BDMs once in API during rebuild https://review.openstack.org/526543
23:57:20 jogo mriedem: doesn't sound very backwards compatible to delete it ... but goodbye filter
#openstack-nova - 2017-12-15
00:14:41 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete https://review.openstack.org/525231
00:15:11 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (2) https://review.openstack.org/526263
00:15:21 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (3) https://review.openstack.org/526557
00:15:31 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in post https://review.openstack.org/526823
00:16:03 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608

Earlier   Later