[CASSANDRASC-10] Read sidecar.yaml from sidecar.config System Property instead of classpath

• Type: Improvement
• Status: Resolved
• Priority: Low
• Resolution: Fixed
• Labels: pull-request-available
• Change Category: Semantic
• Complexity: Low Hanging Fruit

Source Control Link

https://github.com/apache/cassandra-sidecar/commit/14485bd7ad649d9417b4320eab34631251545d0b

Test and Documentation Plan

existing tests should be sufficient.

• Assignee: Andy Tolbert
• Reporter: Andy Tolbert

Authors

Andy Tolbert

• Created: 22/Aug/19 18:42
• Updated: 17/Mar/20 23:28
• Resolved: 23/Aug/19 23:38

Description

I was experimenting earlier with running multiple sidecar instances against multiple local C* nodes (using a ccm cluster) and noticed that the distribution of sidecar did not include the conf directory and instead was loading sidecar.yaml from the classpath. I noticed the lines in build.gradle:

// Config file location should be in file:/// format for local files,
// when we have the fix for adding /conf directory to classpaht, we can get away with below default JvmArg

From this statement I assume the expectation was that eventually the sidecar.config system property would be used to locate the sidecar.yaml file.

Issue Links

• links to: GitHub Pull Request #5
• links to: https://github.com/apache/cassandra-sidecar/pull/5