Skip to content

#15959: groupadd system parameter is not supported on solaris#1052

Merged
slippycheeze merged 7 commits intopuppetlabs:2.7.xfrom
stschulte:ticket/2.7.x/15959_group_systemparam
Sep 4, 2012
Merged

#15959: groupadd system parameter is not supported on solaris#1052
slippycheeze merged 7 commits intopuppetlabs:2.7.xfrom
stschulte:ticket/2.7.x/15959_group_systemparam

Conversation

@stschulte
Copy link
Contributor

On Solaris and HP-UX the groupadd and useradd commands to not support the -r switch to create systemusers and systemgroups.

Do not claim to support feature system_users and system_groups on Solaris and HP-UX.

Tests are now ordered by method, so if modifications have to be done
done in the provider code, it will be easier to track the according spec
tests that have to be changed.
It seams to be more reliable to work with real objects instead
According to

http://docs.oracle.com/cd/E19963-01/html/821-1462/useradd-1m.html
http://nixdoc.net/man-pages/hp-ux/man1/useradd.1m.html

neither Solaris nor HP-UX support the -r switch to create system users
so the useradd provider should not claim to support system_users on
these platforms.
Tests are now ordered by method, so if modifications have to be done
done in the provider code, it will be easier to track the according spec
tests that have to be changed.
It seams to be more reliable to work with real objects instead
According to

http://nixdoc.net/man-pages/hp-ux/man1/groupadd.1m.html
http://docs.oracle.com/cd/E19963-01/html/821-1462/groupadd-1m.html

neither Solaris nor HP-UX support the -r switch to create system groups
so the groupadd provider should not claim to support system_groups on
these platforms.
@stschulte
Copy link
Contributor Author

If you think this change should go in 3.x or master rather then 2.7.x please tell me so I can rebase.

@vrthra
Copy link
Contributor

vrthra commented Aug 18, 2012

your commits seems have the same messages (5e64 -> 6350) (e111 -> c50f5) (03c7-> c404), is that intensional?

@stschulte
Copy link
Contributor Author

@vrthra i first modified the useradd provider (e111f56, 03c7ddf, 5e64c1f) and then I noticed that the same task has to be done for the groupadd provider (rearrange tests, use real objects in the specs, remove the -r switch on solaris) so this is intentional. The commit messages are not the same by the way, one set talks about useradd, the second set of commits about groupadd.

@vrthra
Copy link
Contributor

vrthra commented Aug 20, 2012

Oh sorry, didn't notice useradd/groupadd.

@vrthra
Copy link
Contributor

vrthra commented Aug 25, 2012

@stschulte , I think this should be targeted to 3.x (as you noted in your comment.)

@slippycheeze
Copy link
Contributor

I get 17 test failures running this on the Mac, which I will add to the ticket. I will try and find some time to look into them if you can't, but I don't think that will be for a week or more.

@slippycheeze
Copy link
Contributor

This should, however, target 2.7.x as noted - it is pretty clearly a bug fix worth shipping in the stable release.

@stschulte
Copy link
Contributor Author

@daniel-pittman The specs should now always pick the useradd/groupadd provider. Please let me know if the specs are still failing.

The groupadd and useradd spec will only work if groupadd / useradd are
the default provider for the group / user type.

Explicitly set the provider in the spec tests so they can e.g. also run
on MacOSX.
@stschulte
Copy link
Contributor Author

@daniel-pittman The specs should now always pick the useradd/groupadd provider. Please let me know if the specs are still failing.

slippycheeze added a commit that referenced this pull request Sep 4, 2012
…emparam

#15959: groupadd system parameter is not supported on solaris
@slippycheeze slippycheeze merged commit 609742b into puppetlabs:2.7.x Sep 4, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants