-
Notifications
You must be signed in to change notification settings - Fork 1
test(egress): prove data-plane secret injection #207
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
Open
casey-brooks
wants to merge
46
commits into
main
Choose a base branch
from
noa/issue-206
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
46 commits
Select commit
Hold shift + click to select a range
1e03e95
test(egress): add data-plane secret injection e2e
casey-brooks 635d7c2
fix(egress): use sidecar for ziti tunnel
casey-brooks e232d24
ci(e2e): run focused test from pr ref
casey-brooks 6296936
ci(e2e): use distinct focused artifact name
casey-brooks 7ccae35
test(egress): use supported agent authz role
casey-brooks a15cfdd
fix(e2e): salvage egress dataplane test
casey-brooks 3b8b49f
ci(egress): provision bootstrap egress branch
casey-brooks 0aa8bbd
ci(egress): pin fixed egress runtime
casey-brooks d910e6b
ci(egress): use bootstrap provider staging fix
casey-brooks 009fce0
ci(egress): use published PR image tag
casey-brooks c5c0b6f
ci(egress): wait for PR image
casey-brooks cba5fad
ci(egress): pin host address image
casey-brooks 28670b3
ci(egress): pin address drift image
casey-brooks 29e3dcc
ci(egress): pin service role image
casey-brooks 0b17dcd
ci(e2e): pin ziti reconcile platform
casey-brooks 3929355
chore(ci): pin ziti-management image
casey-brooks 0c9f25a
chore(ci): pin ziti-management image
casey-brooks f18003b
chore(ci): pin current egress image
casey-brooks 1e83324
chore(ci): pin latest egress image
casey-brooks 8b8ef0a
fix(e2e): exec egress workload container
casey-brooks 57d7d80
fix(e2e): pass ziti sidecar image
casey-brooks 0afd798
chore(ci): pin egress gateway image
casey-brooks 5e576d0
fix(egress): resolve ziti enrollment host
casey-brooks c26b633
test(egress): allow ziti service lookup
casey-brooks 6ffa8da
test(egress): log stalled workload pods
casey-brooks fa3a3e3
test(egress): add egress diagnostics helpers
casey-brooks 7922ff5
chore(ci): pin ziti management image
casey-brooks 24875b4
test(egress): log inspect failures
casey-brooks 4ab412c
fix(e2e): use current ziti tunnel image
casey-brooks 7489c0d
chore(ci): pin current ziti management image
casey-brooks 4799e2c
fix(console): wait for organization membership
casey-brooks f787b23
chore(ci): pin current ziti management image
casey-brooks 2f72466
fix(e2e): avoid ziti service lookup
casey-brooks 1b34b29
fix(e2e): keep init DNS on cluster resolver
casey-brooks 3dd768a
fix(e2e): pin usable ziti enrollment image
casey-brooks 09c592e
Revert "fix(e2e): pin usable ziti enrollment image"
casey-brooks ccded66
fix(e2e): use ziti agent enrollment image
casey-brooks 3565ecc
fix(e2e): pin valid ziti management image
casey-brooks 7e17630
fix(e2e): use role-filtered ziti management image
casey-brooks 3a00444
fix(e2e): wait for postman echo body
casey-brooks 03091ab
fix(e2e): pin explicit ziti enrollments
casey-brooks b67ddf7
fix(e2e): repin verified ziti enrollment image
casey-brooks 19b8327
test(e2e): prove ziti enrollment state
casey-brooks c748d03
chore(e2e): repin ziti management image
casey-brooks f102fc9
test(terraform): expect configured egress method
casey-brooks ef29e13
test(console): gate egress UI spec
casey-brooks File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -13,7 +13,7 @@ Validates core platform services through Go E2E tests: gateway authentication, a | |
|
|
||
| - Source directory: `suites/go-core` | ||
| - Test inventory pattern: `tests/*_test.go` | ||
| - Included case count: 92 | ||
| - Included case count: 93 | ||
|
|
||
| ## Actors | ||
|
|
||
|
|
@@ -146,6 +146,7 @@ Validates core platform services through Go E2E tests: gateway authentication, a | |
| | [E2E-GO-CORE-098](#e2e-go-core-098) | `TestEgressGatewayFeaturePath` | @svc_egress | | ||
| | [E2E-GO-CORE-099](#e2e-go-core-099) | `TestEgressGatewayDenyAndNoRulePaths` | @svc_egress | | ||
| | [E2E-GO-CORE-100](#e2e-go-core-100) | `TestEgressGatewayDeploymentWiring` | @svc_egress_gateway | | ||
| | [E2E-GO-CORE-101](#e2e-go-core-101) | `TestEgressGatewayDataPlaneSecretInjection` | @svc_egress, @svc_egress_gateway | | ||
|
|
||
| ## Scenarios | ||
|
|
||
|
|
@@ -1263,4 +1264,16 @@ Validates core platform services through Go E2E tests: gateway authentication, a | |
| - **And** The public internet egress rule allows `0.0.0.0/0` with blocked CIDR exceptions for `10.0.0.0/8`, `172.16.0.0/12`, `192.168.0.0/16`, `169.254.0.0/16`, and `127.0.0.0/8`. | ||
| - **And** A k8s-runner workload can receive and use the egress CA via inline file/env contract. | ||
|
|
||
| **Current framework limitation:** full outbound HTTP forwarding through Egress Gateway is not covered here because the current `egress-gateway` service process exposes only its admin health listener; the pure request-processing runtime exists in source but is not wired to an OpenZiti data-plane listener yet. These cases cover the highest-value feasible path: Egress control-plane rule lookup, Secrets referential integrity, deny/no-rule state, Egress Gateway CA/Ziti wiring, and workload NetworkPolicy defaults. | ||
| ### E2E-GO-CORE-101 | ||
|
|
||
| - **Source:** `suites/go-core/tests/egress_dataplane_test.go` | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [minor] This source path does not match the committed test file ( |
||
| - **Test:** `TestEgressGatewayDataPlaneSecretInjection` | ||
| - **Tags:** @svc_egress, @svc_egress_gateway | ||
|
|
||
| **Scenario:** TestEgressGatewayDataPlaneSecretInjection | ||
|
|
||
| - **Given** An authorized agent workload has no authorization token in its command or environment and an allow EgressRule references a platform Secret for `postman-echo.com:443`. | ||
| - **When** The workload sends an HTTPS request to Postman Echo without an Authorization header. | ||
| - **Then** The request is routed through Egress Gateway. | ||
| - **And** Postman Echo reports that the upstream destination received `Authorization: Bearer <secret-value>`. | ||
| - **And** The echoed query marker matches the test request. | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.