Skip to content
This repository has been archived by the owner on Jul 29, 2021. It is now read-only.

Fix Travis tests #4

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Fix Travis tests #4

wants to merge 3 commits into from

Conversation

pviotti
Copy link
Member

@pviotti pviotti commented Mar 15, 2020

The local and distributed file system tests work locally and yet fail on Travis. Let's try to fix this.

image

@pviotti
Copy link
Member Author

pviotti commented Mar 17, 2020

I've been testing locally with an obsolete version of antidotedb/antidote:latest. Updating it made me realize that the latest versions - even the 0.1.1 - has some data model incompatibility with the version I originally used to develop antidote-fs (0.1.0), which generates this kind of errors:

{{unknown_crdt_type_protobuf,9},[{antidote_pb_codec,decode_type,1,[{file,"/tmp/antidote/_build/default/lib/antidote_pb_codec/src/antidote_pb_codec.erl"},{line,553}]},{antidote_pb_codec,decode_bound_object,1,[{file,"/tmp/antidote/_build/default/lib/antidote_pb_codec/src/antidote_pb_codec.erl"},{line,399}]},{antidote_pb_codec,'-decode_message/1-lc$^2/1-2-',1,[{file,"/tmp/antidote/_build/default/lib/antidote_pb_codec/src/antidote_pb_codec.erl"},{line,213}]},{antidote_pb_codec,decode_message,1,[{file,"/tmp/antidote/_build/default/lib/antidote_pb_codec/src/antidote_pb_codec.erl"},{line,213}]},{antidote_pb_protocol,handle,3,[{file,"/tmp/antidote/_build/default/lib/antidote/src/antidote_pb_protocol.erl"},{line,72}]},{antidote_pb_protocol,loop,2,[{file,"/tmp/antidote/_build/default/lib/antidote/src/antidote_pb_protocol.erl"},{line,54}]}]}

The format of logs and their location in the container changed too - but that's a minor detail.
Unfortunately, the docker container for Antidote v0.1.0 is not published, which makes it difficult to have this project CI tests working at the moment.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants