Add missing include in CondDBTools.cc

[iarspider]

PR description:

Build log: https://cmssdt.cern.ch/SDT/cgi-bin/buildlogs/slc7_amd64_gcc900/CMSSW_12_1_X_2021-08-13-1100/CondCore/Utilities

Error message:

/data/cmsbld/jenkins/workspace/build-any-ib/w/tmp/BUILDROOT/6832eed357397f6b862ceadce6a298f6/opt/cmssw/slc7_amd64_gcc900/cms/cmssw-patch/CMSSW_12_1_X_2021-08-13-1100/src/CondCore/Utilities/src/CondDBTools.cc: In function 'size_t cond::persistency::importIovs(const string&, cond::persistency::Session&, const string&, cond::persistency::Session&, cond::Time_t, cond::Time_t, const string&, const string&, bool, bool, bool)':
  /data/cmsbld/jenkins/workspace/build-any-ib/w/tmp/BUILDROOT/6832eed357397f6b862ceadce6a298f6/opt/cmssw/slc7_amd64_gcc900/cms/cmssw-patch/CMSSW_12_1_X_2021-08-13-1100/src/CondCore/Utilities/src/CondDBTools.cc:97:12: error: 'set' is not a member of 'std'
    97 |       std::set<cond::Hash> pids;
      |            ^~~
/data/cmsbld/jenkins/workspace/build-any-ib/w/tmp/BUILDROOT/6832eed357397f6b862ceadce6a298f6/opt/cmssw/slc7_amd64_gcc900/cms/cmssw-patch/CMSSW_12_1_X_2021-08-13-1100/src/CondCore/Utilities/src/CondDBTools.cc:4:1: note: 'std::set' is defined in header '<set>'; did you forget to '#include <set>'?
    3 | #include "CondCore/CondDB/interface/ConnectionPool.h"
  +++ |+#include <set>
    4 | //

Only affects CMSSW_12_1_X for some reason

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-34869/24672

• This PR adds an extra 12KB to repository

[cmsbuild]

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

It involves the following packages:

• CondCore/Utilities (db)

@ggovi, @cmsbuild can you please review it and eventually sign? Thanks.
@mmusich this is something you requested to watch as well.
@perrotta, @dpiparo, @qliphy you are the release manager for this.

cms-bot commands are listed here

[Dr15Jones]

Please test

[Dr15Jones]

@smuzaffar do we know which pull request broke the IB and how our tests missed it?

[Dr15Jones]

It looks like it was these two pull requests

#34847
#34826

where the latter was tested in an IB that existed before the former was merged into the master.

[smuzaffar]

thats correct @Dr15Jones , combination of #34847 and #34826 caused this issue.
@qliphy , @perrotta, this is trivial technical fix. I would suggest to merge it so that we can start a new working IB

[qliphy]

merge

[smuzaffar]

thanks @qliphy, I have started 17h00 IB.

[cmsbuild]

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-55d01b/17766/summary.html
COMMIT: e346f3f
CMSSW: CMSSW_12_1_X_2021-08-12-2300/slc7_amd64_gcc900
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/34869/17766/install.sh to create a dev area with all the needed externals and cmssw changes.

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:

• @Dr15Jones Make header files in FWCore/Framework/interface consistent #34771
• @smuzaffar [SimCalorimetry] Avoid using private CondFormats/EcalObjects/src/classes.h #34849
• @smuzaffar Coderule script: fixes needed for python3 #34855
• @bsunanda Run3-sim101 Modified the examination of geometry in XML and DB #34857
• @bsunanda Run3-gex87 Modify material budget code relevant to study XML vs DB #34856
• @bsunanda Run3-sim100 Take Vladimir's comments for include statements of CLHEP headers #34842
• @iarspider [GCC11] Fix warning: loop variable binds to a temporary #34852
• @Dr15Jones Performance improvements for DQM startup #34837
• @mmusich [RFC] add commissioning_run as DQM runType and use it in SiStrip online DQM client #34832
• @smuzaffar Replace boost::regex with std::regex #34826

You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-55d01b/17766/git-recent-commits.json
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-55d01b/17766/git-merge-result

Comparison Summary

Summary:

• No significant changes to the logs found
• Reco comparison results: 4 differences found in the comparisons
• DQMHistoTests: Total files compared: 39
• DQMHistoTests: Total histograms compared: 3001486
• DQMHistoTests: Total failures: 6
• DQMHistoTests: Total nulls: 1
• DQMHistoTests: Total successes: 3001457
• DQMHistoTests: Total skipped: 22
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 1.736 KiB( 38 files compared)
• DQMHistoSizes: changed ( 1325.81,... ): 0.870 KiB Physics/NanoAODDQM
• DQMHistoSizes: changed ( 312.0 ): -0.004 KiB MessageLogger/Warnings
• Checked 165 log files, 37 edm output root files, 39 DQM output files
• TriggerResults: no differences found