kubernetes-sigs/cluster-api

Drop internal/log package

Aperta

#7029 aperta il 5 ago 2022

Vedi su GitHubGuida contributor
 (13 commenti) (2 reazioni) (0 assegnatari)Go (1532 fork)auto 404
area/clusterclasshelp wantedkind/cleanuppriority/important-longtermtriage/accepted

Metriche repository

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

Descrizione

internal/log has been introduced while developing the topology controller in order to easy our path towards structured logging.

Now that structured logging is well integrated in klog and we have clear logging guidelines I think we can drop this package (eventually we can add by simply adding a couple of utility func to the topology controller, TBD)

/area topology /kind cleanup

cc @ykakarap @sbueringer @killianmuldoon opinions?

Guida contributor

Direzione di ricerca
Rimuovere il pacchetto internal/log e aggiornare tutti gli import per utilizzare direttamente klog. Assicurarsi che non rimangano riferimenti e verificare che i test passino.
Tech stack
go
Dominio
backend
Tipo issue
Refactoring
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Go
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70