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

Use the latest Express GT as a fallback, and fix http proxy setting in curl for Tier0 DataService query for Event Display applications. #15298

Merged
merged 2 commits into from Aug 10, 2016

Conversation

diguida
Copy link
Contributor

@diguida diguida commented Jul 27, 2016

The fallback GT for the Event display applications is updated to the latest version used at Tier0.
Back-port of commit 8b9abb0 in #15296

Fix http proxy setting in curl for Tier0 DataService query in DQM/Integration/python/config/FrontierCondition_GT_autoExpress_cfi.py: the invalid parameter was causing the event display to always fall back to the default Express GT.
Back-port of commit 349b25c in #15337

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @diguida (Salvatore Di Guida) for CMSSW_8_0_X.

It involves the following packages:

DQM/Integration

@cmsbuild, @dmitrijus, @vanbesien, @davidlange6 can you please review it and eventually sign? Thanks.
@threus, @batinkov this is something you requested to watch as well.
@slava77, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

@diguida diguida changed the title Use the latest Express GT as a fallback for EvD applications. Use the latest Express GT as a fallback for Event Display applications. Jul 27, 2016
@dmitrijus
Copy link
Contributor

+1

@cmsbuild
Copy link
Contributor

cmsbuild commented Jul 27, 2016

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

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_8_0_X IBs after it passes the integration tests. This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @smuzaffar

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

@davidlange6
Copy link
Contributor

what is the 81x PR? @diguida

@diguida
Copy link
Contributor Author

diguida commented Jul 28, 2016

@davidlange6 the PR in 81X, namely #15296, includes also the change in the connection string:

  • d62db6b remove the host domain lookup
  • f568b04 overrides the connection string for the online DQM applications
  • 8b9abb0 is the forward port of this one, i.e. changes the GT.

I can split #15296 into two, if you prefer.

@davidlange6
Copy link
Contributor

no- its fine as it is - Thanks!

…egration/python/config/FrontierCondition_GT_autoExpress_cfi.py.
@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 1, 2016

Pull request #15298 was updated. @cmsbuild, @dmitrijus, @vanbesien, @davidlange6 can you please check and sign again.

@diguida diguida changed the title Use the latest Express GT as a fallback for Event Display applications. Use the latest Express GT as a fallback, and fix http proxy setting in curl for Tier0 DataService query for Event Display applications. Aug 1, 2016
@diguida
Copy link
Contributor Author

diguida commented Aug 1, 2016

Added commit 13c6ade
Fix http proxy setting in curl for Tier0 DataService query in DQM/Integration/python/config/FrontierCondition_GT_autoExpress_cfi.py: the invalid parameter was causing the event display to always fall back to the default Express GT.
Back-port of commit 349b25c in #15337

@dmitrijus
Copy link
Contributor

+1

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 2, 2016

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

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 2, 2016

This pull request is fully signed and it will be integrated in one of the next CMSSW_8_0_X IBs after it passes the integration tests. This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @smuzaffar

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 2, 2016

-1

Tested at: 13c6ade

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

I found follow errors while testing this PR

Failed tests: RelVals

  • RelVals:

When I ran the RelVals I found an error in the following worklfows:
140.53 step1

DAS Error

@davidlange6
Copy link
Contributor

+1

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