dotnet/msbuild

Consider testing command lines to identify worker nodes

Open

#3.125 aberto em 23 de mar. de 2018

Ver no GitHub
 (11 comments) (0 reactions) (0 assignees)C# (1.364 forks)batch import
.NET CoreArea: Performancebackloghelp wantedtriaged

Métricas do repositório

Stars
 (5.062 stars)
Métricas de merge de PR
 (Mesclagem média 11d 7h) (125 fundiu PRs em 30d)

Description

See https://github.com/Microsoft/msbuild/pull/3106#discussion_r176604238

Currently, in .NET Core, MSBuild will find all processes running in the dotnet host and attempt to connect to them as though they were MSBuilds. That shouldn't interfere with the other processes unless they have a named pipe named MSBuild{pid}, but it takes time during the node-connection process.

We should explore narrowing down the candidate processes beforehand. We could for example examine process command lines for /nodereuse:true and/or MSBuild.dll.

Questions that need to be answered:

  • How can this be done at all? AFAIK, there's no unified .NET API to get command lines of other processes on all platforms, so it might have to be platform-specific.
  • Is this faster than the current approach? Examining command lines will take some time.

Guia do colaborador

Direção de pesquisa
Investigue APIs específicas da plataforma (ex.: Win32 Process no Windows, /proc no Linux) para obter as linhas de comando de outros processos e, em seguida, avalie o impacto no desempenho em comparação com a abordagem atual.
Pilha de tecnologia
csharp
Domain
build system
Tipo Issue
Pesquisa
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#.NETPlatform specific process APIs
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
55