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

E2E: refactor Connect tests to use stdlib testing #20278

Merged
merged 1 commit into from
Apr 4, 2024

Conversation

tgross
Copy link
Member

@tgross tgross commented Apr 3, 2024

Migrate our E2E tests for Connect off the old framework in preparation for writing E2E tests for transparent proxy and the updated workload identity workflow. Mark the tests that cover the legacy Consul token submitted workflow.

Ref: #20175


Run against our E2E cluster setup on AWS:

$ go test -v -count=1 ./connect
=== RUN   TestConnect_LegacyACLs
=== RUN   TestConnect_LegacyACLs/ConnectDemo
=== RUN   TestConnect_LegacyACLs/ConnectDemoNamespaced
=== RUN   TestConnect_LegacyACLs/ConnectNativeDemo
=== RUN   TestConnect_LegacyACLs/ConnectIngressGatewayDemo
=== RUN   TestConnect_LegacyACLs/ConnectTerminatingGateway
--- PASS: TestConnect_LegacyACLs (97.62s)
    --- PASS: TestConnect_LegacyACLs/ConnectDemo (29.60s)
    --- PASS: TestConnect_LegacyACLs/ConnectDemoNamespaced (24.57s)
    --- PASS: TestConnect_LegacyACLs/ConnectNativeDemo (14.37s)
    --- PASS: TestConnect_LegacyACLs/ConnectIngressGatewayDemo (14.38s)
    --- PASS: TestConnect_LegacyACLs/ConnectTerminatingGateway (14.44s)
=== RUN   Test_serviceOfSIToken
--- PASS: Test_serviceOfSIToken (0.00s)
=== RUN   TestConnect_ClientRestart
    client_test.go:17: skipping test that does nomad agent restart
--- SKIP: TestConnect_ClientRestart (0.00s)
=== RUN   TestConnect
=== RUN   TestConnect/ConnectDemo
=== RUN   TestConnect/ConnectCustomSidecarExposed
=== RUN   TestConnect/ConnectNativeDemo
=== RUN   TestConnect/ConnectIngressGatewayDemo
=== RUN   TestConnect/ConnectMultiIngress
=== RUN   TestConnect/ConnectTerminatingGateway
=== RUN   TestConnect/ConnectMultiService
--- PASS: TestConnect (109.72s)
    --- PASS: TestConnect/ConnectDemo (22.49s)
    --- PASS: TestConnect/ConnectCustomSidecarExposed (17.32s)
    --- PASS: TestConnect/ConnectNativeDemo (14.31s)
    --- PASS: TestConnect/ConnectIngressGatewayDemo (13.26s)
    --- PASS: TestConnect/ConnectMultiIngress (13.31s)
    --- PASS: TestConnect/ConnectTerminatingGateway (13.40s)
    --- PASS: TestConnect/ConnectMultiService (15.38s)
PASS
ok      github.com/hashicorp/nomad/e2e/connect  207.353s

Migrate our E2E tests for Connect off the old framework in preparation for
writing E2E tests for transparent proxy and the updated workload identity
workflow. Mark the tests that cover the legacy Consul token submitted workflow.

Ref: #20175
@shoenig
Copy link
Contributor

shoenig commented Apr 4, 2024

I think this takes care of most of #19250 no?

Copy link
Contributor

@shoenig shoenig left a comment

Choose a reason for hiding this comment

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

LGTM!

@tgross
Copy link
Member Author

tgross commented Apr 4, 2024

I think this takes care of most of #19250 no?

Halfway... part of the e2e/consul package still hasn't been migrated over. This PR was blocking finishing up tproxy E2E tests. So I wanted to land this first. Once I've got those done, I'll come back and finish the e2e/consul package to close out #19250. (Assigning myself #19250 so I don't forget 😀 )

@tgross tgross merged commit c1f020d into main Apr 4, 2024
24 checks passed
@tgross tgross deleted the e2e-refactor-consul-tests branch April 4, 2024 14:48
philrenaud pushed a commit that referenced this pull request Apr 18, 2024
Migrate our E2E tests for Connect off the old framework in preparation for
writing E2E tests for transparent proxy and the updated workload identity
workflow. Mark the tests that cover the legacy Consul token submitted workflow.

Ref: #20175
Copy link

I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 14, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
theme/consul/connect Consul Connect integration theme/e2e theme/testing Test related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants