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

Missing validate when user update cachegroup's type #5395

Closed
dpham692 opened this issue Dec 28, 2020 · 0 comments · Fixed by #5483
Closed

Missing validate when user update cachegroup's type #5395

dpham692 opened this issue Dec 28, 2020 · 0 comments · Fixed by #5483
Assignees
@zrhoffman
Labels
bug something isn't working as intended Traffic Ops related to Traffic Ops

Comments

@dpham692
Copy link
Contributor

I'm submitting a ...

  • bug report

Traffic Control components affected ...

  • Traffic Ops
  • Traffic Portal

Current behavior:

  1. Create a Topology that contain a EDGE_LOC cache group.
  2. Go to Topology -> Cache Groups and change that Cache Group type from EDGE_LOC to something else like MID_LOC for example.
  3. Notice that now the Topology will have the MID_LOC cache group as a leaf node.

Expected behavior:

Should have a validate that doesn't let user change the type of the cache group if it already assigned to the Topology.

Minimal reproduction of the problem with instructions:

Anything else:
@dpham692 dpham692 added the bug something isn't working as intended label Dec 28, 2020
@mitchell852 mitchell852 added the Traffic Ops related to Traffic Ops label Jan 4, 2021
@zrhoffman zrhoffman mentioned this issue Feb 1, 2021
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zrhoffman zrhoffman

Labels
bug something isn't working as intended Traffic Ops related to Traffic Ops
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Disallow updating the Type of a Cache Group if it is in use by a Topology zrhoffman/trafficcontrol
3 participants
@mitchell852 @zrhoffman @dpham692