Output elapsed time with S3 upload (codecov#148)

blueyed · thomasrockhu · web-flow · commit 9137404b7851 · 2020-09-15T11:06:07.000-04:00
* Output elapsed time with S3 upload

It seems that it fails with "Forbidden" when it takes longer than 15s,
so this at least helps with debugging that.
The Bash uploader gzips the data file, so it is faster to upload and
does not trigger it with the given data file / internet connection.

    (Pdb++) s3.text
    '&lt;?xml version="1.0" encoding="UTF-8"?&gt;\n&lt;Error&gt;&lt;Code&gt;AccessDenied&lt;/Code&gt;&lt;Message&gt;Request has expired&lt;/Message&gt;&lt;Expires&gt;2018-05-19T11:58:04Z&lt;/Expires&gt;&lt;ServerTime&gt;2018-05-19T11:58:10Z&lt;/ServerTime&gt;&lt;RequestId&gt;EFE63A2DA8CE9726&lt;/RequestId&gt;&lt;HostId&gt;LXoeTEXV7h3wN5DICvZg/U6uWIeyOgkJ1JnyEOQCpKNkQtpst4qKhkxuTgPnJeV3uB1jOexYP+s=&lt;/HostId&gt;&lt;/Error&gt;'

* black

Co-authored-by: Thomas Hu &lt;thomasrockhu@users.noreply.github.com&gt;
diff --git a/codecov/__init__.py b/codecov/__init__.py
@@ -1149,6 +1149,7 @@ def main(*argv, **kwargs):
                         )
                         s3.raise_for_status()
                         assert s3.status_code == 200
+                        write("    Uploading to S3 took %s" % s3.elapsed)
                         write("    " + result)
                         success = True
 

Original file line number	Diff line number	Diff line change
`@@ -1149,6 +1149,7 @@ def main(argv, *kwargs):`
`1149`	`1149`	`)`
`1150`	`1150`	`s3.raise_for_status()`
`1151`	`1151`	`assert s3.status_code == 200`
	`1152`	`+ write(" Uploading to S3 took %s" % s3.elapsed)`
`1152`	`1153`	`write(" " + result)`
`1153`	`1154`	`success = True`
`1154`	`1155`