Corrects Designate implementation as it doesn't support duplicate zones

Netflix / denominator

Portably control DNS clouds using java or bash

Apache License 2.0

580 stars 110 forks source link

Corrects Designate implementation as it doesn't support duplicate zones #347

Closed codefromthecrypt closed 9 years ago

codefromthecrypt commented 9 years ago

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

Live tested on hpcloud via...

$ ./gradlew clean test install  -Ddesignate.url=https://region-a.geo-1.identity.hpcloudsvc.com:35357/v2.0/ -Ddesignate.tenantId=XXX -Ddesignate.username=XXX -Ddesignate.password=XXX -Ddesignate.zone=denominator.io.

cloudbees-pull-request-builder commented 9 years ago

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

codefromthecrypt commented 9 years ago

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