Bug 861199

Could you please help doulble confirm whether hot-deployment has been removed for nodejs-0.6? I have checked on devenv_2295 and found the actual results as below:

Steps:
1. Create a nodejs app
2. Add hot deploy marker and do some changes in local git repo
3. git push the changes
4. Do some changes again, then git push again.

Actual results:
To step 3: The app is restarted and changes take effect.
To step4: The app is not restarted, but changes don't take effect except restarting the app manually.

I can confirm that support for hot deployment in the nodejs-0.6 cart has been removed.

There is support for all carts to some degree via the abstract cartridge, which is why it seems to "sort of" work with the marker enabled: however, that's not enough for nodejs in this case. The specific support to handle node was broken, removed due to lack of time to re-implement.

There's still problem for scalable nodejs hot deployment after testing on latest INT(devenv_2409). Here are the steps for your reference:

Steps:
1. Create a scale nodejs application
2. Create the hot deploy marker
$ touch .openshift/markers/hot_deploy
3.Change something in the local git repository
4.Deploy it
5.Change somthing in your local git repository
6.Deploy it again.
7. add 1 line to package.json
"scripts": { "start" : "supervisor <relative-from-repo-dir-path-to>/server.js" }, 
8.Make some change to the git repo and git push

Actual results:
step6: App is not restarted, but changes couldn't take effect.
step8: App is not restarted, but changes couldn't take effect.

Expected results:
App should not be started after step 6 and step8, changes should take effect.

(In reply to comment #2)
> I can confirm that support for hot deployment in the nodejs-0.6 cart has
> been removed.
> 
> There is support for all carts to some degree via the abstract cartridge,
> which is why it seems to "sort of" work with the marker enabled: however,
> that's not enough for nodejs in this case. The specific support to handle
> node was broken, removed due to lack of time to re-implement.

According to commit f75bb21e9456927c4b64a04c65e4f41dc873e8f6 in origin sever, hot deployment for nodejs should be enabled.

There's still problem for non-scale nodejs hot deployment after testing on Stage(devenv-stage_243). Changes changes couldn't take effect.

Test this bug on devenv_2466
1.for non-scale nodejs and no jenkins-client,changes have taken effect
1)create a non-scale nodejs app
2)touch hot_deploy and git push
3) change server.js
self.routes['/test'] = function(req, res) {
            res.send('test');
        };
4)git push
5)check changes

Result:
[sunwei@dhcp-8-229 ~]$ curl http://nodejstest-wsundevbug1114.dev.rhcloud.com/test
test

2.for non-scale nodejs and have jenkins-client,changes have taken effect

1)create a non-scale nodejs app
2)add jenkins-client-1.4 to app
3)touch hot_deploy and git push
4)change server.js
self.routes['/test'] = function(req, res) {
            res.send('test1');
        };
5)git push
6)check changes

Result:
[sunwei@dhcp-8-229 ~]$ curl http://nodejstest-wsundevbug1114.dev.rhcloud.com/test
test1

3.for scale nodejs-0.6 app and have not jenkins-client,changes have not taken effect
1)create a scale nodejs app
2)touch hot_deploy and git push
3)change server.js
 // Routes for /health, /asciimo and /
        self.routes['/health'] = function(req, res) {
            res.send('test');
        };
4)git push
5)check changes

Result:
 [sunwei@dhcp-8-229 ~]$ curl  http://nodejsscaletest-wsundevbug1114.dev.rhcloud.com/health
1

4.for scale nodejs-0.6 and have jenkins-client,and changes have not taken effect.
1)create a scale nodejs app
2)add jenkins-client to app
3)touch hot_deploy and git push
4)change server.js
// Routes for /health, /asciimo and /
        self.routes['/health'] = function(req, res) {
            res.send('testagain');
        };
5)git push
6)check changes

Result:
[sunwei@dhcp-8-229 ~]$ curl  http://nodejsscaletest-wsundevbug1114.dev.rhcloud.com/health
1

This bug seems already fixed.
Please change is status to ON_QA to verify, thanks!

Verified on devenv_2520

Steps:
1. Create scalable nodejs application
rhc app create nodejs1s nodejs-0.6 -s
2. Add hot_deploy marker and git push, hot deployment will take effect from the second time of git push
3. After first git push, modify something in app's local repo, open another terminal, ssh into app's gear, check pid of app's process(supervisor process) remain unchanged before and after git push.
Access app's url after git push to make sure changes take effect.

Result:
App is not restarted during git push, changes were deployed after git push.