knative / client-contrib

Community contributed `kn` plugins.
Apache License 2.0
10 stars 23 forks source link

Update test-infra to stop using serving.yaml #45

Closed daisy-ycguo closed 4 years ago

daisy-ycguo commented 4 years ago

Serving is going to stop releasing serving.yaml, update test-infra to the latest to stop using it.

Refer to @chizhg 's update to Client: https://github.com/knative/client/pull/876

daisy-ycguo commented 4 years ago

@rhuss @maximilien will you help to approve this ? I'd like to check if the integration test failure could be resolved. Thank you.

rhuss commented 4 years ago

/lgtm

rhuss commented 4 years ago

/approve

knative-prow-robot commented 4 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: daisy-ycguo, rhuss

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: - ~~[OWNERS](https://github.com/knative/client-contrib/blob/master/OWNERS)~~ [rhuss] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment