Merge lp:~jtv/gwacl/bug-1213059 into lp:gwacl
Proposed by
Jeroen T. Vermeulen
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Jeroen T. Vermeulen | ||||
Approved revision: | 224 | ||||
Merged at revision: | 220 | ||||
Proposed branch: | lp:~jtv/gwacl/bug-1213059 | ||||
Merge into: | lp:gwacl | ||||
Diff against target: |
65 lines (+14/-6) 1 file modified
example/storage/run.go (+14/-6) |
||||
To merge this branch: | bzr merge lp:~jtv/gwacl/bug-1213059 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Julian Edwards (community) | Approve | ||
Review via email: mp+180782@code.launchpad.net |
Commit message
Add, and use, -location option in storage example/tool.
Description of the change
The -location option is mandatory, because it selects the API endpoint that the program will make its requests to in the first place. (In practice there only seem to be two zones: mainland China and the rest of the world. A storage account in one won't necessarily exist in the other, although a storage account in one location will also be available for other locations in the same zone.)
I also corrected a silly mistake in my previous branch: the "list" operation listed "list" as one of its required arguments. That should have been "container".
Jeroen
To post a comment you must log in.
Perhaps only insist on location for things that need it?