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
CI: integrate hubble-relay and add initial hubble-relay tests #11549
Conversation
test-me-please |
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.
amazing test coverage
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.
Not super familiar with the provisioning scripts, but the CI tests themselves look fine.
Almost all tests failed with known flakes (managed etc, privileged unit tests). test-me-please |
retest-runtime |
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.
Good changes! I have one small request - please add
--build-arg CILIUM_SHA=$(firstword $(GIT_VERSION))
to docker build
call in hubble-relay build step in Makefile and
ARG CILIUM_SHA=""
LABEL cilium-sha=${CILIUM_SHA}
in hubble-relay dockerfile for all intermediate images as well as the final images to ensure that these get cleaned in GKE builds.
d95a9e5
to
9554d1e
Compare
Signed-off-by: Robin Hahling <robin.hahling@gw-computing.net>
Signed-off-by: Robin Hahling <robin.hahling@gw-computing.net>
These tests are the same tests for L3/L4 and L7 flows that target the local hubble server but they target hubble-relay instead. Signed-off-by: Robin Hahling <robin.hahling@gw-computing.net>
9554d1e
to
0163c9b
Compare
test-me-please |
test-me-please |
retest-runtime |
retest-4.9 |
Runtime tests failed on known flake #11512 (https://jenkins.cilium.io/job/Cilium-PR-Runtime-4.9/295/), thus merging. |
This PR integrates hubble-relay into the CI and adds basic tests for hubble-relay.
Ref: #11228