[PATCH 3/3] Enable -owner at fedoraproject.org cc for upgradepath results

Kamil Paral kparal at redhat.com
Thu Sep 30 09:52:37 UTC 2010


----- "James Laska" <jlaska at redhat.com> wrote:

> ---
>  tests/upgradepath/upgradepath.py |    7 +++++++
>  1 files changed, 7 insertions(+), 0 deletions(-)
> 
> diff --git a/tests/upgradepath/upgradepath.py
> b/tests/upgradepath/upgradepath.py
> index 028b8aa..b7457c0 100755
> --- a/tests/upgradepath/upgradepath.py
> +++ b/tests/upgradepath/upgradepath.py
> @@ -133,6 +133,13 @@ class upgradepath(AutoQATest):
>              # compare with higher tags, so version has to be lower or
> equal
>              self.compare(matching_build, hi_tags, operator.le, '<=')
>  
> +            # Include pkg owner in results email if they chose to be
> included
> +            repo = repoinfo.getrepo_by_tag(kojitag)
> +            pkg_name = rpmUtils.miscutils.splitFilename(envr +
> '.noarch')[0]
> +            if repo is not None and
> autoqa.util.check_opt_in(pkg_name, repo['collection_name']):
> +                #FIXME - hardcoded partial email address here -
> obviously sub-par
> +                self.mail_to.append('%s-owner at fedoraproject.org' %
> pkg_name)
> +
>          summary = str(len(envrs) - len(self.envr_list)) + ' OK, ' +
> str(len(self.envr_list)) + ' FAILED'
>          msg = 'SUMMARY: ' + summary
>          self.log.append(msg)
> -- 
> 1.7.3

The two patches seem fine. As for this one, I started a new thread:
https://fedorahosted.org/pipermail/autoqa-devel/2010-September/001197.html



More information about the autoqa-devel mailing list