bcgov / cas-registration

A web app for Registration in OBPS under the Clean Growth branch
Apache License 2.0
1 stars 1 forks source link

Investigate and optimize all backend API routes #827

Open dleard opened 7 months ago

dleard commented 7 months ago

Blocked by #1061

The results of our load testing showed that we have some very ineffecient API routes in the /operations and /user-operator paths. We have created #825 and #826 to tackle the troublesome paths right away as they are blocking our path to prod. The rest of the endpoints appeared to perform well during load testing, but we should (post-release) give all of these API routes a look with the Django Silk endpoint profiler to see if there are any other minor optimizations that we can implement.

This card is a placeholder and should probably be broken up in a way similar to how the k6 tests were done (operator, user, others).

We should also look into making a check with the profiler part of our AC when submitting PR that adds or makes changes to an API endpoint in the backend. Or even better, look at how we might automate this check with some bounds or thresholds.

Note by Sepehr:

AC:

Dianadec commented 7 months ago

placeholder card that should probably be split up. cc @dleard

nanyangpro commented 6 months ago

Re: Refinement discussion - This one is more like an epic. Once we have individual tickets identified, this one will be gone.

dleard commented 6 months ago

This can be better defined once #952 and #1061 are complete. Having our API structure split up first will make it easier to write individual cards for each endpoint.

nanyangpro commented 5 months ago

This will be refined. @andrea-williams