[
https://issues.apache.org/jira/browse/CASSANDRA-15288?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Dinesh Joshi updated CASSANDRA-15288:
-------------------------------------
Fix Version/s: 4.x
Source Control Link:
https://github.com/apache/cassandra-sidecar/commit/14485bd7ad649d9417b4320eab34631251545d0b
Resolution: Fixed
Status: Resolved (was: Ready to Commit)
Committed. Thanks for the patch [~andrew.tolbert]. Thanks [~vinaykumarcse] for
the review.
> Read sidecar.yaml from sidecar.config System Property instead of classpath
> --------------------------------------------------------------------------
>
> Key: CASSANDRA-15288
> URL: https://issues.apache.org/jira/browse/CASSANDRA-15288
> Project: Cassandra
> Issue Type: Improvement
> Components: Sidecar
> Reporter: Andy Tolbert
> Assignee: Andy Tolbert
> Priority: Low
> Labels: pull-request-available
> Fix For: 4.x
>
>
> 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:
> {code:java}
> // 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
> {code}
> From this statement I assume the expectation was that eventually the
> sidecar.config system property would be used to locate the sidecar.yaml file.
--
This message was sent by Atlassian Jira
(v8.3.2#803003)
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org
