dlpar.py: Enhance regex to support optional R suffix in slot format

[maramsmurthy]

• Updated regex for slot_ibm to include optional -R\d+ segment.
  This allows matching both standard and extended slot formats:
  • Standard: U50EE.001.WZS00TL-P3-C14
  • Extended: U50EE.001.WZS00TL-P3-C14-R1
• Change ensures compatibility with newer hardware naming conventions
  that include resource identifiers after the slot number.

Backward Compatibility:

• Existing slot formats without -R remain fully supported.
• No impact on current parsing logic for standard slot names.

Testing Impact:

• Verified regex against both formats to ensure correct matching.
• Additional unit tests recommended for extended slot names to prevent
  regressions.

Summary by CodeRabbit

• Bug Fixes
  • Enhanced PCI slot pattern recognition to support IBM slot naming conventions with revision/reset suffixes, improving compatibility with a broader range of system configurations and ensuring accurate slot identification.

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

[coderabbitai]

Walkthrough

The change modifies the IBM slot parsing regex pattern in avocado/utils/pci.py by extending it with an optional suffix. The regex was updated from r"((\w+)[.])+(\w+)-[PC(\d+)-]*C(\d+)" to r"((\w+)[.])+(\w+)-[PC(\d+)-]*C(\d+)(?:-R\d+)?", adding a non-capturing optional group (?:-R\d+)? to recognize slot strings that may end with "-R" followed by one or more digits. The existing matching behavior is preserved while enabling additional slot format variations.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• Single-file, single-line regex pattern modification
• Straightforward extension using a non-capturing optional group
• No logic changes or behavioral side effects to verify
• Minimal scope with clear intent to support an additional slot name format suffix

Pre-merge checks

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The title references 'dlpar.py' but the actual change is in 'avocado/utils/pci.py'. However, the core objective of enhancing regex for optional R suffix in slot format is accurately captured.	Update the title to reference the correct file: 'avocado/utils/pci.py: Enhance regex to support optional R suffix in slot format' or adjust to reflect the actual file modified.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

Tip

📝 Customizable high-level summaries are now available in beta!

You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.

• Provide your own instructions using the high_level_summary_instructions setting.
• Format the summary however you like (bullet lists, tables, multi-section layouts, contributor stats, etc.).
• Use high_level_summary_in_walkthrough to move the summary from the description to the walkthrough section.

Example instruction:

"Divide the high-level summary into five sections:

1. 📝 Description — Summarize the main change in 50–60 words, explaining what was done.
2. 📓 References — List relevant issues, discussions, documentation, or related PRs.
3. 📦 Dependencies & Requirements — Mention any new/updated dependencies, environment variable changes, or configuration updates.
4. 📊 Contributor Summary — Include a Markdown table showing contributions:
   | Contributor | Lines Added | Lines Removed | Files Changed |
5. ✔️ Additional Notes — Add any extra reviewer context.
   Keep each section concise (under 200 words) and use bullet or numbered lists for clarity."

Note: This feature is currently in beta for Pro-tier users, and pricing will be announced later.

---

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

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

avocado/utils/pci.py (1)

203-205: Regex extension for optional -R suffix looks correct

The updated pattern cleanly extends the previous behavior: existing slot strings still match, and the new optional non-capturing (?:-R\d+)? correctly covers cases like ...-C14-R1 without affecting group() semantics. Given how hardware naming tends to evolve, this looks like a safe, backward‑compatible change.

If not already covered elsewhere, consider adding unit tests for both formats (with and without -R<digits>) to lock this behavior in and avoid future regressions.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 74b7379 and 66fb50d.

📒 Files selected for processing (1)

• avocado/utils/pci.py (1 hunks)

⏰ 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). (16)

• GitHub Check: Linux with Python 3.13
• GitHub Check: Linux with Python 3.12
• GitHub Check: Linux with Python 3.11
• GitHub Check: Linux with Python 3.10
• GitHub Check: Linux with Python 3.9
• GitHub Check: Fedora develop install/uninstall task
• GitHub Check: Egg task ubi:8.8
• GitHub Check: Version task debian:12.4
• GitHub Check: Fedora selftests
• GitHub Check: Version task ubuntu:22.04
• GitHub Check: Podman spawner with 3rd party runner plugin
• GitHub Check: Code Coverage (3.11)
• GitHub Check: Windows with Python 3.11
• GitHub Check: Version task ubi:8.8
• GitHub Check: macOS with Python 3.11
• GitHub Check: Static checks

[maramsmurthy]

As commits which are not part of this branch appeared here due to some issue with branch. Will be raising another PR by clearing these discrepancies.