| *** gouthamr has joined #openstack-shade | 00:27 | |
| *** gouthamr has quit IRC | 01:16 | |
| *** gouthamr has joined #openstack-shade | 02:59 | |
| *** gouthamr has quit IRC | 03:19 | |
| *** yfried has joined #openstack-shade | 07:27 | |
| *** ioggstream has joined #openstack-shade | 08:20 | |
| *** ioggstream has quit IRC | 08:40 | |
| *** ioggstream has joined #openstack-shade | 08:40 | |
| *** openstackgerrit has quit IRC | 09:03 | |
| *** cdent has joined #openstack-shade | 09:45 | |
| *** cdent has quit IRC | 11:22 | |
| *** cdent has joined #openstack-shade | 11:56 | |
| *** ioggstream has quit IRC | 12:42 | |
| *** yfried has quit IRC | 12:54 | |
| *** yfried has joined #openstack-shade | 12:54 | |
| *** gouthamr has joined #openstack-shade | 13:20 | |
| *** cdent has quit IRC | 14:02 | |
| *** cdent has joined #openstack-shade | 14:02 | |
| *** gouthamr has quit IRC | 14:44 | |
| *** cdent has quit IRC | 14:58 | |
| *** ioggstream has joined #openstack-shade | 15:08 | |
| ioggstream | hi @all | 15:08 |
|---|---|---|
| *** cdent has joined #openstack-shade | 15:25 | |
| ioggstream | while uploading a file with create_object I receive a '411 Length required' | 15:32 |
| ioggstream | do I need to set something ? | 15:32 |
| clarkb | ioggstream: guessing that the restification of swift may have dropped an implicit file size header addition from the old swiftclient code path | 15:41 |
| ioggstream | clarkb: is that a feature ? ;) | 15:42 |
| clarkb | https://git.openstack.org/cgit/openstack/keystoneauth/tree/keystoneauth1/adapter.py#n231 is what is being called with a pretty minimal set of kwargs so ya I expect that is what is happening | 15:44 |
| *** yfried has quit IRC | 15:45 | |
| clarkb | ioggstream: I think if you just add a length based on file size to the headers in create_object it will fix it for you (I also wonder if this means we have broken tests somewhere or maybe you have a swift api that is more strict than what we get from devstack) | 15:46 |
| ioggstream | clarkb: now I'm on Helion | 15:51 |
| ioggstream | clarkb: manually adding an header works | 16:02 |
| notmorgan | ioggstream: that seems like a bug forsure then | 16:02 |
| clarkb | notmorgan: ya however I'm beginnign to think it may be a bug in helions swift api | 16:03 |
| notmorgan | at least for the helion profile | 16:03 |
| clarkb | at least this appears to be tested against devstack? | 16:03 |
| notmorgan | clarkb: yah, or we need some magic invocation ot say we need it | 16:03 |
| clarkb | so might be worth pushing on both ends (shade to amke things work and helion to be more swift) | 16:03 |
| ioggstream | notmorgan: clarkb swiftclient works | 16:03 |
| notmorgan | yeah | 16:03 |
| notmorgan | i wonder if helion is running swift in apache | 16:04 |
| notmorgan | or something | 16:04 |
| notmorgan | and that makes it a bit more strict | 16:04 |
| clarkb | ya | 16:04 |
| notmorgan | to be fair... it seems silly not to send a length header | 16:04 |
| clarkb | ++ | 16:04 |
| notmorgan | i'd be inclined to make shade always send it if it wont break things | 16:04 |
| ioggstream | notmorgan: it may depend if you need to chink | 16:06 |
| ioggstream | chunk | 16:06 |
| notmorgan | right. | 16:06 |
| notmorgan | but that should be something shade should know | 16:06 |
| ioggstream | notmorgan: in case the issue will happen again I'll dwell and open a ticket | 16:07 |
| ioggstream | now I've to upload stuff, I'll investigate asap | 16:08 |
| notmorgan | sounds good | 16:08 |
| notmorgan | please do open a bug at the very least and keep us updated as you investigate (we might beat you to a resolution that way if you get swamped) | 16:08 |
| *** yfried has joined #openstack-shade | 16:09 | |
| ioggstream | notmorgan: yep, instead let me know about the neutron stuff :D | 16:12 |
| ioggstream | notmorgan: when you've time | 16:12 |
| *** yfried has quit IRC | 16:32 | |
| clarkb | if it is just a proxy (like apache) being more strict we could enable tls on the shade tests which terminates tls with apache and would get us that extra strictness | 16:46 |
| ioggstream | clarkb: I'll let you know asap | 17:23 |
| ioggstream | see you! | 17:23 |
| *** ioggstream has quit IRC | 17:27 | |
| *** openstackgerrit has joined #openstack-shade | 18:39 | |
| openstackgerrit | Mike Perez proposed openstack-infra/shade master: Accept device_id option when updating ports https://review.openstack.org/449271 | 18:39 |
| mordred | thingee: jinx https://review.openstack.org/#/c/448781/ | 18:42 |
| mordred | thingee: although you actually set the author right on yours | 18:43 |
| mordred | thingee: maybe you add the docstring line from mine to yours and the bug reference in the commit message so it'll flag when it gets pushed to gh ... and I'll abandon mine and land yours instead | 18:43 |
| clarkb | mordred: thingee except docstring still needs update | 18:45 |
| clarkb | oh you said that derp | 18:45 |
| thingee | clarkb: oh right, I meant to add that | 18:52 |
| openstackgerrit | Mike Perez proposed openstack-infra/shade master: Accept device_id option when updating ports https://review.openstack.org/449271 | 18:56 |
| thingee | mordred, clarkb done ^ | 18:56 |
| clarkb | mordred: worth noting that if shade becomes openstack doing ^ becomes a bit murkier on legality | 18:56 |
| thingee | hmm true | 18:56 |
| clarkb | because cla yada yada (but that should improve in the future if we can get gerrit using openstackid) | 18:56 |
| mordred | clarkb: indeed. | 18:58 |
| mordred | although OTOH, I'm pretty confident that the one-word patch there is not legally encumberable | 18:59 |
| clarkb | ya this particular instance is probably fine | 18:59 |
| clarkb | but you could do that with a much bigger patch today and maybe not in the future | 18:59 |
| notmorgan | mordred: never underestimate lawyers and companies lawyering up for the weirdest/most innocuous things | 19:00 |
| mordred | ++ to both | 19:01 |
| *** cdent has quit IRC | 20:48 | |
| *** yfried has joined #openstack-shade | 21:36 | |
| *** yfried has quit IRC | 22:28 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!