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

Rewrite /cachegroup_fallbacks to Go #3848

Closed
4 tasks done
ocket8888 opened this issue Aug 7, 2019 · 5 comments
Closed
4 tasks done

Rewrite /cachegroup_fallbacks to Go #3848

ocket8888 opened this issue Aug 7, 2019 · 5 comments
Labels
tech debt rework due to choosing easy/limited solution Traffic Ops related to Traffic Ops
Milestone

Comments

@ocket8888
Copy link
Contributor

ocket8888 commented Aug 7, 2019

I'm submitting an ...

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

Traffic Control components affected ...

  • Documentation (possibly)
  • Traffic Ops

Current behavior:

/cachegroup_fallbacks is handled by Perl

Expected / new behavior:

/cachegroup_fallbacks should be handled by Go.

Minimum API Version

1.1

Request Methods

  • GET
  • PUT
  • POST
  • DELETE

Minimal reproduction of the problem with instructions:

N/A

@ocket8888 ocket8888 added Traffic Ops related to Traffic Ops tech debt rework due to choosing easy/limited solution labels Aug 7, 2019
@ocket8888 ocket8888 added this to the Go Rewrite milestone Aug 12, 2019
@mitchell852
Copy link
Member

mitchell852 commented Sep 11, 2019

I think this should be deprecated (not rewritten to perl and a deprecation message added to the perl response). This can all be handled using the normal cache group CRUD apis.

@ocket8888
Copy link
Contributor Author

the decision to either deprecate and/or rewrite the endpoint can't be made here, but once the decision is made this issue can be closed (if appropriate)

@rawlinp
Copy link
Contributor

rawlinp commented Sep 13, 2019

+1, definitely shouldn't be rewritten. Go cachegroups API supersedes this.

@mhoppa
Copy link
Contributor

mhoppa commented Jan 29, 2020

#3766 accomplished this @ocket8888 @mitchell852 @rawlinp can you close this?

@ocket8888
Copy link
Contributor Author

Closed by #3766 - strangely, because that PR was merged before this issue was opened.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tech debt rework due to choosing easy/limited solution Traffic Ops related to Traffic Ops
Projects
None yet
Development

No branches or pull requests

4 participants