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
Block server servercapability delete if associated ds requires it #4041
Block server servercapability delete if associated ds requires it #4041
Conversation
Refer to this link for build results (access rights to CI server needed): |
traffic_ops/traffic_ops_golang/server/servers_server_capability.go
Outdated
Show resolved
Hide resolved
Refer to this link for build results (access rights to CI server needed): |
Refer to this link for build results (access rights to CI server needed): |
what about the opposite scenario? blocking ds required capability addition IF assigned servers don't have that capability. i suppose that doesn't belong in this PR but just thought i'd bring it up. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Manual requests worked, tests pass, docs changes look fine.
What does this PR (Pull Request) do?
This PR adds in extra validation in disassociating a server capability from a server. If that server is assigned to a DS that requires that server capability we now block the DELETE.
Which Traffic Control components are affected by this PR?
What is the best way to verify this PR?
Besides running API tests it can be manually tested by:
If this is a bug fix, what versions of Traffic Control are affected?
The following criteria are ALL met by this PR
Additional Information