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

2014-02-25 Thread Pedro Navarro Pérez
I'm not OK with this PR. The POST and DELETE operations are included in OS-KASDM extension: https://github.com/openstack/keystone/blob/master/keystone/contrib/admin_crud/core.py#L84. This is way it's been addressed in the OS-KSADM PR: https://github.com/jclouds/jclouds/pull/303 --- Reply to

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

2014-02-25 Thread Pedro Navarro Pérez
@demobox I think the implementation is incorrect, since these actions are just exposed by Keystone API trough an extension, so depending on the deployments It may be disabled. That's why It should not be implemented in the UserApi, that matches the default actions in the default API, but in a

[jclouds] JCLOUDS-453. Add OpenStack Keystone v2.0 OS-KSADM Admin Extension support (#303)

2014-02-24 Thread Pedro Navarro Pérez
JCLOUDS-453. Add OpenStack Keystone v2.0 OS-KSADM Admin Extension support for Keystone You can merge this Pull Request by running: git pull https://github.com/StackOps/jclouds OSKADM Or you can view, comment on it, or merge it online at: https://github.com/jclouds/jclouds/pull/303 --