-
Notifications
You must be signed in to change notification settings - Fork 152
test: fix python e2e http test #2831
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
Conversation
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jrangelramos, matejvasek The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2831 +/- ##
==========================================
+ Coverage 62.66% 63.49% +0.82%
==========================================
Files 127 127
Lines 15545 15545
==========================================
+ Hits 9742 9870 +128
+ Misses 4892 4736 -156
- Partials 911 939 +28
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
@jrangelramos any ideas why the test is faling? |
/override "Func E2E OnCluster Test / On Cluster Test (ubuntu-latest)" |
@matejvasek: matejvasek unauthorized: /override is restricted to Repo administrators, and the following github teams:knative: knative-release-leads client-wg-leads eventing-wg-leads functions-wg-leads productivity-wg-leads serving-wg-leads. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
@lkingland could you please override the action? |
@dsimansk could you please override the action? |
039b82b
to
0925123
Compare
/override ? |
@dsimansk: /override requires failed status contexts, check run or a prowjob name to operate on.
Only the following failed contexts/checkruns were expected:
If you are trying to override a checkrun that has a space in it, you must put a double quote on the context. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
/override "E2E Test (ubuntu-latest, node)" |
@dsimansk: Overrode contexts on behalf of dsimansk: E2E Test (ubuntu-latest, node) In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
/lgtm |
Changes
Default python functions no longer echoes as response what it receives from request. It just returns "OK", so this fix python tests used on midstream