Repository: knative/client
Pull request #657 (2020-02-12 00:36:15)

Support multiple arguments on revision delete
Addresses #317 


Release note:

- 🧽 Support multiple revision names on `kn revision delete`

knative-prow-robot: [APPROVALNOTIFIER] This PR is **NOT APPROVED**

This pull-request has been approved by: *<a href="https://github.com/knative/client/pull/657#" title="Author self-approved">wslyln</a>*
To complete the [pull request process](https://git.k8s.io/community/contributors/guide/owners.md#the-code-review-process), please assign **navidshaikh**
You can assign the PR to them by writing `/assign @navidshaikh` in a comment when ready.

The full list of commands accepted by this bot can be found [here](https://go.k8s.io/bot-commands?repo=knative%2Fclient).

<details open>
Needs approval from an approver in each of these files:

- **[OWNERS](https://github.com/knative/client/blob/master/OWNERS)**

Approvers can indicate their approval by writing `/approve` in a comment
Approvers can cancel approval by writing `/approve cancel` in a comment
</details>
<!-- META={"approvers":["navidshaikh"]} -->
knative-metrics-robot: The following is the coverage report on the affected files.
Say `/test pull-knative-client-go-coverage` to re-run this coverage report

File | Old Coverage | New Coverage | Delta
---- |:------------:|:------------:|:-----:
[pkg/kn/commands/revision/delete.go](https://storage.cloud.google.com/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-go-coverage/1227391074306101252/artifacts/line-cov.html#file0) | 75.0% | 76.5% | 1.5

knative-prow-robot: @wslyln: The following test **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
pull-knative-client-integration-tests | 0c477a5ff2100a3596ef8a92b8c2180a555c940f | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests/1227391074306101250) | `/test pull-knative-client-integration-tests`

[Full PR test history](https://gubernator.knative.dev/pr/knative_client/657). [Your PR dashboard](https://gubernator.knative.dev/pr/wslyln).

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
knative-metrics-robot: The following is the coverage report on the affected files.
Say `/test pull-knative-client-go-coverage` to re-run this coverage report

File | Old Coverage | New Coverage | Delta
---- |:------------:|:------------:|:-----:
[pkg/kn/commands/revision/delete.go](https://storage.cloud.google.com/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-go-coverage/1227411330349142019/artifacts/line-cov.html#file0) | 75.0% | 76.5% | 1.5

knative-prow-robot: @wslyln: The following test **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
pull-knative-client-integration-tests | e60b1be21b9e6f6029ff674274c460f14016903f | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests/1227411330349142018) | `/test pull-knative-client-integration-tests`

[Full PR test history](https://gubernator.knative.dev/pr/knative_client/657). [Your PR dashboard](https://gubernator.knative.dev/pr/wslyln).

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
knative-prow-robot: @wslyln: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
pull-knative-client-integration-tests | e60b1be21b9e6f6029ff674274c460f14016903f | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests/1227411330349142018) | `/test pull-knative-client-integration-tests`
pull-knative-client-integration-tests-latest-release | e60b1be21b9e6f6029ff674274c460f14016903f | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests-latest-release/1227411330349142020) | `/test pull-knative-client-integration-tests-latest-release`

[Full PR test history](https://gubernator.knative.dev/pr/knative_client/657). [Your PR dashboard](https://gubernator.knative.dev/pr/wslyln).

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
knative-prow-robot: @wslyln: The following test **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
pull-knative-client-go-coverage | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-go-coverage/1227696959440031744) | `/test pull-knative-client-go-coverage`

[Full PR test history](https://gubernator.knative.dev/pr/knative_client/657). [Your PR dashboard](https://gubernator.knative.dev/pr/wslyln).

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
knative-prow-robot: @wslyln: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
pull-knative-client-go-coverage | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-go-coverage/1227696959440031744) | `/test pull-knative-client-go-coverage`
pull-knative-client-integration-tests | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests/1227696959435837440) | `/test pull-knative-client-integration-tests`

[Full PR test history](https://gubernator.knative.dev/pr/knative_client/657). [Your PR dashboard](https://gubernator.knative.dev/pr/wslyln).

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
knative-prow-robot: @wslyln: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
pull-knative-client-go-coverage | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-go-coverage/1227696959440031744) | `/test pull-knative-client-go-coverage`
pull-knative-client-integration-tests | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests/1227696959435837440) | `/test pull-knative-client-integration-tests`
pull-knative-client-build-tests | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-build-tests/1227696959440031745) | `/test pull-knative-client-build-tests`

[Full PR test history](https://gubernator.knative.dev/pr/knative_client/657). [Your PR dashboard](https://gubernator.knative.dev/pr/wslyln).

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
knative-prow-robot: @wslyln: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
pull-knative-client-go-coverage | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-go-coverage/1227696959440031744) | `/test pull-knative-client-go-coverage`
pull-knative-client-integration-tests | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests/1227696959435837440) | `/test pull-knative-client-integration-tests`
pull-knative-client-build-tests | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-build-tests/1227696959440031745) | `/test pull-knative-client-build-tests`
pull-knative-client-integration-tests-latest-release | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests-latest-release/1227696959440031747) | `/test pull-knative-client-integration-tests-latest-release`

[Full PR test history](https://gubernator.knative.dev/pr/knative_client/657). [Your PR dashboard](https://gubernator.knative.dev/pr/wslyln).

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
knative-prow-robot: @wslyln: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
pull-knative-client-go-coverage | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-go-coverage/1227696959440031744) | `/test pull-knative-client-go-coverage`
pull-knative-client-integration-tests | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests/1227696959435837440) | `/test pull-knative-client-integration-tests`
pull-knative-client-build-tests | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-build-tests/1227696959440031745) | `/test pull-knative-client-build-tests`
pull-knative-client-integration-tests-latest-release | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests-latest-release/1227696959440031747) | `/test pull-knative-client-integration-tests-latest-release`
pull-knative-client-unit-tests | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-unit-tests/1227696959440031746) | `/test pull-knative-client-unit-tests`

[Full PR test history](https://gubernator.knative.dev/pr/knative_client/657). [Your PR dashboard](https://gubernator.knative.dev/pr/wslyln).

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
knative-prow-robot: @wslyln: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
pull-knative-client-go-coverage | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-go-coverage/1227696959440031744) | `/test pull-knative-client-go-coverage`
pull-knative-client-integration-tests | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests/1227696959435837440) | `/test pull-knative-client-integration-tests`
pull-knative-client-build-tests | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-build-tests/1227696959440031745) | `/test pull-knative-client-build-tests`
pull-knative-client-integration-tests-latest-release | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests-latest-release/1227696959440031747) | `/test pull-knative-client-integration-tests-latest-release`
pull-knative-client-unit-tests | 07f0d2ce1d7195fc8526c7fe3e2b493e8f7c6292 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-unit-tests/1227696959444226049) | `/test pull-knative-client-unit-tests`

[Full PR test history](https://gubernator.knative.dev/pr/knative_client/657). [Your PR dashboard](https://gubernator.knative.dev/pr/wslyln).

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
knative-prow-robot: @wslyln: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
pull-knative-client-integration-tests | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests/1227696959435837440) | `/test pull-knative-client-integration-tests`
pull-knative-client-build-tests | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-build-tests/1227696959440031745) | `/test pull-knative-client-build-tests`
pull-knative-client-integration-tests-latest-release | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests-latest-release/1227696959440031747) | `/test pull-knative-client-integration-tests-latest-release`
pull-knative-client-unit-tests | 07f0d2ce1d7195fc8526c7fe3e2b493e8f7c6292 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-unit-tests/1227696959444226049) | `/test pull-knative-client-unit-tests`
pull-knative-client-go-coverage | 07f0d2ce1d7195fc8526c7fe3e2b493e8f7c6292 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-go-coverage/1227696959440031748) | `/test pull-knative-client-go-coverage`

[Full PR test history](https://gubernator.knative.dev/pr/knative_client/657). [Your PR dashboard](https://gubernator.knative.dev/pr/wslyln).

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
knative-prow-robot: @wslyln: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
pull-knative-client-integration-tests | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests/1227696959435837440) | `/test pull-knative-client-integration-tests`
pull-knative-client-integration-tests-latest-release | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests-latest-release/1227696959440031747) | `/test pull-knative-client-integration-tests-latest-release`
pull-knative-client-unit-tests | 07f0d2ce1d7195fc8526c7fe3e2b493e8f7c6292 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-unit-tests/1227696959444226049) | `/test pull-knative-client-unit-tests`
pull-knative-client-go-coverage | 07f0d2ce1d7195fc8526c7fe3e2b493e8f7c6292 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-go-coverage/1227696959440031748) | `/test pull-knative-client-go-coverage`
pull-knative-client-build-tests | 07f0d2ce1d7195fc8526c7fe3e2b493e8f7c6292 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-build-tests/1227696959448420352) | `/test pull-knative-client-build-tests`

[Full PR test history](https://gubernator.knative.dev/pr/knative_client/657). [Your PR dashboard](https://gubernator.knative.dev/pr/wslyln).

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
knative-prow-robot: @wslyln: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
pull-knative-client-integration-tests | 805d2a15281e0d09ecc9d890ffb5b131eb624ac8 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests/1227696959435837440) | `/test pull-knative-client-integration-tests`
pull-knative-client-unit-tests | 07f0d2ce1d7195fc8526c7fe3e2b493e8f7c6292 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-unit-tests/1227696959444226049) | `/test pull-knative-client-unit-tests`
pull-knative-client-go-coverage | 07f0d2ce1d7195fc8526c7fe3e2b493e8f7c6292 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-go-coverage/1227696959440031748) | `/test pull-knative-client-go-coverage`
pull-knative-client-build-tests | 07f0d2ce1d7195fc8526c7fe3e2b493e8f7c6292 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-build-tests/1227696959448420352) | `/test pull-knative-client-build-tests`
pull-knative-client-integration-tests-latest-release | 07f0d2ce1d7195fc8526c7fe3e2b493e8f7c6292 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests-latest-release/1227696959461003264) | `/test pull-knative-client-integration-tests-latest-release`

[Full PR test history](https://gubernator.knative.dev/pr/knative_client/657). [Your PR dashboard](https://gubernator.knative.dev/pr/wslyln).

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
knative-metrics-robot: The following is the coverage report on the affected files.
Say `/test pull-knative-client-go-coverage` to re-run this coverage report

File | Old Coverage | New Coverage | Delta
---- |:------------:|:------------:|:-----:
[pkg/kn/commands/revision/delete.go](https://storage.cloud.google.com/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-go-coverage/1227697965905219587/artifacts/line-cov.html#file0) | 75.0% | 76.5% | 1.5

knative-prow-robot: @wslyln: The following test **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
pull-knative-client-integration-tests | c27db89f9f7ae88ff9c8029509124060347bfea7 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests/1227697965905219584) | `/test pull-knative-client-integration-tests`

[Full PR test history](https://gubernator.knative.dev/pr/knative_client/657). [Your PR dashboard](https://gubernator.knative.dev/pr/wslyln).

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
rhuss: /retest

The test should be more stable when #661 is merged.
knative-prow-robot: @wslyln: The following test **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
pull-knative-client-integration-tests | c27db89f9f7ae88ff9c8029509124060347bfea7 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests/1227707778005471232) | `/test pull-knative-client-integration-tests`

[Full PR test history](https://gubernator.knative.dev/pr/knative_client/657). [Your PR dashboard](https://gubernator.knative.dev/pr/wslyln).

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
knative-metrics-robot: The following is the coverage report on the affected files.
Say `/test pull-knative-client-go-coverage` to re-run this coverage report

File | Old Coverage | New Coverage | Delta
---- |:------------:|:------------:|:-----:
[pkg/kn/commands/revision/delete.go](https://storage.cloud.google.com/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-go-coverage/1227723448298508292/artifacts/line-cov.html#file0) | 75.0% | 76.5% | 1.5

knative-metrics-robot: The following is the coverage report on the affected files.
Say `/test pull-knative-client-go-coverage` to re-run this coverage report

File | Old Coverage | New Coverage | Delta
---- |:------------:|:------------:|:-----:
[pkg/kn/commands/revision/delete.go](https://storage.cloud.google.com/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-go-coverage/1227725088699191297/artifacts/line-cov.html#file0) | 75.0% | 76.5% | 1.5

knative-prow-robot: @wslyln: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
pull-knative-client-integration-tests | c27db89f9f7ae88ff9c8029509124060347bfea7 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests/1227707778005471232) | `/test pull-knative-client-integration-tests`
pull-knative-client-build-tests | 65a88fa57b18a00f1490d8f1281d3e95bcb0c32d | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-build-tests/1227725088707579905) | `/test pull-knative-client-build-tests`

[Full PR test history](https://gubernator.knative.dev/pr/knative_client/657). [Your PR dashboard](https://gubernator.knative.dev/pr/wslyln).

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
rhuss: ```
\u0026#xA;# knative.dev/client/test/e2e [knative.dev/client/test/e2e.test]\u0026#xA;test/e2e/revision_test.go:107:96: undefined: nonexistRevision\u0026#xA;test/e2e/revision_test.go:111:73: undefined: nonexistRevision\u0026#xA;
```
knative-prow-robot: @wslyln: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
pull-knative-client-build-tests | 65a88fa57b18a00f1490d8f1281d3e95bcb0c32d | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-build-tests/1227725088707579905) | `/test pull-knative-client-build-tests`
pull-knative-client-integration-tests | 65a88fa57b18a00f1490d8f1281d3e95bcb0c32d | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests/1227725088699191296) | `/test pull-knative-client-integration-tests`

[Full PR test history](https://gubernator.knative.dev/pr/knative_client/657). [Your PR dashboard](https://gubernator.knative.dev/pr/wslyln).

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
knative-prow-robot: @wslyln: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
pull-knative-client-build-tests | 65a88fa57b18a00f1490d8f1281d3e95bcb0c32d | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-build-tests/1227725088707579905) | `/test pull-knative-client-build-tests`
pull-knative-client-integration-tests | 65a88fa57b18a00f1490d8f1281d3e95bcb0c32d | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests/1227725088699191296) | `/test pull-knative-client-integration-tests`
pull-knative-client-integration-tests-latest-release | 65a88fa57b18a00f1490d8f1281d3e95bcb0c32d | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests-latest-release/1227725088707579904) | `/test pull-knative-client-integration-tests-latest-release`

[Full PR test history](https://gubernator.knative.dev/pr/knative_client/657). [Your PR dashboard](https://gubernator.knative.dev/pr/wslyln).

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
knative-metrics-robot: The following is the coverage report on the affected files.
Say `/test pull-knative-client-go-coverage` to re-run this coverage report

File | Old Coverage | New Coverage | Delta
---- |:------------:|:------------:|:-----:
[pkg/kn/commands/revision/delete.go](https://storage.cloud.google.com/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-go-coverage/1227757420294443010/artifacts/line-cov.html#file0) | 75.0% | 76.5% | 1.5

knative-prow-robot: @wslyln: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
pull-knative-client-integration-tests-latest-release | 65a88fa57b18a00f1490d8f1281d3e95bcb0c32d | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests-latest-release/1227725088707579904) | `/test pull-knative-client-integration-tests-latest-release`
pull-knative-client-integration-tests | 696b2de857f01fadeba4e7923c76731154ca19c1 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests/1227757420298637312) | `/test pull-knative-client-integration-tests`

[Full PR test history](https://gubernator.knative.dev/pr/knative_client/657). [Your PR dashboard](https://gubernator.knative.dev/pr/wslyln).

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
knative-prow-robot: @wslyln: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
pull-knative-client-integration-tests | 696b2de857f01fadeba4e7923c76731154ca19c1 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests/1227757420298637312) | `/test pull-knative-client-integration-tests`
pull-knative-client-integration-tests-latest-release | 696b2de857f01fadeba4e7923c76731154ca19c1 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests-latest-release/1227757420294443009) | `/test pull-knative-client-integration-tests-latest-release`

[Full PR test history](https://gubernator.knative.dev/pr/knative_client/657). [Your PR dashboard](https://gubernator.knative.dev/pr/wslyln).

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
wslyln: fixed the tests, and actually checked the tests TestRevisions before pushing :)
knative-metrics-robot: The following is the coverage report on the affected files.
Say `/test pull-knative-client-go-coverage` to re-run this coverage report

File | Old Coverage | New Coverage | Delta
---- |:------------:|:------------:|:-----:
[pkg/kn/commands/revision/delete.go](https://storage.cloud.google.com/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-go-coverage/1228019272433799169/artifacts/line-cov.html#file0) | 75.0% | 76.5% | 1.5

knative-prow-robot: @wslyln: The following tests **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
pull-knative-client-integration-tests-latest-release | 696b2de857f01fadeba4e7923c76731154ca19c1 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests-latest-release/1227757420294443009) | `/test pull-knative-client-integration-tests-latest-release`
pull-knative-client-integration-tests | 676f48410a8f7210e28ea9bd316945916f2f5e49 | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests/1228019272433799173) | `/test pull-knative-client-integration-tests`

[Full PR test history](https://gubernator.knative.dev/pr/knative_client/657). [Your PR dashboard](https://gubernator.knative.dev/pr/wslyln).

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
knative-metrics-robot: The following is the coverage report on the affected files.
Say `/test pull-knative-client-go-coverage` to re-run this coverage report

File | Old Coverage | New Coverage | Delta
---- |:------------:|:------------:|:-----:
[pkg/kn/commands/revision/delete.go](https://storage.cloud.google.com/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-go-coverage/1228040160998854658/artifacts/line-cov.html#file0) | 75.0% | 76.5% | 1.5

knative-prow-robot: @wslyln: The following test **failed**, say `/retest` to rerun all failed tests:

Test name | Commit | Details | Rerun command
--- | --- | --- | ---
pull-knative-client-integration-tests | cf2360352ed62b6354fd03852182efe371ff8c7d | [link](https://prow.knative.dev/view/gcs/knative-prow/pr-logs/pull/knative_client/657/pull-knative-client-integration-tests/1228040160998854659) | `/test pull-knative-client-integration-tests`

[Full PR test history](https://gubernator.knative.dev/pr/knative_client/657). [Your PR dashboard](https://gubernator.knative.dev/pr/wslyln).

<details>

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
</details>
<!-- test report -->
wslyln: /retest
knative-prow-robot: [APPROVALNOTIFIER] This PR is **APPROVED**

This pull-request has been approved by: *<a href="https://github.com/knative/client/pull/657#pullrequestreview-358640216" title="Approved">maximilien</a>*, *<a href="https://github.com/knative/client/pull/657#" title="Author self-approved">wslyln</a>*

The full list of commands accepted by this bot can be found [here](https://go.k8s.io/bot-commands?repo=knative%2Fclient).

The pull request process is described [here](https://git.k8s.io/community/contributors/guide/owners.md#the-code-review-process)

<details >
Needs approval from an approver in each of these files:

- ~~[OWNERS](https://github.com/knative/client/blob/master/OWNERS)~~ [maximilien]

Approvers can indicate their approval by writing `/approve` in a comment
Approvers can cancel approval by writing `/approve cancel` in a comment
</details>
<!-- META={"approvers":[]} -->
knative-prow-robot: [APPROVALNOTIFIER] This PR is **APPROVED**

This pull-request has been approved by: *<a href="https://github.com/knative/client/pull/657#pullrequestreview-358640216" title="Approved">maximilien</a>*, *<a href="https://github.com/knative/client/pull/657#pullrequestreview-358737521" title="Approved">navidshaikh</a>*, *<a href="https://github.com/knative/client/pull/657#" title="Author self-approved">wslyln</a>*

The full list of commands accepted by this bot can be found [here](https://go.k8s.io/bot-commands?repo=knative%2Fclient).

The pull request process is described [here](https://git.k8s.io/community/contributors/guide/owners.md#the-code-review-process)

<details >
Needs approval from an approver in each of these files:

- ~~[OWNERS](https://github.com/knative/client/blob/master/OWNERS)~~ [maximilien,navidshaikh]

Approvers can indicate their approval by writing `/approve` in a comment
Approvers can cancel approval by writing `/approve cancel` in a comment
</details>
<!-- META={"approvers":[]} -->

Patch:
diff --git a/CHANGELOG.adoc b/CHANGELOG.adoc
index fd138675..c7d3bf4c 100644
--- a/CHANGELOG.adoc
+++ b/CHANGELOG.adoc
@@ -20,6 +20,10 @@
 |===
 | | Description | PR
 
+| 🧽
+| Support multiple revisions on `kn revision delete`
+| https://github.com/knative/client/pull/657[#657]
+
 | 🎁
 | Add human readable `kn route describe`
 | https://github.com/knative/client/pull/643[#643]
@@ -48,6 +52,7 @@
 | 🐛
 | Fix `--image` flag to only allow single occurence
 | https://github.com/knative/client/pull/647[#647]
+|===
 
 ## v0.12.0 (2020-01-29)
 
diff --git a/pkg/kn/commands/revision/delete.go b/pkg/kn/commands/revision/delete.go
index eaa3459b..3a856004 100644
--- a/pkg/kn/commands/revision/delete.go
+++ b/pkg/kn/commands/revision/delete.go
@@ -32,8 +32,8 @@ func NewRevisionDeleteCommand(p *commands.KnParams) *cobra.Command {
   # Delete a revision 'svc1-abcde' in default namespace
   kn revision delete svc1-abcde`,
 		RunE: func(cmd *cobra.Command, args []string) error {
-			if len(args) != 1 {
-				return errors.New("'revision delete' requires the revision name given as single argument")
+			if len(args) < 1 {
+				return errors.New("'kn revision delete' requires one or more revision name")
 			}
 			namespace, err := p.GetNamespace(cmd)
 			if err != nil {
@@ -43,11 +43,15 @@ func NewRevisionDeleteCommand(p *commands.KnParams) *cobra.Command {
 			if err != nil {
 				return err
 			}
-			err = client.DeleteRevision(args[0])
-			if err != nil {
-				return err
+
+			for _, name := range args {
+				err = client.DeleteRevision(name)
+				if err != nil {
+					fmt.Fprintf(cmd.OutOrStdout(), "%s.\n", err)
+				} else {
+					fmt.Fprintf(cmd.OutOrStdout(), "Revision '%s' deleted in namespace '%s'.\n", name, namespace)
+				}
 			}
-			fmt.Fprintf(cmd.OutOrStdout(), "Revision '%s' successfully deleted in namespace '%s'.\n", args[0], namespace)
 			return nil
 		},
 	}


Test patch:
diff --git a/pkg/kn/commands/revision/delete_test.go b/pkg/kn/commands/revision/delete_test.go
index 10b1ff74..53ac70dd 100644
--- a/pkg/kn/commands/revision/delete_test.go
+++ b/pkg/kn/commands/revision/delete_test.go
@@ -60,3 +60,22 @@ func TestRevisionDelete(t *testing.T) {
 	}
 	assert.Check(t, util.ContainsAll(output, "Revision", revName, "deleted", "namespace", commands.FakeNamespace))
 }
+
+func TestMultipleRevisionDelete(t *testing.T) {
+	revName1 := "foo-12345"
+	revName2 := "foo-67890"
+	revName3 := "foo-abcde"
+	action, _, output, err := fakeRevisionDelete([]string{"revision", "delete", revName1, revName2, revName3})
+	if err != nil {
+		t.Error(err)
+		return
+	}
+	if action == nil {
+		t.Errorf("No action")
+	} else if !action.Matches("delete", "revisions") {
+		t.Errorf("Bad action %v", action)
+	}
+	assert.Check(t, util.ContainsAll(output, "Revision", revName1, "deleted", "namespace", commands.FakeNamespace))
+	assert.Check(t, util.ContainsAll(output, "Revision", revName2, "deleted", "namespace", commands.FakeNamespace))
+	assert.Check(t, util.ContainsAll(output, "Revision", revName3, "deleted", "namespace", commands.FakeNamespace))
+}
diff --git a/test/e2e/revision_test.go b/test/e2e/revision_test.go
index 534636a7..ea765e5a 100644
--- a/test/e2e/revision_test.go
+++ b/test/e2e/revision_test.go
@@ -38,7 +38,8 @@ func TestRevision(t *testing.T) {
 	})
 
 	t.Run("describe revision from hello service with print flags", func(t *testing.T) {
-		test.revisionDescribeWithPrintFlags(t, "hello")
+		revName := test.findRevision(t, "hello")
+		test.revisionDescribeWithPrintFlags(t, revName)
 	})
 
 	t.Run("update hello service and increase the count of configuration generation", func(t *testing.T) {
@@ -50,7 +51,19 @@ func TestRevision(t *testing.T) {
 	})
 
 	t.Run("delete latest revision from hello service and return no error", func(t *testing.T) {
-		test.revisionDelete(t, "hello")
+		revName := test.findRevision(t, "hello")
+		test.revisionDelete(t, revName)
+	})
+
+	t.Run("delete three revisions with one revision a nonexistent", func(t *testing.T) {
+		// increase count to 2 revisions
+		test.serviceUpdate(t, "hello", []string{"--env", "TARGET=kn", "--port", "8888"})
+
+		existRevision1 := test.findRevisionByGeneration(t, "hello", 1)
+		existRevision2 := test.findRevisionByGeneration(t, "hello", 2)
+		nonexistRevision := "hello-nonexist"
+
+		test.revisionMultipleDelete(t, []string{existRevision1, existRevision2, nonexistRevision})
 	})
 
 	t.Run("delete hello service and return no error", func(t *testing.T) {
@@ -76,18 +89,31 @@ func (test *e2eTest) revisionListWithService(t *testing.T, serviceNames ...strin
 	}
 }
 
-func (test *e2eTest) revisionDelete(t *testing.T, serviceName string) {
-	revName := test.findRevision(t, serviceName)
-
+func (test *e2eTest) revisionDelete(t *testing.T, revName string) {
 	out, err := test.kn.RunWithOpts([]string{"revision", "delete", revName}, runOpts{})
 	assert.NilError(t, err)
 
 	assert.Check(t, util.ContainsAll(out, "Revision", revName, "deleted", "namespace", test.kn.namespace))
 }
 
-func (test *e2eTest) revisionDescribeWithPrintFlags(t *testing.T, serviceName string) {
-	revName := test.findRevision(t, serviceName)
+func (test *e2eTest) revisionMultipleDelete(t *testing.T, revisionNames []string) {
+	existRevision1 := revisionNames[0]
+	existRevision2 := revisionNames[1]
+	nonexistRevision := revisionNames[2]
+
+	out, err := test.kn.RunWithOpts([]string{"revision", "list"}, runOpts{NoNamespace: false})
+	assert.NilError(t, err)
+	assert.Check(t, strings.Contains(out, existRevision1), "Required revision1 does not exist")
+	assert.Check(t, strings.Contains(out, existRevision2), "Required revision2 does not exist")
+
+	out, err = test.kn.RunWithOpts([]string{"revision", "delete", existRevision1, existRevision2, nonexistRevision}, runOpts{NoNamespace: false})
+
+	assert.Check(t, util.ContainsAll(out, "Revision", existRevision1, "deleted", "namespace", test.kn.namespace), "Failed to get 'deleted' first revision message")
+	assert.Check(t, util.ContainsAll(out, "Revision", existRevision2, "deleted", "namespace", test.kn.namespace), "Failed to get 'deleted' second revision message")
+	assert.Check(t, util.ContainsAll(out, "revisions.serving.knative.dev", nonexistRevision, "not found"), "Failed to get 'not found' error")
+}
 
+func (test *e2eTest) revisionDescribeWithPrintFlags(t *testing.T, revName string) {
 	out, err := test.kn.RunWithOpts([]string{"revision", "describe", revName, "-o=name"}, runOpts{})
 	assert.NilError(t, err)