/names

Add "StorageName" function #38

Merged
merged 1 commit into from Jan 28, 2015

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@axw @anastasiamac @wallyworld
@axw
Member

axw commented Jan 28, 2015

This is similar to UnitService; it returns the
storage name from a storage instance ID.
@axw
Add "StorageName" function 
This is similar to UnitService; it returns the
storage name from a storage instance ID.
586e521

axw added a commit to axw/juju that referenced this pull request Jan 28, 2015

@axw
cmd/juju/storage: add "list", tweak "show" 
This commit introduces "juju storage list" to list
all of the storage instances in the environment.
The default output is a tabular format, similar to
"juju status --format tabular".

The existing "juju storage show" has been tweaked
to output as a map, rather than a slice. This makes
the output more like "juju status". We also translate
tags to IDs (unit/service name, storage ID), as tags
are not intended for users' eyes.

The StorageName value is now computed from the storage
ID. Finally, the "location", "available-size" and
"total-size" fields are omitted if unavailable. To
effect this, we make their types pointers, so they
are nullable.

This commit requires juju/names#38
b69fab4

@axw axw referenced this pull request in wallyworld/juju Jan 28, 2015

Merged

cmd/juju/storage: add "list", tweak "show" #12

@anastasiamac
Member

anastasiamac commented Jan 28, 2015

looks Gr8 :D
LGTM
@wallyworld
Owner

wallyworld commented Jan 28, 2015

LGTM

axw added a commit that referenced this pull request Jan 28, 2015

@axw
Merge pull request #38 from axw/storagename 
Add "StorageName" function
384f485

@axw axw merged commit 384f485 into juju:master Jan 28, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment