Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

Override all Delivery Service TTLs in a CDN's Snapshot with a given value#7143

Merged
srijeet0406 merged 11 commits intoapache:masterfrom
zrhoffman:ttl-override
Oct 18, 2022
Merged

Override all Delivery Service TTLs in a CDN's Snapshot with a given value#7143
srijeet0406 merged 11 commits intoapache:masterfrom
zrhoffman:ttl-override

Conversation

@zrhoffman
Copy link
Member

This PR resolves #2388 by adding the TTL Override field to each CDN, which lets you override all TTLs in all Delivery Services of a CDN's snapshot with a single value.


Which Traffic Control components are affected by this PR?

  • Traffic Ops
  • Traffic Portal

What is the best way to verify this PR?

  • Run the TO unit and API tests
  • Run the TP integration tests

PR submission checklist

@zrhoffman zrhoffman added new feature A new feature, capability or behavior Traffic Ops related to Traffic Ops Traffic Portal v1 related to Traffic Portal version 1 labels Oct 17, 2022
@zrhoffman zrhoffman requested a review from srijeet0406 October 17, 2022 22:23
@mitchell852
Copy link
Member

@zrhoffman - how do you set this value? a parameter? a field on the CDN object? (sorry, didn't read the code)

@zrhoffman
Copy link
Member Author

@zrhoffman - how do you set this value? a parameter? a field on the CDN object? (sorry, didn't read the code)

A field directly on the CDN object.

screenshot

It doesn't make sense to have as a global parameter, because global parameters are on the ALL CDN, but operators will commonly want to only upgrade Traffic Routers on a single CDN at a time, and TTLs on a CDN they're not upgrading should not be affected.

@mitchell852
Copy link
Member

A field directly on the CDN object.

@zrhoffman - makes sense. looks good but how do you clear or unset the override value? set it to zero?

@zrhoffman
Copy link
Member Author

zrhoffman commented Oct 18, 2022

@zrhoffman - makes sense. looks good but how do you clear or unset the override value? set it to zero?

By submitting an empty field. Neither TO nor TP accept 0 for the TTL Override field.

@srijeet0406 srijeet0406 merged commit 3aaf465 into apache:master Oct 18, 2022
@zrhoffman zrhoffman deleted the ttl-override branch November 4, 2022 06:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

new feature A new feature, capability or behavior Traffic Ops related to Traffic Ops Traffic Portal v1 related to Traffic Portal version 1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Global DNS TTL adjustment

3 participants