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

Add TO API v5#7063

Merged
srijeet0406 merged 14 commits intoapache:masterfrom
ocket8888:to/api-v5
Sep 13, 2022
Merged

Add TO API v5#7063
srijeet0406 merged 14 commits intoapache:masterfrom
ocket8888:to/api-v5

Conversation

@ocket8888
Copy link
Copy Markdown
Contributor

Adds version 5.0 of the Traffic Ops API.

Everything still uses v4 (which doesn't matter yet as there are no differences between v4 and v5 as of this PR)


Which Traffic Control components are affected by this PR?

  • Documentation
  • Traffic Control Clients (Python, TO Go client (adds new))
  • Traffic Ops
  • Automation (GitHub Actions)

What is the best way to verify this PR?

Make sure all of the new APIv5 TO Go client integration tests pass, make sure the docs build without errors or warnings and are complete and accurate.

PR submission checklist

  • This PR has tests
  • This PR has documentation
  • This PR has a CHANGELOG.md entry
  • This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY

@ocket8888 ocket8888 added new feature A new feature, capability or behavior Traffic Ops related to Traffic Ops documentation related to documentation tests related to tests and/or testing infrastructure medium impact impacts a significant portion of a CDN, or has the potential to do so TC Client (python) related to the Python implementation of a TC client automation related to automated testing/deployment/packaging etc. TO Client (Go) related to the Go implementation of a TC client labels Sep 7, 2022
@ocket8888 ocket8888 force-pushed the to/api-v5 branch 2 times, most recently from 7d6292d to dcf44b8 Compare September 7, 2022 21:40
Comment thread traffic_ops/v5-client/README.md Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Shouldn't this have the Unstable section that you removed from the v4-client/ README.md?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

No, TO APIv5 is not released, so it doesn't need that notice. APIv2 and APIv3 didn't have it, for example.

Comment thread docs/source/api/v4/deliveryserviceserver.rst Outdated
Comment thread docs/source/api/v4/servers.rst Outdated
Comment thread traffic_ops/testing/api/utils/utils.go Outdated
(cherry picked from commit a70d6282e68213383b8491ff0e8315450df79296)
(cherry picked from commit 523b13a0fa946571003f20e6a0342c4701a0017f)
(cherry picked from commit 8592b498ba21110f5e51021b7021569e0591c846)
Also bumps default from now-deprecated 3.0 to current 4.1
@srijeet0406 srijeet0406 merged commit da02502 into apache:master Sep 13, 2022
@ocket8888 ocket8888 deleted the to/api-v5 branch September 13, 2022 18:03
zrhoffman pushed a commit to zrhoffman/trafficcontrol that referenced this pull request Oct 2, 2022
* Add APIv5 routes

(cherry picked from commit a70d6282e68213383b8491ff0e8315450df79296)

* Add v5 API client

(cherry picked from commit 523b13a0fa946571003f20e6a0342c4701a0017f)

* Add APIv5 docs

(cherry picked from commit 8592b498ba21110f5e51021b7021569e0591c846)

* Remove instability notice from client that's no longer unstable

* Remove outdated instability notice

* Add APIv5 support to the client

Also bumps default from now-deprecated 3.0 to current 4.1

* Add APIv5 to tests

* Update CHANGELOG

* Fix rebase issues

* Fix incorrect typing

* fix broken reference

* Fix malformed table

* Fix duplicate label

* Fix incorrect user-agent
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

automation related to automated testing/deployment/packaging etc. documentation related to documentation medium impact impacts a significant portion of a CDN, or has the potential to do so new feature A new feature, capability or behavior TC Client (python) related to the Python implementation of a TC client tests related to tests and/or testing infrastructure TO Client (Go) related to the Go implementation of a TC client Traffic Ops related to Traffic Ops

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants