• Home
  • History
  • Annotate
Name Date Size #Lines LOC

..03-May-2022-

api/H09-Feb-2022-25,49120,691

build/H09-Feb-2022-609470

bus/H09-Feb-2022-398280

cmd/H09-Feb-2022-4,2263,428

components/H09-Feb-2022-3,7482,841

dashboards/H09-Feb-2022-2315

events/H09-Feb-2022-6552

expr/H09-Feb-2022-8,5017,361

extensions/H09-Feb-2022-3024

ifaces/gcsifaces/H09-Feb-2022-4524

infra/H09-Feb-2022-7,6356,000

login/H09-Feb-2022-3,8503,246

macaron/H03-May-2022-2,7261,956

middleware/H09-Feb-2022-3,0452,410

mocks/mock_gcsifaces/H09-Feb-2022-237162

models/H09-Feb-2022-3,8062,912

plugins/H09-Feb-2022-8,9837,363

registry/H09-Feb-2022-4521

schema/H09-Feb-2022-29,68529,180

server/H09-Feb-2022-734627

services/H09-Feb-2022-126,932106,491

setting/H09-Feb-2022-3,6952,934

tests/H09-Feb-2022-11,30910,422

tsdb/H09-Feb-2022-48,16140,546

util/H09-Feb-2022-1,3731,076

web/H09-Feb-2022-1814

README.mdH A D09-Feb-20227.8 KiB10059

ruleguard.rules.goH A D09-Feb-202217.8 KiB502326

README.md

1# Backend
2
3This directory contains the code for the Grafana backend. This document gives an overview of the directory structure, and ongoing refactorings.
4
5For more information on developing for the backend:
6
7- [Backend style guide](/contribute/style-guides/backend.md)
8- [Architecture](/contribute/architecture)
9
10## Central folders of Grafana's backend
11
12| folder | description |
13| ------- | ----------- |
14| /pkg/api | HTTP handlers and routing. Almost all handler funcs are global which is something we would like to improve in the future. Handlers should be associated with a struct that refers to all dependencies. |
15| /pkg/cmd | The binaries that we build: grafana-server and grafana-cli. |
16| /pkg/components | A mix of third-party packages and packages we have implemented ourselves. Includes our packages that have out-grown the util package and don't naturally belong somewhere else. |
17| /pkg/infra | Packages in infra should be packages that are used in multiple places in Grafana without knowing anything about the Grafana domain. |
18| /pkg/services | Packages in services are responsible for persisting domain objects and manage the relationship between domain objects. Services should communicate with each other using DI when possible. Most of Grafana's codebase still relies on global state for this. Any new features going forward should use DI. |
19| /pkg/tsdb | All backend implementations of the data sources in Grafana. Used by both Grafana's frontend and alerting. |
20| /pkg/util | Small helper functions that are used in multiple parts of the codebase. Many functions are placed directly in the util folders which is something we want to avoid. Its better to give the util function a more descriptive package name. Ex `errutil`. |
21
22## Central components of Grafana's backend
23
24| package | description |
25| ------- | ----------- |
26| /pkg/bus | The bus is described in more details under [Communication](/contribute/architecture/backend/communication.md) |
27| /pkg/models | This is where we keep our domain model. This package should not depend on any package outside standard library. It does contain some references within Grafana but that is something we should avoid going forward. |
28| /pkg/registry | Package for managing services. |
29| /pkg/services/alerting | Grafana's alerting services. The alerting engine runs in a separate goroutine and shouldn't depend on anything else within Grafana. |
30| /pkg/services/sqlstore | Currently where the database logic resides. |
31| /pkg/setting | Anything related to Grafana global configuration should be dealt with in this package. |
32
33## Dependency management
34
35Refer to [UPGRADING_DEPENDENCIES.md](https://github.com/grafana/grafana/blob/main/UPGRADING_DEPENDENCIES.md).
36
37## Ongoing refactoring
38
39These issues are not something we want to address all at once but something we will improve incrementally. Since Grafana is released at a regular schedule the preferred approach is to do this in batches. Not only is it easier to review, but it also reduces the risk of conflicts when cherry-picking fixes from main to release branches. Please try to submit changes that span multiple locations at the end of the release cycle. We prefer to wait until the end because we make fewer patch releases at the end of the release cycle, so there are fewer opportunities for complications.
40
41### Global state
42
43Global state makes testing and debugging software harder and it's something we want to avoid when possible. Unfortunately, there is quite a lot of global state in Grafana.
44
45We want to migrate away from this by using the `inject` package to wire up all dependencies either in `pkg/cmd/grafana-server/main.go` or self-registering using `registry.RegisterService` ex https://github.com/grafana/grafana/blob/main/pkg/services/cleanup/cleanup.go#L25.
46
47### Limit the use of the init() function
48
49Only use the init() function to register services/implementations.
50
51### Settings refactoring
52
53The plan is to move all settings to from package level vars in settings package to the [setting.Cfg](https://github.com/grafana/grafana/blob/df917663e6f358a076ed3daa9b199412e95c11f4/pkg/setting/setting.go#L210) struct. To access the settings, services and components can inject this setting.Cfg struct:
54
55[Cfg struct](https://github.com/grafana/grafana/blob/df917663e6f358a076ed3daa9b199412e95c11f4/pkg/setting/setting.go#L210)
56[Injection example](https://github.com/grafana/grafana/blob/df917663e6f358a076ed3daa9b199412e95c11f4/pkg/services/cleanup/cleanup.go#L20)
57
58### Reduce the use of GoConvey
59
60We want to migrate away from using GoConvey. Instead, we want to use stdlib testing, because it's the most common approach in the Go community and we think it will be easier for new contributors. Read more about how we want to write tests in the [style guide](/contribute/style-guides/backend.md).
61
62### Refactor SqlStore
63
64The `sqlstore` handlers all use a global xorm engine variable. Refactor them to use the `SqlStore` instance.
65
66### Avoid global HTTP handler functions
67
68Refactor HTTP handlers so that the handler methods are on the HttpServer instance or a more detailed handler struct. E.g (AuthHandler). This ensures they get access to HttpServer service dependencies (and Cfg object) and can avoid global state.
69
70### Date comparison
71
72Store newly introduced date columns in the database as epochs if they require date comparison. This permits a unified approach for comparing dates against all the supported databases instead of handling dates differently for each database. Also, by comparing epochs, we no longer need error pruning transformations to and from other time zones.
73
74### Avoid use of the simplejson package
75
76Use of the `simplejson` package (`pkg/components/simplejson`) in place of types (Go structs) results in code that is difficult to maintain. Instead, create types for objects and use the Go standard library's [`encoding/json`](https://golang.org/pkg/encoding/json/) package.
77
78### Provisionable*
79
80All new features that require state should be possible to configure using config files. For example:
81
82- [Data sources](https://github.com/grafana/grafana/tree/main/pkg/services/provisioning/datasources)
83- [Alert notifiers](https://github.com/grafana/grafana/tree/main/pkg/services/provisioning/notifiers)
84- [Dashboards](https://github.com/grafana/grafana/tree/main/pkg/services/provisioning/dashboards)
85
86Today its only possible to provision data sources and dashboards but this is something we want to support all over Grafana.
87
88### Use context.Context "everywhere"
89
90The package [context](https://golang.org/pkg/context/) should be used and propagated through all the layers of the code. For example the `context.Context` of an incoming API request should be propagated to any other layers being used such as the bus, service and database layers. Utility functions/methods normally doesn't need `context.Context` To follow best practices, any function/method that receives a context.Context argument should receive it as its first argument.
91
92To be able to solve certain problems and/or implement and support certain features making sure that `context.Context` is passed down through all layers of the code is vital. Being able to provide contextual information for the full life-cycle of an API request allows us to use contextual logging, provide contextual information about the authenticated user, create multiple spans for a distributed trace of service calls and database queries etc.
93
94Code should use `context.TODO` when it's unclear which Context to use or it is not yet available (because the surrounding function has not yet been extended to accept a `context.Context` argument).
95
96
97More details in [Services](/contribute/architecture/backend/services.md), [Communication](/contribute/architecture/backend/communication.md) and [Database](/contribute/architecture/backend/database.md).
98
99[Original design doc](https://docs.google.com/document/d/1ebUhUVXU8FlShezsN-C64T0dOoo-DaC9_r-c8gB2XEU/edit#).
100