alces-software / adminware

A sandbox CLI for running commands remotely across nodes
1 stars 0 forks source link

Remove reference to Batch ID in `run tool` #120

Closed WilliamMcCumstie closed 5 years ago

WilliamMcCumstie commented 5 years ago

Based on #119

This PR removes references to Batch ID from the run command. Instead the Job ID that is unique to each node/command combo is printed. This did require updating when the Job is saved to the db in order to retrieve an ID.

This is the major change required to remove references to Batch ID. There are a few remaining cases where it is mentioned within the old job command. These will be removed when those commands are ported.

Otherwise this PR fixes #115

DavidMarchant commented 5 years ago

Rebased on develop for commit https://github.com/alces-software/adminware/commit/c4aec8d12f0b7eea3167777499c5e2c480fb3c3b

DavidMarchant commented 5 years ago

LGTM