-
Notifications
You must be signed in to change notification settings - Fork 7
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
make TOAD more docker environment for real-use case. #58
Comments
identify user data (jjb files or some) and store it in locally |
@zshi-redhat it seems not to related. Could you please file it in new issue ticket? |
For user data, especially JJB files, we need to take care carefully because ansible-playbook can overwrite it in next provision. |
#83 adds mapping container data to host directory. |
Honestly I think there is a possible need that we want to start breaking out configuration into locally deployed directories, and then mounting those into the image, rather than writing directly to the image. Of course we then need to determine if we're going to continue to support the OpenStack deployment method as well, or say that containers are the only way we're going to allow deployment of TOAD. If we continue to support the OpenStack method, then I think we need to keep "doing it wrong" on the container side of things and configuring the image. If we do that, then I think the container image will really just be a data image that still needs to be "configured" via Ansible (same as we do now), but we'll be able to skip downloading anything from the container image, as all binaries will already be installed. Thoughts? |
Just wanted to say that I like this approach. Unsure about the OpenStack method, definitely defer to others use-case on that one. |
Also thought I'd mention we might be able to use More info @ https://docs.docker.com/engine/reference/commandline/cp/ |
For now, TOAD's docker is just for testing and not matured as docker container environment.
We need to work on following items:
The text was updated successfully, but these errors were encountered: