Merge ~sylvain-pineau/checkbox-ng:submission-service-transport into checkbox-ng:master

[Sylvain Pineau]

New class to create a transport suitable for sending tar.xz submissions to the new service endpoint

[Maciej Kisielewski (kissiel)]

+1! Great stuff.
One comment inline - I'm leaving the decision about this little clean-up up to you.

review: Approve

[Sylvain Pineau (sylvain-pineau)]

New (simplified) version without adding a new class since the new service ignores the header content.

review: Approve

diff --git a/checkbox_ng/certification.py b/checkbox_ng/certification.py
index a8ae820..def7e35 100644
--- a/checkbox_ng/certification.py
+++ b/checkbox_ng/certification.py
@@ -46,9 +46,12 @@ class CertificationTransport(TransportBase):
     Transport for sending data to certification database.
      - POSTs data to a http(s) endpoint
      - Adds a header with a hardware identifier
-     - Data is expected to be in checkbox xml-compatible format.
-       This means it will work best with a stream produced by the
-       xml exporter.
+     - Payload can be in:
+        * checkbox xml-compatible format.
+          This means it will work best with a stream produced by the xml
+          exporter.
+        * LZMA compressed tarball that includes a submission.json and results
+          from checkbox.
    """
 
     def __init__(self, where, options):
diff --git a/checkbox_ng/launcher/subcommands.py b/checkbox_ng/launcher/subcommands.py
index 8baa4e4..d560484 100644
--- a/checkbox_ng/launcher/subcommands.py
+++ b/checkbox_ng/launcher/subcommands.py
@@ -527,6 +527,22 @@ class Launcher(Command, MainLoopStage):
             else:
                 options = ""
             self.transports[transport] = cls(url, options)
+        elif tr_type == 'submission-service':
+            secure_id = self.launcher.transports[transport].get(
+                'secure_id', None)
+            if not secure_id and self.is_interactive:
+                secure_id = input(self.C.BLUE(_('Enter secure-id:')))
+            if secure_id:
+                options = "secure_id={}".format(secure_id)
+            else:
+                options = ""
+            if self.launcher.transports[transport].get('staging', False):
+                url = ('https://submission.staging.canonical.com/'
+                       '1.0/submission/hardware/{}'.format(secure_id))
+            else:
+                url = ('https://submission.canonical.com/'
+                       '1.0/submission/hardware/{}'.format(secure_id))
+            self.transports[transport] = cls(url, options)
 
     def _export_results(self):
         for report in self.launcher.stock_reports:
diff --git a/setup.py b/setup.py
index 1f88d18..84a0285 100755
--- a/setup.py
+++ b/setup.py
@@ -74,6 +74,8 @@ setup(
         'plainbox.transport': [
             'certification='
             'checkbox_ng.certification:CertificationTransport',
+            'submission-service='
+            'checkbox_ng.certification:CertificationTransport',
         ],
     },
     include_package_data=True)