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

Provide the ability to change the name of a topology #5361

Closed
mitchell852 opened this issue Dec 7, 2020 · 3 comments · Fixed by #5495
Closed

Provide the ability to change the name of a topology #5361

mitchell852 opened this issue Dec 7, 2020 · 3 comments · Fixed by #5495
Assignees
Labels
improvement The functionality exists but it could be improved in some way. Traffic Ops related to Traffic Ops Traffic Portal v1 related to Traffic Portal version 1

Comments

@mitchell852
Copy link
Member

mitchell852 commented Dec 7, 2020

I'm submitting a ...

  • improvement request (usability, performance, tech debt, etc.)

Traffic Control components affected ...

  • Traffic Ops
  • Traffic Portal

Current behavior:

Currently you cannot change the name of a topology

New behavior:

Allow the user to change the name of a topology thru the API and cascade that change to all appropriate areas. I.e. ds.toplogy

Minimal reproduction of the problem with instructions:

In TP, navigate to an existing topology and notice that the name field is disabled.

Anything else:

Changing a topology name will probably dirty the snapshot for that CDN in 2 places: the topology section and the delivery service section (if the topology is assigned to a ds)

@mitchell852 mitchell852 added Traffic Ops related to Traffic Ops Traffic Portal v1 related to Traffic Portal version 1 improvement The functionality exists but it could be improved in some way. labels Dec 7, 2020
@srijeet0406
Copy link
Contributor

@mitchell852 I'm guessing this will involve a database migration, since the primary key for the topology table is its name.

@rawlinp
Copy link
Contributor

rawlinp commented Dec 8, 2020

I believe we can actually change the PK and it auto-updates the FK references.

@rimashah25
Copy link
Contributor

I can take this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement The functionality exists but it could be improved in some way. Traffic Ops related to Traffic Ops Traffic Portal v1 related to Traffic Portal version 1
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants