Code review comment for lp:~fginther/cupstream2distro-config/mbs-rebuild-support

Can we strip whitespace from rebuild_job in here (in case somebody specifies rebuild like "job1, job2")?:

46 + for rebuild_job in data.split(','):
47 + rebuild_list.append('-'.join([rebuild_job, 'rebuild']))
48 + ctx['rebuild'] = ','.join(rebuild_list)

The following (needs info only):
795 + rebuild: grail-raring

Would it be too hard to specify only "grail" and the update tool would check the current stack and find out there is a "grail-raring" base job? If it would be too hard maybe we can create a check in the validator for it? This feels a bit error prone/not obvious to me.

Other than that looks good.