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

./admin assets parameter in the config is a liability #706

Closed
szaydel opened this issue Jul 1, 2014 · 2 comments
Closed

./admin assets parameter in the config is a liability #706

szaydel opened this issue Jul 1, 2014 · 2 comments

Comments

@szaydel
Copy link

szaydel commented Jul 1, 2014

I have run into this myself a number of times, and based on what I am seeing from other issues logged this has come up before. This is even more problematic in constrained environments, like when you are running influxdb in a Docker container. If you are launching influxdb bin, from some arbitrary working dir, say / as is common with a Docker container, the admin interface will not work because the ./admin relative path is of course broken. It does help to resolve this by wrapping in a script, etc., but it seems like currently this is a liability and will be problematic for those using zones, containers, etc. who really want a minimalist startup code.

@toddboom
Copy link
Contributor

@szaydel We're going to statically compile the admin assets from now on (see issue #1032), so this will no longer be relevant.

@szaydel
Copy link
Author

szaydel commented Oct 20, 2014

Sounds good! Thanks a lot for a response.

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

No branches or pull requests

2 participants