Merge pull request #18514 from juanvallejo/jvallejo/handle-watch-multiple-reqs

Automatic merge from submit-queue.

UPSTREAM: 59506: fix watch on multiple reqs

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1537789

`oc get <resource> --watch` only supports watching a single resource kind at a time.
This check fails if more than one resource `Info` is returned.

When dealing with large quantities of a single resource kind, or an amount that exceeds the value of `--chunk-size`, more than one request is made to the server causing a resource `Info` to be created for each of the requests, ultimately causing the above check to fail even though we are dealing with the same type of resource.

This patch modifies that check to take into account the GVKs of all infos returned, and only fail if at least one differs.

cc @deads2k @jeremyeder

Author: openshift-merge-robot

test/cmd/get.sh

@@ -37,5 +37,11 @@ os::cmd::expect_success_and_not_text 'oc get users test-user-1' "customlabel=tru
 # test structured and unstructured resources print generically without panic
 os::cmd::expect_success_and_text 'oc get projectrequests -o yaml' 'status: Success'
 os::cmd::expect_success_and_text 'oc get projectrequests,svc,pod -o yaml' 'kind: List'
+# test --wacth does not result in an error when a resource list is served in multiple chunks
+os::cmd::expect_success 'oc create cm cmone'
+os::cmd::expect_success 'oc create cm cmtwo'
+os::cmd::expect_success 'oc create cm cmthree'
+os::cmd::expect_success_and_not_text 'oc get configmap --chunk-size=1 --watch --request-timeout=1s' 'watch is only supported on individual resources'
+os::cmd::expect_success_and_not_text 'oc get configmap --chunk-size=1 --watch-only --request-timeout=1s' 'watch is only supported on individual resources'
 echo "oc get: ok"
 os::test::junit::declare_suite_end