GoogleCloudPlatform/testgrid

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

Ouverte

#517 ouverte le 23 juin 2021

Voir sur GitHubGuide contributeur
 (0 commentaire) (1 réaction) (0 personne assignée)Go (72 forks)auto 404
good first issuehelp wanted

Métriques du dépôt

Stars
 (205 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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.

Guide contributeur

Direction de recherche
Examinez la boucle actuelle dans config merger/main.go qui lit les configurations périodiquement. Ajoutez une logique pour détecter les changements dans la liste des configurations (par exemple, en stockant un hash de la liste et en le comparant à chaque itération). Si modifié, enregistrez l'événement ou mettez à jour la fusion en conséquence. Implémentez une vérification périodique ou un mécanisme de surveillance pour la source de la liste des configurations.
Stack technique
go
Domaine
backenddevops
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Go
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60