search

bcgov / entity

ServiceBC Registry Team working on Legal Entities
Apache License 2.0
23 stars 59 forks source link

Report API - Performance stopgap - upgrade attempt #13599

Closed seeker25 closed 2 years ago

seeker25 commented 2 years ago

I've created this ticket to do some small scale performance upgrades to the Report API as a stop gap until @doug-lovett's new document service is working.

It might be possible to use parts of this in the new document service..

The reasoning behind this is, there are quite a few sentry errors regarding name requests that aren't being resolved on sentry

At least this is what I've gathered from reading the #registries-ops channel

https://sentry.io/organizations/registries/issues/3584795115/?alert_rule_id=799555&alert_timestamp=1663030605469&alert_type=email&environment=production&project=1533020&referrer=alert_email

This is a production defect.

seeker25 commented 2 years ago

Took me about 4h to get this going, get Report API working and get real payloads tested.. Also had to tweak a few things to get it to work correctly.. Ashna will have to look at the 4 scenarios it impacts..

Hopefully it works, even if it doesn't.. we should be able to remove 3/6 of the font files in the template files which should still give us a small performance gain.

seeker25 commented 2 years ago

Bumped this up a bit, we had 3 UX people look at it, Patrick Wei did some minor testing, Ashna did testing.. also working in an Openshift namespace I've never deployed to/used before.