Skip to content
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 tekton loop dsl extension skeleton #799

Merged
merged 1 commit into from
Dec 16, 2021

Conversation

Tomcli
Copy link
Member

@Tomcli Tomcli commented Dec 15, 2021

Which issue is resolved by this Pull Request:
Related #791

Description of your changes:
Add SDK skeleton for Tekton specific DSL extensions.

Environment tested:

  • Python Version (use python --version):
  • Tekton Version (use tkn version):
  • Kubernetes Version (use kubectl version):
  • OS (e.g. from /etc/os-release):

Checklist:

@google-oss-prow
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Tomcli

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@Tomcli
Copy link
Member Author

Tomcli commented Dec 15, 2021

/cc @yhwang

BreakOp_template = components.load_component_from_text(BreakOp_yaml)
BreakOp = BreakOp_template()
return BreakOp

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To be honest I did not follow how the break happens here. Seems it's just a normal task?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pugangxa I think it's related to issue #800.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, right now the pipeline loop break only works when the last-loop-task is skipped. However, the proposed Break() DSL is to break when the condition is met. So I opened #800 to add a new break features to break when the Break() task is being executed.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry I just overlooked that issue. Then it makes sense. Thanks for all your explanation.

@pugangxa
Copy link
Contributor

/lgtm

@google-oss-prow google-oss-prow bot added the lgtm label Dec 16, 2021
@google-oss-prow google-oss-prow bot merged commit b16bd88 into kubeflow:master Dec 16, 2021
wzhanw pushed a commit to wzhanw/kfp-tekton that referenced this pull request Mar 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants