-
Notifications
You must be signed in to change notification settings - Fork 264
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
squid fixes for webhook-http-proxy-test #216
Conversation
Codecov Report
@@ Coverage Diff @@
## master #216 +/- ##
=======================================
Coverage 81.43% 81.43%
=======================================
Files 10 10
Lines 792 792
=======================================
Hits 645 645
Misses 131 131
Partials 16 16 Continue to review full report at Codecov.
|
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👍🏼 ʰᵒᵖᵉ ʸᵒᵘ ᵗᵉˢᵗᵉᵈ ᵒⁿ ʷᶦⁿᵈᵒʷˢ
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.
Nice work. Made a couple of suggestions for readability.
test/e2e/webhook-http-proxy-test
Outdated
squid_worker_pods="" | ||
for i in `seq 1 10`; do | ||
echo "Checking if squid http-proxy has started..." | ||
squid_worker_pods=$(kubectl get pods -o json | jq '.items[] | select( .metadata.name | contains("squid") ) | .metadata.name as $name | select( .spec.nodeName | contains("worker") ) | .spec.nodeName as $nodename | $name' -r || :) |
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.
Minor: could that command be put in a function?
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.
👍
Issue #, if available:
#214
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.