Thanks for the answer, Let me be more concrete: I propose to revert https://issues.apache.org/jira/browse/SLING-9449. The description doesn't really say why throwing the exception is a problem and IMHO the exception helps to prevent undesired target states of the repo. WDYT? Thanks, Konrad
> On 19. Mar 2021, at 15:23, Bertrand Delacretaz <[email protected]> wrote: > > Hi, > > On Tue, Mar 16, 2021 at 10:02 AM Konrad Windszus <[email protected]> wrote: >> ...since https://issues.apache.org/jira/browse/SLING-9449 >> <https://issues.apache.org/jira/browse/SLING-9449> a >> failure of setting principal ACLs just leads to a WARN... > > In general I think any failed repoinit statement should throw an > exception to prevent the repository from starting. > > There might be edge cases though, and depending on how it's been done > so far we might have backwards compatibility issues if we change > behaviors. > > Best is probably to handle things on a case by case basis for existing issues? > > And maybe also document this desired behavior at > https://sling.apache.org/documentation/bundles/repository-initialization.html > for clarity? > > -Bertrand
