Skip to content

K8s 1.24 #318

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

Closed
wants to merge 9 commits into from
Closed

K8s 1.24 #318

wants to merge 9 commits into from

Conversation

perdasilva
Copy link
Contributor

No description provided.

@openshift-ci openshift-ci bot requested review from anik120 and benluddy June 20, 2022 06:33
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 20, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: perdasilva

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 20, 2022
perdasilva and others added 7 commits June 20, 2022 10:10
Signed-off-by: perdasilva <[email protected]>
Upstream-repository: operator-lifecycle-manager
Upstream-commit: 9ced412f3e263b8827680dc0ad3477327cd9a508
Signed-off-by: perdasilva <[email protected]>
- Updates tests to use common Kube and Runtime clients generated at
startup time rather than having them re-generated in each test at
runtime.
- Closes #2570

Signed-off-by: Noah Sapse <[email protected]>
Upstream-repository: operator-lifecycle-manager
Upstream-commit: c610a3eb2f221a2cac8563b68aa4f98e61f888cc
…fix scoped client (#2794)

* fix(vendor): bump k8s version to 1.24 and go version to 1.18

Signed-off-by: Vu Dinh <[email protected]>

* Update clock package from apimachinery to utils

The apimachinery/pkg/util/clock is deprecated and all clock utils is
on k8s.io/utils/clock repo instead.

Signed-off-by: Vu Dinh <[email protected]>

* fix(go.mod): pin opentelemetry packages into specific versions

Signed-off-by: Vu Dinh <[email protected]>
Co-authored-by: timflannagan <[email protected]>

* fix(scoped): update scoped client library to handle token secret

In k8s 1.24, token secret is no longer referenced in ServiceAccount.
By listing all secrets in the namespace and then filter them with
SA name via kubernetes.io/service-account.name annotation, the
token secret can be retrieved successfully.

Signed-off-by: Vu Dinh <[email protected]>

* fix(e2e): Fix several RBAC-related e2e test cases

Create token secret for ServiceAccount to ensure those SA
is valid for scoped client use.

Signed-off-by: Vu Dinh <[email protected]>

Upstream-repository: operator-lifecycle-manager
Upstream-commit: fd9017345aa0fbd7aaed7f0937ff23fca33a9df0

Co-authored-by: timflannagan <[email protected]>

Signed-off-by: perdasilva <[email protected]>
* bump k8s deps to 1.24 and go version to 1.18

Signed-off-by: Bryce Palmer <[email protected]>

* revert Go 1.18 bump

Signed-off-by: Bryce Palmer <[email protected]>

* revert go 1.18 in workflows

Signed-off-by: Bryce Palmer <[email protected]>

Upstream-repository: api
Upstream-commit: 725ffcedf4774ea71acd9d4a5abfc0f93d1968ed
* changes to bump to k8s 1.24

Signed-off-by: Bryce Palmer <[email protected]>

* changes to bump to k8s 1.24

Signed-off-by: Bryce Palmer <[email protected]>

* update prometheus CRDs to have preserveUnknownFields: false

Signed-off-by: Bryce Palmer <[email protected]>

Upstream-repository: operator-registry
Upstream-commit: 043db4f20c13db8535f6991dd9afd590e7427228
Signed-off-by: Jordan Keister <[email protected]>

Upstream-repository: operator-registry
Upstream-commit: 0899512627ff2fbf6f65e213f0358429959d8963
@perdasilva perdasilva force-pushed the k8s-1.24 branch 3 times, most recently from 3ce9495 to a6de3ea Compare June 20, 2022 14:18
…lakiness (openshift#245)

* Revert "fix removed_apis_test"

This reverts commit beb9613e0dc43e74ddcc583dfca3e59dd19c2660.

* refactor removed_apis_test to test on sorted error strings to remove flakiness

Signed-off-by: perdasilva <[email protected]>
Upstream-repository: api
Upstream-commit: cb9dc0ee2c6a5c31e04ce1ab0c2eee72d9125ffb
@perdasilva perdasilva force-pushed the k8s-1.24 branch 2 times, most recently from 2579284 to 41dec0e Compare June 20, 2022 15:56
Signed-off-by: perdasilva <[email protected]>
@perdasilva
Copy link
Contributor Author

/restest e2e-gcp-console-olm

@perdasilva
Copy link
Contributor Author

/retest ci/prow/e2e-gcp-console-olm

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 20, 2022

@perdasilva: The /retest command does not accept any targets.
The following commands are available to trigger required jobs:

  • /test e2e-gcp
  • /test e2e-gcp-console-olm
  • /test e2e-gcp-olm
  • /test e2e-upgrade
  • /test images
  • /test unit-api
  • /test unit-olm
  • /test unit-psm
  • /test unit-registry
  • /test verify

The following commands are available to trigger optional jobs:

  • /test e2e-gcp-olm-flaky

Use /test all to run all jobs.

In response to this:

/retest ci/prow/e2e-gcp-console-olm

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.

@perdasilva
Copy link
Contributor Author

/retest-required

@perdasilva
Copy link
Contributor Author

/retest required

@perdasilva
Copy link
Contributor Author

/retest-required

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 20, 2022

@perdasilva: The /retest command does not accept any targets.
The following commands are available to trigger required jobs:

  • /test e2e-gcp
  • /test e2e-gcp-console-olm
  • /test e2e-gcp-olm
  • /test e2e-upgrade
  • /test images
  • /test unit-api
  • /test unit-olm
  • /test unit-psm
  • /test unit-registry
  • /test verify

The following commands are available to trigger optional jobs:

  • /test e2e-gcp-olm-flaky

Use /test all to run all jobs.

In response to this:

/retest required

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-ci
Copy link
Contributor

openshift-ci bot commented Jun 21, 2022

@perdasilva: all tests passed!

Full PR test history. Your PR dashboard.

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. I understand the commands that are listed here.

@openshift-ci openshift-ci bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 21, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 21, 2022

@perdasilva: PR needs rebase.

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.

@perdasilva perdasilva closed this Jun 21, 2022
openshift-bot pushed a commit to openshift-bot/operator-framework-olm that referenced this pull request Mar 15, 2024
…penshift#318)

Bumps google.golang.org/protobuf from 1.31.0 to 1.33.0.

Upstream-repository: api
Upstream-commit: 4454fc8910dbe30bee53c9cfb0764fa46cb0c5d5

---
updated-dependencies:
- dependency-name: google.golang.org/protobuf
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants