-
Notifications
You must be signed in to change notification settings - Fork 23
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
Implement server & worker init & final scripts #248
Comments
Worker is implemented and tested. Server will get implemented next week. |
Worker looks to be mostly stable. There are some paradoxes in when to apply worker finalization in the workflow, but otherwise the implementation appears solid. |
Worker is being merged in. Sever remains to be done. |
From @asparke2 on August 10, 2017 1:12 Sweet! |
Defining server finalization as major, and server initialization as minor. |
Per Pivotal, it looks like this should be assigned to me. It would be helpful if I could get a concrete example of what we are trying to do. I discussed with @nllong running code on only the server docker container as part of the deployment process - presumably this is "initialization". Is "finalization" a final stage of setup or part of the destroy process for the server container? And are we only talking about Docker here? |
@nllong @rHorsey Notes from our call: Analysis finalization should be triggered from web_background but run on web_1, which has superior resources and memory management. Consider triggering this via a HTTP action for the analysis. Since this may take longer than a standard HTTP request and we don't want it to time out, we will still need the actual script(s) to run in the background on web_1, meaning that we need to use Resque on web_1. So:
Now we can all think what's wrong with this... |
I guess there is no reason for the http call since we are going to need to use background task queue regardless. So:
|
From @asparke2 on May 11, 2017 16:5
Necessary for ResStock and ComStock
Copied from original issue: NREL/OpenStudio#2608
The text was updated successfully, but these errors were encountered: