fix(nvcf-unbound): nest format param under params in PodMonitor

[mesutoezdil]

TL;DR

In the logger-metrics endpoint, format: sat at the same indent level as params:, so the CRD schema pruned it and the format=prometheus scrape parameter was never sent. Nested it under params, matching the container-cache chart. helm lint and helm template verified.

Issues

Closes #295

Summary by CodeRabbit

• Bug Fixes
  • Corrected Prometheus metrics configuration to ensure monitoring endpoints are interpreted properly.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 556c2fd7-189a-4652-b1ce-5c0fc64f9899

📥 Commits

Reviewing files that changed from the base of the PR and between 845c594 and dcf5ead.

📒 Files selected for processing (1)

• deploy/helm/nvcf-unbound/templates/podmonitor.yaml

🚧 Files skipped from review as they are similar to previous changes (1)

• deploy/helm/nvcf-unbound/templates/podmonitor.yaml

---

📝 Walkthrough

Walkthrough

The PodMonitor configuration now nests format: [prometheus] under params: for the first metrics endpoint.

Changes

PodMonitor scrape configuration

Layer / File(s)	Summary
Nest format under endpoint parameters deploy/helm/nvcf-unbound/templates/podmonitor.yaml	Moves the Prometheus format setting from the endpoint level into the params: block.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: balajinvda

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly describes the main PodMonitor fix and matches the change set.
Linked Issues check	✅ Passed	The change nests format under params as required by issue #295, restoring the intended query parameter.
Out of Scope Changes check	✅ Passed	The PR only changes the PodMonitor nesting and introduces no unrelated code changes.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Comment @coderabbitai help to get the list of available commands.}

[balajinvda]

LGTM — correct fix: format nested under params so the PodMonitor scrape param isn't schema-pruned. Matches the container-cache chart. CI green.