-
Notifications
You must be signed in to change notification settings - Fork 125
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 canary rollout documentation and example #137
Conversation
✅ Deploy Preview for elastic-nobel-0aef7a ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
@alexagriffith Can you help sign off your commit following https://github.com/kserve/website/pull/137/checks?check_run_id=6411042959 Also need to link your tutorial on the main index |
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.
Looks good! I think we just need to add entries here in order for users to be able to navigate to the pages from the website.
Signed-off-by: agriffith50 <agriffith50@bloomberg.net>
Signed-off-by: agriffith50 <agriffith50@bloomberg.net>
/lgtm |
remove shell since it causes weird formatting on the website
add bash keyword and remove the extra tick causing formatting issues
thanks @alexagriffith !! /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: alexagriffith, yuzisun 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 |
* add canary rollout documentation Signed-off-by: agriffith50 <agriffith50@bloomberg.net> * make docs findable on website Signed-off-by: agriffith50 <agriffith50@bloomberg.net> * remove "shell" remove shell since it causes weird formatting on the website * fixing format issue add bash keyword and remove the extra tick causing formatting issues Co-authored-by: agriffith50 <agriffith50@bloomberg.net> Signed-off-by: Dan Sun <dsun20@bloomberg.net>
"Fixes #issue-number" or "Add description of the problem this PR solves"
Proposed Changes
This PR adds and updates the canary documentation from the kserve/kserve repo https://github.com/kserve/kserve/tree/master/docs/samples/v1beta1/rollout - it adds a rollout tab to the Kserve website as well. That way, when more rollout options are available, we can add them there. There is also a step by step example of how to implement a simple canary rollout strategy.