grafana/mimir

Mimir distributed chart does not support minDomains for topologySpreadConstraints

Open

#6,140 opened on Sep 26, 2023

View on GitHub
 (2 comments) (0 reactions) (0 assignees)Go (808 forks)auto 404
good first issuehelm

Repository metrics

Stars
 (5,181 stars)
PR merge metrics
 (PR metrics pending)

Description

Describe the bug

The mimir distributed chart does not support the minDomains field

To Reproduce

Steps to reproduce the behavior:

  1. Add minDomains to a topologySpreadConstraint in mimir distributed helm chart
  2. Template the output and see that the topologySpreadConstraint does not have minDomains in it.

Expected behavior

I expect the minDomains field to be present in the topologySpreadConstraint field

Environment

  • EKS
  • HELM

Additional Context

Contributor guide

Research direction
Study the Helm chart structure for topologySpreadConstraints, locate where the constraints are defined, add support for minDomains by conditioning on its presence, and test with helm template.
Tech stack
gokubernetes
Domain
infrastructuredevops
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
GitHelmKubernetes
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
65