-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Replaced the contents of the fed config by fedtest #12317
Replaced the contents of the fed config by fedtest #12317
Conversation
More than 6 years ago the work was started to replace FED by something new. This was done by placing a second configuration next to FED, called FEDTest and instruction the shifters (already in 2009) to look at FEDTest instead of FED. In the meanwhile FED was completely broken and FEDTest was partly broken. FEDTest was now fixed in cms-sw#12240. We placed the contents of FEDTest into FED so that this can be the main default plot to look at again. We should stop running FEDTest in the near future and adapt the instructions at that moment.
A new Pull Request was created by @vanbesien (Broen van Besien) for CMSSW_7_4_X. Replaced the contents of the fed config by fedtest It involves the following packages: DQM/Integration @cmsbuild, @danduggan, @vanbesien, @deguio can you please review it and eventually sign? Thanks. |
please test |
+1 |
The tests are being triggered in jenkins. |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs after it passes the integration tests and once validation in the development release cycle CMSSW_7_6_X is complete. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_7_6_X is complete. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_7_6_X is complete. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
More than 6 years ago the work was started to replace FED by something
new. This was done by placing a second configuration next to FED, called
FEDTest and instructing the shifters (already in 2009) to look at
FEDTest instead of FED. In the meanwhile FED was completely broken and
FEDTest was partly broken. FEDTest was now fixed in #12240.
We placed the contents of FEDTest into FED so that this can be the main
default plot to look at again. We should stop running FEDTest in the
near future and adapt the instructions at that moment.