CodeRabbit Review Feedback#739

Open

AnnaZivkovic wants to merge 29 commits intooutrigger-project:mainfrom

AnnaZivkovic:coderabbit-fixes

Commits on Apr 14, 2026

fix: add log level validation and support 0-10 range
AnnaZivkovic
committed
fix: change early return to continue in resource processing loop
AnnaZivkovic
committed
fix: return unmodified pod on PPC list error (true fail-open)

AnnaZivkovic
and
claude
committed
fix: make PodPlacementConfig.spec.plugins optional for backward compatibility

AnnaZivkovic
and
claude
committed
fix: enforce singleton name 'cluster' in ClusterPodPlacementConfig webhook

AnnaZivkovic
and
claude
committed
fix: make cache-empty assertion eventual to prevent test flakes

AnnaZivkovic
and
claude
committed
fix: close HTTP response body in suite test
AnnaZivkovic
committed
fix: use DeferCleanup to restore CPPC in fallback architecture test

AnnaZivkovic
and
claude
committed
fix: add separate metric for stale ENoExecEvents (NotFound cases)

AnnaZivkovic
and
claude
committed
fix: propagate failed ENoExecEvent delete errors during cleanup

AnnaZivkovic
and
claude
committed
docs: update vendoring instructions to use make vendor

AnnaZivkovic
and
claude
committed
fix: preserve Kubernetes patch version in fallback discovery

AnnaZivkovic
and
claude
committed
fix: exclude prerelease tags from version discovery

AnnaZivkovic
and
claude
committed
fix: error on go version mismatch instead of downgrading

AnnaZivkovic
and
claude
committed
fix: improve upgrade automation script robustness and portability

AnnaZivkovic
and
claude
committed
docs: correct namespace exclusion behavior in CLAUDE.md

AnnaZivkovic
and
claude
committed
fix: correct typo in EnoexecCounterInvalid metric help text

AnnaZivkovic
and
claude
committed
fix: correct typo in copyright header

AnnaZivkovic
and
claude
committed
docs: add language tags to fenced code blocks in upgrade README

AnnaZivkovic
and
claude
committed
docs: document fail-open behavior in pullSecretDataList

AnnaZivkovic
and
claude
committed
fix: use Eventually for cache assertion after CPPC deletion

AnnaZivkovic
and
claude
committed
test: mark placeholder tests as pending to prevent false passes

AnnaZivkovic
and
claude
committed
test: add GinkgoRecover to manager startup goroutines

AnnaZivkovic
and
claude
committed
fix: separate mktemp declaration to catch failures

AnnaZivkovic
and
claude
committed
fix: add gosec nolint for validated int8 conversion

AnnaZivkovic
and
claude
committed
fix: add nil check for cppc in shouldIgnorePod

AnnaZivkovic
and
claude
committed
fix: suppress linter warnings for unexported fields with json tags
AnnaZivkovic
committed
fix: replace deprecated BearerTokenFile with Authorization in ServiceMonitor

AnnaZivkovic
and
claude
committed
fix: replace deprecated NewSimpleClientset with NewClientset

AnnaZivkovic
and
claude
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

CodeRabbit Review Feedback#739

CodeRabbit Review Feedback#739
AnnaZivkovic wants to merge 29 commits intooutrigger-project:mainfrom
AnnaZivkovic:coderabbit-fixes

Commits on Apr 14, 2026

fix: add log level validation and support 0-10 range

fix: change early return to continue in resource processing loop

fix: return unmodified pod on PPC list error (true fail-open)

fix: make PodPlacementConfig.spec.plugins optional for backward compatibility

fix: enforce singleton name 'cluster' in ClusterPodPlacementConfig webhook

fix: make cache-empty assertion eventual to prevent test flakes

fix: close HTTP response body in suite test

fix: use DeferCleanup to restore CPPC in fallback architecture test

fix: add separate metric for stale ENoExecEvents (NotFound cases)

fix: propagate failed ENoExecEvent delete errors during cleanup

docs: update vendoring instructions to use make vendor

fix: preserve Kubernetes patch version in fallback discovery

fix: exclude prerelease tags from version discovery

fix: error on go version mismatch instead of downgrading

fix: improve upgrade automation script robustness and portability

docs: correct namespace exclusion behavior in CLAUDE.md

fix: correct typo in EnoexecCounterInvalid metric help text

fix: correct typo in copyright header

docs: add language tags to fenced code blocks in upgrade README

docs: document fail-open behavior in pullSecretDataList

fix: use Eventually for cache assertion after CPPC deletion

test: mark placeholder tests as pending to prevent false passes

test: add GinkgoRecover to manager startup goroutines

fix: separate mktemp declaration to catch failures

fix: add gosec nolint for validated int8 conversion

fix: add nil check for cppc in shouldIgnorePod

fix: suppress linter warnings for unexported fields with json tags

fix: replace deprecated BearerTokenFile with Authorization in ServiceMonitor

fix: replace deprecated NewSimpleClientset with NewClientset