Comment 8 for bug 1678018

Revision history for this message
Matthew Oliver (matt-0) wrote :

Like clay, I put snowmen everywhere: http://paste.openstack.org/show/606008/

And was also getting _full_path issues, and ssync sender stuff.

This patch seems to fix it for me. It makes sure the metadata from the diskfile is encoded to utf8 so it is handled the same in repl and EC (thanks to Alistair's notes).

And then decoded to unicode in _full_path so we don't get errors there.

I'm sure there are better ways, but this seems to work and it hasn't broken any unit tests.. although I haven't yet created a unit test that fails. This also gets around not needing to have the key utf8 string-ified (current gerrit patch), because the metadata has already been encoded.

It may be overkill, but this is where I got to this arvo, and I need to finish up now (dinner). So haven't simplified further.