Skip to content
This repository has been archived by the owner on Jan 10, 2019. It is now read-only.

Set LD_LIBRARY_PATH so Cassandra runs on Test/Continuous Clusters #125

Merged
merged 1 commit into from
Jul 30, 2015
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,8 @@ public final class CassandraCluster {
);

private static final Map<String, String> executorEnv = unmodifiableHashMap(
tuple2("JAVA_OPTS", "-Xms256m -Xmx256m")
tuple2("JAVA_OPTS", "-Xms256m -Xmx256m"),
tuple2("MESOSPHERE_LD_LIBRARY_PATH", "/opt/mesosphere/lib")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On non dcos clusters (most major linux distributions), LD library path is typically located under "/usr/local/lib". I'm not very familiar with this code, but I wanted to double check that the default value is correct here.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This value is appended to the path at executor start time. You're right that it's normally not there and will only be there in DCOS.

I want to test this out on DCOS and mesos to make sure that it works in both locations then I'll merge it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This has been tested on DCOS and mesos clusters now.

);

private static final TaskResources EXECUTOR_RESOURCES = taskResources(0.1, 384, 256);
Expand Down Expand Up @@ -606,6 +607,7 @@ private CassandraNodeExecutor buildCassandraNodeExecutor(@NotNull final String e
final CassandraConfigRole configRole = configuration.getDefaultConfigRole();

final List<String> command = newArrayList(
"export LD_LIBRARY_PATH=$MESOSPHERE_LD_LIBRARY_PATH:$LD_LIBRARY_PATH && ",
javaExec,
// "-agentlib:jdwp=transport=dt_socket,server=y,suspend=y,address=5005",
"-XX:+PrintCommandLineFlags",
Expand Down