Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Default REAPER_HTTP_MANAGEMENT_ENABLE environment variable #1428

Merged
merged 1 commit into from
Oct 17, 2023

Conversation

Miles-Garnsey
Copy link
Contributor

Add default REAPER_HTTP_MANAGEMENT_ENABLE="false" value to docker environment to ensure proper startup when not explicitly set.

At present, the docker container fails to start because this value is not defined anywhere and Dropwizards throws:

docker run australia-southeast1-docker.pkg.dev/k8ssandra/miles/cassandra-reaper:http-management3
WARN  [2023-10-17 00:46:54,064] com.datastax.driver.core.utils.UUIDs: PID returned through native call was 0, JNR versions incompatible?  Falling back to JMX.
io.dropwizard.configuration.UndefinedEnvironmentVariableException: Cannot resolve variable 'REAPER_HTTP_MANAGEMENT_ENABLE' (enableSubstitutionInVariables=false).
        at io.dropwizard.configuration.EnvironmentVariableSubstitutor.substitute(EnvironmentVariableSubstitutor.java:37)
        at org.apache.commons.text.StringSubstitutor.replace(StringSubstitutor.java:893)
        at io.dropwizard.configuration.SubstitutingSourceProvider.open(SubstitutingSourceProvider.java:41)
        at io.dropwizard.configuration.BaseConfigurationFactory.build(BaseConfigurationFactory.java:81)
        at io.dropwizard.cli.ConfiguredCommand.parseConfiguration(ConfiguredCommand.java:137)
        at io.dropwizard.cli.ConfiguredCommand.run(ConfiguredCommand.java:85)
        at io.dropwizard.cli.Cli.run(Cli.java:78)
        at io.dropwizard.Application.run(Application.java:94)
        at io.cassandrareaper.ReaperApplication.main(ReaperApplication.java:99)

@github-actions
Copy link

No linked issues found. Please add the corresponding issues in the pull request description.
Use GitHub automation to close the issue when a PR is merged

…ironment to ensure proper startup when not explicitly set.
@Miles-Garnsey Miles-Garnsey merged commit 7162ea3 into master Oct 17, 2023
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants