Merge ubuntu-sponsoring:subdir-for-jsons into ubuntu-sponsoring:master

[Sebastien Bacher]

Write the json reports in a subdir

It's easier to list them this way. Keep the old location for compatibility

Currently browsing http://reqorts.qa.ubuntu.com/reports/sponsoring/ loads the index.html. I'm working on creating some sponsoring KPI and it would be nice to be able to iterate over the records from a directory rather than coding the list of filename.

The reports are also still written to the old location for compatibility reasons in case other scripts or service rely on the current url.

I've deleted the remove use since opening the file in 'w' mode is enough to replace the content

[Brian Murray (brian-murray)]

This looks good to me.

review: Approve

diff --git a/sponsors-page.py b/sponsors-page.py
index e2d76a6..6c06437 100755
--- a/sponsors-page.py
+++ b/sponsors-page.py
@@ -568,8 +568,15 @@ def generate_page(items, devs, packages, other_teams=[], team=None):
     json_items = []
     for item in items:
         json_items += [item.prepare_json()]
-    if os.path.exists(json_filename):
-        os.remove(json_filename)
+
+    # create a subdir for the json reports
+    if not os.path.exists("jsons"):
+        os.mkdir("jsons")
+    f = open(os.path.join("jsons", json_filename), "w")
+    f.write(json.dumps(json_items))
+    f.close()
+
+    # still write to the old location for compatibility
     f = open(json_filename, "w")
     f.write(json.dumps(json_items))
     f.close()