Skip to content

feat: Initial commit#1

Merged
abaczek merged 1 commit into
mainfrom
first_release
Jul 10, 2025
Merged

feat: Initial commit#1
abaczek merged 1 commit into
mainfrom
first_release

Conversation

@abaczek

@abaczek abaczek commented Jul 10, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@mfd-intel-bot

Copy link
Copy Markdown

We don't publish DEVs .whl.
To build .whl, run 'pip install git+https://github.com/intel/mfd-libibverbs-utils@first_release'

1 similar comment
@mfd-intel-bot

Copy link
Copy Markdown

We don't publish DEVs .whl.
To build .whl, run 'pip install git+https://github.com/intel/mfd-libibverbs-utils@first_release'

@github-advanced-security

Copy link
Copy Markdown

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

Comment thread .github/workflows/manual_release.yml Dismissed
Comment thread .github/workflows/pull_requests.yml Dismissed
@adrianlasota adrianlasota requested a review from Copilot July 10, 2025 14:53

This comment was marked as outdated.

Comment thread README.md Outdated
Comment thread README.md Outdated
Signed-off-by: Baczek, Arkadiusz <arkadiusz.baczek@intel.com>
@mfd-intel-bot

Copy link
Copy Markdown

We don't publish DEVs .whl.
To build .whl, run 'pip install git+https://github.com/intel/mfd-libibverbs-utils@first_release'

1 similar comment
@mfd-intel-bot

Copy link
Copy Markdown

We don't publish DEVs .whl.
To build .whl, run 'pip install git+https://github.com/intel/mfd-libibverbs-utils@first_release'

@adrianlasota adrianlasota requested a review from Copilot July 10, 2025 15:00

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Initial commit of the mfd-libibverbs-utils project, introducing core functionality for interacting with libibverbs-based RDMA tools, accompanying tests, documentation, and CI workflows.

  • Introduce IBVDevices and IBVDevinfo classes for listing RDMA devices and parsing detailed device info.
  • Add unit tests and fixtures covering normal, error, and edge cases.
  • Set up Sphinx documentation and GitHub Actions workflows for testing and publishing.

Reviewed Changes

Copilot reviewed 34 out of 37 changed files in this pull request and generated 2 comments.

File Description
mfd_libibverbs_utils/ibv_devices.py Implements IBVDevices for ibv_devices parsing
mfd_libibverbs_utils/ibv_devinfo.py Implements IBVDevinfo for ibv_devinfo parsing
README.md Project overview, usage examples, supported OS list
Comments suppressed due to low confidence (1)

mfd_libibverbs_utils/ibv_devices.py:46

  • [nitpick] The header-skip logic (startswith(header_lines)) depends on prefix matching; consider explicitly skipping the first two lines or matching the header row exactly to avoid accidentally filtering a valid device whose name matches those prefixes.
        for dev in re.finditer(pattern=ibv_dev_regex, string=output, flags=re.MULTILINE):

Comment thread README.md
Comment thread sphinx-doc/generate_docs.py
@abaczek abaczek merged commit a69b55e into main Jul 10, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants