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

Fix tasks tags in kfp workflows. #236

Merged
merged 11 commits into from
Jun 13, 2024
Merged

Fix tasks tags in kfp workflows. #236

merged 11 commits into from
Jun 13, 2024

Conversation

revit13
Copy link
Collaborator

@revit13 revit13 commented Jun 6, 2024

Why are these changes needed?

Currently the task tags in kfp workflow are not updated correctly. This PR tries to fix that

@revit13 revit13 marked this pull request as draft June 6, 2024 12:10
@revit13 revit13 requested a review from daw3rd June 6, 2024 12:32
@revit13 revit13 marked this pull request as ready for review June 8, 2024 18:42
Signed-off-by: Revital Sur <eres@il.ibm.com>
Signed-off-by: Revital Sur <eres@il.ibm.com>
Signed-off-by: Revital Sur <eres@il.ibm.com>
Signed-off-by: Revital Sur <eres@il.ibm.com>
Signed-off-by: Revital Sur <eres@il.ibm.com>
Signed-off-by: Revital Sur <eres@il.ibm.com>
Signed-off-by: Revital Sur <eres@il.ibm.com>
.make.versions Outdated
@@ -13,29 +13,45 @@ RELEASE_VERSION_SUFFIX=.dev6
# Data prep lab wheel version
DPK_LIB_VERSION=0.2.0$(RELEASE_VERSION_SUFFIX)
DPK_LIB_KFP_VERSION=0.2.0$(RELEASE_VERSION_SUFFIX)
DPK_LIB_KFP_VERSION_v2=0.2.0$(RELEASE_VERSION_SUFFIX)
Copy link
Member

Choose a reason for hiding this comment

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

I think we need a separate image for v2

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed. Thanks

Copy link
Member

@roytman roytman left a comment

Choose a reason for hiding this comment

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

LGTM, but would like that @daw3rd or @blublinsky will take a look.

@daw3rd
Copy link
Member

daw3rd commented Jun 10, 2024

LGTM

Signed-off-by: Revital Sur <eres@il.ibm.com>
Signed-off-by: Revital Sur <eres@il.ibm.com>
@revit13 revit13 merged commit abab08c into IBM:dev Jun 13, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants