-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[cherry-pick] Update tektoncd/plumbing. #4009
[cherry-pick] Update tektoncd/plumbing. #4009
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: sbwsg 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:
Approvers can indicate their approval by writing |
/test check-pr-has-kind-label |
hey @jerop! tiny pedantic request: could you explain a bit more (ideally in the commit message) about why this is needed and how it fixes things? you linked to tektoncd/plumbing@3c5af40 and the explanation there makes this sound like a cleanup that maybe will in the future be relevant the commit said:
so im guessing the situation might be that we are now using 1.19 so this is a problem? or is it something else? Thanks!! 🙏 |
3919933
to
d683fae
Compare
thanks for the reviews!! @bobcatfish I've added more detail to the commit messages and PR description:
|
This is pulls in tektoncd/plumbing@3c5af40 to fix integration tests. (cherry picked from commit 55ae856) We need to cherry pick this change into v0.24 to run integration tests for a Workspaces in Sidecars fix. We need to fix the Workspaces in Sidecars to be serialized as workspaces not Workspaces. This is change is made in tektoncd#4008. Its integration tests are failing because basic authentication was removed for GKE cluster versions >= 1.19 so clusters cannot be created with basic authentication enabled. So we need to cherry pick this change to disable basic auth and create clusters to run the tests for the Workspaces in Sidecars fix.
d683fae
to
90395ab
Compare
Note that this is also due to the fact that, the version of k8s we use for a given release is not tied to the release (aka if we were to release 0.10 today, we would run against a 1.19, whereas at the time of the release we were using 1.15 or something). /lgtm |
In response to this:
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. |
Changes
This is pulls in tektoncd/plumbing@3c5af40 to fix integration tests.
(cherry picked from commit 55ae856)
We need to cherry pick this change into v0.24 to run integration tests for a Workspaces in Sidecars fix. We need to fix the Workspaces in Sidecars to be serialized as workspaces not Workspaces. This is change is made in #4008. Its integration tests are failing because basic authentication was removed for GKE cluster versions >= 1.19 so clusters cannot be created with basic authentication enabled. So we need to cherry pick this change to disable basic auth and create clusters to run the tests for the Workspaces in Sidecars fix.
/kind bug
Submitter Checklist
As the author of this PR, please check off the items in this checklist:
functionality, content, code)
Release Notes