Netflix / denominator

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

Corrects CloudDNS implementation as it doesn't support duplicate zones #348

Closed codefromthecrypt closed 9 years ago

codefromthecrypt commented 9 years ago

Note: this impacts CLI usage, as formerly you'd pass a integer id to affect clouddns zones. Now, you pass the zone name.

Live tested, and had some throttling related failures, but nothing functional...

$ ./gradlew clean test install -Dclouddns.username=XXXX -Dclouddns.apiKey=XXXX -Dclouddns.zone=denominator.com
codefromthecrypt commented 9 years ago

@everett-toews fyi

codefromthecrypt commented 9 years ago

cc also @quinncomendant

cloudbees-pull-request-builder commented 9 years ago

NetflixOSS » denominator » denominator-pull-requests #93 SUCCESS This pull request looks good

codefromthecrypt commented 9 years ago

zone addressing reverted in https://github.com/Netflix/denominator/pull/349