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

feat(backend):Parameterizing v2 Launcher Image #149

Closed
wants to merge 1 commit into from
Closed

feat(backend):Parameterizing v2 Launcher Image #149

wants to merge 1 commit into from

Conversation

DharmitD
Copy link
Member

@DharmitD DharmitD commented Oct 4, 2023

Which issue is resolved by this Pull Request:
Resolves #130

Description of your changes:
Parameterizing the v2 launcher image so it's possible to extend/provide fixes to the code.
Cherry picks kubeflow#1359

Testing instructions:

  1. Saved the code changes locally and built this image: quay.io/rhn_support_ddalvi/odh-ml-pipelines-api-server-container-live:v2-2
  2. Deployed DSP v2 and replaced the API Server image with the above image.

Checklist:

@DharmitD DharmitD changed the title Parameterizing launcher image feat(backend):Parameterizing v2 Launcher Image Oct 4, 2023
@rimolive
Copy link

rimolive commented Oct 5, 2023

/lgtm

@rimolive
Copy link

rimolive commented Oct 5, 2023

/approve

@rimolive
Copy link

rimolive commented Oct 5, 2023

/approve cancel

@openshift-ci
Copy link

openshift-ci bot commented Oct 5, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: DharmitD

The full list of commands accepted by this bot can be found 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

@DharmitD
Copy link
Member Author

DharmitD commented Oct 5, 2023

Closing this PR as this has been duplicated in https://github.com/opendatahub-io/data-science-pipelines/pull/153

@DharmitD DharmitD closed this Oct 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants