GitHub user jadunham1 opened a pull request:
https://github.com/apache/libcloud/pull/702
DimensionData: Adding ability to remove a backup clients + small updates
Added a new method ex_remove_client_from_target
After comment from @tonybaloney noticed that the translation functions
should be privatized instead, can't find any use outside of the classes
currently, so I privatized them. Helps the code look cleaner.
Added both positive/negative test coverage around new backup call.
Added test coverage around privatized translation functions
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jadunham1/libcloud
feature/remove_backup_client
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/libcloud/pull/702.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #702
----
commit 641ba3fec2a5d4cc8aa4ed48f5b7f52533862cd6
Author: Jeffrey Dunham <jeffrey.a.dun...@gmail.com>
Date: 2016-02-11T18:48:45Z
Removal of client
commit 379f000f1af806dd4aa84c1703f02771b03bbe83
Author: Jeffrey Dunham <jeffrey.a.dun...@gmail.com>
Date: 2016-02-11T20:17:28Z
Moving location_to_location_id to private staticmethod
commit cb70a5f0a373202d0b0fffa1114cdf1ee5978bfe
Author: Jeffrey Dunham <jeffrey.a.dun...@gmail.com>
Date: 2016-02-12T06:24:13Z
DimensionData: Adding ability to remove backup clients/privatize
translation functions
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
