| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-19 | |||
| 17:18:34 | mdbooth | https://bugs.launchpad.net/oslo.versionedobjects/+bug/1552927 | |
| 17:18:35 | openstack | Launchpad bug 1552927 in oslo.versionedobjects "hasattr() function behavior differences between Py27 and Py34" [Undecided,New] | |
| 17:18:43 | mdbooth | lyarwood: Already been reported ^^^^ | |
| 17:19:34 | mdbooth | obj.attr_is_set()... | |
| 17:21:16 | lyarwood | mdbooth: ah! | |
| 17:21:42 | mdbooth | Just testing that | |
| 17:22:34 | mdbooth | lyarwood: Works, at least for that test | |
| 17:22:48 | lyarwood | mdbooth: yeah works for me | |
| 17:23:48 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: QEMU native LUKS decryption for encrypted volumes https://review.openstack.org/523958 | |
| 17:35:28 | claudiub | mdbooth: gonna send you a link soon. already 60 seconds passed while trying to execute unittests which shouldn't last more than 10 seconds. :) | |
| 17:42:40 | claudiub | mdbooth: https://github.com/bclau/nova/tree/autospec-issue | |
| 17:43:22 | mdbooth | lyarwood: I'd have left you a +1 if Jenkins had voted already. I'd appreciate if you consider the readability of the new test, though. | |
| 17:43:53 | mdbooth | My eyeballs aren't good at computing diffs. | |
| 17:43:54 | claudiub | execution time: vanilla: ~10 seconds. oslotest mock.MagicMock(autospec): ~14.5 sec. mock.create_autospec: I dunno, a few hundred seconds, still running. | |
| 17:44:54 | mdbooth | claudiub: That's great info, and definitely something I hadn't appreciated. | |
| 17:45:11 | mdbooth | I think I've used create_autospec in a few places in the past. | |
| 17:45:53 | claudiub | it's definetely great, but the fact that it autospecs literally everything can be problematic. | |
| 17:46:19 | claudiub | and I have sent pull requests to python 3 and the mock library, but no response for at least half a year | |
| 17:46:35 | claudiub | and there hasn't been any updates to the mock library since more than 12-13 months | |
| 17:47:03 | claudiub | Ran 364 tests in 434.626s | |
| 17:53:05 | openstackgerrit | David Rabel proposed openstack/nova stable/pike: Fix format in live-migration-usage.rst https://review.openstack.org/535871 | |
| 18:00:27 | mdbooth | Incoming | |
| 18:01:03 | openstackgerrit | Matthew Booth proposed openstack/nova master: Remove redundant swap_volume tests https://review.openstack.org/531179 | |
| 18:01:04 | openstackgerrit | Matthew Booth proposed openstack/nova master: Give volume DriverBlockDevice classes a common prefix https://review.openstack.org/526346 | |
| 18:01:04 | openstackgerrit | Matthew Booth proposed openstack/nova master: Add DriverLocalImageBlockDevice https://review.openstack.org/526347 | |
| 18:01:05 | openstackgerrit | Matthew Booth proposed openstack/nova master: Expose driver_block_device fields as attributes https://review.openstack.org/528362 | |
| 18:01:05 | openstackgerrit | Matthew Booth proposed openstack/nova master: Add local_root to block_device_info https://review.openstack.org/529029 | |
| 18:01:06 | openstackgerrit | Matthew Booth proposed openstack/nova master: Pass DriverBlockDevice to driver.attach_volume https://review.openstack.org/528363 | |
| 18:01:06 | openstackgerrit | Matthew Booth proposed openstack/nova master: Use real block_device_info data in libvirt tests https://review.openstack.org/527916 | |
| 18:01:07 | openstackgerrit | Matthew Booth proposed openstack/nova master: Fix libvirt volume tests passing invalid disk_info https://review.openstack.org/529328 | |
| 18:01:07 | openstackgerrit | Matthew Booth proposed openstack/nova master: Pass disk_info dict to libvirt_info https://review.openstack.org/529329 | |
| 18:01:08 | openstackgerrit | Matthew Booth proposed openstack/nova master: Expose volume host type and path independent of libvirt config https://review.openstack.org/530786 | |
| 18:01:09 | openstackgerrit | Matthew Booth proposed openstack/nova master: Don't generate fake disk_info in swap_volume https://review.openstack.org/530787 | |
| 18:01:09 | openstackgerrit | Matthew Booth proposed openstack/nova master: Local disk serial numbers for the libvirt driver https://review.openstack.org/529380 | |
| 18:15:36 | lyarwood | mdbooth: ack, I might rewrite both tests using the conf objects and format_dom calls | |
| 18:38:02 | openstackgerrit | Jim Rollenhagen proposed openstack/nova master: Ironic: negotiate microversion to allow downgrade to Pike https://review.openstack.org/535786 | |
| 19:13:34 | cfriesen | found an interesting glance/nova interworking issue. if doing a snapshot, and cinder/glance are both on ceph, nova tries to shortcut the snapshot then set the location in glance. But by default glance won't let you set the location of an image, so this fails. | |
| 19:16:09 | cfriesen | correction, not cinder, but when instance backend is ceph. | |
| 19:57:39 | cburgess | cfriesen Sounds like nova is doing the right thing but glance just isn't configured properly. Sounds like maybe a release note or something is needed? | |
| 19:57:54 | cburgess | cfriesen I suspect cinder will run into the same issue. | |
| 20:22:17 | fried_rolls | you gotta be effin kidding me. The bottom three patches, all +W'd, got zuul -1 with (three *different*) spurious failures. The fourth patch - the first one with a -1 - passed. | |
| 20:24:05 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/535919 | |
| 20:25:32 | ildikov | fried_rice: mine got a time_out error on the gate, sooo close, but no... | |
| 20:26:46 | fried_rice | Boo. I feel your pain. | |
| 20:43:30 | fried_rice | figleaf regarding https://review.openstack.org/#/c/526540/17/nova/scheduler/client/report.py@390 -- the only possible reason I can see for it having been split out before was LOG.warning vs .error. | |
| 20:43:59 | fried_rice | figleaf But given that the error text is (at least) as descriptive as what was there before, are you cool with it? | |
| 20:55:35 | openstackgerrit | Ihar Hrachyshka proposed openstack/nova master: DNM testing whether lib/neutron switch breaks this repo https://review.openstack.org/535947 | |
| 21:02:27 | cfriesen | cburgess: I've been diving down into the rathole of the nova/glance interaction. turns out glance's policy stuff is all mucked up, so that enabling nova's snapshot in this case requires glance to be configured insecurely. It might be possible to use service tokens to fix it, but nova doesn't send them by default. | |
| 21:02:39 | cfriesen | speaking of which...anyone know why send_service_user_token is False by default? | |
| 21:03:19 | cfriesen | cburgess: the glance folks suggested a workaround of a separate glance node that is configured less secure, which is only accessible by other openstack components. kind of icky if you ask me. | |
| 21:08:02 | figleaf | fried_rice: It just seemed like a bigger behavior change than it set out to be | |
| 21:08:13 | figleaf | fried_rice: it's not a showstopper by any stretch | |
| 21:08:20 | fried_rice | coolcool. | |
| 21:09:58 | leakypipes | fried_rice: k, your patches are now reviewed up to the WIPs. | |
| 21:10:11 | fried_rice | leakypipes Thanks dude. Starting to work through 'em. | |
| 21:10:17 | leakypipes | yup, np | |
| 21:11:52 | openstackgerrit | Matthew Edmonds proposed openstack/nova master: Fix bad autospec usage https://review.openstack.org/535806 | |
| 21:15:21 | edmondsw | claudiub ^ fixed a pep8 error. Interested in how you feel about that patch | |
| 21:16:20 | edmondsw | I'm not against using the fixture you got in oslotests if the performance is better, but this might be quicker to merge and doesn't preclude your changes coming later | |
| 21:16:44 | edmondsw | doesn't have a significant performance impact at this small scale that I've detected | |
| 21:20:26 | edmondsw | cfriesen send_service_user_token is False by default simply because you have to configure credentials for that to work | |
| 21:21:06 | edmondsw | I'd like for send_service_user_token to go away, and just have the code detect whether it has credentials or not and use that as the determining factor, but that's a nit | |
| 21:22:37 | openstackgerrit | Hongbin Lu proposed openstack/nova master: Handle RPC timeout on get_console_output https://review.openstack.org/525335 | |
| 21:28:03 | cburgess | cfriesen Thats very icky, especially considering that its worked this way for a while (the nova side I mean). | |
| 21:28:20 | cburgess | nova merged this stuff shortly after Tokyo | |
| 21:37:56 | cfriesen | cburgess: I think it might be related to the glance v1/v2 stuff | |
| 21:38:06 | cburgess | cfriesen Hmm yeah I think you might be right. | |
| 21:59:39 | openstackgerrit | Jim Rollenhagen proposed openstack/nova master: Ironic: negotiate microversion to allow downgrade to Pike https://review.openstack.org/535786 | |
| 23:00:00 | fried_rice | leakypipes Still around? | |
| 23:04:19 | fried_rice | leakypipes Cancel, I see it now. | |
| 23:28:24 | openstackgerrit | Eric Fried proposed openstack/nova master: Raise on API errors getting aggregates/traits https://review.openstack.org/526540 | |
| 23:28:24 | openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.populate_from_iterable https://review.openstack.org/520756 | |
| 23:28:25 | openstackgerrit | Eric Fried proposed openstack/nova master: Track tree-associated providers in report client https://review.openstack.org/526541 | |
| 23:28:25 | openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient.set_aggregates_for_provider https://review.openstack.org/533808 | |
| 23:28:26 | openstackgerrit | Eric Fried proposed openstack/nova master: report client: get_provider_tree_and_ensure_root https://review.openstack.org/521098 | |
| 23:28:26 | openstackgerrit | Eric Fried proposed openstack/nova master: ComputeDriver.update_provider_tree() https://review.openstack.org/521187 | |
| 23:28:27 | openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient._delete_provider https://review.openstack.org/535463 | |
| 23:28:27 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: SchedulerReportClient.update_from_provider_tree https://review.openstack.org/533821 | |
| 23:28:28 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | |
| 23:28:28 | openstackgerrit | Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260 | |
| 23:28:29 | openstackgerrit | Eric Fried proposed openstack/nova master: Move refresh time from report client to prov tree https://review.openstack.org/535517 | |
| 23:50:14 | Spazmotic | Hrm. | |
| 23:50:40 | fried_rice | Do tell | |
| 23:50:49 | Spazmotic | Just reading over what leaky said | |
| 23:51:07 | fried_rice | Yeah, I still think there's a hole there. | |
| 23:51:16 | fried_rice | I meant to put it on the PTG agenda, but I think I forgot. | |
| 23:51:20 | fried_rice | ... | |
| 23:51:59 | Spazmotic | https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/handlers/allocation.py#L335-L343 | |
| 23:52:24 | Spazmotic | This is what I see.. It's if the set allocations POST call doesn't receive any data, it pulls all the consumers allocations that exist and runs allocation.used = 0 | |
| 23:52:28 | fried_rice | Spazmotic You going to be at the PTG? | |
| 23:52:44 | Spazmotic | Nah i'll be in Korea and honestly i think i'm too new to the scene to cause a fuss :D | |
| 23:53:08 | Spazmotic | I will re-read the code tonight, maybe I just don't understand it properly | |
| 23:55:33 | fried_rice | Spazmotic Oh, and the other thing was a way to add/update/delete individual pieces of allocation rather than the whole payload. | |
| 23:55:42 | fried_rice | Spazmotic I'm not sure there's really a good use case for that. | |
| 23:55:54 | fried_rice | ...IF we have proper concurrency management | |
| 23:56:25 | fried_rice | The amount of data just isn't that onerous. And you'd want to have the existing allocation before you tried sending an update anyway. If for no other reason than to have the current generation :) | |
| 23:56:48 | Spazmotic | Hmm. I guess I can understand that to ensure no double allocations. | |
| 23:57:59 | Spazmotic | I think the only use case that I could think of immediately was the ability to manage the allocations of a resource provider without having to touch multi-tenant full libraries of allocations, which just feels a bit top heavy for that ability. But without a redesign of how taht works it seems unavoidable for the immediate future. | |
| #openstack-nova - 2018-01-20 | |||
| 00:00:46 | Spazmotic | I will head to the gym and think on it. Poor brain. | |
| 00:00:58 | fried_rice | Spazmotic You don't have to muck with allocations for anything other than the single consumer you're dealing with. | |