fix a typo in --era specification

[hengne]

fix a typo in --era specification noticed by @boudoul

[davidlange6]

@hengne - maybe a "_" could be added to HLTDR250ns (e.g., HLTDR2_50ns) to avoid the possibility for further confusion

[cmsbuild]

A new Pull Request was created by @hengne (Hengne Li) for CMSSW_8_0_X.

fix a typo in --era specification

It involves the following packages:

Configuration/PyReleaseValidation

@danduggan, @boudoul, @franzoni, @fabozzi, @cmsbuild, @srimanob 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.
This pull requests was done for a production branch and will require explicit ORP approval on friday or L1 override.

[hengne]

@davidlange6 i think that's a good idea. this will require to change all the workflows that using that step, which has a potential to introduce new bugs, and better be tested well before a PR to change.

[davidlange6]

Right - it means a query replace in 3 places according to the search in git. (and fortunately a test will crash if not ok in this case)

[hengne]

@davidlange6 I can do this change and test carefully with the current 760pre7 regular Relval update PR that I am currently preparing. Sounds good? I hope I can have it done by today or during the weekend.

[boudoul]

@cmsbuild , please test

[davidlange6]

sounds good. Thanks-

On Oct 23, 2015, at 10:58 AM, Hengne Li notifications@github.com wrote:

@davidlange6 I can do this change and test carefully with the current 760pre7 regular Relval update PR that I am currently preparing. Sounds good? I hope I can have it done by today or during the weekend.

—
Reply to this email directly or view it on GitHub.

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/9151/console
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/9156/console

[cmsbuild]

+1
Tested at: 3f23db9
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-12065/9156/summary.html

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
3f23db9
8b1004f
a328daa
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-12065/9156/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-12065/9156/git-merge-result

[boudoul]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_8_0_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar

[cmsbuild]

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

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_8_0_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar