Skip to content

Tenanted Alertmanagers are not currently instrumented #1631

@jtlisi

Description

@jtlisi

Currently a non-active registry is passed to each alertmanager tenant:

// New creates a new Alertmanager.
func New(cfg *Config) (*Alertmanager, error) {
	am := &Alertmanager{
		cfg:    cfg,
		logger: log.With(cfg.Logger, "user", cfg.UserID),
		stop:   make(chan struct{}),
	}

	// TODO(cortex): Build a registry that can merge metrics from multiple users.
	// For now, these metrics are ignored, as we can't register the same
	// metric twice with a single registry.
	localRegistry := prometheus.NewRegistry()

We should investigate ways to instrument each alertmanager since it would provide significant improvement in the observability of the alertmanager.

Metadata

Metadata

Assignees

Labels

component/rulesBits & bobs todo with rules and alerts: the ruler, config service etc.keepaliveSkipped by stale bottype/observabilityTo help know what is going on inside Cortex

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions