-
Notifications
You must be signed in to change notification settings - Fork 834
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
Add executor changes for HTTP/gRPC multiplexing #1772
Conversation
Signed-off-by: glindsell <gl@seldon.io>
Thu Apr 30 16:18:28 UTC 2020 impatient try |
Thu Apr 30 16:18:35 UTC 2020 impatient try |
/test integration |
Fri May 1 13:35:32 UTC 2020 impatient try |
Signed-off-by: glindsell <gl@seldon.io>
Fri May 1 13:56:35 UTC 2020 impatient try |
Fri May 1 13:56:37 UTC 2020 impatient try |
/retest |
Fri May 1 15:20:37 UTC 2020 impatient try |
/test integration |
Mon May 4 09:01:45 UTC 2020 impatient try |
/test integration |
Tue May 5 13:13:30 UTC 2020 impatient try |
Signed-off-by: glindsell <gl@seldon.io>
Wed May 6 08:58:27 UTC 2020 impatient try |
Wed May 6 08:58:30 UTC 2020 impatient try |
/retest |
Wed May 6 09:45:40 UTC 2020 impatient try |
/test integration |
Thu May 7 08:19:49 UTC 2020 impatient try |
Signed-off-by: glindsell <gl@seldon.io>
Mon May 11 14:05:27 UTC 2020 impatient try |
Mon May 11 14:06:38 UTC 2020 impatient try |
/test integration |
Mon May 18 15:45:43 UTC 2020 impatient try |
/test integration |
Tue May 19 12:42:24 UTC 2020 impatient try |
/test integration |
Wed May 20 08:15:41 UTC 2020 impatient try |
/test integration |
Wed May 20 09:55:30 UTC 2020 impatient try |
/test integration |
Wed May 20 13:19:18 UTC 2020 impatient try |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cliveseldon 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 |
Thu May 21 11:07:56 UTC 2020 impatient try |
Thu May 21 11:08:25 UTC 2020 impatient try |
Mon May 25 07:16:16 UTC 2020 impatient try |
Mon May 25 07:16:18 UTC 2020 impatient try |
Tue May 26 08:42:10 UTC 2020 impatient try |
Tue May 26 08:42:16 UTC 2020 impatient try |
This reverts commit ad5b6a5.
* Revert "Add executor changes for HTTP/gRPC multiplexing (#1772)" This reverts commit ad5b6a5. * Fix executor main conflict Signed-off-by: glindsell <gl@seldon.io> * Remove reduntant code in executor main Signed-off-by: glindsell <gl@seldon.io> * Fix log message Signed-off-by: glindsell <gl@seldon.io> * Move Kafka serving Signed-off-by: glindsell <gl@seldon.io> * remove grpcurl dependency Signed-off-by: glindsell <gl@seldon.io> * Reintroduce getExecutorGrpcPort Signed-off-by: glindsell <gl@seldon.io> * Add missing constants from operator seldondeployment engine Signed-off-by: glindsell <gl@seldon.io> * Remove port selection for executor in operator and change port name to grpc Signed-off-by: glindsell <gl@seldon.io> * go mod tidy Signed-off-by: glindsell <gl@seldon.io> * go.mod go.sum and licenses changes Signed-off-by: glindsell <gl@seldon.io> * Add transport grpc to metadata grpc test Signed-off-by: glindsell <gl@seldon.io>
failed to trigger Pull Request pipeline
|
Executor changes work when tested locally.
Operator changes still needed.
Fixes #1762
Fixes #1628
Signed-off-by: glindsell gl@seldon.io