rage / mooc.fi

https://mooc.fi
5 stars 6 forks source link

Feat: add ability to set connection limit and other connection string params #1210

Closed mipyykko closed 1 year ago

mipyykko commented 1 year ago

When creating the Prisma client, we use our own prebuilt connection string with parameters that can be set by envs.

Added connection limits to helm configuration.

codecov[bot] commented 1 year ago

Codecov Report

Merging #1210 (a3a58eb) into master (4d665be) will decrease coverage by 0.01%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1210      +/-   ##
==========================================
- Coverage   64.19%   64.18%   -0.01%     
==========================================
  Files         124      124              
  Lines        5315     5314       -1     
  Branches     1234     1233       -1     
==========================================
- Hits         3412     3411       -1     
  Misses       1774     1774              
  Partials      129      129              
Flag Coverage Δ
backend 64.18% <100.00%> (-0.01%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
backend/services/knex.ts 100.00% <ø> (ø)
backend/prisma/extensions.ts 56.94% <100.00%> (ø)
backend/prisma/index.ts 86.66% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more