-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
update developer guide to renew deployment section #771
Conversation
/assign @IronPan |
c5ded6f
to
ea590eb
Compare
@IronPan, could you please have a look? |
thank you @jinchihe |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: IronPan 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 |
Maybe duplicated with #770 |
Thanks @IronPan |
* add py unittests tekton run * disable lint messages based on tests * try fixing syntax error * revert and disable syntax-error * ignore specific line lints * resolve run_e2e_workflow lint * remove ks workflow condition * remove tests related to _get_actual_junit_files * resolve f string for python2 * reformat the string * remove unnecessary pipfile * fix logging string in get_kf_testing_cluster module * skip test for e2e workflow * delete e2e workflow test * remove unused import * add todo issues in the plintrc file
* add uid into big data passing midpath and update uid replacement to also replace with Tekton uid variable * use taskrun name as midpath * regenerate tests * fix tests * fix tests * fix lint * fix tests * update midpath from uid to original pipelinerun * fix lint * replace original pipelinerun name in api to make metadata writer to work. Also add big data passing format to pipeline annotations * update sdk loop logic to pass workspaces to sub-pipelines * fix tests
The old deployment way (bootstrapper.yaml/deploy.sh) has been removed, the pipeline is included in kubeflow now, but the developer guide is not updated accordingly.
The PR to renew the deployment section.
This change is