nacx commented on code in PR #202:
URL: https://github.com/apache/jclouds/pull/202#discussion_r1547349446
##########
apis/ec2/src/main/java/org/jclouds/ec2/features/ElasticIPAddressApi.java:
##########
@@ -144,6 +144,28 @@ void releaseAddressInRegion(
@EndpointParam(parser = RegionToEndpointOrProviderIfNull.class)
@Nullable String region,
@FormParam("PublicIp") String publicIp);
+ /**
+ * Releases an elastic IP address associated with your identity.
+ *
+ * @param region
+ * Elastic IP addresses are tied to a Region and cannot be mapped
across Regions.
+ * @param allocationId
+ * The Allocation ID (e.g., eipalloc-0ca038968f2a2c986) of the IP
address that you are releasing from your identity.
+ *
+ * @see #allocateAddress
+ * @see #describeAddresses
+ * @see #associateAddress
+ * @see #disassociateAddress
+ * @see <a
href="http://docs.amazonwebservices.com/AWSEC2/latest/APIReference/index.html?ApiReference-query-ReleaseAddress.html";
+ */
+ @Named("ReleaseAddress")
+ @POST
+ @Path("/")
+ @FormParams(keys = ACTION, values = "ReleaseAddress")
+ void releaseAddressInRegionByAllocationId(
+ @EndpointParam(parser = RegionToEndpointOrProviderIfNull.class)
@Nullable String region,
+ @FormParam("AllocationId") String allocationId);
Review Comment:
Can you please add the corresponding unit test to the
`ElasticIPAddressApiTest`?
##########
providers/aws-ec2/src/main/java/org/jclouds/aws/ec2/features/RouteTableApi.java:
##########
@@ -276,4 +282,21 @@ boolean deleteRoute(
FluentIterable<RouteTable> describeRouteTables(
@EndpointParam(parser = RegionToEndpointOrProviderIfNull.class)
@Nullable String region,
@BinderParam(BindRouteTableIdsToIndexedFormParams.class) String...
routeTableIds);
+
+ /**
+ * Describes route tables.
+ * @param region The region to search for route tables.
+ * @param filter One or more filters utilized to search for RouteTable
instances
+ *
+ * @link <a
href="https://docs.aws.amazon.com/AWSEC2/latest/APIReference/API_DescribeRouteTables.html";>...</a>
+ */
+ @Named("DescribeRouteTables")
+ @POST
+ @FormParams(keys = ACTION, values = "DescribeRouteTables")
+ @XMLResponseParser(DescribeRouteTablesResponseHandler.class)
+ @Fallback(Fallbacks.EmptyFluentIterableOnNotFoundOr404.class)
+ FluentIterable<RouteTable> describeRouteTablesWithFilter(
+ @EndpointParam(parser = RegionToEndpointOrProviderIfNull.class)
@Nullable String region,
+ @BinderParam(BindFiltersToIndexedFormParams.class) Multimap<String,
String> filter);
Review Comment:
Can you please add the corresponding unit tests to the
`RouteTableApiMockTest`?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@jclouds.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
