Skip to content

Patch to fix ptest issue in libstoragemgmt#16472

Draft
aninda-al wants to merge 1 commit intomicrosoft:3.0-devfrom
Kanishk-Bansal:fix-ptest-lsm-local-disk-link-speed-get-issue
Draft

Patch to fix ptest issue in libstoragemgmt#16472
aninda-al wants to merge 1 commit intomicrosoft:3.0-devfrom
Kanishk-Bansal:fix-ptest-lsm-local-disk-link-speed-get-issue

Conversation

@aninda-al
Copy link
Copy Markdown
Contributor

@aninda-al aninda-al commented Apr 6, 2026

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

  • The toolchain has been rebuilt successfully (or no changes were made to it)
  • The toolchain/worker package manifests are up-to-date
  • Any updated packages successfully build (or no packages were changed)
  • Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
  • Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
  • All package sources are available
  • cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
  • LICENSE-MAP files are up-to-date (./LICENSES-AND-NOTICES/SPECS/data/licenses.json, ./LICENSES-AND-NOTICES/SPECS/LICENSES-MAP.md, ./LICENSES-AND-NOTICES/SPECS/LICENSE-EXCEPTIONS.PHOTON)
  • All source files have up-to-date hashes in the *.signatures.json files
  • sudo make go-tidy-all and sudo make go-test-coverage pass
  • Documentation has been updated to match any changes to the build system
  • Ready to merge

Summary

What does the PR accomplish, why was it needed?
One of the ptest was failing for this package libstoragemgmt
Failure Description: lsm_local_disk_link_speed_get() returned unexpected code 1 (LSM_ERR_LIB_BUG); test only accepts LSM_ERR_OK, LSM_ERR_NO_SUPPORT, or LSM_ERR_PERMISSION_DENIED
Added patch to fix the ptest failure due to unsupported link speed in test environment
Upstream patch not available

AI Analysis relevance (In %) / Reasoning

AI Analysis relevance (In %): 100

Change Log
  • Change : SPECS/libstoragemgmt/libstoragemgmt.spec
  • Added : SPECS/libstoragemgmt/0001-fix-ptest-issue-link-speed-no-support.patch
Does this affect the toolchain?

NO

Associated issues
Links to CVEs
Test Methodology

local build
image

libstoragemgmt-1.9.8-2.azl3.src.rpm.log
libstoragemgmt-1.9.8-2.azl3.src.rpm.test.log

  • Pipeline build id: xxxx

@microsoft-github-policy-service microsoft-github-policy-service bot added Packaging 3.0-dev PRs Destined for AzureLinux 3.0 labels Apr 6, 2026
@Kanishk-Bansal Kanishk-Bansal added the ptest package testing (%check section in spec) label Apr 6, 2026
@Kanishk-Bansal
Copy link
Copy Markdown
Contributor

Build

@aninda-al aninda-al changed the title [Work item #18108] Added patch to fix ptest failure due to unsupported link speed in test environment Patch to fix ptest issue in libstoragemgmt Apr 7, 2026
@aninda-al
Copy link
Copy Markdown
Contributor Author

Buddy build is failing due to a known issue. There is already a PR for this issue #15477

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3.0-dev PRs Destined for AzureLinux 3.0 Packaging ptest package testing (%check section in spec)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants