Project

General

Profile

Actions

User story #3752

closed

User Management: add an option to remove user home directory when deleting user account

Added by Dennis Cabooter over 11 years ago. Updated almost 3 years ago.

Status:
Rejected
Priority:
3
Assignee:
-
Category:
Techniques
UX impact:
Suggestion strength:
User visibility:
Effort required:
Name check:
Fix check:
Regression:

Description

I've created a new feature in the userManagement technique. The feature makes it possible to pass the "-r" option to userdel and it works. However, I can't get the reporting right.

diff -Nru systemSettings/userManagement/userManagement/2.0/metadata.xml systemSettings/userManagement/userManagement/2.1/metadata.xml
--- systemSettings/userManagement/userManagement/2.0/metadata.xml       2013-06-11 16:09:35.000000000 +0200
+++ systemSettings/userManagement/userManagement/2.1/metadata.xml       2013-07-17 10:25:35.000000000 +0200
@@ -70,6 +70,10 @@
           <VALUE>remove</VALUE>
         </ITEM>
         <ITEM>
+          <LABEL>Remove and delete home dir</LABEL>
+          <VALUE>removehome</VALUE>
+        </ITEM>
+        <ITEM>
           <LABEL>Check only (account should exist)</LABEL>
           <VALUE>checkhere</VALUE>
         </ITEM>
diff -Nru systemSettings/userManagement/userManagement/2.0/userManagement.st systemSettings/userManagement/userManagement/2.1/userManagement.st
--- systemSettings/userManagement/userManagement/2.0/userManagement.st  2013-07-17 09:53:30.000000000 +0200
+++ systemSettings/userManagement/userManagement/2.1/userManagement.st  2013-07-17 13:21:08.000000000 +0200
@@ -91,6 +91,8 @@

       "usermanagement_user_remove_${usergroup_user_index}" expression => strcmp("${usergroup_user_action[${usergroup_user_index}]}","remove");

+      "usermanagement_user_removehome_${usergroup_user_index}" expression => strcmp("${usergroup_user_action[${usergroup_user_index}]}","removehome");
+
       "usermanagement_user_checkpres_${usergroup_user_index}" expression => strcmp("${usergroup_user_action[${usergroup_user_index}]}","checkhere");

       "usermanagement_user_checkabs_${usergroup_user_index}" expression => strcmp("${usergroup_user_action[${usergroup_user_index}]}","checknothere");
@@ -157,6 +159,12 @@
         comment => "Delete the user ${usergroup_user_login[${usergroup_user_index}]}",
         ifvarclass => "usermanagement_user_exists_${usergroup_user_index}.usermanagement_user_remove_${usergroup_user_index}";

+      "/usr/sbin/userdel" 
+        args => "-r ${usergroup_user_login[${usergroup_user_index}]}",
+        classes => cf2_if_else("usermanagement_login_remove_${usergroup_user_index}_repaired", "usermanagement_login_remove_${usergroup_user_index}_error"),
+        comment => "Delete the user ${usergroup_user_login[${usergroup_user_index}]}",
+        ifvarclass => "usermanagement_user_exists_${usergroup_user_index}.usermanagement_user_removehome_${usergroup_user_index}";
+
   files:

       "/etc/passwd"

Related issues 2 (0 open2 closed)

Has duplicate Rudder - User story #1938: User Management: add an option to remove user home directory when deleting user accountRejected2011-10-13Actions
Has duplicate Rudder - User story #3971: Add an option to remove home folder when deleting a user with userManagement techniqueRejectedActions
Actions #1

Updated by Nicolas CHARLES over 11 years ago

  • Project changed from Rudder to 24
  • Category set to Techniques

Dennis,

Oh, indeed you need to change the reports part.
In the reports part, you have a # remove user section, with three reports (success, repaired and error), where reports are made if a combinaison of class is defined (or not defined), and if the class usermanagement_user_remove_${usergroup_user_index} exists

You could dupliate these lines, changing the text in the reports to say that it delete the home folder as well, and replace usermanagement_user_remove_${usergroup_user_index} by usermanagement_user_removehome_${usergroup_user_index}

Does that help you ?

Actions #2

Updated by Jonathan CLARKE over 11 years ago

  • Subject changed from New feature userManagement technique to User Management: add an option to remove user home directory when deleting user account
  • Status changed from New to Discussion
  • Assignee set to Dennis Cabooter
Actions #3

Updated by Vincent MEMBRÉ about 11 years ago

  • Status changed from Discussion to 13
  • Assignee changed from Dennis Cabooter to Vincent MEMBRÉ
  • Target version set to 2.4.10

Dennis, I'll create a pull request from your patch and I'll add Nicolas suggestion.

Thanks for your contribution Dennis!

Actions #4

Updated by Nicolas PERRON about 11 years ago

  • Target version changed from 2.4.10 to 2.4.11
Actions #5

Updated by Nicolas PERRON about 11 years ago

  • Target version changed from 2.4.11 to 2.4.12
Actions #6

Updated by Nicolas PERRON about 11 years ago

  • Target version changed from 2.4.12 to 2.4.13
Actions #7

Updated by Vincent MEMBRÉ almost 11 years ago

  • Target version changed from 2.4.13 to 2.6.11

Since 2.4 is not maintained anymore, retargeting to branch 2.6

Actions #8

Updated by Vincent MEMBRÉ almost 11 years ago

  • Target version changed from 2.6.11 to 2.6.12
Actions #9

Updated by Vincent MEMBRÉ almost 11 years ago

  • Target version changed from 2.6.12 to 2.6.13
Actions #10

Updated by Vincent MEMBRÉ over 10 years ago

  • Target version changed from 2.6.13 to 2.6.14
Actions #11

Updated by Jonathan CLARKE over 10 years ago

  • Target version changed from 2.6.14 to 2.6.16
Actions #12

Updated by Jonathan CLARKE over 10 years ago

  • Target version changed from 2.6.16 to 2.6.17
Actions #13

Updated by Nicolas PERRON over 10 years ago

  • Target version changed from 2.6.17 to 2.6.18
Actions #14

Updated by Matthieu CERDA about 10 years ago

  • Target version changed from 2.6.18 to 2.6.19
Actions #15

Updated by Vincent MEMBRÉ about 10 years ago

  • Target version changed from 2.6.19 to 2.6.20
Actions #16

Updated by François ARMAND almost 10 years ago

  • Status changed from 13 to Discussion
  • Target version changed from 2.6.20 to 2.10.10

To be sure, the PR wasn't ever done, right ?

So, what are we doing with that? Is it still needed in 2.10 and up?

Actions #17

Updated by Vincent MEMBRÉ almost 10 years ago

  • Target version changed from 2.10.10 to 2.10.11
Actions #18

Updated by Vincent MEMBRÉ almost 10 years ago

  • Assignee changed from Vincent MEMBRÉ to Benoît PECCATTE
Actions #19

Updated by Vincent MEMBRÉ almost 10 years ago

Benoit can you please look at it please !

Actions #20

Updated by Vincent MEMBRÉ almost 10 years ago

  • Target version changed from 2.10.11 to 2.10.12
Actions #21

Updated by Benoît PECCATTE almost 10 years ago

  • Project changed from 24 to Rudder
  • Category changed from Techniques to Techniques
Actions #22

Updated by Vincent MEMBRÉ almost 10 years ago

  • Target version changed from 2.10.12 to 2.10.13
Actions #23

Updated by Vincent MEMBRÉ over 9 years ago

  • Target version changed from 2.10.13 to 2.10.14
Actions #24

Updated by Vincent MEMBRÉ over 9 years ago

  • Target version changed from 2.10.14 to 2.10.15
Actions #25

Updated by Vincent MEMBRÉ over 9 years ago

  • Target version changed from 2.10.15 to 2.10.16
Actions #26

Updated by Vincent MEMBRÉ over 9 years ago

  • Target version changed from 2.10.16 to 2.10.17
Actions #27

Updated by Vincent MEMBRÉ over 9 years ago

  • Target version changed from 2.10.17 to 2.10.18
Actions #28

Updated by Vincent MEMBRÉ about 9 years ago

  • Target version changed from 2.10.18 to 2.10.19
Actions #29

Updated by Vincent MEMBRÉ about 9 years ago

  • Target version changed from 2.10.19 to 2.10.20
Actions #30

Updated by Vincent MEMBRÉ about 9 years ago

  • Target version changed from 2.10.20 to 2.11.18
Actions #31

Updated by Vincent MEMBRÉ almost 9 years ago

  • Target version changed from 2.11.18 to 2.11.19
Actions #32

Updated by Vincent MEMBRÉ almost 9 years ago

  • Target version changed from 2.11.19 to 2.11.20
Actions #33

Updated by Vincent MEMBRÉ over 8 years ago

  • Target version changed from 2.11.20 to 2.11.21
Actions #34

Updated by Vincent MEMBRÉ over 8 years ago

  • Target version changed from 2.11.21 to 2.11.22
Actions #35

Updated by Vincent MEMBRÉ over 8 years ago

  • Target version changed from 2.11.22 to 2.11.23
Actions #36

Updated by Vincent MEMBRÉ over 8 years ago

  • Target version changed from 2.11.23 to 2.11.24
Actions #37

Updated by Vincent MEMBRÉ over 8 years ago

  • Target version changed from 2.11.24 to 308
Actions #38

Updated by Vincent MEMBRÉ over 8 years ago

  • Target version changed from 308 to 3.1.14
Actions #39

Updated by Vincent MEMBRÉ about 8 years ago

  • Target version changed from 3.1.14 to 3.1.15
Actions #40

Updated by Vincent MEMBRÉ about 8 years ago

  • Target version changed from 3.1.15 to 3.1.16
Actions #41

Updated by Vincent MEMBRÉ about 8 years ago

  • Target version changed from 3.1.16 to 3.1.17
Actions #42

Updated by Vincent MEMBRÉ about 8 years ago

  • Target version changed from 3.1.17 to 3.1.18
Actions #43

Updated by Vincent MEMBRÉ almost 8 years ago

  • Target version changed from 3.1.18 to 3.1.19
Actions #44

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 3.1.19 to 3.1.20
Actions #45

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 3.1.20 to 3.1.21
Actions #46

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 3.1.21 to 3.1.22
Actions #47

Updated by Benoît PECCATTE over 7 years ago

  • Assignee deleted (Benoît PECCATTE)
Actions #48

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 3.1.22 to 3.1.23
Actions #49

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 3.1.23 to 3.1.24
Actions #50

Updated by Vincent MEMBRÉ about 7 years ago

  • Target version changed from 3.1.24 to 3.1.25
Actions #51

Updated by Benoît PECCATTE about 7 years ago

  • Target version changed from 3.1.25 to 4.1.9
Actions #52

Updated by Vincent MEMBRÉ about 7 years ago

  • Target version changed from 4.1.9 to 4.1.10
Actions #53

Updated by Benoît PECCATTE almost 7 years ago

  • Target version changed from 4.1.10 to Ideas (not version specific)
Actions #54

Updated by Alexis Mousset over 6 years ago

  • Has duplicate User story #3971: Add an option to remove home folder when deleting a user with userManagement technique added
Actions #55

Updated by Alexis Mousset almost 3 years ago

This won’t be added to that technique, please use the technique editor for that. If you are missing some capabilities in it, please open a ticket for that need.

Actions #56

Updated by Alexis Mousset almost 3 years ago

  • Status changed from Discussion to Rejected
Actions

Also available in: Atom PDF