sling-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bertrand Delacretaz (JIRA)" <>
Subject [jira] [Commented] (SLING-6984) Allow to disable service user
Date Fri, 18 Aug 2017 13:38:00 GMT


Bertrand Delacretaz commented on SLING-6984:

@maret the below patch breaks the integration tests. probably because the service used doesn't

I think trying to disable a non-existing service user should just log an INFO message but
not fail, as in many cases I suppose what people mean is "disable service user if it exists".

Index: bundles/extensions/repoinit/it/src/main/provisioning/repoinit.txt
--- bundles/extensions/repoinit/it/src/main/provisioning/repoinit.txt	(revision 1805421)
+++ bundles/extensions/repoinit/it/src/main/provisioning/repoinit.txt	(working copy)
@@ -48,6 +48,8 @@
 create service user thirdUserFromProvisioningModel
 disable service user thirdUserFromProvisioningModel : "Disabled for testing reason"
+disable service user nonExisting3733bbc56b8a8364828d580821d97b88933c071d : "This should just
log a message, we really mean 'disable service user if it exists'"
 register namespace (slingtest)
 register nodetypes

> Allow to disable service user
> -----------------------------
>                 Key: SLING-6984
>                 URL:
>             Project: Sling
>          Issue Type: Improvement
>          Components: Repoinit
>    Affects Versions: Repoinit Parser 1.1.0, Repoinit JCR 1.1.4
>            Reporter: Timothee Maret
>            Assignee: Timothee Maret
>             Fix For: Repoinit Parser 1.1.2, Repoinit JCR 1.1.6
> Currently the repoinit module does not support -removing- disabling service users.
> Disbling -Removing- service user is required when one user is no longer needed due to
service user sharing among components.
> The syntax could be the one proposed by [~anchela]
> {code}
> disable service user <serviceUserId> <disableMessage>
> {code}

This message was sent by Atlassian JIRA

View raw message