Browse files

Clarify CCI transaction output

* Rename the "provisioning" column to "active_transaction"
It's slightly more accurate, and it's just a preference of mine not to
introduce new terminology. If users interact with the API in other ways,
it'll be called a transaction, right? Or is the terminology changing
globally?

* Display "<None>" for an empty active transaction
The empty string means that transaction would always have to be the last
column, because you couldn't count on awk column offsets in selecting
individual fields. Also, it makes it hard to grep for CCI that _don't_
currently have an active transaction.
  • Loading branch information...
1 parent 04869f9 commit a165784c6172baba0312174a9a7769004fc77fb1 @quiteliderally committed Mar 13, 2013
Showing with 2 additions and 2 deletions.
  1. +2 −2 SoftLayer/CLI/modules/cci.py
View
4 SoftLayer/CLI/modules/cci.py
@@ -53,7 +53,7 @@ def execute(client, args):
t = Table([
'id', 'datacenter', 'host',
'cores', 'memory', 'primary_ip',
- 'backend_ip', 'provisioning',
+ 'backend_ip', 'active_transaction',
])
t.sortby = args.get('--sortby') or 'host'
@@ -77,7 +77,7 @@ def execute(client, args):
guest.get('primaryIpAddress', '-'),
guest.get('primaryBackendIpAddress', '-'),
guest.get('activeTransaction', {}).get(
- 'transactionStatus', {}).get('friendlyName', ''),
+ 'transactionStatus', {}).get('friendlyName', '<None>'),
])
return t

0 comments on commit a165784

Please sign in to comment.