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

Temporarily enable CI to test against enterprise instance #1355

Merged
merged 15 commits into from May 22, 2024

Conversation

sebasslash
Copy link
Contributor

Description

We are temporarily enabling CI to test against an enterprise-configured instance (TFE). These do not suffer from the same concurrency issues as our cloud-configured test instances. What this means for the time being is that cloud-only tests must be run manually and the results added to the PR description/comment.

Remember to:

Testing plan

  1. Describe how to replicate
  2. the conditions
  3. under which your code performs its purpose,
  4. including example Terraform configs where necessary.

External links

Include any links here that might be helpful for people reviewing your PR. If there are none, feel free to delete this section.

Output from acceptance tests

Please run applicable acceptance tests locally and include the output here. See testing.md to learn how to run acceptance tests.

If you are an external contributor, your contribution(s) will first be reviewed before running them against the project's CI pipeline.

$ TESTARGS="-run TestAccTFEWorkspace" make testacc

...

@sebasslash sebasslash requested a review from a team as a code owner May 10, 2024 15:40
@sebasslash sebasslash force-pushed the sebasslash/enable-onprem-ci branch from dd3da3a to 355a477 Compare May 10, 2024 18:47
@brandonc brandonc merged commit 036e812 into main May 22, 2024
9 checks passed
@brandonc brandonc deleted the sebasslash/enable-onprem-ci branch May 22, 2024 20:22
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.

None yet

2 participants