Fixing sample model

[jland-redhat]

The gateway did infact need the route: {} in order to actually create the HTTPRoute

Summary by CodeRabbit

• Documentation
  • Updated the simulator model sample YAML to reflect the current router structure, introducing an explicit route field under router for clarity and alignment with the latest schema.
  • Improves accuracy of examples and helps users configure simulator models correctly using the updated configuration format.
  • No runtime or API changes; impact is limited to sample documentation and configuration guidance.

[coderabbitai]

Walkthrough

A single YAML configuration file was updated to change the router structure: replacing a bare router key with an object containing an empty route field under spec.model. No other fields were modified.

Changes

Cohort / File(s)	Summary
Simulator model config `docs/samples/models/simulator/model.yaml`	Updated `spec.model.router` from a bare key to an object with `route: {}` nested under `router`; no other changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

I twitch my whiskers at the YAML night,
A tiny route appears—oh what a sight!
Router now with hollow brace,
Paths await their future place.
I thump, I hop, configs align—
Carrots, commits, and pipelines fine. 🥕✨

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title Check	❓ Inconclusive	The title “Fixing sample model” indicates a change to a sample model but is too generic to convey the specific update of adding a required route: {} entry under the router spec. It refers to a real part of the change but lacks clarity on what was fixed, making it ambiguous for readers scanning the pull request history.	Please revise the title to specify the exact change, for example “Add required route: {} field to simulator sample model” so that it clearly reflects the main update.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fixing-model

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 21d3dfa and 1e3d3ab.

📒 Files selected for processing (1)

• docs/samples/models/simulator/model.yaml (1 hunks)

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[chaitanya1731]

LGTM

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: chaitanya1731, jland-redhat

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [chaitanya1731,jland-redhat]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment