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

TO /api/x/steering output should be sorted #5461

Closed
rawlinp opened this issue Jan 22, 2021 · 0 comments · Fixed by #5471
Closed

TO /api/x/steering output should be sorted #5461

rawlinp opened this issue Jan 22, 2021 · 0 comments · Fixed by #5471
Assignees
Labels
bug something isn't working as intended Traffic Ops related to Traffic Ops Traffic Router related to Traffic Router

Comments

@rawlinp
Copy link
Contributor

rawlinp commented Jan 22, 2021

I'm submitting a ...

  • bug report

Traffic Control components affected ...

  • Traffic Ops
  • Traffic Router

Current behavior:

Currently, the output of /api/x/steering is unsorted. This causes the response to be different each time it's requested, which causes TR to reload the steering data even though the steering data has not actually changed. This leads to a lot of unnecessary/misleading log output in TR.

Expected behavior:

Responses from /api/x/steering should be sorted by xml_id so that they only change if the actual steering data has changed.

Minimal reproduction of the problem with instructions:

Request /api/3.0/steering multiple times. Observe that the ordering changes with each request.

@rawlinp rawlinp added bug something isn't working as intended Traffic Ops related to Traffic Ops Traffic Router related to Traffic Router labels Jan 22, 2021
@mattjackson220 mattjackson220 self-assigned this Jan 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something isn't working as intended Traffic Ops related to Traffic Ops Traffic Router related to Traffic Router
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants