forked from kubernetes-sigs/controller-runtime
-
Notifications
You must be signed in to change notification settings - Fork 0
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
[pull] master from kubernetes-sigs:master #143
Open
pull
wants to merge
462
commits into
alexeldeib:master
Choose a base branch
from
kubernetes-sigs:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Bump k8s.io packages to v1.24.2 * go mod tidy
* bump github.com/evanphx/json-patch to v5.6.0 * bump github.com/fsnotify/fsnotify to v1.5.4 * bump github.com/go-logr/logr to v1.2.3 * bump github.com/go-logr/zapr to v1.2.3 * bump github.com/google/go-cmp to v0.5.8 * bump go.uber.org/zap to v1.21.0 * bump github.com/prometheus/client_golang to v1.12.2 * bump github.com/onsi/gomega to v1.19.0 * bump golang.org/x/sys to v0.0.0-20220615213510-4f61da869c0c * bump golang.org/x/time to v0.0.0-20220609170525-579cf78fd858 * import evanphx/json-patch via module
📖 nit: fix in client package documentation
Only configure the local warning handler.
🐛 Remove no-op clientgo reflector metrics
⚠️ Do not mutate the global warning handler
✨ Provide access to admission.Request in custom validator/defaulter
* Fix broken kind logging If gvk.Kind is "APIService", I expect "apiService", but the current code produces and logs "aPIService". Because it is not easy to do this kind of thing, it is better to log the kind as is. * fixup! Fix broken kind logging
Currently, a panic occcurence in a webhook handler is not recovered and crashes the webhook server. This change adds an option to Webhook to recover panics similar to how it is handled in Reconciler. It ensures that panics are converted to normal error response.
✨ webhook: add an option to recover from panics in handler
* fix docs: typo in OpenAPI There was a typo in docs of `pkg/client/fake/doc.go`: s/OpeanAPI/OpenAPI/ * Update pkg/client/fake/doc.go
Signed-off-by: jesus m. rodriguez <[email protected]>
✨ Bump k8s v0.25.0-alpha.3
The event message should be the message passed to the log statement rather than a value.
🏃 Update golangci-lint to 1.47.3
🐛 Fix log depth for DelegatingLogSink
🐛 Rearange `EventBroadcaster` log statement.
✨ Implement IgnoreAlreadyExists
📖 update doc for reconcile example
Kubernetes v1.25 has been released, so we can update our dependencies.
🌱 Update k8s API to v1.25
🌱 Use runtime.Unstructured interface instead of Unstructured struct
📖 Add inteon to reviewers
Signed-off-by: Tim Ramlot <[email protected]>
🌱Return an error if no httpClient is provided for `NewDynamicRESTMapper`
This commit adds a rest mapper that will lazily query the provided client for discovery information to do REST mappings.
Instead of creating the instance of the mapper directly, we will use WithExperimentalLazyMapper option for Dynamic Restmapper.
✨ Provide a truly lazy restmapper
Signed-off-by: Vince Prignano <[email protected]>
Signed-off-by: yanggang <[email protected]>
✨ Expose Cache/Client options on Cluster
🌱 code clean for pkg/envtest/server.go
🌱 Bump golangci-lint to v1.51.1
🌱 verify.sh: verify generate & modules (in CI)
🌱 Bump controller-tools to v0.11.3 and apidiff to v0.5.0
Signed-off-by: Vince Prignano <[email protected]>
🌱 Re-add tests for component config
Bumps [github.com/onsi/ginkgo/v2](https://github.com/onsi/ginkgo) from 2.8.0 to 2.8.1. - [Release notes](https://github.com/onsi/ginkgo/releases) - [Changelog](https://github.com/onsi/ginkgo/blob/master/CHANGELOG.md) - [Commits](onsi/ginkgo@v2.8.0...v2.8.1) --- updated-dependencies: - dependency-name: github.com/onsi/ginkgo/v2 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]>
…ithub.com/onsi/ginkgo/v2-2.8.1 🌱 Bump github.com/onsi/ginkgo/v2 from 2.8.0 to 2.8.1
Bumps [go.uber.org/goleak](https://github.com/uber-go/goleak) from 1.2.0 to 1.2.1. - [Release notes](https://github.com/uber-go/goleak/releases) - [Changelog](https://github.com/uber-go/goleak/blob/master/CHANGELOG.md) - [Commits](uber-go/goleak@v1.2.0...v1.2.1) --- updated-dependencies: - dependency-name: go.uber.org/goleak dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]>
…o.uber.org/goleak-1.2.1 🌱 Bump go.uber.org/goleak from 1.2.0 to 1.2.1
- If the user sets the GVK, make sure that's in the list of the GVK returned from the Scheme. - Always print out the multiple GVKs when erroring out - Add more comments on where to find more information about this issue Signed-off-by: Vince Prignano <[email protected]>
🐛 Allow to set GracefulShutdownTimeout to -1, disabling timeouts
recocnized -> recognized
📖 Fix a typo in a godoc comment
🌱 GVKForObject should handle multiple GVKs in Scheme gracefully
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by pull[bot]
Can you help keep this open source service alive? 💖 Please sponsor : )