This is an archive of the discontinued LLVM Phabricator instance.

[llvm][docs] Update "Adding a Builder" docs
ClosedPublic

Authored by DavidSpickett on Apr 21 2023, 3:11 AM.

Details

Summary
  • Move step 8 to later, after worker credentials have been added to the buildmaster.
  • Added command for starting the worker, in addition to creating the worker. The latter only sets up the directories.
  • Noted that in step 6, it is expected that you get a refused connection.
  • Stated that the connection should be tried once, and the worker then stopped.

We could mention that repeated connections with invalid
credentials will result in an IP ban, but it's probably
detail people don't need here.

If it did happen, then you would not know until you tried
the later steps. At which point you are already in contact
with Galina, who is the person who would help you with that
issue in any case.

Diff Detail

Event Timeline

DavidSpickett created this revision.Apr 21 2023, 3:11 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 21 2023, 3:11 AM
DavidSpickett requested review of this revision.Apr 21 2023, 3:11 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 21 2023, 3:11 AM
DavidSpickett added a reviewer: gkistanova.EditedApr 21 2023, 3:20 AM

I didn't think it was worth mentioning the IP blocking, but I am not the one answering the emails about it so my opinion isn't so important here. Let me know if I should add that.

gkistanova accepted this revision.Apr 21 2023, 11:01 AM

Thanks, David! Looks good.

This revision is now accepted and ready to land.Apr 21 2023, 11:01 AM
This revision was automatically updated to reflect the committed changes.