Bug 1677545 - Correctly handle last chunk on incremental load

[spadgett]

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

We weren't calling bulkAddToList on the last chunk of data since response.metadata.continue wasn't defined.

@alecmerdler Let me know if this looks right. I'd your input on how to change k8s-actions-spec.ts to check this.

/cc @alecmerdler @rhamilto

[spadgett]

/retest

[spadgett]

/retest

[spadgett]

lgtm, tagging since all but 3 lines of this code are from @alecmerdler :)

(see second commit)

[spadgett]

/retest

[spadgett]

/retest

[spadgett]

/cherrypick release-3.11

[openshift-cherrypick-robot]

@spadgett: #1204 failed to apply on top of branch "release-3.11":

error: Failed to merge in the changes.
Using index info to reconstruct a base tree...
M	frontend/public/module/k8s/k8s-actions.ts
Falling back to patching base and 3-way merge...
Auto-merging frontend/public/module/k8s/k8s-actions.ts
CONFLICT (content): Merge conflict in frontend/public/module/k8s/k8s-actions.ts
Patch failed at 0001 Bug 1677545 - Correctly handle last chunk on incremental load

Details

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.