Skip to content

Automate the k8s sameness tests add peering (#2725)#2794

Merged
wilkermichael merged 2 commits intorelease/1.2.xfrom
mw/net-5344-add-tproxy-scenarios-backport
Aug 18, 2023
Merged

Automate the k8s sameness tests add peering (#2725)#2794
wilkermichael merged 2 commits intorelease/1.2.xfrom
mw/net-5344-add-tproxy-scenarios-backport

Conversation

@wilkermichael
Copy link
Copy Markdown
Contributor

Changes proposed in this PR:

How I've tested this PR:

How I expect reviewers to test this PR:

Checklist:

* added fixtures

* removed fixtures
- intentions only gets added now if acls are enabled
- payment-service-resolver is only for locality aware which isn't in scope for this PR

* updated sameness tests to include peering
- refactored with some helper functions for members (now TestClusters)
- made names more uniform, tend more towards the cluster-01-a/cluster-02-a/etc. nomenclature

* added 4 clusters to cni make target

* disable proxy lifecycle
* add additional tproxy static-client
- this doesn't specify an upstream so that tproxy will be able to handle routing

* add tproxy coverage
- add control-flow to handle using the virtual host name when tproxy is enabled
@wilkermichael wilkermichael added pr/no-backport signals that a PR will not contain a backport label pr/no-changelog PR does not need a corresponding .changelog entry labels Aug 18, 2023
@wilkermichael wilkermichael merged commit 910d34b into release/1.2.x Aug 18, 2023
@wilkermichael wilkermichael deleted the mw/net-5344-add-tproxy-scenarios-backport branch August 18, 2023 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr/no-backport signals that a PR will not contain a backport label pr/no-changelog PR does not need a corresponding .changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants