Fix lazy-download handling of file systems that are mounted more than once

[dan131riley]

When checking for a local file system for lazy download, handle the case where a file system is listed more than once in /proc/self/mounts by checking same-length path matches until a local file system is found. Workaround for issue found with Tier0 merge jobs on VMs where / is the only file system:

https://hypernews.cern.ch/HyperNews/CMS/get/edmFramework/3689.html

[cmsbuild]

A new Pull Request was created by @dan131riley (Dan Riley) for CMSSW_8_1_X.

It involves the following packages:

Utilities/StorageFactory

@cmsbuild, @smuzaffar, @Dr15Jones, @davidlange6 can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @wddgit this is something you requested to watch as well.
@slava77, @smuzaffar you are the release manager for this.

cms-bot commands are listed here #13028

[Dr15Jones]

please test

[cmsbuild]

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

[Dr15Jones]

+1

[cmsbuild]

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

[Dr15Jones]

@davidlange6 this is a change we are testing to solve the Tier 0 lazy-download problem.

[cmsbuild]

+1
Tested at: f6eaecd
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-16409/16117/summary.html

[cmsbuild]

Comparison job queued.

[davidlange6]

Right - how do we test that it works before this reaches the tier0?

On Nov 1, 2016, at 3:37 PM, Chris Jones notifications@github.com wrote:

@davidlange6 this is a change we are testing to solve the Tier 0 lazy-download problem.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

[cmsbuild]

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

[davidlange6]

+1