[jira] [Updated] (CASSANDRA-15288) Read sidecar.yaml from sidecar.config System Property instead of classpath
[
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
[jira] [Updated] (CASSANDRA-15288) Read sidecar.yaml from sidecar.config System Property instead of classpath
[
https://issues.apache.org/jira/browse/CASSANDRA-15288?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Dinesh Joshi updated CASSANDRA-15288:
-
Status: Ready to Commit (was: Review In Progress)
+1
> 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
>
> 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
[jira] [Updated] (CASSANDRA-15288) Read sidecar.yaml from sidecar.config System Property instead of classpath
[
https://issues.apache.org/jira/browse/CASSANDRA-15288?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Andy Tolbert updated CASSANDRA-15288:
-
Test and Documentation Plan: existing tests should be sufficient.
Status: Patch Available (was: In Progress)
> 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
>
> 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
[jira] [Updated] (CASSANDRA-15288) Read sidecar.yaml from sidecar.config System Property instead of classpath
[
https://issues.apache.org/jira/browse/CASSANDRA-15288?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Andy Tolbert updated CASSANDRA-15288:
-
Reviewers: Dinesh Joshi, Vinay Chella, Andy Tolbert (was: Andy Tolbert,
Dinesh Joshi, Vinay Chella)
Dinesh Joshi, Vinay Chella, Andy Tolbert (was: Dinesh Joshi,
Vinay Chella)
Status: Review In Progress (was: Patch Available)
> 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
>
> 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
[jira] [Updated] (CASSANDRA-15288) Read sidecar.yaml from sidecar.config System Property instead of classpath
[
https://issues.apache.org/jira/browse/CASSANDRA-15288?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Andy Tolbert updated CASSANDRA-15288:
-
Change Category: Semantic
Complexity: Low Hanging Fruit
Status: Open (was: Triage Needed)
> 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
>
> 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
[jira] [Updated] (CASSANDRA-15288) Read sidecar.yaml from sidecar.config System Property instead of classpath
[
https://issues.apache.org/jira/browse/CASSANDRA-15288?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Andy Tolbert updated CASSANDRA-15288:
-
Reviewers: Dinesh Joshi, Vinay Chella
> 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
>
> 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
[jira] [Updated] (CASSANDRA-15288) Read sidecar.yaml from sidecar.config System Property instead of classpath
[
https://issues.apache.org/jira/browse/CASSANDRA-15288?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Andy Tolbert updated CASSANDRA-15288:
-
Priority: Low (was: Normal)
> 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
>
> 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
[jira] [Updated] (CASSANDRA-15288) Read sidecar.yaml from sidecar.config System Property instead of classpath
[
https://issues.apache.org/jira/browse/CASSANDRA-15288?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Andy Tolbert updated CASSANDRA-15288:
-
Labels: pull-request-available (was: )
> 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: Normal
> Labels: pull-request-available
>
> 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
