-
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
[enhancement] break out ARA results to volume and nginx proxy #131
Comments
Just an FYI that I really like it. I like it busted out into a volume, and I like nginx to serve that content. |
One potential question will be around how this affects our OpenStack deployments, and whether we're still going to support that approach. A lot of these Docker changes could start to affect how we deploy in other topologies. |
Yes, please try to make it compatible with the OpenStack approach, and also keep backwards functionality, to avoid redeploying existing environments. |
OK so things aren't really broken out the way I originally had it envisioned in my head. Of course I think that would be the more "correct" way of doing it, but we have a lot of rules we break already in the name of science^Wspeed :) Basically all I'm going to propose for now is actually moving trafik to point at NGINX running on the Jenkins Master as a proxy. This is the same way it works in the VM deployment, and requires the least amount of changes on our Docker deployment. |
Currently, logs are recovered from the jenkins slave and SCP'd back to the
/srv/static/
directory on the jenkins master. These files are then exposed via an NGINX proxy located on the master which performs redirects using a hostname and path.Moving things forward in the Docker realm, we should actually create a volume in
docker-compose.yml
to expose the/srv/static/
path into the volume. Then create an NGINX container thatvolumes_from
jenkins master.We can then also run ARA behind the NGINX proxy, and provide it access to the same volume, which ideally should permit loading the full web interface, loading the job data in to the sqlite proxy. That is likely a future enhancement and not part of the scope of this change.
The text was updated successfully, but these errors were encountered: