Greptile Summary

This PR adds support for Kubernetes extended resources to the @kubernetes decorator, allowing users to request arbitrary node-advertised resources (e.g. example.com/foo) alongside CPU/memory/GPU. The extended_resources dict is merged into both qos_requests and qos_limits — which is correct, since Kubernetes requires equal requests and limits for extended resources — and the change is consistently applied across the direct Kubernetes job path, the JobSet path, and the Argo Workflows path.

• Adds extended_resources: Dict[str, str] parameter (default {}) to KubernetesDecorator, serialized as JSON for CLI forwarding and properly deserialized on the executor side.
• The or {} null guard in kubernetes_job.py, kubernetes_jobsets.py, and argo_workflows.py safely handles None propagation from callers that omit the argument.
• One latent foot-gun: if extended_resources contains a key that matches the GPU vendor resource string (e.g. nvidia.com/gpu), the GPU-specific limits block that merges last will silently create a requests/limits mismatch for that key, causing a Kubernetes API validation rejection. This is a user-error scenario but worth documenting at the merge site.

Confidence Score: 5/5

Safe to merge — the feature is correctly threaded through all execution paths (direct job, jobset, Argo) and the null-safety guard is in place everywhere it matters.

All changed files correctly propagate the new parameter end-to-end. The or {} guard prevents None-unpacking errors. The only observation is an undocumented interaction where specifying a GPU vendor key in extended_resources at the same time as the gpu parameter would produce a requests/limits mismatch and a Kubernetes API rejection — but this requires a deliberate (and unusual) user action and fails loudly rather than silently.

No files require special attention — the change is consistent across all three execution paths.

Important Files Changed

_{Reviews (2): Last reviewed commit: "Apply suggestion from @valayDave" | Re-trigger Greptile}

Codecov Report

❌ Patch coverage is 10.00000% with 9 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (master@9088ec4). Learn more about missing BASE report.

@@            Coverage Diff            @@
##             master    #2242   +/-   ##
=========================================
  Coverage          ?   29.01%           
=========================================
  Files             ?      381           
  Lines             ?    52523           
  Branches          ?     9266           
=========================================
  Hits              ?    15242           
  Misses            ?    36246           
  Partials          ?     1035           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.