| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-09-07 | |||
| 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 | |
| 13:42:29 | sdague | if people left old things around, meh | |
| 13:43:07 | openstackgerrit | Andreas Jaeger proposed openstack/nova master: Fix broken URLs https://review.openstack.org/501402 | |
| 13:43:38 | mriedem | who wants to look at https://review.openstack.org/#/c/501514/ and up? regression in pike that needs to be backported | |
| 13:43:42 | mriedem | vexxhost hit it yesterday | |
| 13:44:08 | jaypipes | mriedem: just FYI, we're evacuating this evening so I'll be OOO tomorrow and part of this afternoon. | |
| 13:44:23 | mriedem | jaypipes: ok, good luck | |
| 13:44:25 | jaypipes | mriedem: and I do mean evacuate, not rebuild. | |
| 13:44:29 | sdague | mikal: the general rule of thumb, assume someone is upgrading with pip install -U on the commit. Do they need to do something besides that to have usable environment | |
| 13:44:39 | mriedem | jaypipes: i hear ohio still has room for you | |
| 13:44:44 | mriedem | if you'll take it back | |
| 13:44:47 | jaypipes | mriedem: that's where we're headed. | |
| 13:45:04 | mriedem | nice | |
| 13:45:15 | mriedem | just in time for your beloved browns season to begin | |
| 13:45:22 | sdague | jaypipes: well, that means jeni's icecream when you arrive | |
| 13:45:24 | dansmith | jaypipes: good luck | |
| 13:45:30 | jaypipes | sdague: heh :) | |
| 13:45:43 | jaypipes | dansmith: thx. still smoldering where you are? | |
| 13:45:56 | dansmith | jaypipes: yeah | |
| 13:45:58 | sdague | jaypipes: we stopped in columbus north market on our drive to TN for the eclipse. Could not leave without some jeni's | |
| 13:46:14 | sdague | jaypipes: safe travels | |
| 13:46:21 | jaypipes | sdague: indeed. Jeni's is awesome. Graeters is also very good :) | |
| 13:46:26 | jaypipes | sdague: thx | |
| 13:46:36 | jaypipes | dansmith: how close to you are the fires? | |
| 13:46:54 | sdague | jaypipes: we going to see you in CO still? | |