Skip to content
This repository was archived by the owner on Aug 17, 2023. It is now read-only.

Cherry-pick of #439 and #440 to v1.2-branch#441

Merged
k8s-ci-robot merged 2 commits into
kubeflow:v1.2-branchfrom
pvaneck:autoscaler-check-cherrypick
Nov 11, 2020
Merged

Cherry-pick of #439 and #440 to v1.2-branch#441
k8s-ci-robot merged 2 commits into
kubeflow:v1.2-branchfrom
pvaneck:autoscaler-check-cherrypick

Conversation

@pvaneck

@pvaneck pvaneck commented Nov 11, 2020

Copy link
Copy Markdown
Member

This cherry-picks #439 and #440 in to v1.2-branch.

In anticipation of an update of the manifest knative version
to v0.14.3, this PR removes a check for the autoscaler-hpa
deployment which was removed.
@pvaneck

pvaneck commented Nov 11, 2020

Copy link
Copy Markdown
Member Author

/cc @PatrickXYS

@PatrickXYS

Copy link
Copy Markdown
Member

/lgtm
/approve

Thanks @pvaneck !

@PatrickXYS

Copy link
Copy Markdown
Member
           HTTP response body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"deployments.apps \"metadata-ui\" not found","reason":"NotFound","details":{
"name":"metadata-ui","group":"apps","kind":"deployments"},"code":404}

This is because this PR is not yet cherry-picked #440

@pvaneck

pvaneck commented Nov 11, 2020

Copy link
Copy Markdown
Member Author

@PatrickXYS Want me to cherry-pick that in this PR?

@PatrickXYS

Copy link
Copy Markdown
Member

@pvaneck I sent a PR as of now, #442, can you lgtm?

@k8s-ci-robot k8s-ci-robot removed the lgtm label Nov 11, 2020
@pvaneck pvaneck changed the title Cherry-pick of Remove autoscaler-hpa knative deployment check (#439) to v1.2-branch Cherry-pick of #439 and #440 to v1.2-branch Nov 11, 2020
@pvaneck

pvaneck commented Nov 11, 2020

Copy link
Copy Markdown
Member Author

@PatrickXYS I cherry-picked #440 here as well.

@PatrickXYS

PatrickXYS commented Nov 11, 2020

Copy link
Copy Markdown
Member

/lgtm
/approve
/hold
Hold until test succeed

Thanks @pvaneck !

@k8s-ci-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: PatrickXYS, pvaneck

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

@PatrickXYS

Copy link
Copy Markdown
Member

@pvaneck Feel free to unhold to merge the PR

@pvaneck

pvaneck commented Nov 11, 2020

Copy link
Copy Markdown
Member Author

/unhold

@k8s-ci-robot k8s-ci-robot merged commit 1337086 into kubeflow:v1.2-branch Nov 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants