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

Add workflow for OCI AWS tests & document the setup #775

Merged
merged 2 commits into from
Aug 5, 2024
Merged

Conversation

darkowlzz
Copy link
Contributor

@darkowlzz darkowlzz commented May 24, 2024

  • Add a new workflow integration-aws for running the OCI AWS integration tests.
  • Add setup docs for configuring the AWS account and GitHub repository for running the tests.

Depends on fluxcd/test-infra#41.

⚠️ The AWS and GitHub requirements as documented in the test README under IAM and CI setup needs to be fulfilled before or soon after merging this. Else, the CI jobs triggered due to the workflow cron configuration would fail.

Example run: https://github.com/darkowlzz/pkg/actions/runs/9276292128/job/25523036576

Part of fluxcd/flux2#4619

@darkowlzz darkowlzz changed the title oci/tests/int: Add IAM policy document for AWS Add workflow for OCI AWS test & document the setup May 28, 2024
@darkowlzz darkowlzz marked this pull request as ready for review May 28, 2024 21:20
@darkowlzz darkowlzz requested review from stefanprodan and a team as code owners May 28, 2024 21:20
@darkowlzz darkowlzz changed the title Add workflow for OCI AWS test & document the setup Add workflow for OCI AWS tests & document the setup May 28, 2024
@darkowlzz darkowlzz added area/testing Testing related issues and pull requests area/oci OCI related issues and pull requests labels May 28, 2024
- Add a new workflow integration-aws for running the AWS integration
  tests for OCI.
- Add setup docs for configuring the AWS account and GitHub repository
  for running the tests.

Signed-off-by: Sunny <[email protected]>
Modify the workflow configurations accordingly to share accound ID and
region with the cleanup job.

Signed-off-by: Sunny <[email protected]>
Copy link
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

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

LGTM

I've setup the needed AWS secrets in GHA

@darkowlzz darkowlzz merged commit 94e75b7 into main Aug 5, 2024
12 checks passed
@darkowlzz darkowlzz deleted the int-aws-perms branch August 5, 2024 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/oci OCI related issues and pull requests area/testing Testing related issues and pull requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants