Skip to content

Replace github-status-test webhook with same functionalities in PyTorch bot #7549

@huydhn

Description

@huydhn

An issue surfaces recently when we try to onboard repo from meta-pytorch onto HUD https://fburl.com/workplace/j063m9ot. This usually is a 2-step processes:

  1. Onboard the repo to PyTorch bot app
  2. Add github-status-test webhook to receive all the events (although we only process workflow_job and workflow_run https://github.com/pytorch/test-infra/blob/main/aws/lambda/github-status-test/lambda_function.py#L24)

(1) is ok, but (2) raises some questions on why it's needed because its functionalities can be done in PyTorch bot without the need to ask meta-pytorch admins to tweak the repo webhook configs.

github-status-test does several things:

  1. Upload logs to S3 (and trigger log classifier).
  2. Archive the raw workflow_job and workflow_run events, also on S3.
  3. Allow Dev Infra to backfill historical workflow runs and jobs in the event of an outage on our infra. This is rarely used.

Given these 3, should we fold every onto PyTorch bot and deprecate github-status-test? This lambda also has a very bad naming that is hard to get rid off :)

cc @malfet @seemethere @d4l3k

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    Projects

    Status

    No status

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions