-
Notifications
You must be signed in to change notification settings - Fork 816
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
Set the Default Envelope for Workflow Functions #1550
Set the Default Envelope for Workflow Functions #1550
Conversation
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
It seems the CI pipeline status report is stuck for days. |
Description
Set the Default Envelope for Workflow Functions
Fixes #1549
Problems to Fix
Currently:
The envelope of the workflow function is not set correctly.
While the models' envelope are set to the default envelope specified in config.properties, the workflow funcitons' envelope is null.
Type of change
Please delete options that are not relevant.
~~ New feature (non-breaking change which adds functionality)~~
Breaking change (fix or feature that would cause existing functionality to not work as expected)This change requires a documentation updateFeature/Issue validation/testing
Please describe the tests [UT/IT] that you ran to verify your changes and relevant result summary. Provide instructions so it can be reproduced.
Please also list any relevant details for your test configuration.
config.properties
. Run the container with the workflow mnist I provided above, or the example workflow dog_breed_classification. The prediction will succeed.service_envelope=kservev2
, run the container with the workflow in2.
. The workflow function is registered with null envelope.Step 3.
, The workflow function is registered with kserve v2 envelope.Checklist: