modules/plague/templates/sync-committers.sh.erb Change

Mike McGrath mmcgrath at redhat.com
Sat Sep 13 23:18:00 UTC 2008


On Sat, 13 Sep 2008, Ricky Zhou wrote:

> Sorry, I completely forgot we were in a change freeze now.  I just
> pushed a change (diff below) to make the sync-committers script not
> fail.  Should I revert this, or does it look all right?
>
> diff --git a/modules/plague/templates/sync-committers.sh.erb b/modules/plague/templates/sync-committers.sh.erb
> index d3c8e63..4e51d44 100644
> --- a/modules/plague/templates/sync-committers.sh.erb
> +++ b/modules/plague/templates/sync-committers.sh.erb
> @@ -12,7 +12,7 @@ fi
>
>  for user in `cat ${users}|cut -d, -f2`
>   do
> -   plague-user-manager.py ${userdb} add $user own_jobs > $cruft 2>&1
> +   /usr/bin/plague-user-manager ${userdb} add $user own_jobs > $cruft 2>&1
>     if [ $? != 0 ]; then
>          if ! grep -qi 'already exists' $cruft; then
>               echo "Error adding user $user was:"
>

I'll +1 it just in case.  This could be considered an outage so I think no
harm done.

	-Mike




More information about the infrastructure mailing list