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

Update TR to use TO API v3.1 by default#6778

Merged
mattjackson220 merged 3 commits intoapache:masterfrom
rawlinp:update-tr-req-version
May 2, 2022
Merged

Update TR to use TO API v3.1 by default#6778
mattjackson220 merged 3 commits intoapache:masterfrom
rawlinp:update-tr-req-version

Conversation

@rawlinp
Copy link
Copy Markdown
Contributor

@rawlinp rawlinp commented Apr 25, 2022

Closes: #6653


Which Traffic Control components are affected by this PR?

  • Traffic Router

What is the best way to verify this PR?

Ensure the TR tests still pass and that TR successfully polls data from TO.

PR submission checklist

@rawlinp rawlinp added Traffic Router related to Traffic Router tech debt rework due to choosing easy/limited solution labels Apr 25, 2022
@rawlinp rawlinp added this to the 7.0.0 milestone Apr 25, 2022
@zrhoffman zrhoffman added the improvement The functionality exists but it could be improved in some way. label Apr 25, 2022
Copy link
Copy Markdown
Contributor

@mattjackson220 mattjackson220 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great to me! code looks good and tested locally in CiaB

@mattjackson220 mattjackson220 merged commit 5de5ff8 into apache:master May 2, 2022
@rawlinp rawlinp deleted the update-tr-req-version branch May 2, 2022 19:02
@asf-ci asf-ci mentioned this pull request Jun 1, 2022
4 tasks
zrhoffman pushed a commit to zrhoffman/trafficcontrol that referenced this pull request Oct 2, 2022
* Update TR to use TO API v3.1 by default

Closes: apache#6653

* Please weasel

* Make weasel check non-TO-API-specific
zrhoffman pushed a commit that referenced this pull request Nov 3, 2022
* Update TR to use TO API v3.1 by default

Closes: #6653

* Please weasel

* Make weasel check non-TO-API-specific

(cherry picked from commit 5de5ff8)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

improvement The functionality exists but it could be improved in some way. tech debt rework due to choosing easy/limited solution Traffic Router related to Traffic Router

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TR should default to using TO API 3.1

3 participants