GoogleCloudPlatform/testgrid

Config Merger only reads configuration on startup; should be more often

Aperta

#517 aperta il 23 giu 2021

Vedi su GitHubGuida contributor
 (0 commenti) (1 reazione) (0 assegnatari)Go (72 fork)auto 404
good first issuehelp wanted

Metriche repository

Star
 (205 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

The Config Merger is a controller that takes multiple configurations and merges them together. It requires a list of configurations to merge together.

https://github.com/GoogleCloudPlatform/testgrid/blob/651499da486f08c6ba6fcfa7efb1535318c22fa5/cmd/config_merger/main.go#L93

It then periodically reads those configs and merges them together. Changes in the config are picked up in this loop, but changes to the config list are not.

https://github.com/GoogleCloudPlatform/testgrid/blob/651499da486f08c6ba6fcfa7efb1535318c22fa5/cmd/config_merger/main.go#L111-L125

This has caused issues when the config is changed, and Config Merger silently ignores those changes until it's rebooted. See https://github.com/GoogleCloudPlatform/oss-test-infra/issues/919

Additional logic to determine if the config list has changed and to either log this event or do less work if it hasn't changed would also be nice.

Guida contributor

Direzione di ricerca
Esamina il ciclo attuale in config merger/main.go che legge le configurazioni periodicamente. Aggiungi logica per rilevare cambiamenti nell'elenco delle configurazioni (ad esempio, memorizzando un hash dell'elenco e confrontandolo ad ogni iterazione). Se cambiato, registra l'evento o aggiorna il merge di conseguenza. Implementa un controllo periodico o un meccanismo di watch per la fonte dell'elenco delle configurazioni.
Tech stack
go
Dominio
backenddevops
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Go
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60