[Work Item #19220] Resolved ptests failure in git-lfs package#17224
[Work Item #19220] Resolved ptests failure in git-lfs package#17224SumitJenaHCL wants to merge 5 commits into
ptests failure in git-lfs package#17224Conversation
b2a6914 to
2f50ce4
Compare
|
Buddy build has passed. |
Kanishk-Bansal
left a comment
There was a problem hiding this comment.
remove CGO_ENABLED=0
removed. |
Kanishk-Bansal
left a comment
There was a problem hiding this comment.
similar changes already merged in 70e4aa0
kgodara912
left a comment
There was a problem hiding this comment.
We may need to make change in build section as well. Because in build section, if the module is compiled with different crypto backend, then it can't run check section with different backend. Please check once and put the change in build section which will ensure consistent crypto backend for both build and test.
Added requested changes. Buddy build has passed after necessary changes. |
Merge Checklist
All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)
*-staticsubpackages, etc.) have had theirReleasetag incremented../cgmanifest.json,./toolkit/scripts/toolchain/cgmanifest.json,.github/workflows/cgmanifest.json)./LICENSES-AND-NOTICES/SPECS/data/licenses.json,./LICENSES-AND-NOTICES/SPECS/LICENSES-MAP.md,./LICENSES-AND-NOTICES/SPECS/LICENSE-EXCEPTIONS.PHOTON)*.signatures.jsonfilessudo make go-tidy-allandsudo make go-test-coveragepassSummary
What does the PR accomplish, why was it needed?
This is to fix ptests failures in TLS handshake issue due to openssl for package git-lfs
Change Log
Does this affect the toolchain?
NO
Associated issues
Links to CVEs
Test Methodology