The GitHub Actions job "Flink Kubernetes Operator CI" on 
flink-kubernetes-operator.git/FLINK-33548 has succeeded.
Run started by GitHub user vsantwana (triggered by gyfora).

Head commit for run:
cd4636088297f6c93e848a07008d8a61c055d343 / Santwana Verma <[email protected]>
[FLINK-33548] Address review: resources fallback, autoscaler tuning, and 
validation hardening

Fixes issues found while reviewing the ResourceRequirements change:

- KubernetesScalingRealizer no longer NPEs when memory is configured only
  via the new resources field; it now reads/writes memory through whichever
  field is in effect and scales limits.memory proportionally so the memory
  limit factor is preserved across tuning.
- FlinkConfigBuilder gates the resources-over-resource precedence on actual
  content (hasResourceRequirements) instead of a plain null check, so an
  empty "resources: {}" block no longer suppresses the deprecated resource
  field.
- DefaultValidator rejects negative cpu/ephemeral-storage requirements
  (previously accepted via getAmountInBytes), and its memory-not-defined
  errors now reference both resource.memory and resources.requests.memory.
- Reuse FlinkConfigBuilder.getRequestOrLimit from the autoscaler instead of
  duplicating the request-or-limit lookup.
- Complete the Apache license header in the e2e resource-requirements CR.

Adds tests for the autoscaler (resources-only, limits-only, limit-factor
preservation), the empty-resources fallback, requests-only/zero-cpu limit
factors, and negative resource validation.

Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>

Report URL: 
https://github.com/apache/flink-kubernetes-operator/actions/runs/27932503552

With regards,
GitHub Actions via GitBox

Reply via email to