Skip to content

Commit

Permalink
Fix order in which included spec blocks get resolved
Browse files Browse the repository at this point in the history
  • Loading branch information
eacharles committed Jul 1, 2024
1 parent 2a0aef7 commit 0a2205d
Showing 1 changed file with 3 additions and 6 deletions.
9 changes: 3 additions & 6 deletions src/lsst/cmservice/client/loaders.py
Original file line number Diff line number Diff line change
Expand Up @@ -106,11 +106,8 @@ def _upsert_spec_block(
},
)

for include_key, include_val in include_data.items():
if include_key in block_data and isinstance(include_val, Mapping):
block_data[include_key].update(include_val)
else:
block_data[include_key] = include_val
block_data = include_data.copy()
update_include_dict(block_data, config_values.copy())

handler = block_data.pop("handler", None)
if spec_block is None:
Expand Down Expand Up @@ -397,7 +394,7 @@ def campaign_cl(
camp_config.setdefault(key, {})
val = kwargs.get(key, None)
if val:
camp_config[key].update(val)
update_include_dict(camp_config[key], val)

production = self._parent.production.get_row_by_name(parent_name)
if not production:
Expand Down

0 comments on commit 0a2205d

Please sign in to comment.