Re: [jclouds] Add keystone user add and delete methods. (#290)

2014-02-18 Thread Austin
Closed #290. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/290

Re: [jclouds] Add keystone user add and delete methods. (#290)

2014-02-14 Thread Austin
Thanks! I'll give that a try from here and see if I can get the live tests working. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/290#issuecomment-35120286

Re: [jclouds] Add keystone user add and delete methods. (#290)

2014-02-13 Thread Austin
Fixed javadoc style comments. I'm not particularly tied to the fallback behavior on user deletion; I'm open to other options. Once we figure out what behavior we want, I'll update the javadoc so that it better describes the behavior. --- Reply to this email directly or view it on GitHub:

Re: [jclouds] Add keystone user add and delete methods. (#290)

2014-02-13 Thread Austin
@@ -86,4 +88,37 @@ public void testUsersByName() { } } + + public void testAddDeleteUser() { I wrote this as one test because the test for the delete operation requires that the add operation succeed, and that we know the ID of the user that was created. We should be

[jclouds] Add keystone user add and delete methods. (#290)

2014-02-12 Thread Austin
Add support for the keystone user add and delete API. I have NOT yet tested this against a live environment. You can merge this Pull Request by running: git pull https://github.com/trainman419/jclouds master Or you can view, comment on it, or merge it online at:

Re: [jclouds] Add keystone user add and delete methods. (#290)

2014-02-12 Thread Austin
Addressed javadoc issue. I reformatted the existing comments as well to keep things consistent. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/290#issuecomment-34939055

Re: [jclouds] Add keystone user add and delete methods. (#290)

2014-02-12 Thread Austin
Added live tests. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/290#issuecomment-34941215