Skip to content

Clewellyn nava/bb2 4668/integration test gha#1591

Draft
clewellyn-nava wants to merge 71 commits into
masterfrom
clewellyn-nava/BB2-4668/integration-test-gha
Draft

Clewellyn nava/bb2 4668/integration test gha#1591
clewellyn-nava wants to merge 71 commits into
masterfrom
clewellyn-nava/BB2-4668/integration-test-gha

Conversation

@clewellyn-nava
Copy link
Copy Markdown
Contributor

@clewellyn-nava clewellyn-nava commented May 13, 2026

JIRA Ticket:
BB2-4668

What Does This PR Do?

Expands our verification GitHub Action to run the integration tests in a Codebuild instance as a part of PR checks

What Should Reviewers Watch For?

Make sure that local development hasn't been affected by the changes (particularly around the --env-file flags), verify that integration tests still run normally in your own local container, and check to see that the logs from the GHA are not bleeding information.

If you're reviewing this PR, please check for these things in particular:

Validation

If your local stack and the GHA look good, it's good.

What Security Implications Does This PR Have?

Please indicate if this PR does any of the following:

  • Adds any new software dependencies
  • Modifies any security controls
  • Adds new transmission or storage of data
  • Any other changes that could possibly affect security?
  • Yes, one or more of the above security implications apply. This PR must not be merged without the ISSO or team
    security engineer's approval.

Any Migrations?

  • Yes, there are migrations
    • The migrations should be run PRIOR to the code being deployed
    • The migrations should be run AFTER the code is deployed
    • There is a more complicated migration plan (downtime,
      etc)
  • No migrations


echo "TARGET_ENV: $TARGET_ENV"

set -e -o pipefail
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

May need to remove this.

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.

2 participants