Merge lp:~jpds/arsenal/css-subscribed into lp:arsenal/trunk

[Jonathan Davies]

This branch allows one to specify a subscriber field to JSON files. Only bugs with that subscriber will be thus listed.

[Brad Figg (brad-figg)]

On 03/20/2012 03:15 PM, Jonathan Davies wrote:
> Jonathan Davies has proposed merging lp:~jpds/arsenal/css-subscribed into lp:arsenal.
>
> Requested reviews:
> Brad Figg (brad-figg): code
>
> For more details, see:
> https://code.launchpad.net/~jpds/arsenal/css-subscribed/+merge/98529
>
> This branch allows one to specify a subscriber field to JSON files. Only bugs with that subscriber will be thus listed.

This fails for .json files that do not have a "subscriber" entry
in them. That will need to be fixed.

Brad
--
Brad Figg <email address hidden> http://www.canonical.com

[lp:~jpds/arsenal/css-subscribed]

963. By Jonathan Davies on 2012-03-21

Make reports/cbd set search_subscriber to None instead of an empty string.

[Jonathan Davies (jpds)]

> This fails for .json files that do not have a "subscriber" entry
> in them. That will need to be fixed.

Fixed in r963.

[lp:~jpds/arsenal/css-subscribed]

964. By Jonathan Davies on 2012-03-21

Finalize CSS bug reports.

Unmerged revisions

964. By Jonathan Davies on 2012-03-21

Finalize CSS bug reports.

963. By Jonathan Davies on 2012-03-21

Make reports/cbd set search_subscriber to None instead of an empty string.

962. By Jonathan Davies on 2012-03-20

Fixed Dbg output for subscriber.

961. By Jonathan Davies on 2012-03-20

Improved search_subscriber code.

960. By Jonathan Davies on 2012-03-20

Corrected subscribers → subscriber typo change.

959. By Jonathan Davies on 2012-03-20

Added reports/rls-mgr/rls-mgr-p-css-tracking.json.

958. By Jonathan Davies on 2012-03-20

Adapted cbd to take a subscriber option.

=== modified file 'reports/cbd'
--- reports/cbd	2012-03-13 20:58:18 +0000
+++ reports/cbd	2012-03-21 14:43:19 +0000
@@ -309,6 +309,12 @@
         Dbg.verbose('search: reporters: \'%s\'\n' % ','.join(reporters))
 
         try:
+            subscriber = search_criteria['subscriber']
+        except KeyError:
+            subscriber = ''
+        Dbg.verbose('search: subscriber: \'%s\'\n' % subscriber)
+
+        try:
             assignees = search_criteria['assignees']
         except KeyError:
             assignees = ''
@@ -318,13 +324,19 @@
             print('You must choose either assignees or reporters not both!')
             exit(1)
 
+        if subscriber:
+            search_subscriber = self.lp.launchpad.people[subscriber]
+        else:
+            search_subscriber = None
+
         Dbg.leave("CollectBugData.distro_tasks_search")
         if reporters != '':
             tasks = []
             for reporter in reporters:
                 search_reporter = self.lp.launchpad.people[reporter]
                 if search_reporter.is_team is False:
-                    for task in distro.search_tasks(tags=search_tags,
+                    for task in distro.search_tasks(bug_subscriber=search_subscriber,
+                        tags=search_tags,
                         tags_combinator=search_tags_combinator,
                         status=search_status, bug_reporter=search_reporter,
                         modified_since=last_run_time):
@@ -333,7 +345,9 @@
                     for participant in search_reporter.participants:
                         if participant.is_team:
                             continue
-                        for task in distro.search_tasks(tags=search_tags,
+                        for task in distro.search_tasks(
+                            bug_subscriber=search_subscriber,
+                            tags=search_tags,
                             tags_combinator=search_tags_combinator,
                             status=search_status, bug_reporter=participant,
                             modified_since=last_run_time):
@@ -344,7 +358,9 @@
             for assignee in assignees:
                 search_assignee = self.lp.launchpad.people[assignee]
                 if search_assignee.is_team is False:
-                    for task in distro.search_tasks(tags=search_tags,
+                    for task in distro.search_tasks(
+                        bug_subscriber=search_subscriber,
+                        tags=search_tags,
                         tags_combinator=search_tags_combinator,
                         status=search_status, assignee=search_assignee,
                         modified_since=last_run_time):
@@ -353,14 +369,18 @@
                     for participant in search_assignee.participants:
                         if participant.is_team:
                             continue
-                        for task in distro.search_tasks(tags=search_tags,
+                        for task in distro.search_tasks(
+                            bug_subscriber=search_subscriber,
+                            tags=search_tags,
                             tags_combinator=search_tags_combinator,
                             status=search_status, assignee=participant,
                             modified_since=last_run_time):
                             tasks.append(task)
             return tasks
         else:
-            return distro.search_tasks(tags=search_tags,
+            return distro.search_tasks(
+                bug_subscriber=search_subscriber,
+                tags=search_tags,
                 tags_combinator=search_tags_combinator, status=search_status,
                 modified_since=last_run_time)
 

=== added directory 'reports/css'
=== added file 'reports/css/rls-mgr-p-css-tracking.json'
--- reports/css/rls-mgr-p-css-tracking.json	1970-01-01 00:00:00 +0000
+++ reports/css/rls-mgr-p-css-tracking.json	2012-03-21 14:43:19 +0000
@@ -0,0 +1,35 @@
+{
+    "launchpad_config": {
+        "launchpad_client_name": "rls-mgr-reports", 
+        "read_only": false
+    }, 
+    "search_criteria": [
+        {
+            "distribution": "ubuntu", 
+            "series": [
+                "precise", 
+                "NONE"
+            ], 
+            "status": [
+                "New", 
+                "Confirmed", 
+                "Triaged", 
+                "Incomplete", 
+                "Incomplete (with response)", 
+                "Incomplete (without response)", 
+                "In Progress", 
+                "Fix Committed"
+            ], 
+            "subscriber": "canonical-support", 
+            "tags": [
+                "rls-mgr-p-tracking"
+            ]
+        }
+    ], 
+    "tasks": {
+        "865672": [], 
+        "926263": [], 
+        "949782": []
+    }, 
+    "update_timestamp": "Tuesday, 20. March 2012 21:49 UTC"
+}
\ No newline at end of file

=== added file 'reports/css/rls-p-css-tracking.json'
--- reports/css/rls-p-css-tracking.json	1970-01-01 00:00:00 +0000
+++ reports/css/rls-p-css-tracking.json	2012-03-21 14:43:19 +0000
@@ -0,0 +1,32 @@
+{
+    "launchpad_config": {
+        "launchpad_client_name": "rls-mgr-reports", 
+        "read_only": false
+    }, 
+    "search_criteria": [
+        {
+            "distribution": "ubuntu", 
+            "series": [
+                "precise", 
+                "NONE"
+            ], 
+            "status": [
+                "New", 
+                "Confirmed", 
+                "Triaged", 
+                "Incomplete", 
+                "Incomplete (with response)", 
+                "Incomplete (without response)", 
+                "In Progress", 
+                "Fix Committed"
+            ], 
+            "subscriber": "canonical-support", 
+            "tags": [
+                "rls-p-tracking"
+            ]
+        }
+    ], 
+    "tasks": {
+    }, 
+    "update_timestamp": "Tuesday, 20. March 2012 21:49 UTC"
+}