dotnet/msbuild

Consider testing command lines to identify worker nodes

Open

#3 125 ouverte le 23 mars 2018

Voir sur GitHub
 (11 commentaires) (0 réactions) (0 assignés)C# (1 364 forks)batch import
.NET CoreArea: Performancebackloghelp wantedtriaged

Métriques du dépôt

Stars
 (5 062 stars)
Métriques de merge PR
 (Merge moyen 11j 7h) (125 PRs mergées en 30 j)

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.

Guide contributeur

Direction de recherche
Étudiez les API spécifiques à la plateforme (par exemple Win32 Process sur Windows, /proc sur Linux) pour récupérer les lignes de commande d'autres processus, puis évaluez l'impact sur les performances par rapport à l'approche actuelle.
Stack technique
csharp
Domaine
build system
Type d'issue
Recherche
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#.NETPlatform specific process APIs
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
55