Skip to content

Commit 4787986

Browse files
authored
Add the sub-command "update-server" (#140)
* Added the sub-command "update-server" Co-authored-by: Stephen Fleming <[email protected]>
1 parent c2e8c7b commit 4787986

File tree

2 files changed

+10
-0
lines changed

2 files changed

+10
-0
lines changed

README.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,9 @@ requires `column`, `curl`, `mail`, and [jq](https://stedolan.github.io/jq/)
9595
where a completed state is one of: `Succeeded`, `Failed`, `Aborted`
9696
* *`-s STATUS`* If provided, will only remove jobs with the given `STATUS` from the local list.
9797

98+
#### Update cromwell server
99+
* `update-server`
100+
* Change the cromwell server that new jobs will be submitted to.
98101

99102
### Features:
100103
* Running `submit` will create a new folder in the `~/.cromshell/${CROMWELL_URL}/` directory named with the cromwell job id of the newly submitted job.

cromshell

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -204,6 +204,8 @@ function usage()
204204
echo -e " Will remove all jobs from the local list that are in a completed state,"
205205
echo -e " where a completed state is one of: $(echo ${TERMINAL_STATES} | tr ' ' ',' )"
206206
echo -e " -s STATUS If provided, will only remove jobs with the given STATUS from the local list."
207+
echo -e " Update cromwell server:"
208+
echo -e " update-server Change which cromwell server jobs will be submitted to."
207209
echo -e ""
208210
echo -e "Return values:"
209211
echo -e " 0 SUCCESS"
@@ -1840,6 +1842,11 @@ if ${ISINTERACTIVESHELL} ; then
18401842
# Get our sub-command:
18411843
SUB_COMMAND=${1}
18421844
shift
1845+
1846+
# Use the update-server sub-command to modify CROMWELL_NEEDS_SETUP here.
1847+
if [[ "${SUB_COMMAND}" == "update-server" ]] ; then
1848+
CROMWELL_NEEDS_SETUP=true
1849+
fi
18431850

18441851
# Check if we need to set this up.
18451852
# Note: because of how `notify` works, we can't require the setup for the notify action.

0 commit comments

Comments
 (0)