Skip to content

Loading…

Hi #8

Merged
merged 1 commit into from

2 participants

@srhea

I just found your library and tried to use ValetExample. It worked fine the first time I posted a resource, but it failed the second time on the same resource. First the lookup failed to find the existing entry, then the delete failed to clear it out. This small patch fixes the example.

Sean

@uriahcarpenter uriahcarpenter merged commit 09f9e3b into Widen:master
@uriahcarpenter
Widen Enterprises, Inc. member

Thanks for the patch!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Dec 14, 2011
  1. @srhea
Showing with 2 additions and 2 deletions.
  1. +2 −2 src/main/java/com/widen/examples/ValetExample.java
View
4 src/main/java/com/widen/examples/ValetExample.java
@@ -57,7 +57,7 @@ private void run()
//Simple query service to search for existing resources within the zone
NameQueryService queryService = new NameQueryByRoute53APIService(driver, zone);
- NameQueryService.LookupRecord lookup = queryService.lookup(resource, RecordType.A);
+ NameQueryService.LookupRecord lookup = queryService.lookup(resource + "." + domain, RecordType.A);
//Holds the commands to run within the update transaction
List<ZoneUpdateAction> actions = new ArrayList<ZoneUpdateAction>();
@@ -66,7 +66,7 @@ private void run()
{
//if the resource exists it must be deleted within the update transaction
- ZoneUpdateAction delete = new ZoneUpdateAction.Builder().withData(resource, RecordType.A, lookup.values).withTtl(lookup.ttl).buildDeleteAction();
+ ZoneUpdateAction delete = new ZoneUpdateAction.Builder().withData(resource + "." + domain, RecordType.A, lookup.values).withTtl(lookup.ttl).buildDeleteAction();
//ZoneUpdateAction.deleteAction(resource, RecordType.A, 600, lookup.getFirstValue());
actions.add(delete);
}
Something went wrong with that request. Please try again.