Skip to content

Add L2 CRCR deployment configuration#614

Merged
atalman merged 5 commits into
mainfrom
crcr-l2
May 22, 2026
Merged

Add L2 CRCR deployment configuration#614
atalman merged 5 commits into
mainfrom
crcr-l2

Conversation

@can-gaa-hou
Copy link
Copy Markdown
Collaborator

Summary

This PR is an extension of L1 (#433), and it only adds another AWS Lambda function and some environment variables for handling result callbacks from DownStream CI. It is also associated with L2 implementation (pytorch/test-infra#7967) and should only be merged after L2 implementation is completed.

File to change

.github/workflows/
├── crcr-on-pr.yml                  # Add result callback environment variables
└── crcr-deploy-prod.yml            # Add result callback environment variables
crcr/
├── Terrafile                       # Add result callback zip file download from test-infra
└── aws/ 
    ├── variables.tf                # Add result callback environment variables
    ├── outputs.tf                  # Add result callback AWS Lambda function
    ├── secrets.tf                  # Add result callback secrets
    ├── security.tf                 # Add security configuration
    └── callback.tf                 # Add result callback AWS Lambda function configuration

Test

Multiple deployments and verifications have been completed on a personal AWS environment.

cc @fffrog

Comment thread crcr/Terrafile Outdated
Copy link
Copy Markdown
Collaborator

@fffrog fffrog left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work, thank you.

@can-gaa-hou
Copy link
Copy Markdown
Collaborator Author

can-gaa-hou commented May 22, 2026

This PR can be merged once the following PR is merged:

@fffrog fffrog requested a review from atalman May 22, 2026 03:29
Copy link
Copy Markdown

@subinz1 subinz1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed against our RFC (RFC-0054) and HUD PR (test-infra#8069) — everything lines up: auth secrets, HUD API integration, Redis status tracking, and the dependency order on test-infra#7967. Great work @can-gaa-hou, thanks for putting this together!

Comment thread crcr/Terrafile Outdated
Co-authored-by: Thanh Ha <zxiiro@gmail.com>
@atalman atalman enabled auto-merge May 22, 2026 15:10
@atalman atalman added this pull request to the merge queue May 22, 2026
Merged via the queue into main with commit 1d11798 May 22, 2026
12 checks passed
@atalman atalman deleted the crcr-l2 branch May 22, 2026 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants