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

chore(e2e): Use public version of enos #4872

Merged
merged 2 commits into from
Jun 10, 2024
Merged

chore(e2e): Use public version of enos #4872

merged 2 commits into from
Jun 10, 2024

Conversation

moduli
Copy link
Collaborator

@moduli moduli commented Jun 4, 2024

This PR updates the e2e test suite to utilize the public version of enos. Enos (both the CLI and the Terraform Provider) were moved to public repositories and, as a result, no longer require a user to need a TFC token.

For anyone with an existing setup, you can run the following

# Install enos from the new location
brew tap hashicorp/tap && brew update && brew install hashicorp/tap/enos

# Remove the TFC token from your enos vars

Follow-up

  • Update enterprise's enos-run-ent.yml to remove tfc_api_token
  • Inform others of this change

https://hashicorp.atlassian.net/browse/ICU-13987

@moduli moduli added pr/no-milestone Ignores the Milestone Check backport/0.16.x labels Jun 4, 2024
@moduli moduli requested review from ryancragun and stasryzhov June 4, 2024 16:18
@moduli moduli marked this pull request as draft June 4, 2024 18:14
enos/README.md Outdated Show resolved Hide resolved
@moduli moduli force-pushed the moduli-e2e-public-enos branch from 438e957 to fafb276 Compare June 5, 2024 16:25
@@ -104,7 +104,7 @@ module "aws_target" {
}

module "vault" {
source = "app.terraform.io/hashicorp-qti/aws-vault/enos"
source = "./modules/aws_vault"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

For now, I'm going to just copy the contents of the of the shared terraform provider as opposed to updating to use the copy from the vault repo. There have been many a number of changes to this module that exist in the vault repo, so I'd like to save that for a separate activity.

@moduli moduli force-pushed the moduli-e2e-public-enos branch from fafb276 to 397da5b Compare June 5, 2024 16:47
@moduli moduli marked this pull request as ready for review June 5, 2024 17:01
@moduli moduli requested a review from ryancragun June 5, 2024 17:01
@moduli moduli force-pushed the moduli-e2e-public-enos branch from 397da5b to ab427c5 Compare June 10, 2024 14:18
@moduli moduli merged commit 333b9e8 into main Jun 10, 2024
65 checks passed
@moduli moduli deleted the moduli-e2e-public-enos branch June 10, 2024 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/no-milestone Ignores the Milestone Check
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants