update edmPickEvents since das_client can not be imported

[davidlange6]

some work needed I think. - but this version works again
@VinInn

[cmsbuild]

A new Pull Request was created by @davidlange6 (David Lange) for CMSSW_8_1_X.

It involves the following packages:

PhysicsTools/Utilities

@cmsbuild, @davidlange6 can you please review it and eventually sign? Thanks.
@slava77, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

[davidlange6]

please test

[cmsbuild]

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

[cmsbuild]

+1
Tested at: a113d4a
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-15479/14537/summary.html

[cmsbuild]

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

[davidlange6]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_8_1_X IBs (tests are also fine). This pull request will be automatically merged.

[nsmith-]

If das_client.py in the CMSSW python path gets updated to a new version (v02.17.04-giojec or newer), then this patch would work:

diff --git a/PhysicsTools/Utilities/scripts/edmPickEvents.py b/PhysicsTools/Utilities/scripts/edmPickEvents.py
index 6771946..305a874 100755
--- a/PhysicsTools/Utilities/scripts/edmPickEvents.py
+++ b/PhysicsTools/Utilities/scripts/edmPickEvents.py
@@ -87,7 +87,7 @@ def getFileNames (event):
     files = []
     # Query DAS
     query = "file dataset=%(dataset)s run=%(run)i lumi=%(lumi)i | grep file.name" % event
-    jsondict = das_client.get_data('https://cmsweb.cern.ch', query, 0, 0, False)
+    jsondict = das_client.get_data('https://cmsweb.cern.ch', query, 0, 0, False, ckey=das_client.x509(), cert=das_client.x509())
     status = jsondict['status']
     if status != 'ok':
         print "DAS query status: %s"%(status)

Also, I noticed the method in das_client shell script to pick newest version will prefer v02.17.04 to v02.17.04-giojec. Perhaps that is intentional to limit to full releases.