Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Metadata Bug Fixes #81

Merged
merged 2 commits into from Apr 5, 2013

Conversation

Projects
None yet
2 participants
Contributor

sudorandom commented Apr 5, 2013

Resolves issue #80

$ sl my network public
:...............:...................:
:          Name : Value             :
:...............:...................:
: mac addresses : 06:8a:0d:8c:d8:80 :
:        router : fcr04.dal05       :
:         vlans : 1134              :
:      vlan ids : 236267            :
:...............:...................:
$ sl my datacenter
dal05
$ sl my user_data
No user metadata.
$ echo $?
2
  • Since the REST API gives errors (if you're using json) in json format, I made make_request raise an exception (if it can) based on the json body instead of just HTTP Error codes.
  • Fixed datacenter CLI doc block
  • Fixed datacenter_id CLI doc block
  • Improves user_metadata error when there is no metadata
  • Fixes typo/refactor issue that broke sl my network public/private

CrackerJackMack added a commit that referenced this pull request Apr 5, 2013

@CrackerJackMack CrackerJackMack merged commit dee204b into softlayer:master Apr 5, 2013

@CrackerJackMack CrackerJackMack referenced this pull request Apr 5, 2013

Closed

metadata is broken #80

@sudorandom sudorandom deleted the sudorandom:issue_80 branch Dec 17, 2013

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