Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cli record add remove #48

Merged
merged 3 commits into from
Mar 6, 2013
Merged

Cli record add remove #48

merged 3 commits into from
Mar 6, 2013

Conversation

codefromthecrypt
Copy link
Contributor

for issue #11

This allows the cli to add and remove records. --ttl is optional on record add, and when it is specified, it will replace whatever the old ttl is on existing records.

There's no impact if you remove a record already gone, or add one that already exists, unless you specify an updated --ttl.

Examples (keeping in mind mock is not durable)

Adding multiple records:

$ denominator -p mock record -z denominator.io. add -n www1.myzone.com. -t A -d 1.1.1.1 -d 1.1.1.2
;; in zone denominator.io. adding to rrset www1.myzone.com. A values: [{address=1.1.1.1},{address=1.1.1.2}]
;; ok

Adding a record and replacing ttl:

$ denominator -p mock record -z denominator.io. add -n www1.myzone.com. -t A --ttl 3600 -d 1.1.1.1
;; in zone denominator.io. adding to rrset www1.myzone.com. A values: [{address=1.1.1.1}] applying ttl 3600
;; ok

Removing a record:

$ denominator -p mock record -z denominator.io. remove -n www1.myzone.com. -t A -d 1.1.1.1
;; in zone denominator.io. removing from rrset www1.myzone.com. A values: [{address=1.1.1.1}]
;; ok

@cloudbees-pull-request-builder

denominator-pull-requests #20 SUCCESS
This pull request looks good

codefromthecrypt pushed a commit that referenced this pull request Mar 6, 2013
@codefromthecrypt codefromthecrypt merged commit cd27221 into master Mar 6, 2013
@codefromthecrypt codefromthecrypt deleted the cli-record-crud branch March 6, 2013 00:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants