-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Disable proofd file server for XRootD>=5 #178
Conversation
A new Pull Request was created by @smuzaffar (Malik Shahzad Muzaffar) for branch cms/v6-14-00-patches/8ba6b0f. @cmsbuild, @smuzaffar, @aandvalenzuela, @iarspider can you please review it and eventually sign? Thanks. |
please test for CMSSW_10_6_X |
-1 Failed Tests: UnitTests Unit TestsI found errors in the following unit tests: ---> test TestFWCoreIntegrationInterProcess had ERRORS Comparison SummarySummary:
|
+externals @perrotta @rappoccio any objections on merging this for CMSSW 10.6.X . This change allows us to backport xrootd 5 to 10.6.X |
This pull request is fully signed and it will be integrated in one of the next cms/v6-14-00-patches/8ba6b0f IBs (but tests are reportedly failing). This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
+1 Yes, please merge this. Once we get them all together, I assume the checks will pass? |
@rappoccio the failed unit test is not related to this PR, I think it's just a glitch. I will restart the test just in case |
please test |
-1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-b76671/32569/summary.html External BuildI found compilation error when building: -- Configuring incomplete, errors occurred! See also "/data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/BUILD/el8_amd64_gcc11/lcg/root/6.26.11-fe38957ec289fe627966ac63e6dbb7ac/build/CMakeFiles/CMakeOutput.log". See also "/data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/BUILD/el8_amd64_gcc11/lcg/root/6.26.11-fe38957ec289fe627966ac63e6dbb7ac/build/CMakeFiles/CMakeError.log". error: Bad exit status from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/tmp/rpm-tmp.w9YCH3 (%build) RPM build errors: line 37: It's not recommended to have unversioned Obsoletes: Obsoletes: lcg+root+6.26.11-fe38957ec289fe627966ac63e6dbb7ac Macro expanded in comment on line 350: %{pkginstroot}/lib |
no need to test it any more. Anyway test should have been requested for CMSSW_10_6_X :-) |
Patch is needed in order to backport the XrootD>=5 in to root 6.12/14. This basically disables the XProofD file server and client support for XRootD versions>=5. Thanks to Christopher H Green and Kyle Knoepfel for providing this patch.