[jira] [Commented] (SLING-6984) Allow to disable service user

2017-08-18 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6984?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16133021#comment-16133021 ] Bertrand Delacretaz commented on SLING-6984: The ITs do pass as is, they fail if I apply the

[jira] [Commented] (SLING-6984) Allow to disable service user

2017-08-18 Thread Timothee Maret (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6984?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16133014#comment-16133014 ] Timothee Maret commented on SLING-6984: --- [~bdelacretaz] I re-ran the IT tests successfully locally.

[jira] [Commented] (SLING-6984) Allow to disable service user

2017-08-18 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6984?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16132193#comment-16132193 ] Bertrand Delacretaz commented on SLING-6984: @maret the below patch breaks the integration

[jira] [Commented] (SLING-6984) Allow to disable service user

2017-08-18 Thread Timothee Maret (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6984?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16132161#comment-16132161 ] Timothee Maret commented on SLING-6984: --- Thanks [~bdelacretaz]! Commit at

[jira] [Commented] (SLING-6984) Allow to disable service user

2017-08-18 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6984?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16132152#comment-16132152 ] Bertrand Delacretaz commented on SLING-6984: Thanks, looks good to me! > Allow to disable

[jira] [Commented] (SLING-6984) Allow to disable service user

2017-08-18 Thread Timothee Maret (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6984?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16132127#comment-16132127 ] Timothee Maret commented on SLING-6984: --- Thanks [~bdelacretaz]! I have implemented the remaining in

[jira] [Commented] (SLING-6984) Allow to disable service user

2017-08-18 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6984?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16132061#comment-16132061 ] Bertrand Delacretaz commented on SLING-6984: The parser now requires a non-empty reason for

[jira] [Commented] (SLING-6984) Allow to disable service user

2017-08-18 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6984?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16132047#comment-16132047 ] Bertrand Delacretaz commented on SLING-6984: bq. The implementation will enable the user if a

[jira] [Commented] (SLING-6984) Allow to disable service user

2017-08-18 Thread Timothee Maret (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6984?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16132009#comment-16132009 ] Timothee Maret commented on SLING-6984: --- Thanks [~bdelacretaz]! The Jackrabbit API provides a

[jira] [Commented] (SLING-6984) Allow to disable service user

2017-08-18 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6984?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16131954#comment-16131954 ] Bertrand Delacretaz commented on SLING-6984: Note also that we'll need to update

[jira] [Commented] (SLING-6984) Allow to disable service user

2017-08-17 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6984?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16130056#comment-16130056 ] Bertrand Delacretaz commented on SLING-6984: I have fixed the failing integration tests and

[jira] [Commented] (SLING-6984) Allow to disable service user

2017-08-15 Thread Timothee Maret (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6984?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16127382#comment-16127382 ] Timothee Maret commented on SLING-6984: --- [~anchela] I have updated the description and title. Thanks