linaro-pm-wg team mailing list archive

[Mattias Backman <mattias.backman@xxxxxxxxxx>]

On Tue, Dec 6, 2011 at 10:54 AM, Данило Шеган <danilo@xxxxxxxxxxxxx> wrote:
> Данило Шеган has proposed merging lp:~danilo/launchpad-work-items-tracker/linaro-config into lp:~linaro/launchpad-work-items-tracker/linaro-config.
>
> Requested reviews:
>  Linaro (linaro)
>
> For more details, see:
> https://code.launchpad.net/~danilo/launchpad-work-items-tracker/linaro-config/+merge/84589
>
> This automatically fetches all the linaro-infrastructure subprojects and puts
> them in the config file.
>
> Ideally, workitem-tracker would directly support project groups, and we'd
> only specify 'linaro-infrastructure' in the config file, instead of having
> code in the config.
>
> I don't have the time look into extending workitem-tracker to support that now,
> though, but for the same reason I won't mind if somebody tells me this branch
> should not be merged: the biggest problem right now is the fact that every single
> load of the config file might do a page load over network, thus taking much longer.

We load the file three times for every scheduled run so it's not going
to have any impact. I'm more concerned with having code in the config
file as you say. When all the other teams want to do this it can get
messy.

It will be pretty easy to lift this code into work-items-tracker and
specify "extra_project_groups" or something in the config. I can have
a go at that after lunch.

>
> I am not sure if this is a serious enough issue that we should really fix this
> in workitem tracker itself.
> --
> https://code.launchpad.net/~danilo/launchpad-work-items-tracker/linaro-config/+merge/84589
> Your team Linaro is requested to review the proposed merge of lp:~danilo/launchpad-work-items-tracker/linaro-config into lp:~linaro/launchpad-work-items-tracker/linaro-config.
>
> === modified file '11.12.cfg'
> --- 11.12.cfg   2011-11-25 10:00:27 +0000
> +++ 11.12.cfg   2011-12-06 09:53:23 +0000
> @@ -101,13 +101,6 @@
>     'u-boot-linaro': 'trunk',
>  # android
>     'linaro-android': 'trunk',
> -# infrastructure
> -    #'linaro-infrastructure': None,
> -    'linaro-android-infrastructure': 'trunk',
> -    'linaro-ci': 'engineering',
> -    'linaro-image-tools': 'trunk',
> -    'linaro-infrastructure-misc': 'trunk',
> -    'svammel': 'trunk',
>  # validation
>     'lava-android-test': 'trunk',
>     'lava-celery': 'trunk',
> @@ -129,6 +122,21 @@
>     'linaro-octo-armhf': 'trunk',
>  }
>
> +
> +####################################################
> +# Infrastructure subprojects fetched directly from #
> +# linaro-infrastructure project group.             #
> +####################################################
> +import urllib, simplejson, os.path
> +urllib.urlcleanup()
> +data = simplejson.loads(urllib.urlopen(
> +        'https://api.launchpad.net/1.0/linaro-infrastructure/projects').read())
> +print data
> +infra_projects = {
> +    product['name']:os.path.basename(product['development_focus_link'])
> +    for product in data['entries']}
> +extra_projects.update(infra_projects)
> +
>  # Ubuntu release to track
>  release = 'oneiric'
>
>
>

-- 
https://code.launchpad.net/~danilo/launchpad-work-items-tracker/linaro-config/+merge/84589
Your team Linaro is requested to review the proposed merge of lp:~danilo/launchpad-work-items-tracker/linaro-config into lp:~linaro/launchpad-work-items-tracker/linaro-config.