removed '/updateID' route.

routes/api.js

@@ -18,43 +18,7 @@ var tagPrefix = settings.tagPrefix || 'clwV';
 var workers = require('./worker_manager.js');
 
 
-
-// var runnerTimeout = function(runner, time){
-// 	time = time || 60000; // 1 minutes
-
-// 	if(runner.hasOwnProperty('timeout')){
-// 		clearTimeout(runner.timeout);
-// 	}
-
-// 	return runner.timeout = setTimeout(runnerFree, time, runner);
-// };
-
-// var runnerFree = function(runner){
-// 	lxc.stop(runner.name, runner.worker.ip);
-// 	runner.worker.usedrunners--;
-// 	if(runner.hasOwnProperty('timeout')){
-// 		clearTimeout(runner.timeout);
-// 	}
-// 	delete label2runner[runner.label];
-
-// 	console.log(`Runner freed ${runner.label}.`, runner.worker);
-// 	workers.startRunners({worker: runner.worker});
-// };
-
-// var getAvailrunner = function(runner){
-// 	for(let worker of workers){
-// 		if(worker.availrunners.length === 0) continue;
-// 		if(runner && runner.worker.index <= worker.index) break;
-// 		// if(runner) runnerFree(runner);
-// 		if(runner) runner.free();
-
-// 		return worker.getRunner();
-// 	}
-
-// 	if(runner) return runner;
-// };
-
-var run = function(req, res, runner, count){
+var attemptRun = function(req, res, runner, count){
 	count = count || 0;
 	console.log(`Runner starting attempt ${count}.`);
 
@@ -64,6 +28,7 @@ var run = function(req, res, runner, count){
 		return res.json({error: 'No runners, try again soon.'});
 	}
 
+	// TODO: Configurable
 	if(count > 2){
 		console.log(`Runner attempt failed, to many requests!`);
 		return res.status(400).json({error: 'Runner restarted to many times'});
@@ -81,7 +46,10 @@ var run = function(req, res, runner, count){
 
 	return request.post(httpOptions, function(error, response, body){
 		// console.log('runner response:', arguments)
-		if(error || response.statusCode !== 200) return run(req, res, workers.getAvailableRunner(), ++count);
+		if(error || response.statusCode !== 200) {
+			return attemptRun(req, res, workers.getAvailableRunner(), ++count);
+		}
+		
 		body = JSON.parse(body);
 
 		if(req.query.once){
@@ -91,7 +59,7 @@ var run = function(req, res, runner, count){
 			return runner.free();
 		}
 
-		label2runner[runner.label] = runner;
+		workers.setRunner(runner);
 		body['ip'] = runner.label;
 		body['rname'] = runner.name;
 		body['wname'] = runner.worker.name;
@@ -103,10 +71,11 @@ var run = function(req, res, runner, count){
 };
 
 console.log('========STARTING===========')
+// TODO: Make this a function
 setInterval(workers.checkBalance, 15000);
 workers.destroyByTag();
 
-
+// Why is this a GET?
 router.get('/stop/:name', function(req, res, next){
 	return lxc.stop(req.params.name, function(data){
 		console.log('stop', arguments);
@@ -117,79 +86,13 @@ router.get('/stop/:name', function(req, res, next){
 		}
 	});
 });
+
+// Why is this a GET?
 router.get('/destroyByTag', function(req, res, next) {
 	workers.destroyByTag();
 	res.send('?');
 });
 
-router.post('/updateID', function(req, res, next){
-	var newWorkers = {
-		workers: [],
-		image: req.query.image,
-		target: req.query.target || workers.length,
-		size: req.query.size || workers.settings.size,
-		version: workers.settings.version+1,
-		min: req.query.min || workers.settings,
-		minAvail: req.query.minAvail || workers.settings
-	};
-
-	doapi.tagCreate(tagPrefix+newWorkers.version);
-	workers.destroyByTag(tagPrefix+newWorkers.version);
-
-	for(var i=0; i<newWorkers.target; i++){
-
-		doapi.dropletToActive({
-			name: 'clw'+newWorkers.version+'-'+(Math.random()*100).toString().slice(-4),
-			newWorkers: newWorkers,
-			image: newWorkers.image,
-			size: newWorkers.size,
-			onCreate: function(data, args){
-				doapi.dropletSetTag(tagPrefix+args.newWorkers.version, data.droplet.id);
-			},
-			onActive: function(droplet, args){
-				workers.startRunners({
-					worker: workers.makeWorkerObj(droplet),
-					newWorkers: args.newWorkers,
-					onStart: function(runner, args){
-						args.newWorkers.workers.push(args.worker);
-						console.log('onStart', args.worker.name);
-						args.onStart = function(){};
-					},
-					onDone: function(args){
-						console.log('new workers:', args.newWorkers.workers.length);
-						doapi.domianAddRecord({
-							domain: "codeland.us",
-							type: "A",
-							name: "*."+args.worker.name+".workers",
-							data: args.worker.publicIP
-						});
-						
-						if(args.newWorkers.workers.length >= args.newWorkers.target){
-							console.log('upgrade complete!')
-							workers.settings.image = args.newWorkers.image;
-							workers.settings.size = args.newWorkers.size;
-							workers.settings.min = args.newWorkers.min;
-							workers.settings.minAvail = args.newWorkers.minAvail;
-
-							workers.forEach(function(worker){
-								worker.availrunners.forEach(function(runner){
-									lxc.stop(runner.name, runner.worker.ip);
-								});
-								worker.availrunners = [];
-							});
-
-							workers.add(args.newWorkers.workers);
-							workers.settingsSave();
-							workers.checkBalance();
-						}
-					}
-
-				});
-			}
-		});
-	}
-	res.json({status: "maybe?"});
-});
 
 router.get('/liststuff', function(req, res, next){
 	var obj = util.inspect(workers, {depth: 4});
@@ -197,14 +100,14 @@ router.get('/liststuff', function(req, res, next){
 		<h1>Workers</h1>
 		<pre>${obj}</pre>
 		<h1>label2runner</h1>
-		<pre>${util.inspect(label2runner)}</pre>
+		<pre>${util.inspect(workers.runnerMap)}</pre>
 		<h1>DO calls</h1>
 		${doapi.calls}
 	`);
 });
 
 router.get('/ping/:runner', function(req, res, next){
-	var runner = label2runner[req.params.runner];
+	var runner = workers.getRunner(req.params.runner);
 	// runnerTimeout(runner);
 	runner.setTimeout();
 	res.json({res:''});
@@ -213,7 +116,7 @@ router.get('/ping/:runner', function(req, res, next){
 router.post('/run/:runner?', function (req, res, next){
 	console.log(`Request runner route!`);
 	var runner = workers.getAvailrunner(workers.getRunner(req.params.runner));
-	return run(req, res, runner);
+	return attemptRun(req, res, runner);
 });
 
 module.exports = router;