Netflix / denominator

Portably control DNS clouds using java or bash
Apache License 2.0
580 stars 110 forks source link

cli add record does not include new record with new rdata when rrset has old data. #269

Closed hasika closed 9 years ago

hasika commented 9 years ago

denominator-cli ResourceRecordSetCommands class' add command filters out new data.

Line 261-

ImmutableList<Map<String, Object>> newRData = ImmutableList.copyOf(filter(rrset.records(), not(in(toAdd.records()))));

Fix -

ImmutableList<Map<String, Object>> newRData = ImmutableList.copyOf(filter(toAdd.records(), not(in(rrset.records()))));

codefromthecrypt commented 9 years ago

crazy latency, but I plan to spend a bit of time in denominator. ping back if you'd still like this addressed!