launch driver process from egg working.

[unwin]

bin/nosetests ion/agents/instrument/test/test_driver_process.py
Also will work with ion/services/sa/test/test_driver_egg.py once that code is brought in from Ian.

[swarbhanu]

Since, REPO_BASE = 'http://sddevrepo.oceanobservatories.org/releases/' at the beginning of the file, line 369 will result in two '/' before the egg_name. Should we not have the REPO_BASE = 'http://sddevrepo.oceanobservatories.org/releases' ? Thanks.

[unwin]

made requested change and pushed it to my fork.

On Jan 22, 2013, at 10:35 AM, swarbhanu notifications@github.com wrote:

In ion/agents/instrument/driver_process.py:

•    log.debug("_egg_path")
  

•    if not CACHE_DIR:
  

•        raise ServerError("CACHE_DIR is %s'%s'" % (type(CACHE_DIR), CACHE_DIR))
  

•    elif not egg_name:
  

•        raise ServerError("egg_name for path is %s'%s'" % (type(egg_name), egg_name))
  

•    return CACHE_DIR + "/" + egg_name
  
• def _egg_remotepath(self, egg_name):

•    log.debug("_egg_remotepath" + str(egg_name))
  

•    if not REPO_BASE:
  

•        raise ServerError("REPO_BASE is %s'%s'" % (type(REPO_BASE), REPO_BASE))
  

•    elif not egg_name:
  

•        raise ServerError("egg_name for remotepath is %s'%s'" % (type(egg_name), egg_name))
  

•    return REPO_BASE + "/" + egg_name
  

  Since, REPO_BASE = 'http://sddevrepo.oceanobservatories.org/releases/' at the beginning of the file, line 369 will result in two '/' before the egg_name. Should we not have the REPO_BASE = 'http://sddevrepo.oceanobservatories.org/releases' ? Thanks.

—
Reply to this email directly or view it on GitHub.