stackabletech / stackabletech/operator-rs

User defined role overrides are ignored by compute_files()

Offen
#339 0 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
Rust
Sterne
167
Forks
19
Ø Merge
1 T. 6 Std.
Gemergte PRs (30 T.)
9

Beschreibung

Description

Given a fictional CRD with role level configuration overrides like this:

brokers:
  configOverrides:
    server.properties:
      auto.create.topics.enable: "false"
  roleGroups:
    default:
      replicas: 1

and an implementation of Configuration::compute_files() that sets auto.create.topics.enable to true,
then the user defined override is ignored and the value of the property will be true.

Note that this doesn't happen with configuration overrides at group level!

The expectation of overrides is, that whatever the user has set , wins over everything else.

A test that demonstrates the bug is here: https://github.com/stackabletech/operator-rs/pull/338

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne mit Configuration::compute_files() und dem Test, der den Fehler in Pull Request #338 demonstriert. Vergleiche, wie Konfigurationsüberschreibungen auf Rollenebene und Gruppenebene behandelt werden, und überprüfe anschließend, dass die benutzerdefinierte Rollenüberschreibung Vorrang vor dem Wert der Implementierung hat und der Test erfolgreich ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
kubernetes, rust
Bereich
infrastructure
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.