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.

Sign up for GitHub

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

Jump to bottom

Control default TTL #357

Open
codefromthecrypt opened this issue Mar 30, 2015 · 1 comment
Open

Control default TTL #357

codefromthecrypt opened this issue Mar 30, 2015 · 1 comment

Comments

@codefromthecrypt
Copy link
Contributor

ResourceRecordSet.ttl can be null to suggest inherit a default. This could use provider-specific functionality or instead lookup the SOAData.ttl and apply it.

This was attempted in #351, but not practical due to lack of support.

  • route53 - rrsets require an explicit ttl param, so denominator hard-codes
  • ultradns - default TTL per record type in the SOAP API, see: saveAccountPreferences. regardless, we currently hard-code
  • google cloud dns - rrsets require an explicit ttl param
  • dynect - default is SOA.ttl
  • clouddns - default is zone.ttl
  • designate - default is zone.ttl

Using the SOAData.ttl in lieu of provider support would make this feature expensive to implement, as that would imply we need to lookup or cache the SOA for any RRSet update that set the ttl to null. We could also make a denominator-specific default obtained from configuration.
codefromthecrypt pushed a commit that referenced this issue Mar 30, 2015
Reverts Zone.ttl
4ccab85 
Reverts default zone ttl as it is not supportable across all providers.

See #357
@codefromthecrypt codefromthecrypt mentioned this issue Mar 30, 2015
Merged
codefromthecrypt pushed a commit that referenced this issue Mar 31, 2015
Adds Zone.ttl
c7eeffd 
Zone.ttl is the SOA ttl. This is not always the default for new records.

See #357
@codefromthecrypt codefromthecrypt mentioned this issue Mar 31, 2015
Merged
codefromthecrypt pushed a commit that referenced this issue Mar 31, 2015
Adds Zone.ttl
0b5f323 
Zone.ttl is the SOA ttl. This is not always the default for new records.

See #357
codefromthecrypt pushed a commit that referenced this issue Mar 31, 2015
Adds Zone.ttl
b599e10 
Zone.ttl is the SOA ttl. This is not always the default for new records.

See #357
codefromthecrypt pushed a commit that referenced this issue Apr 1, 2015
Reverts Zone.ttl
5859f54 
Reverts default zone ttl as it is not supportable across all providers.

See #357
codefromthecrypt pushed a commit that referenced this issue Apr 1, 2015
Adds Zone.ttl
dae431d 
Zone.ttl is the SOA ttl. This is not always the default for new records.

See #357
@codefromthecrypt codefromthecrypt changed the title Add Zone default ttl Control default TTL Apr 1, 2015
@codefromthecrypt
Copy link
Contributor Author

Cleared out comments after reworking title and description

MisterK pushed a commit to MisterK/denominator that referenced this issue Apr 26, 2015
Reverts Zone.ttl
7ed83f7 
Reverts default zone ttl as it is not supportable across all providers.

See Netflix#357
MisterK pushed a commit to MisterK/denominator that referenced this issue Apr 26, 2015
Adds Zone.ttl
679caeb 
Zone.ttl is the SOA ttl. This is not always the default for new records.

See Netflix#357
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@codefromthecrypt