Repository navigation
DNM : Bump/dep/test kind - #1830
Open
sameerforge wants to merge 2 commits into
Open
sameerforge wants to merge 2 commits into
sameerforge wants to merge 2 commits into
Conversation
sameerforge
force-pushed
the
bump/dep/test-kind
branch
3 times, most recently
from
May 20, 2026 06:15
aff651b to
edfe506
Compare
sameerforge
force-pushed
the
bump/dep/test-kind
branch
from
June 17, 2026 06:26
edfe506 to
061a289
Compare
sameerforge
force-pushed
the
bump/dep/test-kind
branch
from
July 16, 2026 10:18
6852ee3 to
cb7337c
Compare
Signed-off-by: Sameer <sameer.khan@broadcom.com>
The packaging aggregated API server is a thin proxy whose storage is the host cluster's InternalPackage(Metadata) CRDs, and its custom REST watch does not synthesize the WatchList "initial-events-end" bookmark. With WatchList enabled, the embedded apiserver's SetListOptionsDefaults injects sendInitialEvents + resourceVersionMatch=NotOlderThan into watch requests with resourceVersion=0. Host clusters that ship the WatchList feature gate disabled (e.g. Kubernetes 1.33, which turned it off by default) then reject the forwarded options with 422 Unprocessable Entity, failing a plain "watch&resourceVersion=0" (TestK8APIEndpoints). Disabling the feature gate in the packaging apiserver process fixes this across every host version: raw watch&resourceVersion=0 requests are served as a plain watch (no injection), and WatchList-capable clients (client-go informers) receive a 422 for the streaming request and transparently fall back to list+watch. Note: an earlier approach that stripped these options only in the REST watch path was rejected because it returned a successful watch stream with no bookmark, which hangs WatchList informers (including kapp-controller's own) and stalls the shared manager cache. Signed-off-by: Sameer <sameer.khan@broadcom.com> Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Signed-off-by: Sameer <sameer.khan@broadcom.com>
sameerforge
force-pushed
the
bump/dep/test-kind
branch
from
July 16, 2026 10:22
cb7337c to
9ff81ff
Compare
This branch has not been deployed
This file contains hidden or 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
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.
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?
Additional Notes for your reviewer:
Review Checklist:
a link to that PR
change
Additional documentation e.g., Proposal, usage docs, etc.: