Skip to content
Snippets Groups Projects

Fixes WARNING Failed merging new config value

Merged Fixes WARNING Failed merging new config value
emoyse/athena:fixes-for-InDetConfig into master
All threads resolved!
Merged Edward Moyse requested to merge emoyse/athena:fixes-for-InDetConfig into master
All threads resolved!

With these changes, the previous Failed merging errors:

  17966 (InDet::InDetBoundaryCheckTool/InDetBoundaryCheckTool)
  53898 (InDet::InDetTrackSummaryHelperTool/InDetSummaryHelper)
  17966 (PixelDCSCondStateAlg/PixelDCSCondStateAlg)
  17966 (PixelDCSCondStatusAlg/PixelDCSCondStatusAlg)
  17966 (PixelDeadMapCondAlg/PixelDeadMapCondAlg)
   8983 (TileBadChanTool/TileBadChanTool)
   8983 (Trk::Navigator/AtlasNavigator)

reduce to these two:

bash-4.2$ grep "WARNING Failed merging new config value" log.RAWtoALL 
Py:conf2toConfigurable WARNING Failed merging new config value (/PIXEL/DCS/FSMSTATE) and old config value (ConditionStore+) for (ReadKeyState) property of PixelDCSCondStateAlg/PixelDCSCondStateAlg (PixelDCSCondStateAlg/PixelDCSCondStateAlg) old (new). Will take value from NEW configuration, but this should be checked!
Py:conf2toConfigurable WARNING Failed merging new config value (/PIXEL/DCS/FSMSTATUS) and old config value (ConditionStore+) for (ReadKeyStatus) property of PixelDCSCondStatusAlg/PixelDCSCondStatusAlg (PixelDCSCondStatusAlg/PixelDCSCondStatusAlg) old (new). Will take value from NEW configuration, but this should be checked!

These need an expert to look into them.

For the ones this MR fixes, there are two types - real ones, from incompletely setup new configuration, and false positives due to GaudiConf2 reporting DataHandle values differently.

Ping: @tadej @wlampl @goetz @elmsheus

Edited by Edward Moyse

Merge request reports

Pipeline #2338466 passed

Stage: external

Pipeline passed for 6156c1ab on emoyse:fixes-for-InDetConfig

Approval is optional

Merged by Edward MoyseEdward Moyse 4 years ago (Feb 23, 2021 4:38pm UTC)

Merge details

  • Changes merged into master with 1f135d78 (commits were squashed).
  • Did not delete the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply