diff --git a/leapp/repository/__init__.py b/leapp/repository/__init__.py index ec91b714..98e6855a 100644 --- a/leapp/repository/__init__.py +++ b/leapp/repository/__init__.py @@ -190,7 +190,15 @@ def mapped_actor_data(data): }) return data - # TODO: Do we need to add configs here as well? + # Note: `configs` are not present here because we are not yet making + # them globally accessible. This is to force people to copy the config + # schema to their Actors instead of importing them from other Actors. + # That copy, in turn, is a good idea so the framework can return an + # error if two Actors share the same config but they have different + # schemafor it (for instance, if Actor Foo and Bar were sharing the + # same config entry but Actor Foo updated the entry in a later version. + # We need to error so Actor Bar can either be ported to the new + # definition or use a different config entry for it's needs.) return { 'repo_dir': self._repo_dir, 'actors': [mapped_actor_data(a.serialize()) for a in self.actors],