LCORE-1508: Updated config docs

[tisnik]

Description

LCORE-1508: Updated config docs

Type of change

• Refactor
• New feature
• Bug fix
• CVE fix
• Optimization
• Documentation Update
• Configuration Update
• Bump-up service version
• Bump-up dependent library
• Bump-up library or tool used for development (does not change the final image)
• CI configuration change
• Konflux configuration change
• Unit tests improvement
• Integration tests improvement
• End to end tests improvement
• Benchmarks improvement

Tools used to create PR

• Assisted-by: N/A
• Generated by: N/A

Related Tickets & Documents

• Related Issue #LCORE-1508

Summary by CodeRabbit

• Documentation

  • Updated configuration documentation to reflect URL validation enhancements for configuration fields.

• Chores

  • Updated build process tooling integration.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 7e942108-f1af-4820-905e-b479d641f764

📥 Commits

Reviewing files that changed from the base of the PR and between 55c4543 and 39c63d9.

⛔ Files ignored due to path filters (2)

• docs/config.png is excluded by !**/*.png
• docs/config.svg is excluded by !**/*.svg

📒 Files selected for processing (2)

• Makefile
• docs/config.puml

---

Walkthrough

Two minor updates: the Makefile recipe for generating the PlantUML diagram now invokes pyreverse through uv run, and the OkpConfiguration.rhokp_url field type in the generated diagram is updated from Optional[str] to Optional[AnyHttpUrl] to reflect a model change.

Changes

Cohort / File(s)	Summary
Build Configuration Makefile	Updated docs/config.puml target to execute pyreverse via uv run instead of directly, preserving the subsequent file rename step.
Documentation Diagram docs/config.puml	Updated OkpConfiguration.rhokp_url field type declaration from Optional[str] to Optional[AnyHttpUrl].

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title references the ticket number LCORE-1508 but does not clearly describe what was actually changed in the configuration documentation.	Revise the title to be more specific about the changes, such as 'Update OkpConfiguration.rhokp_url type and pyreverse execution method' or similar to reflect the actual modifications made.

✅ 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 changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

📝 Coding Plan

• Generate coding plan for human review comments

---

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.}