Skip to content

HCPCP-2204: Acceptance Tests - Vault #1169

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

Merged
merged 9 commits into from
Jan 13, 2025

Conversation

jasonpilz
Copy link
Member

@jasonpilz jasonpilz commented Jan 9, 2025

🛠️ Description

  • This PR onboards the Vault group of Acceptance Tests to be able to run independently of the rest of the suite.
    1. Adds a new Actions workflow - TestAcc - Vault - which is configured to run on-demand, called by the TestAcc workflow, or automatically on PR when any related implementation changes.
    2. Utilizes it's own dedicated HCP provider account, by seeding new creds via GH Secrets.
    3. Parallelizes the tests, and updates HVN resource names to be unique to support this update.
    4. ⚠️ 2 Tests are skipped and will need triage by the cloud vault team. The current errors are outputted for reference in the action run.

Tests Migrated

TestAccVaultClusterAWS
TestAccVaultClusterAzure
TestAccVaultPlugin
TestAccPerformanceReplication_ValidationsAws

Supporting ENV Variables

VAULT_HCP_ORGANIZATION_ID
VAULT_HCP_PROJECT_ID
VAULT_HCP_CLIENT_ID
VAULT_HCP_CLIENT_SECRET

Related

Output from acceptance testing:

@jasonpilz jasonpilz self-assigned this Jan 9, 2025
@jasonpilz jasonpilz added the pr/no-changelog Auto-pinning label Jan 9, 2025
@jasonpilz jasonpilz changed the title Jasonpilz/hcpcp 2204 testacc vault HCPCP-2204: Acceptance Tests - Vault Jan 9, 2025
@jasonpilz jasonpilz marked this pull request as ready for review January 10, 2025 18:19
@jasonpilz jasonpilz requested review from a team as code owners January 10, 2025 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/no-changelog Auto-pinning
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants