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 a better message on DS create/update/delete to indicate if a CDN snapshot and/or queue updates is required #4351

Open
3 of 16 tasks
mitchell852 opened this issue Jan 29, 2020 · 1 comment
Labels
improvement The functionality exists but it could be improved in some way. medium difficulty the estimated level of effort to resolve this issue is medium medium impact impacts a significant portion of a CDN, or has the potential to do so Traffic Ops related to Traffic Ops Traffic Portal v1 related to Traffic Portal version 1

Comments

@mitchell852
Copy link
Member

mitchell852 commented Jan 29, 2020

I'm submitting a ...

  • bug report
  • new feature / enhancement request
  • improvement request (usability, performance, tech debt, etc.)
  • other

Traffic Control components affected ...

  • CDN in a Box
  • Documentation
  • Grove
  • Traffic Control Client
  • Traffic Monitor
  • Traffic Ops
  • Traffic Ops ORT
  • Traffic Portal
  • Traffic Router
  • Traffic Stats
  • Traffic Vault
  • unknown

Current behavior:

Currently when you successfully create/edit/delete a delivery service you get a very simple message like "DS create/updated/deleted successfully". However, creating, updating and deleting a delivery service requires more than simply persisting the change to the database. In some cases it requires a cdn snapshot and/or an ATS config file update via a queue updates of all servers in the cdn for the change to take affect.

Expected / new behavior:

IMO it would be very helpful if the success message was something like:

  • DS updated successfully. A CDN snapshot is required for the changes to take affect.

  • DS updated successfully. Queue updates on all servers in the cdn for the changes to take affect.

  • DS updated successfully. Queue updates on all servers in the cdn and perform a CDN snapshot for the changes to take affect.

  • DS created successfully. Queue updates on all servers in the cdn then perform a CDN snapshot for the changes to take affect.

  • DS deleted successfully. Perform a CDN snapshot then queue updates on all servers in the cdn for the changes to take affect.

Of course in the case of an update this would require inspecting which properties changed to determine the appropriate message.

In fact, this more informative message could apply to any delivery service related change that requires a queue updates and/or a cdn snapshot such as:

  • updating ds regexes
  • updating ds static dns entries
  • updating ds ssl keys
  • updating ds url sig keys
  • updating ds uri signing keys
  • updating ds/server assignments
  • updating ds origins

Minimal reproduction of the problem with instructions:

create/update or delete a DS via the TO API or TP and observe the very non-informative message that is returned from the API and presented in TP.

@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 Jan 29, 2020
@mitchell852 mitchell852 added high impact impacts the basic function, deployment, or operation of a CDN medium difficulty the estimated level of effort to resolve this issue is medium labels Nov 30, 2021
@mitchell852
Copy link
Member Author

it's very unclear what the user is supposed to do after making changes to a ds. should i queue updates? snapshot? both? i'm going to mark this as high impact as i believe it to be "high value" from a user perspective but i know that's very debatable.

@ocket8888 ocket8888 added medium impact impacts a significant portion of a CDN, or has the potential to do so and removed high impact impacts the basic function, deployment, or operation of a CDN labels Jul 19, 2022
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. medium difficulty the estimated level of effort to resolve this issue is medium medium impact impacts a significant portion of a CDN, or has the potential to do so Traffic Ops related to Traffic Ops Traffic Portal v1 related to Traffic Portal version 1
Projects
None yet
Development

No branches or pull requests

2 participants