-
Notifications
You must be signed in to change notification settings - Fork 7.6k
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
ambient: integrate ztunnel builds #41633
ambient: integrate ztunnel builds #41633
Conversation
Change-Id: Icfbdcafa953d3af16307490d552a00d44e857866
Change-Id: Ic91b929bf9afd3fff272f3acc6c439a0246ff8c3
Change-Id: Icbf04c969e9eca0c07ea336fd0f7859e48a79496
Change-Id: I46d9b96895e12c40774c18507bba1ad6571590c5
/test all |
@@ -244,9 +244,6 @@ spec: | |||
httpGet: | |||
port: 15020 # TODO: use 15021, but it spams logs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should probably make this 15021 (should work for envoy) then switch the ztunnel code to use 15021 as well.
In the meantime I suspect it won't get marked as ready
/test integ-ambient |
Change-Id: Iba1f99f5942653b512ab966e450568e2bfbf53d1
Change-Id: I65f6a4e4979fad618bc46b83062a19b069dc7125
cp -f "${2}" "${TARGET_OUT_LINUX}/ztunnel" | ||
} | ||
|
||
function maybe_build_ztunnel() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not download like envoy proxy? IMO this will couple ztunnel and istio
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"maybe" - by default we download it.
Rust proxy is 100x easier to build locally. If I'm quickly iterating on both proxy and control plane, this allows triggering the build of both in one step.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fix lint
/retest |
This sets up the Istio repo to depend on https://github.com/isito/ztunnel
Usage
../ztunnel
and try to build from rust.make docker.ztunnel
for rapid iteration with control plane + ztunnel changes.