Uh oh!
There was an error while loading. Please reload this page.
Bump msgpack version - #171
Conversation
Signed-off-by: Nilesh Patra <npatra974@gmail.com>
71c3df5 to
5b798ffComparecoveralls
commented
Nov 19, 2020
1 similar comment
coveralls
commented
Nov 19, 2020
repeatedly
commented
Jan 25, 2021
@arcivanov Do you have any concern for this patch? |
arcivanov
commented
Jan 26, 2021
Yes, I do @repeatedly. Specifically due to #157. #157 provides a description and logs of a specific in-FluentD unpacking failure requiring msgpack pinning pre-1.0. The Fluent Logger 1.0 has a harness rewrite that would allow compatibility testing with whatever versions of fluentd/-bit but I simply don't have time to work on it right now. |
| # TODO: have to process string encoding properly. currently we assume | ||
| # that all encoding is utf-8. | ||
| return list(Unpacker(self._buf, encoding='utf-8')) | ||
| return list(Unpacker(self._buf)) |
There was a problem hiding this comment.
Changing the mock server to "make tests pass" does nothing to alleviate concerns in #157
bgruening
commented
Mar 10, 2021
We are starting to get unresolvable venvs because of the upper-pin. Is there anything we can do to help here? |
arcivanov
commented
Mar 10, 2021
I'll remove the pin today in that case. |
fixesfluent#171 Signed-off-by: Arcadiy Ivanov <arcadiy@ivanov.biz>
Remove Python 2.7 to remove the msgpack encoding ambiguity issue. Remove Python 3.4 support just because it's dead. fixesfluent#171 Signed-off-by: Arcadiy Ivanov <arcadiy@ivanov.biz>
Remove Python 2.7 to remove the msgpack encoding ambiguity issue. Remove Python 3.4 support just because it's dead. fixesfluent#171 Signed-off-by: Arcadiy Ivanov <arcadiy@ivanov.biz>
Remove Python 2.7 to remove the msgpack encoding ambiguity issue. Remove Python 3.4 support just because it's dead. fixesfluent#171 Signed-off-by: Arcadiy Ivanov <arcadiy@ivanov.biz>
Remove Python 2.7 to remove the msgpack encoding ambiguity issue. Remove Python 3.4 support just because it's dead. fixesfluent#171 Signed-off-by: Arcadiy Ivanov <arcadiy@ivanov.biz> Signed-off-by: Matt Cipperly <mcipperly@fb.com>
Remove Python 2.7 to remove the msgpack encoding ambiguity issue. Remove Python 3.4 support just because it's dead. fixesfluent#171 Signed-off-by: Arcadiy Ivanov <arcadiy@ivanov.biz>
Remove Python 2.7 to remove the msgpack encoding ambiguity issue. Remove Python 3.4 support just because it's dead. fixesfluent#171 Signed-off-by: Arcadiy Ivanov <arcadiy@ivanov.biz>
Remove Python 2.7 to remove the msgpack encoding ambiguity issue. Remove Python 3.4 support just because it's dead. fixesfluent#171 Signed-off-by: Arcadiy Ivanov <arcadiy@ivanov.biz>
Since the
encodingoption is always UTF-8 with msgpack>=1, I've remove the same.Otherwise it fails with:
After the change, it goes green: