Skip to content
This repository has been archived by the owner on Oct 13, 2023. It is now read-only.

Commit

Permalink
Fix assembly streams parsing
Browse files Browse the repository at this point in the history
Having `streams` overrides in assembly definition doesn't work because
of wrong field is parsed in `assembly_streams_config` function.
  • Loading branch information
vfreex committed May 26, 2023
1 parent aa8b080 commit fab6270
Showing 1 changed file with 1 addition and 11 deletions.
12 changes: 1 addition & 11 deletions doozerlib/assembly.py
Expand Up @@ -210,17 +210,7 @@ def assembly_streams_config(releases_config: Model, assembly: typing.Optional[st
:param assembly: The name of the assembly
:param streams_config: The streams config to merge into a new streams config (original Model will not be altered)
"""
if not assembly or not isinstance(releases_config, Model):
return streams_config

_check_recursion(releases_config, assembly)
target_assembly = releases_config.releases[assembly].assembly

if target_assembly.basis.assembly: # Does this assembly inherit from another?
# Recursively apply ancestor assemblies
streams_config = assembly_streams_config(releases_config, target_assembly.basis.assembly, streams_config)

target_assembly_streams = target_assembly.group
target_assembly_streams = _assembly_config_struct(releases_config, assembly, 'streams', {})
if not target_assembly_streams:
return streams_config

Expand Down

0 comments on commit fab6270

Please sign in to comment.