Skip to content

Cherry pick #1396 #1410 to v1.1-branch#1419

Merged
k8s-ci-robot merged 3 commits into
kubeflow:v1.1-branchfrom
PatrickXYS:v1.1-branch
Jul 25, 2020
Merged

Cherry pick #1396 #1410 to v1.1-branch#1419
k8s-ci-robot merged 3 commits into
kubeflow:v1.1-branchfrom
PatrickXYS:v1.1-branch

Conversation

@PatrickXYS

Copy link
Copy Markdown
Member

Which issue is resolved by this Pull Request:

Description of your changes:
Cherry-pick kubeflow/manifests AWS changes to v1.1-branch for release purpose.

Checklist:

  • Unit tests have been rebuilt:
    1. cd manifests/tests
    2. make generate-changed-only
    3. make test

Yao Xiao added 2 commits July 24, 2020 12:50
* Migrate AWS manifest to v3 pattern

* Clean up tests files

* Add istio namespace to istio ingress

* Update KFP pipeline test case for aws stack
* Changes needed for V3 change on AWS

* Adopt AWS ECR image for jupyter-web-app components and generate all tests against manifest V3 changes

* Adopt Cognito change against V3

* Adopt change for multi-user KFP
@google-cla google-cla Bot added the cla: yes label Jul 24, 2020
@kubeflow-bot

Copy link
Copy Markdown
Contributor

This change is Reviewable

@PatrickXYS

Copy link
Copy Markdown
Member Author

/cc @Jeffwan

@PatrickXYS

Copy link
Copy Markdown
Member Author

/test kubeflow-manifests-presubmit

@Jeffwan

Jeffwan commented Jul 24, 2020

Copy link
Copy Markdown
Member

Let's try to fix the test failure.

@Jeffwan

Jeffwan commented Jul 24, 2020

Copy link
Copy Markdown
Member

/cc @jlewi @animeshsingh @yanniszark @hougangliu

We'd like to cherry-pick v3 manifest with multi-user pipeline on AWS to 1.1. please help approve the change.

@Jeffwan

Jeffwan commented Jul 24, 2020

Copy link
Copy Markdown
Member

/lgtm

@PatrickXYS

Copy link
Copy Markdown
Member Author

/cc @terrytangyuan Can you take a look?

@terrytangyuan terrytangyuan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/approve

@k8s-ci-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: terrytangyuan

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details 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

@k8s-ci-robot k8s-ci-robot merged commit d8be7f9 into kubeflow:v1.1-branch Jul 25, 2020
@PatrickXYS PatrickXYS deleted the v1.1-branch branch July 25, 2020 18:38
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.

5 participants