Some bugs to report

[jimmy]

Hi,

I am working my way through the sample rollout.cfg and have
encountered a few problems. So far this is what I have found.


Two typos

in the file steps/050-users

29 # Now all the users
30 if (i_should("users")) {
31 v "Checking groups: ". join(" ", keys %users);

in line 31, it should be v "Checking users: "

Using the sample rollout.cfg for reference, I noted that the user
account creation step never works, The relevant lines in the
rollout.cfg file

58 group => {
59 # Make sure the "admin" group exists
60 admin => { },
61 },
62 # Create 2 test users
63 user => {
64 # One is an admin
65 testadmin => {
66 name => "Test Admin",
67 groups => [ "admin", "cdrom" ], # Add the user to
these 2 groups
68 # This password is only set when the user is first
created
69 password => '$1$sOVcUMAy$janTja1meiuCfZfG4iJw8/',
70 },

Note line number 63 , it says user => However in steps/050-
users

8 my %groups = flatten_hash(c("$hostname/group"));
9 my %users = flatten_hash(c("$hostname/users"));

in line 9 it should be $hostname/user" . With this correction
user account creation works.


This is using the source file dparrish-rollout-1262707.tar.gz I
downloaded yesterday.

rgds

Jimmy

[David Parrish]

On Wed, Aug 25, 2010 at 1:40 PM, jimmy <jimmy...@gmail.com> wrote:
>
> Hi,
>
> I am working my way through the  sample  rollout.cfg   and have
> encountered a few  problems.  So far this is what I have found.

Thanks for letting me know.

> in the  file   steps/050-users
>
>  29  # Now all the users
>    30  if (i_should("users")) {
>    31    v "Checking groups: ". join(" ", keys %users);
>
> in line 31,  it should  be    v "Checking  users: "

Fixed.

> Note line  number  63 ,  it says   user =>      However  in  steps/050-
> users
>
>     8  my %groups = flatten_hash(c("$hostname/group"));
>     9  my %users = flatten_hash(c("$hostname/users"));
>
> in  line 9    it should be   $hostname/user"  .  With this correction
> user account creation works.

Fixed.

Thanks, the latest source at http://github.com/dparrish/rollout has these fixes.