dotnet/tye

Make it clearer that proxy containers are not where project services are running

Open

#380 aperta il 17 apr 2020

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)C# (532 fork)batch import
buggood first issuehelp wanted

Metriche repository

Star
 (5310 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Looking at a log for the voting app:

[13:42:21 INF] Running image redis for redis_06610a82-7
[13:42:21 INF] Running image postgres for postgres_190bd61a-2
[13:42:21 INF] Running image mcr.microsoft.com/dotnet/core/sdk:3.1 for vote-proxy_105de2c0-e
[13:42:21 INF] Running image mcr.microsoft.com/dotnet/core/sdk:3.1 for worker-proxy_70879c25-7

This confused me for a second because I thought my projects were being run as containers, even though it's just the proxy. I'd recommend we change the log message to something like:

[13:42:21 INF] Running proxy image mcr.microsoft.com/dotnet/core/sdk:3.1 for worker-proxy_70879c25-7 to communicate with other docker images.

Or something to that effect.

Guida contributor

Direzione di ricerca
Cerca nel codebase i messaggi di log mostrati nell'issue, in particolare le righe che stampano 'Running image' per i contenitori proxy. La posizione probabile è nel codice dell'orchestratore locale (ad esempio sotto src/). Modifica il messaggio di log per indicare chiaramente che si tratta di un contenitore proxy, ad esempio 'Running proxy image ...' o aggiungi 'to communicate with other docker images' come suggerito. Assicurati di aggiornare tutti i messaggi di log proxy simili.
Tech stack
csharp
Dominio
developer experience
Tipo issue
Documentazione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
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.NET
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90