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

Traffic Portal Invalidate Content page: warning about origin needing to support 304s #5698

Closed
traeak opened this issue Apr 1, 2021 · 2 comments · Fixed by #6332
Closed
Assignees
Labels
documentation related to documentation improvement The functionality exists but it could be improved in some way. Traffic Portal v1 related to Traffic Portal version 1

Comments

@traeak
Copy link
Contributor

traeak commented Apr 1, 2021

I'm submitting a ...

  • improvement request (usability, performance, tech debt, etc.)

Traffic Control components affected ...

  • Traffic Portal

Current behavior:

Recently we issued a large scale invalidation request against one of our delivery services. Unfortunately we didn't realize that the origin was not configured to handle If-Modified-Since or If-None-Match headers and always sent back 200s instead of 304s. This resulted in a storm of requests going to the origin, and a new mini storm starting every time an edge cache ATS was restarted.

Perhaps some message or some documentation should be added to ensure that whoever uses Invalidations realizes that they are designed to be used with origins that properly handle IMS requests.

New behavior:

Minimal reproduction of the problem with instructions:

Anything else:

@traeak traeak added the improvement The functionality exists but it could be improved in some way. label Apr 1, 2021
@mitchell852 mitchell852 added Traffic Portal v1 related to Traffic Portal version 1 documentation related to documentation labels Apr 1, 2021
@mitchell852
Copy link
Member

mitchell852 commented Apr 1, 2021

Seems easy enough to create a warning in both of these places in TP:

https://tp.domain.tld/#!/jobs/new
https://tp.domain.tld/#!/delivery-services/{{id}}/jobs?type={{ds_type}}

something along the lines of "Delivery service origin(s) must support IMS in order to safely invalidate content"

Also, that should be added to the documentation:

https://traffic-control-cdn.readthedocs.io/en/latest/admin/quick_howto/content_invalidation.html
https://traffic-control-cdn.readthedocs.io/en/latest/admin/traffic_portal/usingtrafficportal.html

and anywhere else in the documentation that discusses content invalidation.

@tcfdev
Copy link
Collaborator

tcfdev commented May 25, 2021

Throw this one on my docket!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation related to documentation improvement The functionality exists but it could be improved in some way. Traffic Portal v1 related to Traffic Portal version 1
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants