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

Change ORT to Calculate parent_pending Itself #5117

Open
rob05c opened this issue Oct 7, 2020 · 2 comments
Open

Change ORT to Calculate parent_pending Itself #5117

rob05c opened this issue Oct 7, 2020 · 2 comments
Labels
cache-config Cache config generation improvement The functionality exists but it could be improved in some way.

Comments

@rob05c
Copy link
Member

rob05c commented Oct 7, 2020

Currently, ORT uses the parent_pending flag in a TO endpoint to determine whether any parents of the current server have updates pending.

We should change ORT to calculate this itself. It has all the data to do so.

The calculation itself is logic (mostly SQL) performed on the data. Logic that becomes more complex with the move from DeliveryServiceServers to Topologies.

This is just one more thing that should be done in ORT because it can, and because ORT is canaryable, where the TO Monolith is not. Not a huge issue, but a code/canary/rollback improvement.

I'm submitting a ...

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

Traffic Control components affected ...

  • Traffic Ops ORT

Current behavior:

TO performs logic to determine parents pending, and TO requests that.

New behavior:

ORT performs logic to determine parents pending.

Minimal reproduction of the problem with instructions:

N/A. Behavior does not change.

Anything else:
@rob05c rob05c added Traffic Ops ORT *DEPRECATED* related to the traffic_ops_ort.pl script improvement The functionality exists but it could be improved in some way. labels Oct 7, 2020
@rob05c
Copy link
Member Author

rob05c commented Oct 7, 2020

Related to #3687

@rob05c
Copy link
Member Author

rob05c commented Oct 7, 2020

Related to #3689

This was referenced Oct 7, 2020
Closed
Closed
@mitchell852 mitchell852 added high impact impacts the basic function, deployment, or operation of a CDN and removed high impact impacts the basic function, deployment, or operation of a CDN labels Jan 25, 2021
@mitchell852 mitchell852 added cache-config Cache config generation and removed Traffic Ops ORT *DEPRECATED* related to the traffic_ops_ort.pl script labels Jun 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
cache-config Cache config generation improvement The functionality exists but it could be improved in some way.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mitchell852 @rob05c