Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integrate github app workflow with lambda. #302

Merged

Conversation

santoshmahto7
Copy link
Contributor

A new github app "iocost-issue-creater" is created and installed in iocost-benchmark organization which has a permission to file issue on iocost-benachmarks repository.

When a user uploads the benchmark result with lambda url, lambda saves the benchmark result in aws s3 bucket and create a issue in iocost-benchmarks projects.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jan 31, 2024
A new github app "iocost-issue-creater" is created and installed
with iocost-benchmark organization which has a permission to
file issue on iocost-benachmarks repository.

When a user uploads the benchmark result with lambda url, lambda
saves the benchmark result in aws s3 bucket and create a issue
in iocost-benchmarks projects.

Signed-off-by: Santosh Mahto <[email protected]>
@santoshmahto7
Copy link
Contributor Author

@htejun @davide125 : I don't see any option to add reviewers or self merge this pull request. It seems i have very limited permission in this repo.
Could any one of you review or land this pull request ?

@htejun htejun merged commit a5d64c4 into facebookexperimental:main Feb 2, 2024
2 checks passed
@santoshmahto7 santoshmahto7 deleted the santo/githubapp_push branch February 21, 2024 06:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants