Skip to content

Conversation

@smuzaffar
Copy link
Contributor

This should fix the issue with ibeos-lfn-sort when called recursively. With this change following commands now return the correct results

> export CMSSW_USE_IBEOS=true
>  dasgoclient --limit 0 --query 'file dataset=/Cosmics/Run2011A-v1/RAW run=160960 site=T2_CH_CERN' | head -2
root://eoscms.cern.ch//store/user/cmsbuild/store/data/Run2011A/Cosmics/RAW/v1/000/160/960/049F6443-8E53-E011-A943-003048F117EA.root
/store/data/Run2011A/Cosmics/RAW/v1/000/160/960/062C43C2-8353-E011-BC2B-001617C3B76E.root
>  dasgoclient --limit 0 --query 'file dataset=/Cosmics/Run2011A-v1/RAW run=160960 site=T2_CH_CERN' | ibeos-lfn-sort | head -2
root://eoscms.cern.ch//store/user/cmsbuild/store/data/Run2011A/Cosmics/RAW/v1/000/160/960/049F6443-8E53-E011-A943-003048F117EA.root
/store/data/Run2011A/Cosmics/RAW/v1/000/160/960/062C43C2-8353-E011-BC2B-001617C3B76E.root
[muzaffar@cmsdev40 src]$ git diff

FYI @kpedro88

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 17, 2025

cms-bot internal usage

@cmsbuild
Copy link
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-49402/46854

  • There are other open Pull requests which might conflict with changes you have proposed:

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @smuzaffar for master.

It involves the following packages:

  • Utilities/General (core)

@Dr15Jones, @cmsbuild, @makortel, @smuzaffar can you please review it and eventually sign? Thanks.
@wddgit this is something you requested to watch as well.
@ftenchini, @mandrenguyen, @sextonkennedy you are the release manager for this.

cms-bot commands are listed here

@cmsbuild
Copy link
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-49402/46855

  • There are other open Pull requests which might conflict with changes you have proposed:

@cmsbuild
Copy link
Contributor

Pull request #49402 was updated. @Dr15Jones, @cmsbuild, @makortel, @smuzaffar can you please check and sign again.

@smuzaffar
Copy link
Contributor Author

smuzaffar commented Nov 17, 2025

closing it as duplicate of #49401

@smuzaffar smuzaffar closed this Nov 17, 2025
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.

2 participants