microsoft / omi

Open Management Infrastructure
Other
368 stars 116 forks source link

Don't delete and re-create the user 'omi' if it is already exist #657

Closed deepakjain111 closed 4 years ago

deepakjain111 commented 4 years ago

In Linux: as part of installation, don't delete and re-create the user 'omi' if it is already exist irrespective of user type is normal or system user or any other attribute.

sarojcare commented 4 years ago

Is there any dependency on omi group? Do we need to make validation that omi user existing is also part of omi group?

deepakjain111 commented 4 years ago

Is there any dependency on omi group? Do we need to make validation that omi user existing is also part of omi group?

I don't think so. We don't need that validation.

sarojcare commented 4 years ago

Whether pbuild and unit testing has done?

Also the comment line should not be longer than 70 chars. In case we need to have longer string please put it in a file with each line not having 70 chars and use it to commit.

deepakjain111 commented 4 years ago

Whether pbuild and unit testing has done?

Also the comment line should not be longer than 70 chars. In case we need to have longer string please put it in a file with each line not having 70 chars and use it to commit.

I have done unit testing and will ask testing team to test on all Linux system