[SLING-6398] Repoinit should not attempt to create access control entries when no changes are needed

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: Repoinit JCR 1.1.2
• Component/s: Repoinit
• Assignee: Robert Munteanu
• Reporter: Robert Munteanu
• Created: 14/Dec/16 14:15
• Updated: 19/Dec/16 11:21
• Resolved: 15/Dec/16 10:14

Description

I have a more complex Sling setup based on the recent Oak multiplexing additions.

The repository is split bewteen

• /libs and /apps, read-only
• the rest of the repository, read-write

When the provisioning model contains ACL definitions, they are processed directly without checking if they exist. In turn, Oak updates the definitions, even if equivalent ones exist. This causes the repoinit part to fail if it refers to ACLs for the read-only part of the repository.

I would propose that the repoinit statements check if the ACL really needs to be replaced or if it can be skipped. This also has the advantage of making it symmetric with the checks for service users and paths and also should slightly reduce provisioning time.

bdelacretaz - would that work for you?