[Bug] Fix config verification after SOL introduced

[podkidyshev]

Summary

verify-configs command uses mocked System object. The recent SOL feature fails on merging scenario case config with system config because mocked objects doesn't have sol property

Test Plan

• Automated CI

Additional Notes

N/A

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Enterprise

Run ID: 05394c58-355e-4eef-af52-c7c67e1245c5

📥 Commits

Reviewing files that changed from the base of the PR and between 3369e5d and c1ce5dd.

📒 Files selected for processing (3)

• .github/workflows/ci.yml
• src/cloudai/cli/cli.py
• src/cloudai/cli/handlers.py

💤 Files with no reviewable changes (1)

• .github/workflows/ci.yml

Included review availability: Your plan provides up to 12 included reviews per hour; 10 remain after this review.

---

📝 Walkthrough

Walkthrough

Changes

Configuration verification

Layer / File(s)	Summary
Verification reporting and exit status src/cloudai/cli/handlers.py, src/cloudai/cli/cli.py	Configuration failures now log errors with debug tracebacks. Scenario parsing initializes System with sol={}. The CLI returns status code 1 on failure and 0 on success.
Targeted CI smoke validation .github/workflows/ci.yml	The smoke test removes the editable reinstall and broad configuration scan. Targeted verification commands remain.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to c1ce5

The PR makes a localized configuration-verification fix, and no actionable merge-blocking risk remains beyond normal checks and review.

Suggested reviewers: shreyaskommuri

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly identifies the bug fix for configuration verification after introducing SOL.
Description check	✅ Passed	The description explains the SOL-related mock configuration failure and identifies automated CI testing.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch ipod/sol-verify-conf-fix

---

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

[jj10306]

lgtm!

[rutayan-nv]

LGTM. Good catch on the swallowed exit code — that was hiding real failures.