mirror of
https://github.com/meteor/meteor.git
synced 2026-05-02 03:01:46 -04:00
096df9d62d
This commit moves parent pid process from the webapp package to the boot script. This means that daemonized apps without webapp will also exit when the runner exits, if run from the runner. (For example, several self-tests such as 'autoupdate' no longer leak node processes.) This is controlled via the $METEOR_PARENT_PID environment variable instead of from command line arguments, in order to make fewer assumptions about how Meteor apps process arguments. This also drops the old --keepalive support (which already has stopped being used by the dev mode runner or any MDG deployment platforms). Neither --parent-pid nor --keepalive were documented beforehand, and --keepalive was already deprecated before 1.0. These flags used to also incidentally trigger printing the LISTENING line; this is now controlled by $METEOR_PRINT_ON_LISTEN. Fixes #3315.
393 lines
10 KiB
JavaScript
393 lines
10 KiB
JavaScript
var selftest = require('../selftest.js');
|
|
var Sandbox = selftest.Sandbox;
|
|
var utils = require('../utils.js');
|
|
var net = require('net');
|
|
var Future = require('fibers/future');
|
|
var _ = require('underscore');
|
|
var files = require('../files.js');
|
|
|
|
var MONGO_LISTENING =
|
|
{ stdout: " [initandlisten] waiting for connections on port" };
|
|
|
|
var SIMPLE_WAREHOUSE = {
|
|
v1: { },
|
|
v2: { recommended: true },
|
|
v3: { }
|
|
};
|
|
|
|
selftest.define("run", function () {
|
|
var s = new Sandbox({ fakeMongo: true });
|
|
var run;
|
|
|
|
// Starting a run
|
|
s.createApp("myapp", "standard-app");
|
|
s.cd("myapp");
|
|
s.set("METEOR_TEST_TMP", files.mkdtemp());
|
|
run = s.run();
|
|
run.match("myapp");
|
|
run.match("proxy");
|
|
run.tellMongo(MONGO_LISTENING);
|
|
run.match("MongoDB");
|
|
run.match("your app");
|
|
run.waitSecs(5);
|
|
run.match("running at");
|
|
run.match("localhost");
|
|
|
|
// File change
|
|
s.write("empty.js", "");
|
|
run.waitSecs(2);
|
|
run.match("restarted");
|
|
s.write("empty.js", " ");
|
|
run.waitSecs(2);
|
|
run.match("restarted");
|
|
// XXX want app to generate output so that we can see restart counter reset
|
|
|
|
// Crashes
|
|
s.write("crash.js", "process.exit(42);");
|
|
run.waitSecs(5);
|
|
run.match("with code: 42");
|
|
run.waitSecs(5);
|
|
run.match("is crashing");
|
|
s.unlink("crash.js");
|
|
run.waitSecs(5);
|
|
run.match("Modified");
|
|
run.waitSecs(5);
|
|
run.match("restarted");
|
|
s.write("empty.js", "");
|
|
run.waitSecs(5);
|
|
// We used to see the restart counter reset but right now restart messages
|
|
// don't coalesce due to intermediate use of the progress bar.
|
|
run.match("restarted");
|
|
s.write("crash.js", "process.kill(process.pid, 'SIGKILL');");
|
|
run.waitSecs(5);
|
|
run.match("from signal: SIGKILL");
|
|
run.waitSecs(5);
|
|
run.match("is crashing");
|
|
|
|
// Bundle failure
|
|
s.unlink("crash.js");
|
|
s.write("junk.js", "]");
|
|
run.waitSecs(5);
|
|
run.match("Modified");
|
|
run.match("prevented startup");
|
|
run.match("Unexpected token");
|
|
run.match("file change");
|
|
|
|
// Back to working
|
|
s.unlink("junk.js");
|
|
run.waitSecs(5);
|
|
run.match("restarted");
|
|
|
|
// Crash just once, then restart successfully
|
|
s.write("crash.js",
|
|
"var fs = Npm.require('fs')\n" +
|
|
"var path = Npm.require('path')\n" +
|
|
"var crashmark = path.join(process.env.METEOR_TEST_TMP, 'crashed');\n" +
|
|
"try {\n" +
|
|
" fs.readFileSync(crashmark);\n" +
|
|
"} catch (e) {\n" +
|
|
" fs.writeFileSync(crashmark);\n" +
|
|
" process.exit(137);\n" +
|
|
"}\n");
|
|
run.waitSecs(5);
|
|
run.match("with code: 137");
|
|
run.waitSecs(5);
|
|
run.match("restarted");
|
|
run.stop();
|
|
|
|
// How about a bundle failure right at startup
|
|
s.unlink("crash.js");
|
|
s.write("junk.js", "]");
|
|
run = s.run();
|
|
run.tellMongo(MONGO_LISTENING);
|
|
run.waitSecs(5);
|
|
run.match("prevented startup");
|
|
run.match("Unexpected token");
|
|
run.match("file change");
|
|
s.unlink("junk.js");
|
|
run.waitSecs(5);
|
|
run.match("restarted");
|
|
run.stop();
|
|
|
|
// XXX --port, --production, --raw-logs, --settings, --program
|
|
});
|
|
|
|
selftest.define("run --once", function () {
|
|
var s = new Sandbox({ fakeMongo: true });
|
|
var run;
|
|
|
|
s.createApp("onceapp", "once");
|
|
s.cd("onceapp");
|
|
|
|
// Basic run --once
|
|
s.set("RUN_ONCE_OUTCOME", "exit");
|
|
run = s.run("--once");
|
|
run.tellMongo(MONGO_LISTENING);
|
|
run.waitSecs(5);
|
|
run.match("once test\n");
|
|
run.expectExit(123);
|
|
|
|
// run --once, exit on signal
|
|
s.set("RUN_ONCE_OUTCOME", "kill");
|
|
run = s.run("--once");
|
|
run.tellMongo(MONGO_LISTENING);
|
|
run.waitSecs(5);
|
|
run.match("once test\n");
|
|
run.matchErr("Killed (SIGKILL)\n");
|
|
run.expectExit(255);
|
|
|
|
// run --once, bundle failure
|
|
s.set("RUN_ONCE_OUTCOME", "exit");
|
|
s.write("junk.js", "]");
|
|
run = s.run("--once");
|
|
run.waitSecs(5);
|
|
run.matchErr("Build failed");
|
|
run.matchErr("Unexpected token");
|
|
run.expectExit(254);
|
|
s.unlink("junk.js");
|
|
|
|
// file changes don't make it restart
|
|
s.set("RUN_ONCE_OUTCOME", "hang");
|
|
run = s.run("--once");
|
|
run.tellMongo(MONGO_LISTENING);
|
|
run.waitSecs(5);
|
|
run.match("once test\n");
|
|
s.write('empty.js', 'null');
|
|
var originalRelease = s.read('.meteor/release');
|
|
s.write('.meteor/release', 'v1');
|
|
utils.sleepMs(2000); // sorry, hard to avoid
|
|
run.stop();
|
|
run.forbidAll("updated");
|
|
s.unlink('empty.js');
|
|
s.write('.meteor/release', originalRelease);
|
|
|
|
// Try it with a real Mongo. Make sure that it actually starts one.
|
|
s = new Sandbox;
|
|
s.createApp("onceapp", "once");
|
|
s.cd("onceapp");
|
|
s.set("RUN_ONCE_OUTCOME", "mongo");
|
|
run = s.run("--once");
|
|
run.waitSecs(30);
|
|
run.expectExit(86);
|
|
});
|
|
|
|
selftest.define("run errors", function () {
|
|
var s = new Sandbox;
|
|
s.createApp("myapp", "standard-app");
|
|
s.cd("myapp");
|
|
|
|
// Prevent mongod from starting up. (Note that "127.0.0.1" matches the
|
|
// interface that mongo uses.)
|
|
var proxyPort = utils.randomPort();
|
|
var mongoPort = proxyPort + 1;
|
|
var f = new Future;
|
|
var server = net.createServer().listen(mongoPort, "127.0.0.1", f.resolver());
|
|
f.wait();
|
|
|
|
var run = s.run("-p", proxyPort);
|
|
_.times(2, function () {
|
|
run.waitSecs(30);
|
|
run.match("Unexpected mongo exit code 48. Restarting.");
|
|
});
|
|
run.waitSecs(3);
|
|
run.match("Can't start Mongo server");
|
|
run.match("MongoDB exited because its port was closed");
|
|
run.match("running in the same project.\n");
|
|
run.expectEnd();
|
|
run.forbid("Started MongoDB");
|
|
run.expectExit(254);
|
|
|
|
f = new Future;
|
|
server.close(f.resolver());
|
|
f.wait();
|
|
|
|
// This time, prevent the proxy from starting. (This time, leaving out the
|
|
// interface name matches.)
|
|
f = new Future;
|
|
server = net.createServer().listen(proxyPort, f.resolver());
|
|
f.wait();
|
|
|
|
run = s.run("-p", proxyPort);
|
|
run.waitSecs(3);
|
|
run.match(/Can't listen on port.*another Meteor/);
|
|
run.expectExit(254);
|
|
|
|
f = new Future;
|
|
server.close(f.resolver());
|
|
f.wait();
|
|
});
|
|
|
|
selftest.define("update during run", ["checkout"], function () {
|
|
var s = new Sandbox({
|
|
warehouse: SIMPLE_WAREHOUSE,
|
|
fakeMongo: true
|
|
});
|
|
var run;
|
|
|
|
s.createApp("myapp", "packageless", { release: 'METEOR@v1' });
|
|
s.cd("myapp");
|
|
|
|
// If the app version changes, we exit with an error message.
|
|
run = s.run();
|
|
run.tellMongo(MONGO_LISTENING);
|
|
run.waitSecs(10);
|
|
run.match('localhost:3000');
|
|
s.write('.meteor/release', 'METEOR@v2');
|
|
run.matchErr('to Meteor v2 from Meteor v1');
|
|
run.expectExit(254);
|
|
|
|
// But not if the release was forced (case 1)
|
|
s.write('.meteor/release', 'METEOR@v1');
|
|
run = s.run("--release", "METEOR@v3");
|
|
run.tellMongo(MONGO_LISTENING);
|
|
run.waitSecs(2);
|
|
run.match('localhost:3000');
|
|
s.write('.meteor/release', 'METEOR@v2');
|
|
s.write('empty.js', '');
|
|
run.waitSecs(2);
|
|
run.match('restarted');
|
|
run.stop();
|
|
run.forbidAll("updated");
|
|
|
|
// But not if the release was forced (case 2)
|
|
s.write('.meteor/release', 'METEOR@v1');
|
|
run = s.run("--release", "METEOR@v1");
|
|
run.tellMongo(MONGO_LISTENING);
|
|
run.waitSecs(2);
|
|
run.match('localhost:3000');
|
|
s.write('.meteor/release', 'METEOR@v2');
|
|
s.write('empty.js', '');
|
|
run.waitSecs(2);
|
|
run.match('restarted');
|
|
run.stop();
|
|
run.forbidAll("updated");
|
|
|
|
// Nor do we do it if you're running from a checkout
|
|
s = new Sandbox({ fakeMongo: true });
|
|
s.createApp("myapp", "standard-app");
|
|
s.cd("myapp");
|
|
|
|
s.write('.meteor/release', 'METEOR@v1');
|
|
run = s.run();
|
|
run.tellMongo(MONGO_LISTENING);
|
|
run.waitSecs(2);
|
|
run.match('localhost:3000');
|
|
s.write('.meteor/release', 'METEOR@v2');
|
|
s.write('empty.js', '');
|
|
run.waitSecs(2);
|
|
run.match('restarted');
|
|
run.stop();
|
|
run.forbidAll("updated");
|
|
});
|
|
|
|
selftest.define("run with mongo crash", ["checkout"], function () {
|
|
var s = new Sandbox({ fakeMongo: true });
|
|
var run;
|
|
|
|
s.createApp("myapp", "standard-app");
|
|
s.cd("myapp");
|
|
|
|
// Kill mongod three times. See that it gives up and quits.
|
|
run = s.run();
|
|
run.tellMongo(MONGO_LISTENING);
|
|
run.waitSecs(2);
|
|
run.match('localhost:3000/\n');
|
|
run.tellMongo({exit: 23});
|
|
run.read('Unexpected mongo exit code 23. Restarting.\n');
|
|
run.tellMongo({exit: 46});
|
|
run.read('Unexpected mongo exit code 46. Restarting.\n');
|
|
run.tellMongo({exit: 47});
|
|
run.read('Unexpected mongo exit code 47. Restarting.\n');
|
|
run.read("Can't start Mongo server.\n");
|
|
run.read("MongoDB exited due to excess clock skew\n");
|
|
run.expectEnd();
|
|
run.expectExit(254);
|
|
|
|
// Now create a build failure. Make sure that killing mongod three times
|
|
// *also* successfully quits even if we're waiting on file change.
|
|
s.write('bad.js', ']');
|
|
run = s.run();
|
|
run.tellMongo(MONGO_LISTENING);
|
|
run.waitSecs(2);
|
|
run.match("prevented startup");
|
|
run.match("file change.\n");
|
|
run.tellMongo({exit: 23});
|
|
run.match('Unexpected mongo exit code 23. Restarting.\n');
|
|
run.tellMongo({exit: 46});
|
|
run.read('Unexpected mongo exit code 46. Restarting.\n');
|
|
run.tellMongo({exit: 47});
|
|
run.read('Unexpected mongo exit code 47. Restarting.\n');
|
|
run.read("Can't start Mongo server.\n");
|
|
run.read("MongoDB exited due to excess clock skew\n");
|
|
run.expectEnd();
|
|
run.expectExit(254);
|
|
});
|
|
|
|
// Test that when the parent runner process is SIGKILLed, the child
|
|
// process exits also.
|
|
selftest.define("run and SIGKILL parent process", function () {
|
|
var s = new Sandbox();
|
|
var run;
|
|
|
|
s.createApp("myapp", "app-prints-pid");
|
|
s.cd("myapp");
|
|
|
|
run = s.run();
|
|
run.waitSecs(30);
|
|
var match = run.match(/My pid is (\d+)/);
|
|
var childPid;
|
|
if (! match || ! match[1]) {
|
|
selftest.fail("No pid printed");
|
|
}
|
|
childPid = match[1];
|
|
|
|
process.kill(run.proc.pid, "SIGKILL");
|
|
// This sleep should be a little more time than the interval at which
|
|
// the child checks if the parent is still alive, in
|
|
// packages/webapp/webapp_server.js.
|
|
utils.sleepMs(3500);
|
|
|
|
// Send the child process a signal of 0. If there is no error, it
|
|
// means that the process is still running, which is not what we
|
|
// expect.
|
|
var caughtError;
|
|
try {
|
|
process.kill(childPid, 0);
|
|
} catch (err) {
|
|
caughtError = err;
|
|
}
|
|
|
|
if (! caughtError) {
|
|
selftest.fail("Child process " + childPid + " is still running");
|
|
}
|
|
|
|
run.stop();
|
|
|
|
// Test that passing a bad pid in $METEOR_PARENT_PID logs an error and exits
|
|
// immediately.
|
|
s.set("METEOR_BAD_PARENT_PID_FOR_TEST", "t");
|
|
run = s.run();
|
|
run.waitSecs(30);
|
|
run.match("must be a valid process ID");
|
|
run.match("Your application is crashing");
|
|
run.stop();
|
|
});
|
|
|
|
selftest.define("'meteor run --port' requires a port", function () {
|
|
var s = new Sandbox();
|
|
var run;
|
|
|
|
s.createApp("myapp", "app-prints-pid");
|
|
s.cd("myapp");
|
|
|
|
run = s.run("run", "--port", "example.com");
|
|
run.waitSecs(30);
|
|
run.matchErr("--port must include a port");
|
|
run.expectExit(1);
|
|
|
|
run = s.run("run", "--port", "http://example.com");
|
|
run.waitSecs(30);
|
|
run.matchErr("--port must include a port");
|
|
run.expectExit(1);
|
|
});
|