-
Notifications
You must be signed in to change notification settings - Fork 56
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
Support PSGI reload #236
Support PSGI reload #236
Conversation
[test] |
IIUC the hot deploy option for the other languages, including perl with mod_perl, is a runtime variable. IOW |
@yselkowitz ok, good point. I will look to it. |
@michal-josef-spacek Why do you close #221 and create a newer one? |
[test-all] |
@phracek I rebased this PR to upstream and I created force push of PR. Force push closed PR, I don't know why. |
Heh I totally missed possibility of reopening. |
3466bfb
to
5d28908
Compare
[test] |
TODO: Move PSGI_RELOAD from assembly to run. |
[test-all] |
1 similar comment
[test-all] |
[test] |
a908274
to
c436cd6
Compare
[test] |
Container tests are green. [test-openshift] |
Only nitpicking. Nothing else. |
[test] |
[test-all] |
@yselkowitz Seems that this is functional as you need. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't follow, why you have moved PLACKUP_ARGS from assemble
to run
? Can you please describe it a bit more?
|
[test-all] |
[test-openshift] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks for the PR!
Again rebase of #221