Odis Combiner to k8s

[aaronmgdr]

Description

Migrate ODIS Combiner from firebase function to k8s. Add Prometheus metrics

Other changes

• Removes some unused config fields from Signer
• Includes changes from Transition ODIS to cloud function Gen 2 celo-monorepo#10511
• Removes usage of Performance library from the Combiner in favor of prometheus timers
• Adds prometheus metric for tracking tail signer latency

Tested

CI + e2e and load testing prior to release

Related issues

• Fixes Modify ODIS Combiner to run in container #39
• Fixes Add Prometheus metrics to ODIS Combiner code #38

Backwards compatibility

Backwards compatible with clients thanks to proxying via the old Combiner. Not backwards compatible with old combiner monitoring and alerting / infrastructure

Documentation

NA

[changeset-bot]

🦋 Changeset detected

Latest commit: 6c46f2f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 0 packages

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[socket-security]

New, updated, and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Packages	Version	New capabilities	Transitives	Size	Publisher
node-fetch	2.6.9	None	+0	162 kB	node-fetch-bot
prettier	2.8.7	None	+0	11.2 MB	prettier-bot
@opentelemetry/auto-instrumentations-node	0.38.0	environment	+252	34.4 MB	dyladan
jest-snapshot	29.7.0...29.5.0	None	+54/-41	11.1 MB	simenb
jest	29.7.0...29.5.0	None	+96/-89	13 MB	simenb
@types/express	4.17.18...4.17.17	None	+28/-36	340 kB	types
@types/node-fetch	2.6.6...2.6.4	None	+2/-0	97.6 kB	types
babel-jest	29.7.0...29.5.0	None	+53/-45	10.9 MB	simenb
@types/jest	29.5.5...29.5.4	None	+11/-8	365 kB	types
@babel/core	7.23.2...7.21.4	environment	+31/-25	10.1 MB	nicolo-ribaudo
@babel/runtime	7.23.2...7.21.0	None	+1/-1	285 kB	nicolo-ribaudo
@typescript-eslint/eslint-plugin	6.7.5...6.7.4	None	+13/-12	7.39 MB	jameshenry
@typescript-eslint/parser	6.7.5...6.7.4	None	+10/-9	4.52 MB	jameshenry
jest-circus	29.7.0...29.5.0	None	+73/-62	12 MB	simenb
@types/supertest	2.0.14...2.0.12	None	+3/-3	44.7 kB	types
eslint-plugin-prettier	5.0.1...5.0.0	None	+5/-5	3.11 MB	jounqin
eslint	8.51.0...8.50.0	None	+4/-4	3.08 MB	eslintbot
@opentelemetry/sdk-trace-web	1.17.1...1.15.2	None	+0/-4	225 kB	pichlermarc

🚮 Removed packages: @opentelemetry/[email protected], @opentelemetry/[email protected], @types/[email protected], @types/[email protected], [email protected], [email protected], [email protected]

[soloseng]

Left a few comments.

Few general questions:

• Is this PR deprecating odis-3-combiner-errors metric in favor of odis-3-sig-request-errors? I dont see that we are logging any odis-3-combiner-errors

• Will this PR also include the publishing of the common package? If so, the changeset should be added for the common package. Same goes for the signer package.