migrate: ignore resources that cannot be listed and updated

[enj]

This change updates FindAllCanonicalResources to check the verbs asserted by discovery for listing and updating capability. This prevents us from trying to update aggregated resources such as pod metrics.

Bug 1631517

Signed-off-by: Monis Khan [email protected]

/kind bug
/assign @deads2k
/cc @jupierce

[enj]

/cherrypick release-3.11

[openshift-cherrypick-robot]

@enj: once the present PR merges, I will cherry-pick it on top of release-3.11 in a new PR and assign it to you.

In response to this:

/cherrypick release-3.11

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[enj]

/cherrypick release-3.10

[openshift-cherrypick-robot]

@enj: once the present PR merges, I will cherry-pick it on top of release-3.10 in a new PR and assign it to you.

In response to this:

/cherrypick release-3.10

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[enj]

/cherrypick release-3.9

[openshift-cherrypick-robot]

@enj: once the present PR merges, I will cherry-pick it on top of release-3.9 in a new PR and assign it to you.

In response to this:

/cherrypick release-3.9

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[deads2k]

/lgtm

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: deads2k, enj

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:

• pkg/oc/OWNERS [deads2k,enj]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[openshift-cherrypick-robot]

@enj: #21071 failed to apply on top of branch "release-3.9":

error: Failed to merge in the changes.
Using index info to reconstruct a base tree...
A	pkg/oc/cli/admin/migrate/migrator.go
Falling back to patching base and 3-way merge...
Auto-merging pkg/oc/admin/migrate/migrator.go
CONFLICT (content): Merge conflict in pkg/oc/admin/migrate/migrator.go
Patch failed at 0001 migrate: ignore resources that cannot be listed and updated

In response to this:

/cherrypick release-3.9

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[openshift-cherrypick-robot]

@enj: new pull request created: #21075

In response to this:

/cherrypick release-3.11

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[openshift-cherrypick-robot]

@enj: #21071 failed to apply on top of branch "release-3.10":

error: Failed to merge in the changes.
Using index info to reconstruct a base tree...
A	pkg/oc/cli/admin/migrate/migrator.go
Falling back to patching base and 3-way merge...
Auto-merging pkg/oc/admin/migrate/migrator.go
CONFLICT (content): Merge conflict in pkg/oc/admin/migrate/migrator.go
Patch failed at 0001 migrate: ignore resources that cannot be listed and updated

In response to this:

/cherrypick release-3.10

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.