Repository navigation
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
crobinso
left a comment
There was a problem hiding this comment.
looks fine with the comment tweaks. PR is in draft. do you still need it?
f0a6b97 to
a0825da
Compare
Use a regular expression so the negative test accepts libvirt errors with or without the word "some". Document aw_bits support on newer libvirt versions. Assisted-by: Codex (~15% contribution) Signed-off-by: hholoubk <hholoubk@redhat.com>
a0825da to
9056ade
Compare
Libvirt now reports "doesn't support some additional attributes" instead of "doesn't support additional attributes". Update the expected error string so caching_mode, eim, and iotlb variants pass on RHEL 10.3.
Assisted-by: Claude AI (cursor) ~80%