Resources to launch GitHub Automation App

opensearch-project / opensearch-metrics

OpenSearch Metrics

https://metrics.opensearch.org

Apache License 2.0

2 stars 4 forks source link

Resources to launch GitHub Automation App #78

Closed prudhvigodithi closed 2 weeks ago

prudhvigodithi commented 2 weeks ago

Description

Resources to launch GitHub Automation App

This PR needs to be merged https://github.com/opensearch-project/automation-app/pull/12 1st.
After the above PR is merged, the new CDK stacks will create a GitHub app secret (update the secret value) and resources to launch the GitHub app.
- Ec2 autoscaling group with right launch configuration to run the app.
- Permissions to the Ec2 role to fetch the GitHub app secret and list to the GitHub events.
- Runs 2 docker containers to support github-merged-pulls-monitor and github-workflow-runs-monitor.
- Permissions to index the data to metrics cluster using sigv4.

Issues Resolved

Part of https://github.com/opensearch-project/opensearch-build/issues/4941 and https://github.com/opensearch-project/automation-app/issues/9

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. For more information on following Developer Certificate of Origin and signing off your commits, please check here.

prudhvigodithi commented 2 weeks ago

Adding @peterzhuamazon @bshien to review the PR, as mentioned this PR needs to be reviewed and merged https://github.com/opensearch-project/automation-app/pull/12 1st. Thank you @getsaurabh02

codecov[bot] commented 2 weeks ago

Codecov Report

Attention: Patch coverage is 96.29630% with 1 line in your changes missing coverage. Please review.

Project coverage is 83.27%. Comparing base (d8b52e6) to head (2444c66). Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
infrastructure/lib/stacks/gitHubAutomationApp.ts	96.29%	1 Missing :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #78 +/- ## ============================================ + Coverage 82.98% 83.27% +0.29% Complexity 202 202 ============================================ Files 53 54 +1 Lines 1187 1214 +27 Branches 48 51 +3 ============================================ + Hits 985 1011 +26 - Misses 180 181 +1 Partials 22 22 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

prudhvigodithi commented 2 weeks ago

Since the PR is merged https://github.com/opensearch-project/automation-app/pull/12, we can move forward with this change. Thanks

prudhvigodithi commented 2 weeks ago

Thanks @bshien and @peterzhuamazon I have updated the PR with your suggestions, please check.

prudhvigodithi commented 2 weeks ago

Merging this PR as there are known issues with Mend Security Check.