Commit 0ed19d7
committed
Decrease coupling between services
Previously, the different services had many unnesscary dependencies,
such as the database cannot run without cleanup, and that the it cannot
accept BEP uploads without starting the gRPC BES server, and Graphql
API. This change removes many unnesscary dependencies and decreases the
coupling between different services.
Some of the changes:
- Reorganizes bb_portal.proto
- Moves the database and database cleanup configuration to its own top
level configuration.
- Separate BES/BEP uploading and the Graphql API into two different
services.
- Replace instanceNameAuthorizer with individual authorizers for each
service and blob access.
- Make it possible to
- not serve the frontend
- not serve the graphql api
- run without database cleanup
- run without a http server
- run without the BES gRPC server
- run database cleanup without the BES1 parent ba8e3f4 commit 0ed19d7
33 files changed
Lines changed: 1142 additions & 724 deletions
File tree
- cmd/bb_portal
- config
- internal
- api
- grpc/bes
- http
- bepuploader
- graphql_api_service
- loghandler
- bep
- database
- buildeventrecorder
- dbauthservice
- dbcleanupservice
- pkg
- authmetadataextraction
- frontend
- grpcweb
- blobstoreservice
- buildqueuestateproxy
- schedulerservice
- proto/configuration/bb_portal
- test/integrationtest
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
| 10 | + | |
10 | 11 | | |
| 12 | + | |
11 | 13 | | |
12 | 14 | | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
13 | 18 | | |
14 | 19 | | |
15 | 20 | | |
| 21 | + | |
16 | 22 | | |
| 23 | + | |
17 | 24 | | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
18 | 29 | | |
| 30 | + | |
19 | 31 | | |
20 | 32 | | |
21 | 33 | | |
22 | 34 | | |
23 | 35 | | |
24 | 36 | | |
| 37 | + | |
25 | 38 | | |
26 | 39 | | |
27 | 40 | | |
| |||
0 commit comments