onyx

FYI: alternative Onyx :onyx: chat is at <https://gitter.im/onyx-platform/onyx> ; log can be found at <https://clojurians-log.clojureverse.org/onyx/index.html>
2018-07-01T11:04:17.000044Z

@dave.dixon i think Onyx upgraded ZK (specifically, curator)to a more recent version in 0.13, but there were some issues with curator-test which was left on the old version

2018-07-01T11:04:49.000003Z

i'm using the latest version of curator/ZK myself (because I need a recent guava) and haven't experienced any issues so far

sparkofreason 2018-07-01T14:48:16.000010Z

@lmergen Thanks. Do you know what version of ZK server you're running?

2018-07-01T14:48:43.000043Z

3.3.6

sparkofreason 2018-07-01T14:53:17.000052Z

Interesting. I'll poke around in the deps analysis and see if something else might be interfering. I'll also give 3.3.6 a try if I can.

sparkofreason 2018-07-01T16:15:07.000057Z

Onyx won't start at all with ZK 3.3.6, just hangs when trying to connect.

sparkofreason 2018-07-01T16:17:56.000043Z

The version of Kafka I'm running also seemed to have trouble with 3.3.6.

lucasbradstreet 2018-07-01T17:42:07.000006Z

@lmergen did you mean 3.3.6? That’s really old, but you say you’re using the latest version of ZK?

2018-07-01T17:51:04.000022Z

hmmm I’m using the latest tag on docker

2018-07-01T17:53:32.000033Z

oops, i read it wrong indeed

2018-07-01T17:53:51.000036Z

3.4.12 it is

lucasbradstreet 2018-07-01T17:55:38.000027Z

I wonder if the issue is something to do with curator test and resolution with tools-dep then

lucasbradstreet 2018-07-01T17:57:01.000026Z

Maybe we need some instructions to switch everything over together with exclusions, or just force users to provide both. Dependencies with zk and kafka can be tough with all of the versions floating out there in prod deployments

2018-07-01T17:58:25.000023Z

don’t forget about us poor google cloud users :)

2018-07-01T17:58:37.000060Z

i still need to override guava

2018-07-01T18:00:49.000071Z

i’ve documented it here though => https://github.com/solatis/onyx-google-pubsub

lucasbradstreet 2018-07-01T18:01:15.000062Z

Right yep

lucasbradstreet 2018-07-01T18:01:22.000064Z

How’s it working out for you?

2018-07-01T18:02:05.000111Z

good! running stable since release, 3 node cluster

lucasbradstreet 2018-07-01T18:02:40.000070Z

Great :)

lucasbradstreet 2018-07-01T18:04:33.000019Z

At some point we can bring it into the official onyx org, and give you commit access. That is, assuming you’re interested.

2018-07-01T18:05:16.000039Z

yes would be good, i would like to make a few more changes in usability before that

2018-07-01T18:05:50.000092Z

there are some things regarding authentication i would like to improve

lucasbradstreet 2018-07-01T18:09:03.000014Z

Great. Yeah, better to wait until you’ve iterated on it enough :)

sparkofreason 2018-07-01T18:16:06.000036Z

@lmergen Are you using onyx-kafka, or the kafka-clients lib?

2018-07-01T18:16:27.000038Z

not using kafka

2018-07-01T18:17:04.000052Z

@lucasbradstreet exactly, i’ll give you a nudge when i think it’s fully stable.

sparkofreason 2018-07-01T18:31:33.000058Z

@lucasbradstreet Not sure if it's relevant, but here are the logs ZK gives when the exception occurs.

sparkofreason 2018-07-01T18:51:08.000013Z

Got it working after excluding ZK enough places and referring 3.4.12.

lucasbradstreet 2018-07-01T18:52:02.000011Z

Great. That was what I was about to suggest once I saw “unknown type 15” in the logs and did some googling

lucasbradstreet 2018-07-01T18:52:18.000017Z

It’d be helpful to know which libs you needed to exclude it from - especially anything we can improve on our side.

2018-07-01T18:55:21.000076Z

on that note, @lucasbradstreet -- would it be possible to move curator-test to the :test profile? or is this related to the onyx-test-env ?

lucasbradstreet 2018-07-01T18:56:36.000034Z

Yeah, I think that’s a good idea. I’ve been hesitant about breaking user code but I think it probably has to be done

lucasbradstreet 2018-07-01T18:59:00.000057Z

I have some backlog work to do with migrating all our projects to circleci 2, but if you want to send us a PR after doing a little testing I’d appreciate it

2018-07-01T18:59:17.000013Z

sure

sparkofreason 2018-07-01T19:07:27.000073Z

I'll try and figure out the minimal exclusions.

sparkofreason 2018-07-01T20:29:54.000040Z

Here's what I wound up with.