cleanup per review by @fperez

* '127' in url -> url.startswith('127.')
* engines_started_okay -> engines_started_ok

IPython/parallel/apps/ipclusterapp.py

@@ -321,7 +321,7 b' class IPClusterEngines(BaseParallelApplication):'
         )
         return launcher
 
-    def engines_started_okay(self):
+    def engines_started_ok(self):
         self.log.info("Engines appear to have started successfully")
         self.early_shutdown = 0
     
@@ -330,7 +330,7 b' class IPClusterEngines(BaseParallelApplication):'
         self.engine_launcher.start(self.n)
         self.engine_launcher.on_stop(self.engines_stopped_early)
         if self.early_shutdown:
-            ioloop.DelayedCallback(self.engines_started_okay, self.early_shutdown*1000, self.loop).start()
+            ioloop.DelayedCallback(self.engines_started_ok, self.early_shutdown*1000, self.loop).start()
 
     def engines_stopped_early(self, r):
         if self.early_shutdown and not self._stopping:

IPython/parallel/engine/engine.py

@@ -214,7 +214,7 b' class EngineFactory(RegistrationFactory):'
 
     def abort(self):
         self.log.fatal("Registration timed out after %.1f seconds"%self.timeout)
-        if '127' in self.url:
+        if self.url.startswith('127.'):
             self.log.fatal("""
             If the controller and engines are not on the same machine,
             you will have to instruct the controller to listen on an external IP (in ipcontroller_config.py):