Remove ovirt output limitation from some v2v cases

[mxie91]

Summary by CodeRabbit

Release Notes

• Tests
  • Expanded output mode support for configuration variants to work across multiple conversion targets.
  • Reorganized test suite configurations with standardized naming conventions and removed hypervisor-specific restrictions for improved test flexibility.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

This pull request modifies two test configuration files to relax restrictions and standardize naming conventions. In convert_from_file.cfg, three config blocks have their explicit output_mode restrictions removed, allowing broader output mode configurations. In function_test_esx.cfg, variant entries are renamed to use underscores instead of hyphens (e.g., no-copy-illegal → no_copy_illegal), hypervisor-specific filtering constraints are removed, and nested variant names are updated for consistency.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• Verify that all references to renamed variants are updated consistently throughout the file (no stale references to hyphenated names remain)
• Ensure the removal of output_mode and hypervisor-specific filtering constraints doesn't unintentionally broaden the scope of affected test cases
• Confirm that nested variant renaming (e.g., no_rhv_disk_uuid sub-variants) is applied consistently across all occurrences in function_test_esx.cfg

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately reflects the main change: removing ovirt (rhev) output mode limitations from v2v test configuration cases.
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
• Post copyable unit tests in a comment

---

📜 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 875c970 and e257367.

📒 Files selected for processing (2)

• v2v/tests/cfg/convert_from_file.cfg (0 hunks)
• v2v/tests/cfg/function_test_esx.cfg (4 hunks)

💤 Files with no reviewable changes (1)

• v2v/tests/cfg/convert_from_file.cfg

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)

• GitHub Check: Python 3.11
• GitHub Check: Python 3.9
• GitHub Check: Python 3.8
• GitHub Check: Python 3.12

🔇 Additional comments (2)

v2v/tests/cfg/function_test_esx.cfg (2)

1061-1061: Selector correctly updated to reference the renamed variant.

The negative_test selector properly references no_copy_illegal (with underscore) to match the renamed variant at line 551, maintaining consistency between variant definition and usage.

---

551-583: Variant naming standardization is complete and verified—all references have been properly updated.

Verification confirms that all old hyphenated variant names (no-copy-illegal, no-rhv-disk-uuid, etc.) have been completely removed from the codebase, and all new underscored names are in place. The variant selector at line 1061 has been correctly updated to reference no_copy_illegal. CLI option strings correctly retain hyphens (--no-copy, -oo rhv-disk-uuid). The refactoring is consistent and complete.

---

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

[mxie91]

@xiaodwan please help review, thanks!