Cleaning runthematrix76 x

[boudoul]

This PR is intended to synchronize runthematrix with 75X (see PR #10558)
It removes the option magfield added by mistake by @srimanob
Some check should be further done with the baseDataSetRelease but testing already as it is now

[cmsbuild]

A new Pull Request was created by @boudoul (boudoul) for CMSSW_7_6_X.

Cleaning runthematrix76 x

It involves the following packages:

Configuration/PyReleaseValidation

@cmsbuild, @srimanob, @boudoul, @franzoni can you please review it and eventually sign? Thanks.
@ghellwig, @Martin-Grunewald this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[boudoul]

@cmsbuild , please test

[cmsbuild]

The tests are being triggered in jenkins.

[cmsbuild]

+1
Tested at: 775a6a7
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-10559/6938/summary.html

[cmsbuild]

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

[boudoul]

removed duplicate

[boudoul]

please test

[cmsbuild]

The tests are being triggered in jenkins.

[cmsbuild]

Pull request #10559 was updated. @cmsbuild, @srimanob, @boudoul, @franzoni can you please check and sign again.

[cmsbuild]

+1
Tested at: 6b5e716
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-10559/4020/summary.html

[cmsbuild]

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

[srimanob]

Sorry, I don't follow up the comment that "It removes the option magfield added by mistake by me."
@boudoul , why don't we need magField for FastSim?
However, this line is proposed by FastSim group,
https://github.com/lveldere/cmssw/blob/8dd97e3968c6b2bdd0047e3b8a16c556c4a2d388/Configuration/PyReleaseValidation/python/relval_steps.py#L613

@lveldere Could you please comment?

In addition, this line is also in 74X,
https://github.com/cms-sw/cmssw/blob/CMSSW_7_4_X/Configuration/PyReleaseValidation/python/relval_steps.py#L615

[boudoul]

@srimanob and @lveldere : from CMSSW_7_4_3 and CMSSW_7_5_0_pre5 : the magfield is read from the GT and not anymore from xml, see here (*) for reference - If it is also in runthematrix74X also we should remove it from there too.

(*) https://hypernews.cern.ch/HyperNews/CMS/get/swDevelopment/3214.html

[srimanob]

OK, Thanks. I will fix 74X.