Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-09-07
02:59:03 openstackgerrit Jeremy Liu proposed openstack/nova master: Implement keymgr list() method https://review.openstack.org/501533
03:03:59 openstackgerrit jichenjc proposed openstack/nova master: Add quiesce and unquiesce in support matrix https://review.openstack.org/482390
03:12:09 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Tweak connection_info translation for the new Cinder attach/detach API https://review.openstack.org/493324
03:12:09 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Add attachment_complete call to volume/cinder.py https://review.openstack.org/493323
03:12:10 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
04:21:04 openstackgerrit Takashi NATSUME proposed openstack/nova master: Update contributor guide for Queens https://review.openstack.org/500454
04:23:29 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/489029
04:43:52 openstackgerrit Jackie Truong proposed openstack/python-novaclient master: Microversion 2.54 - Add trusted_certificates param https://review.openstack.org/500396
04:46:34 openstackgerrit Jackie Truong proposed openstack/python-novaclient master: Microversion 2.54 - Add trusted_certificates param https://review.openstack.org/500396
05:10:02 openstackgerrit Andreas Jaeger proposed openstack/nova master: Fix broken URLs https://review.openstack.org/501402
06:06:14 openstackgerrit Rajesh Tailor proposed openstack/nova master: Host addition host-aggregate should be case-sensitive https://review.openstack.org/498334
06:49:59 openstackgerrit jichenjc proposed openstack/nova master: remove CONF.vendordata_driver https://review.openstack.org/501510
07:02:36 Dinesh_Bhor stephenfin: Hi, you there?
07:20:57 openstackgerrit AJAY KALAMBUR proposed openstack/nova master: Fix incorrect port unbind request in nova on instance delete https://review.openstack.org/387687
08:23:27 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612
08:38:20 openstackgerrit Merged openstack/nova master: Update contributor guide for Queens https://review.openstack.org/500454
08:46:59 bauzas good morning Novaers
08:47:11 bauzas a bit late due to paperwork...
08:49:23 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226
08:51:22 openstackgerrit jichenjc proposed openstack/nova master: remove CONF.vendordata_driver https://review.openstack.org/501510
08:57:47 stephenfin Dinesh_Bhor: I am now (forgot to open HexChat). What's up?
08:59:40 stephenfin mikal: RE: removing UML support, I removed it from the docs because it didn't work, hadn't been tested in years, and we saw that it was unused by anyone and everyone (was originally added to test some esoteric features in the gate, iirc)
08:59:53 stephenfin mikal: tl;dr: It's dead. We can remove all references to it
09:03:43 Dinesh_Bhor stephenfin: about this: https://review.openstack.org/#/c/466799/2/nova/tests/unit/api/openstack/test_wsgi.py
09:04:03 Dinesh_Bhor stephenfin: Earlier the behaviors was in response headers the integers values were allowed.
09:04:44 Dinesh_Bhor stephenfin: and I can see it RFC as well that response headers can contain integer values: https://tools.ietf.org/html/rfc2616#section-6.2
09:05:00 Dinesh_Bhor stephenfin: https://tools.ietf.org/html/rfc2616#section-14.6
09:06:07 Dinesh_Bhor stephenfin: Are we restricting integer values now? IF yes, then is it going to be impact from user's point of view.
09:06:31 stephenfin Ohhhh...
09:07:40 stephenfin Dinesh_Bhor: So I'm pretty sure I found the plain-text bit in another RFC (which I should have linked). Clearly it was either wrong or had been supplanted though
09:08:16 stephenfin Dinesh_Bhor: Fortunately that didn't merge, so you can leave your -1 and I'll address the comments if possible. If not, I'll abandon the patch
09:09:31 Dinesh_Bhor stephenfin: If we decide to not allow integers then it's ok. sure I will add my comment.
09:12:09 stephenfin Dinesh_Bhor: Yeah, I'll investigate and see why I had to change that test. If nova spits the dummy out on integers, which are clearly valid, that would not be good :)
09:12:20 stephenfin Thanks for bringing that up too. Good catch
09:13:07 Dinesh_Bhor stephenfin: Okay, np,
09:20:44 openstackgerrit Rajesh Tailor proposed openstack/nova master: Host addition host-aggregate should be case-sensitive https://review.openstack.org/498334
09:22:56 stephenfin mikal: Do you think this change could be restored now? https://review.openstack.org/#/c/397835/
09:23:46 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: deprecate image handler class path option https://review.openstack.org/497201
09:23:47 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475
09:53:40 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612
09:58:46 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi: Exception Error logs shown in Citrix XenServer CI https://review.openstack.org/478437
10:00:09 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add memoize function using oslo.cache https://review.openstack.org/472773
10:02:06 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226
10:28:04 gibi What is using compute.api.HostAPI? I see that xen and vmware implements the backend but I don't see any usage of this compute.api
10:53:50 openstackgerrit Andreas Jaeger proposed openstack/nova master: Fix broken URLs https://review.openstack.org/501402
12:39:10 awaugama t
12:41:54 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Enable cold migration with target host https://review.openstack.org/489031
12:59:31 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Add documentation for emulator_thread_policy https://review.openstack.org/501721
12:59:41 stephenfin sahid, bauzas, jaypipes: Could ye look at that ^ simple doc fix
13:00:39 bauzas ack
13:03:00 bauzas stephenfin: https://en.wikipedia.org/wiki/RRRrrrr!!!
13:03:12 bauzas stephenfin: https://review.openstack.org/#/c/501721/1
13:03:22 openstackgerrit Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297
13:03:22 openstackgerrit Balazs Gibizer proposed openstack/nova master: Send soft_delete from context manager https://review.openstack.org/476459
13:03:26 stephenfin Ha!
13:04:21 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Add documentation for emulator_thread_policy https://review.openstack.org/501721
13:04:22 stephenfin bauzas: Done :D
13:06:07 bauzas stephenfin: it's a cult film y'know
13:07:16 stephenfin bauzas: I figured as much. Had a Monty Python feel about
13:10:08 mriedem this is blocked an upper-constraints change, can we get this in https://review.openstack.org/#/c/501400/ ?
13:10:11 mriedem needs another +2
13:12:06 stephenfin mriedem: Yup, looks grand to me
13:12:17 efried 'cept for the typos
13:13:46 stephenfin efried: If you're brave enough to -1 it for that, I'll give you a cookie :P
13:13:59 stephenfin efried: mriedem might give you something else though
13:14:09 efried I only -1 for typos in docs
13:14:20 efried ("typo's in doc's"?)
13:14:35 stephenfin Yup, likewise. I kid :)
13:18:58 mikal stephenfin: yeah, I think we could take another pass at that now
13:18:59 sahid stephenfin: https://review.openstack.org/#/c/501721/2 the explnation does not make a lot of sense
13:20:24 stephenfin sahid: Yup, good call. I'll rework shortly
13:20:32 bauzas sahid: I tend to fast-approve doc changes but it helps, but feel free to open a follow-up and I'll review it
13:20:49 bauzas s/but/because
13:20:50 stephenfin bauzas, sahid: It's actually blocked by a base patch so it won't merge :/
13:21:04 bauzas stephenfin: ack, then push another rev and I'll +2
13:21:12 stephenfin +2 to fast-approving docs too. Fast iteration is really helpful
13:21:28 bauzas I mean, doc bugs don't hurt
13:21:45 bauzas while doc amendments really help
13:21:48 stephenfin Well, not as much as no docs does
13:21:57 bauzas yup that
13:22:21 bauzas unless of course if you state in the doc something like "rm -rf /"
13:22:59 sdague mikal: ok, looking
13:23:31 stephenfin mikal: Sweet. I'll revive that shortly so
13:24:40 bauzas mriedem: new school year, new constraints. I'll need to bail out from the nova meeting around 1420UTC (ie. 20 mins after you start=
13:24:56 bauzas but I should be back 20 mins after
13:25:26 bauzas hopefully the daylight shift will solve the problem in a couple of weeks :)
13:25:42 mriedem ok
13:33:25 openstackgerrit Andreas Jaeger proposed openstack/nova master: Fix broken URLs https://review.openstack.org/501402
13:33:54 sdague mikal: https://review.openstack.org/#/c/489486 ... that needs some more explaining, the the cryptsetup add isn't clear to me
13:37:54 dansmith mriedem: you going to fix the typo in that second one or just steam on?
13:38:04 mriedem doing it now
13:38:14 openstackgerrit Matt Riedemann proposed openstack/nova master: Make ConductorTaskTestCase run with 2 cells https://review.openstack.org/501514
13:38:14 openstackgerrit Matt Riedemann proposed openstack/nova master: Track which cell each instance is created in and use it consistently https://review.openstack.org/501452
13:38:15 openstackgerrit Matt Riedemann proposed openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails https://review.openstack.org/501408
13:39:31 mriedem alex raised a good question here https://review.openstack.org/#/c/501408/8/nova/conductor/manager.py@1049
13:40:06 mriedem in that if the build request is already deleted after we create the instance in the cell but before we actually send the instance to the host to build, we cleanup the instance (local delete) but don't cleanup dest node allocations
13:40:34 mriedem separate bug, but a bug nonetheless
13:41:01 mikal sdague: so I need a reno when I remove a command from rootwrap's filters? Or only when I add one?
13:41:04 openstackgerrit Merged openstack/nova master: Fix test_rpc_consumer_isolation for oslo.messaging 5.31.0 https://review.openstack.org/501400
13:42:20 sdague the adds are the critical bit. That's a config file which won't work if you don't add those lines

Earlier   Later