Skip to content
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

roll back to PREvious CRAB client version. #6167

Merged
merged 1 commit into from Aug 12, 2020

Conversation

belforte
Copy link
Contributor

@mrodozov this will allow users to run crab client on older CMSSW w/o the hack described in
https://hypernews.cern.ch/HyperNews/CMS/get/cernCompAnnounce/1444/1/1/2/1/1.html
they will have to use crab from IB, but that's rather straightforward
https://twiki.cern.ch/twiki/bin/view/CMSPublic/CMSCrabClient#Using_CRABClient_from_nightly_CM

For crab-prod I will rather change the code to have both this fix and the fixes from current (alas broken) version.

Sorry for the extra work,

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @belforte (Stefano Belforte) for branch IB/CMSSW_11_2_X/master.

@cmsbuild, @smuzaffar, @mrodozov, @tulamor can you please review it and eventually sign? Thanks.
cms-bot commands are listed here

@smuzaffar
Copy link
Contributor

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 11, 2020

The tests are being triggered in jenkins.

@cmsbuild
Copy link
Contributor

+1
Tested at: 09b086a
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8c67ad/8717/summary.html
CMSSW: CMSSW_11_2_X_2020-08-11-1100
SCRAM_ARCH: slc7_amd64_gcc820

@cmsbuild
Copy link
Contributor

Comparison job queued.

@cmsbuild
Copy link
Contributor

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8c67ad/8717/summary.html

Comparison Summary:

  • No significant changes to the logs found
  • Reco comparison results: 2 differences found in the comparisons
  • DQMHistoTests: Total files compared: 35
  • DQMHistoTests: Total histograms compared: 2612401
  • DQMHistoTests: Total failures: 5
  • DQMHistoTests: Total nulls: 1
  • DQMHistoTests: Total successes: 2612347
  • DQMHistoTests: Total skipped: 48
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 0.004 KiB( 34 files compared)
  • DQMHistoSizes: changed ( 10224.0 ): 0.004 KiB MessageLogger/Warnings
  • Checked 149 log files, 22 edm output root files, 35 DQM output files

@mrodozov
Copy link
Contributor

@belforte I merge this one and close the previous, correct ?

@smuzaffar smuzaffar merged commit cbaeec5 into cms-sw:IB/CMSSW_11_2_X/master Aug 12, 2020
@belforte
Copy link
Contributor Author

HI @smuzaffar @mrodozov I am puzzled that this PR does not seem to have made it to latest IB, differnetly from those for crab-prod and crab-dev (which were merged later btw).
When looking in CVMFS files have been updated

-rw-r--r--. 1 cvmfs cvmfs 13 Aug 13 04:04 /cvmfs/cms-ib.cern.ch/week1/share/cms/crab/1.0/etc/crab-dev.latest
-rw-r--r--. 1 cvmfs cvmfs 13 Aug 13 04:04 /cvmfs/cms-ib.cern.ch/week1/share/cms/crab/1.0/etc/crab-pre.latest
-rw-r--r--. 1 cvmfs cvmfs 13 Aug 13 04:04 /cvmfs/cms-ib.cern.ch/week1/share/cms/crab/1.0/etc/crab-prod.latest

but crab-pre.latest still points to previous tag

belforte@lxplus780/TC3> cat /cvmfs/cms-ib.cern.ch/week1/share/cms/crab/1.0/etc/crab-dev.latest 
v3.200812.00
belforte@lxplus780/TC3> cat /cvmfs/cms-ib.cern.ch/week1/share/cms/crab/1.0/etc/crab-prod.latest 
v3.200812.00
belforte@lxplus780/TC3> cat /cvmfs/cms-ib.cern.ch/week1/share/cms/crab/1.0/etc/crab-pre.latest 
v3.200719.00
belforte@lxplus780/TC3> 

while it should be v3.200531.00
will this simply be updated tomorrow ? Or did something go wrong and is action needed ?

thanks

@smuzaffar
Copy link
Contributor

@belforte , as the reverted version(200531) is smaller than the previous version (200719) so it will not become the default version for pre. You need to provide a new tag which should be greater than 200719 in otder to make it default for pre.

@belforte
Copy link
Contributor Author

belforte commented Aug 13, 2020 via email

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

Successfully merging this pull request may close these issues.

None yet

4 participants