Comment 3 for bug 1663166

Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

spineau: ePierre: I've found the root cause of https://bugs.launchpad.net/plainbox/+bug/1663166
spineau: ePierre: to render the text data we're using ByteStringStreamTranslator
spineau: ePierre: but we're using filling it with shutil.copyfileobj
spineau: ePierre: so there are (in our case) two calls to the write method
ePierre: spineau, \o/
spineau: ePierre: it mean two chunks, and guess what cut right in the middle of the sigil char
spineau: ePierre: copyfileobj(data, translating_stream, -1) in transport.py fixes the issue
ePierre: spineau, you mean stop using chunks and instead copying all of it at once?
spineau: ePierre: yes, or stop using the unicode checkboxes
ePierre: spineau, knowing that, it's kind of incredible this kind of issue doesn't happen more often...
ePierre: spineau, for your info, the session attached to the issue is a very legit, full test session
ePierre: spineau, which is 90% of what QA team tests
spineau: ePierre: indeed. I think we're using StreamTransport only for the ttext exporter so I'd assume we won't fill all the system memory if we use -1
ePierre: so I think it's fair to assume that if copying the whole thing at once works well with this session, it should work well with anything
ePierre: spineau, ok. Maybe it would be wise to check if this would have an inpact on ubuntu core or smalled devices that we've tested...