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

DE Sandbox - GiHub Actions Role #7135

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

zerj9
Copy link

@zerj9 zerj9 commented Mar 7, 2025

Pull Request Objective

Create roles for GitHub Actions runners. These will be used for integrations tests in the create-a-derived-table repo that target the analytical-platform-data-engineering-sandbox-a account. New runners will need to be created.

Checklist

Note

Each items should be checked. Skipping below checks could delay your PR review!

  • I have reviewed the style guide
    and ensured that my code complies with it
  • All checks have passed (or override label applied, if I've
    used the override-static-analysis label, I've explained why)
  • I have self-reviewed my code
  • I have reviewed the checks and can attest they're as expected

Additional Comments

The runners have not been created yet

@zerj9 zerj9 requested review from a team as code owners March 7, 2025 11:51
@zerj9 zerj9 requested a review from Aj-MoJ March 7, 2025 11:51
@zerj9 zerj9 changed the title Create oidc provider and iam role for github action runner in ap compute DE Sandbox - GiHub Actions Role Mar 7, 2025
Copy link
Contributor

github-actions bot commented Mar 7, 2025

Terraform Component 🧱: aws-analytical-platform-data-engineering-sandbox-a-github-actions-roles

Checkov 🛂: success

Trivy 🛂: success

Static Analysis Override Label 🏷️: false

Pusher: @moj-data-platform-robot, Action: pull_request, Working Directory: terraform/aws/analytical-platform-data-engineering-sandbox-a/github-actions-roles, Workflow: Terraform, Marker: aws-analytical-platform-data-engineering-sandbox-a-github-actions-roles_static_analysis

Copy link
Contributor

github-actions bot commented Mar 7, 2025

Terraform Component 🧱: aws-analytical-platform-data-engineering-sandbox-a-github-actions-roles

Terraform Initialization ⚙️: success

Terraform Validation 🤖: success

Terraform Plan 🛠️: success

Pusher: @moj-data-platform-robot, Action: pull_request, Working Directory: terraform/aws/analytical-platform-data-engineering-sandbox-a/github-actions-roles, Workflow: Terraform, Marker: aws-analytical-platform-data-engineering-sandbox-a-github-actions-roles_plan

@jacobwoffenden jacobwoffenden added the do-not-merge Don't merge yet! label Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge Don't merge yet!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants