Skip to content

Bump sigs.k8s.io/controller-runtime from 0.9.0-beta.5 to 0.9.0 #2487

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

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Jun 9, 2021

Bumps sigs.k8s.io/controller-runtime from 0.9.0-beta.5 to 0.9.0.

Release notes

Sourced from sigs.k8s.io/controller-runtime's releases.

v0.9.0

Changes since v0.8.3

🚨 Significant Changes 🚨

Envtest Refactor & Support for 1.20+ Clusters (#1486)

This adds support for newer Kubernetes clusters, particularly those that require the use of authentication & the secure port, as well as some refactors & deprecation.

By and large, these changes should be transparent for most users.

As a result of the removal of the insecure port in newer k8s versions, authentication & authorization are now on by default in envtest (and the secure port is used by default). The built-in REST config now returns an admin account, meaning perceived impact should be minimal in most cases -- if you're using the built-in Environment.Config, things should just work.

However, if you manually construct new REST configs from raw URLs, things will break. You'll need to migrate to using Environment.AddUser instead.

Additionally, a number of options on ControlPlane & APIServer around the secure & insecure ports are deprecated -- see the docs for more details.

Envtest Setup Tool (#1488)

A new tool has been introduced for setting up envtest binaries at sigs.k8s.io/controller-runtime/tools/setup-envtest. This replaces the existing hack/setup-envtest.sh script, so if you were depending on that, you should switch. See the docs for more information.

Kubernetes Dependencies

Now at v1.21.1.

⚠️ Breaking Changes

  • Update Kubernetes v1.21 dependencies and use Go 1.16 (#1389) moderate impact: if you're on an old Go version, it's time to upgrade 😉
  • Envtest should modify CRDs appropriately when using webhooks (#1525) minimal impact: one of the methods on WebhookInstallOptions had a signature change. Mostly internal, expected that this method is unlikely to be called directly.
  • Allow setting NewClientFunc w/o implementing an interface (#1473) moderate impact: If you're constructing a custom client, this'll change how you do so, but should make it easier in the long run.
  • Support global controller options in component config (#1371) minimal impact: adds new method to Manager interface, generally only implemented by CR)
  • Fix race in the delegating logger (#1361) minimal impact: exposed piece of internals was changed)
  • Fakeclient: Handle Finalizers (#869) moderate impact: Finalizers in the fakeclient now behave similiar to the kube api, i.E. an object with a finalizer will not actually be deleted upon Delete and Updating an object with a DeletionTimestamp to remove the finalizer will result in deletion

✨ New Features

  • Structured args in Testing (#1541) -- the use of templated args (Environment.KubeAPIServerFlags) is now deprecated -- use GetAPIServer().Configure() instead.
  • Clean up pkg/internal/testing (#1540)
  • Cleanup Webhook server setup (#1504)
  • EnvTest Binaries Setup Tool (#1488)
  • Add SelectorsByObject option to cache (#1435)
  • Simple helper for unmanaged webhook server (#1429)
  • pkg/log/zap: clarify zap level vs. logr verbosity (#1485)
  • Add ClientWithWatch for use in CLIs (#1460)
  • Add client.StrategicMergeFrom (#1406)

... (truncated)

Commits
  • a905949 Merge pull request #1552 from DirectXMan12/docs/flags-deprecation
  • 2c1b1dd Mark all templated flag pieces as deprecated
  • abb406d Merge pull request #1551 from DirectXMan12/docs/fix-clientbuilder-ref
  • da8716a Remove reference to ClientBuilder in Cluster docs
  • d4ecab6 Merge pull request #1550 from randomvariable/dependencies-06-07
  • 96d15b7 Update dependencies for 2021/06/07
  • fca94d5 Merge pull request #1542 from DirectXMan12/infra/add-missing-license-headers
  • 66a2e62 Make sure all files have a copyright notice
  • ab7825e Merge pull request #1486 from DirectXMan12/feature/envtest-secure-port
  • e77a2fc Disable lll
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [sigs.k8s.io/controller-runtime](https://github.com/kubernetes-sigs/controller-runtime) from 0.9.0-beta.5 to 0.9.0.
- [Release notes](https://github.com/kubernetes-sigs/controller-runtime/releases)
- [Commits](kubernetes-sigs/controller-runtime@v0.9.0-beta.5...v0.9.0)

---
updated-dependencies:
- dependency-name: sigs.k8s.io/controller-runtime
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot added dependencies Pull requests that update a dependency file go Pull requests that update Go code labels Jun 9, 2021
@k8s-ci-robot
Copy link
Contributor

@dependabot[bot]: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

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.

@k8s-ci-robot k8s-ci-robot added do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Jun 9, 2021
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign timothysc after the PR has been reviewed.
You can assign the PR to them by writing /assign @timothysc in a comment when ready.

The full list of commands accepted by this bot can be found 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

@k8s-ci-robot
Copy link
Contributor

Hi @dependabot[bot]. Thanks for your PR.

I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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.

@k8s-ci-robot k8s-ci-robot added needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Jun 9, 2021
@randomvariable
Copy link
Member

/close

Getting bumped in #2485

@k8s-ci-robot
Copy link
Contributor

@randomvariable: Closed this PR.

In response to this:

/close

Getting bumped in #2485

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.

@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Jun 9, 2021

OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting @dependabot ignore this major version or @dependabot ignore this minor version. You can also ignore all major, minor, or patch releases for a dependency by adding an ignore condition with the desired update_types to your config file.

If you change your mind, just re-open this PR and I'll resolve any conflicts on it.

@dependabot dependabot bot deleted the dependabot/go_modules/sigs.k8s.io/controller-runtime-0.9.0 branch June 9, 2021 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. dependencies Pull requests that update a dependency file do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. go Pull requests that update Go code needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants