Remove pkg/pool

[imjasonh]

Fixes #2556

Changes

🗑️ Remove pkg/pool

/kind cleanup

Release Note

pkg/pool is removed, use errgroup instead

Docs

NONE

cc @dprotaso

[codecov]

Codecov Report

Merging #2568 (dd67813) into main (7b8b060) will decrease coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2568      +/-   ##
==========================================
- Coverage   81.40%   81.32%   -0.09%     
==========================================
  Files         163      162       -1     
  Lines        9740     9695      -45     
==========================================
- Hits         7929     7884      -45     
  Misses       1574     1574              
  Partials      237      237              

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[dprotaso]

/lgtm
/approve
/override Downstream Tekton / Unit Test (tektoncd/chains) (pull_request)
/override Downstream Tekton / Unit Test (tektoncd/pipeline) (pull_request)

[knative-prow]

@dprotaso: /override requires a failed status context, check run or a prowjob name to operate on.
The following unknown contexts/checkruns were given:

• Downstream Tekton / Unit Test (tektoncd/chains) (pull_request)
• Downstream Tekton / Unit Test (tektoncd/pipeline) (pull_request)

Only the following failed contexts/checkruns were expected:

• EasyCLA
• Unit Test (tektoncd/chains)
• Unit Test (tektoncd/pipeline)
• build-tests_pkg_main
• integration-tests_pkg_main
• tide
• unit-tests_pkg_main

In response to this:

/lgtm
/approve
/override Downstream Tekton / Unit Test (tektoncd/chains) (pull_request)
/override Downstream Tekton / Unit Test (tektoncd/pipeline) (pull_request)

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.

[dprotaso]

/override Unit Test (tektoncd/chains)
/override Unit Test (tektoncd/pipeline)

[knative-prow]

@dprotaso: Overrode contexts on behalf of dprotaso: Unit Test (tektoncd/chains), Unit Test (tektoncd/pipeline)

In response to this:

/override Unit Test (tektoncd/chains)
/override Unit Test (tektoncd/pipeline)

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.

[knative-prow]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dprotaso, imjasonh

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:

• pool/OWNERS [dprotaso]

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

[dprotaso]

override Tekton - since I assume you'll fix those :)

[dprotaso]

/retest

flake - TestDialTLSWithBackoffSuccess

[imjasonh]

Chains fix: tektoncd/chains#517 -- bumps cosign and knative/pkg deps
Pipeline fix: tektoncd/pipeline#5311 -- bumps knative/pkg dep, accounts for new changeset.Get behavior