mirror of https://github.com/apache/jclouds.git
Merge Keystone optional admin API provider with optional extension provider
This commit is contained in:
parent
8ae33f2315
commit
2debb53e83
|
@ -43,7 +43,6 @@ import org.jclouds.openstack.keystone.v2_0.features.TokenApi;
|
||||||
import org.jclouds.openstack.keystone.v2_0.features.TokenAsyncApi;
|
import org.jclouds.openstack.keystone.v2_0.features.TokenAsyncApi;
|
||||||
import org.jclouds.openstack.keystone.v2_0.features.UserApi;
|
import org.jclouds.openstack.keystone.v2_0.features.UserApi;
|
||||||
import org.jclouds.openstack.keystone.v2_0.features.UserAsyncApi;
|
import org.jclouds.openstack.keystone.v2_0.features.UserAsyncApi;
|
||||||
import org.jclouds.openstack.keystone.v2_0.functions.PresentWhenAdminURLExistsForIdentityService;
|
|
||||||
import org.jclouds.openstack.keystone.v2_0.handlers.KeystoneErrorHandler;
|
import org.jclouds.openstack.keystone.v2_0.handlers.KeystoneErrorHandler;
|
||||||
import org.jclouds.openstack.keystone.v2_0.suppliers.RegionIdToAdminURIFromAccessForTypeAndVersion;
|
import org.jclouds.openstack.keystone.v2_0.suppliers.RegionIdToAdminURIFromAccessForTypeAndVersion;
|
||||||
import org.jclouds.openstack.keystone.v2_0.suppliers.RegionIdToAdminURISupplier;
|
import org.jclouds.openstack.keystone.v2_0.suppliers.RegionIdToAdminURISupplier;
|
||||||
|
@ -101,7 +100,6 @@ public class KeystoneRestClientModule<S extends KeystoneApi, A extends KeystoneA
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
protected void configure() {
|
protected void configure() {
|
||||||
bind(ImplicitOptionalConverter.class).to(PresentWhenAdminURLExistsForIdentityService.class);
|
|
||||||
install(new FactoryModuleBuilder().implement(RegionIdToAdminURISupplier.class,
|
install(new FactoryModuleBuilder().implement(RegionIdToAdminURISupplier.class,
|
||||||
RegionIdToAdminURIFromAccessForTypeAndVersion.class).build(RegionIdToAdminURISupplier.Factory.class));
|
RegionIdToAdminURIFromAccessForTypeAndVersion.class).build(RegionIdToAdminURISupplier.Factory.class));
|
||||||
}
|
}
|
||||||
|
|
|
@ -1,46 +0,0 @@
|
||||||
/**
|
|
||||||
* Licensed to jclouds, Inc. (jclouds) under one or more
|
|
||||||
* contributor license agreements. See the NOTICE file
|
|
||||||
* distributed with this work for additional information
|
|
||||||
* regarding copyright ownership. jclouds licenses this file
|
|
||||||
* to you under the Apache License, Version 2.0 (the
|
|
||||||
* "License"); you may not use this file except in compliance
|
|
||||||
* with the License. You may obtain a copy of the License at
|
|
||||||
*
|
|
||||||
* http://www.apache.org/licenses/LICENSE-2.0
|
|
||||||
*
|
|
||||||
* Unless required by applicable law or agreed to in writing,
|
|
||||||
* software distributed under the License is distributed on an
|
|
||||||
* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
|
|
||||||
* KIND, either express or implied. See the License for the
|
|
||||||
* specific language governing permissions and limitations
|
|
||||||
* under the License.
|
|
||||||
*/
|
|
||||||
package org.jclouds.openstack.keystone.v2_0.functions;
|
|
||||||
|
|
||||||
import javax.inject.Singleton;
|
|
||||||
|
|
||||||
import org.jclouds.internal.ClassMethodArgsAndReturnVal;
|
|
||||||
import org.jclouds.rest.functions.ImplicitOptionalConverter;
|
|
||||||
|
|
||||||
import com.google.common.base.Optional;
|
|
||||||
|
|
||||||
/**
|
|
||||||
*
|
|
||||||
* @author Adrian Cole
|
|
||||||
*
|
|
||||||
*/
|
|
||||||
@Singleton
|
|
||||||
public class PresentWhenAdminURLExistsForIdentityService implements ImplicitOptionalConverter {
|
|
||||||
|
|
||||||
@Override
|
|
||||||
public Optional<Object> apply(ClassMethodArgsAndReturnVal input) {
|
|
||||||
//TODO: log and return absent when the admin url for identity service isn't available
|
|
||||||
return Optional.of(input.getReturnVal());
|
|
||||||
}
|
|
||||||
|
|
||||||
public String toString() {
|
|
||||||
return "presentWhenAdminURLExistsForIdentityService()";
|
|
||||||
}
|
|
||||||
|
|
||||||
}
|
|
|
@ -19,7 +19,6 @@
|
||||||
package org.jclouds.openstack.v2_0.functions;
|
package org.jclouds.openstack.v2_0.functions;
|
||||||
|
|
||||||
import static com.google.common.base.Preconditions.checkNotNull;
|
import static com.google.common.base.Preconditions.checkNotNull;
|
||||||
import static com.google.common.base.Preconditions.checkState;
|
|
||||||
|
|
||||||
import java.net.URI;
|
import java.net.URI;
|
||||||
import java.util.Set;
|
import java.util.Set;
|
||||||
|
@ -74,8 +73,11 @@ public class PresentWhenExtensionAnnotationNamespaceEqualsAnyNamespaceInExtensio
|
||||||
} else {
|
} else {
|
||||||
throw new RuntimeException(String.format("expecting zero or one args %s", input));
|
throw new RuntimeException(String.format("expecting zero or one args %s", input));
|
||||||
}
|
}
|
||||||
}
|
|
||||||
return Optional.absent();
|
return Optional.absent();
|
||||||
|
} else {
|
||||||
|
// No extension annotation, should check whether to return absent
|
||||||
|
return Optional.of(input.getReturnVal());
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
|
|
Loading…
Reference in New Issue