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

Jump to bottom

Remove Obsolete Geometry Scenarios #9952

Merged
merged 1 commit into from Jul 7, 2015
Merged

Remove Obsolete Geometry Scenarios #9952

merged 1 commit into from Jul 7, 2015

Conversation

ianna
Copy link
Contributor

@ianna ianna commented Jun 29, 2015

Note: if any of the removed scenarios is indeed needed it should be restarted from either run 1 or run 2 default scenario.

@ianna
* Remove obsolete scenarios.
633e5ec 
Note: if any of the removed scenarios is indeed needed it should be restarted from either run 1 or run 2 default scenario.
@ianna
Copy link
Contributor Author

ianna commented Jun 29, 2015

+1

@cmsbuild cmsbuild added this to the Next CMSSW_7_5_X milestone Jun 29, 2015
@cmsbuild
Copy link
Contributor

A new Pull Request was created by @ianna (Ianna Osborne) for CMSSW_7_5_X.

Remove Obsolete Geometry Scenarios

It involves the following packages:

Configuration/Geometry
Configuration/StandardSequences
Geometry/CMSCommonData

@cmsbuild, @franzoni, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @Martin-Grunewald, @makortel, @GiacomoSguazzoni, @rovere, @VinInn, @appeltel, @cerati, @dgulhan 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.

@cmsbuild
Copy link
Contributor

-1
Tested at: 633e5ec
The relvals timed out after 2 hours.

you can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-9952/5808/summary.html

@ianna
Copy link
Contributor Author

ianna commented Jun 30, 2015

@cmsbuild - the error is unrelated to this PR.

@ianna
Copy link
Contributor Author

ianna commented Jul 1, 2015

@cmsbuild - please, rerun the tests

@ianna
Copy link
Contributor Author

ianna commented Jul 6, 2015

please test

@ianna
Copy link
Contributor Author

ianna commented Jul 6, 2015

@cmsbuild please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Jul 6, 2015

The tests are being triggered in jenkins.

@cmsbuild
Copy link
Contributor

cmsbuild commented Jul 6, 2015

+1
Tested at: 633e5ec
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-9952/43/summary.html

@cmsbuild
Copy link
Contributor

cmsbuild commented Jul 6, 2015

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

davidlange6 added a commit that referenced this pull request Jul 7, 2015
@davidlange6
Merge pull request #9952 from ianna/geometry-scenarios-cleanup
b919ca0 
Remove Obsolete Geometry Scenarios
@davidlange6 davidlange6 merged commit b919ca0 into cms-sw:CMSSW_7_5_X Jul 7, 2015
@ianna ianna deleted the geometry-scenarios-cleanup branch October 26, 2015 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
comparison-available geometry-approved operations-pending orp-pending pending-signatures tests-approved
Projects
None yet
Milestone
CMSSW_7_5_X
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@ianna @cmsbuild @davidlange6