Skip to content

Commit 359ae8d

Browse files
committed
[QC-1274] Fix missing ccdb url in PostProcessing (#2528)
- a bad merge in the original PR removed this line.
1 parent dd61ab5 commit 359ae8d

File tree

3 files changed

+6
-0
lines changed

3 files changed

+6
-0
lines changed

Framework/src/PostProcessingConfig.cxx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@ PostProcessingConfig::PostProcessingConfig(const std::string& id, const boost::p
3737
moduleName = config.get<std::string>("qc.postprocessing." + id + ".moduleName");
3838
className = config.get<std::string>("qc.postprocessing." + id + ".className");
3939
detectorName = config.get<std::string>("qc.postprocessing." + id + ".detectorName", "MISC");
40+
ccdbUrl = config.get<std::string>("qc.config.conditionDB.url", "");
4041
consulUrl = config.get<std::string>("qc.config.consul.url", "");
4142
// if available, use the source repo as defined in the postprocessing task, otherwise the general QCDB
4243
auto sourceRepo = config.get_child_optional("qc.postprocessing." + id + ".sourceRepo");

Framework/test/testPostProcessingConfig.cxx

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,4 +50,6 @@ BOOST_AUTO_TEST_CASE(test_configuration_read)
5050
BOOST_CHECK_EQUAL(ppconfig.stopTriggers[0], "once");
5151

5252
BOOST_CHECK_EQUAL(ppconfig.customParameters.size(), 4);
53+
54+
BOOST_CHECK_EQUAL(ppconfig.ccdbUrl, "ccdb-test.cern.ch:8080");
5355
}

Framework/test/testSharedConfig.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,9 @@
1414
},
1515
"monitoring": {
1616
"url": "infologger:///debug?qc"
17+
},
18+
"conditionDB": {
19+
"url": "ccdb-test.cern.ch:8080"
1720
}
1821
},
1922
"tasks": {

0 commit comments

Comments
 (0)