GoogleContainerTools/skaffold

Expose option to suppress output from hooks runner

Aperta

#6344 aperta il 2 ago 2021

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)Go (1416 fork)batch import
area/hookshelp wantedkind/feature-requestpriority/p3

Metriche repository

Star
 (12.822 stelle)
Metriche merge PR
 (Merge medio 3g 6h) (16 PR mergiate in 30 g)

Descrizione

From https://github.com/GoogleContainerTools/skaffold/pull/6337#discussion_r681211816:

we definitely can't use the deploy mute setting since the application logs are manually muted in-between dev loops using this setting. This will have the effect of never actually printing the container hook log messages, since the container hooks are always executed in-between dev loops.

As is, output from executing hooks scripts will always be surfaced to users, even when the logs for individual phases are suppressed. We should either:

  • tie the output of the hooks to the associated phase (e.g. pre-/post-deploy hooks tied to the deploy phase; or,
  • add an option to explicitly suppress output from hooks themselves, independent of their associated phase

Option 1 seems a bit cleaner, but Option 2 seems useful for users who have hooks with particularly verbose output.

Guida contributor

Direzione di ricerca
Esamina la discussione nella PR #6337 e verifica come l'output dell'hook viene attualmente presentato nel codebase, specialmente in relazione alla soppressione dei log di fase.
Tech stack
go
Dominio
backend
Tipo issue
Funzionalità
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
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70