Jaeger: Strip whitespace from value for servers in configmap

[joshushua]

Requirement

Listing the IP addresses of each node of a Cassandra cluster.

Problem - what in Jaeger blocks you from solving the requirement?

In the configmap.yml file, the key-value pair of _servers: XX.XX.XXX.XX_ for one IP entry works fine. But when providing more than one entry, like _servers: XX.XX.XXX.XX, XX.XXX.XX.XXX, XX.XXX.XXX.XX_, the Collector's gocql reveals an error for the second IP: _dns error: lookup XX.XXX.XX.XXX: no such host_ and reveals the same error for the third IP.

When I remove the whitespace between the comma and the next IP, the thing works fine and the Collector successfully connects to the Cassandra cluster.

Proposal - what do you suggest to solve the problem or improve the existing situation?

Strip white spaces from the value, so that humans can still use whitespaces when writing and reading these values.

Any open questions to address

Will you please include this type of Cassandra example in your documentation? Something beyond the all-in-one example which is too simple for production scenarios. I couldn't find anything about clusters and raw IP addresses for the cluster.

[jpkrohling]

For those looking to send their first contribution to the project, this is probably the right place to apply the fix:

https://github.com/jaegertracing/jaeger/blob/master/plugin/storage/cassandra/options.go

Make sure to add a new test to https://github.com/jaegertracing/jaeger/blob/master/plugin/storage/cassandra/options_test.go

[jpkrohling]

And don't forget to read the CONTRIBUTING.md!

[karlpokus]

I'll have a PR ready shortly.

[black-adder]

reopening to address the documentation ask

[Abhi-khandelwal]

Hi @black-adder, I'm new to this community, can I work on this issue ?

[yurishkuro]

@Abhi-khandelwal by all means, but note that at this point it's a documentation task, not coding.

[Abhi-khandelwal]

Ohk @yurishkuro , I'll try to fix some other beginner coding issues. :)

[aloeinc]

@jpkrohling I'm coming from Outreachy, am interested in data tracing research, and would enjoy solving this documentation issue. Would you recommend any other tasks I could complete in order to exemplify my skills?

[jpkrohling]

@aloeankh sorry, I just saw your message. The application phase for Outreachy is over (or almost), but if you'd still like to contribute, the right place for this would probably be in the Cassandra-specific documentation, here:

https://www.jaegertracing.io/docs/1.14/deployment/#cassandra
(source: https://github.com/jaegertracing/documentation/blob/master/content/docs/next-release/deployment.md#cassandra )