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
Use CLI-based Helm installation for ingress tests #25609
Conversation
1d3434f
to
3898fca
Compare
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.
Thanks and lgtm ✔️
3898fca
to
242e44c
Compare
242e44c
to
f50f2d7
Compare
f50f2d7
to
0e16fdb
Compare
Commit 0e16fdb9a0362cf62cd53156ea1992df18dbacf8 does not contain "Signed-off-by". Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin |
Signed-off-by: Daniel Hawton <daniel.hawton@solo.io>
0e16fdb
to
8c26829
Compare
Commit 0e16fdb9a0362cf62cd53156ea1992df18dbacf8 does not contain "Signed-off-by". Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin |
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.
LGTM from a docs perspective
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.
LGTM ✔️
The changes are limited to gha Conformance Ingress, which is passed in this PR. All reviews are in, marking this ready to merge. Thanks a lot for your contribution 🎖️ |
Move
conformance-ingress
workflow to cli-based helm install mode in an effort to complete another item in #25156Locally act wasn't cooperating so may need CI to troubleshoot any potential issues.