Skip to content
This repository has been archived by the owner on Apr 20, 2023. It is now read-only.

Output elapsed time with S3 upload #148

Merged
merged 3 commits into from Sep 15, 2020
Merged

Conversation

blueyed
Copy link
Contributor

@blueyed blueyed commented May 19, 2018

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
'<?xml version="1.0" encoding="UTF-8"?>\n<Error><Code>AccessDenied</Code><Message>Request has expired</Message><Expires>2018-05-19T11:58:04Z</Expires><ServerTime>2018-05-19T11:58:10Z</ServerTime><RequestId>EFE63A2DA8CE9726</RequestId><HostId>LXoeTEXV7h3wN5DICvZg/U6uWIeyOgkJ1JnyEOQCpKNkQtpst4qKhkxuTgPnJeV3uB1jOexYP+s=</HostId></Error>'

blueyed and others added 2 commits May 19, 2018 14:05
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
    '<?xml version="1.0" encoding="UTF-8"?>\n<Error><Code>AccessDenied</Code><Message>Request has expired</Message><Expires>2018-05-19T11:58:04Z</Expires><ServerTime>2018-05-19T11:58:10Z</ServerTime><RequestId>EFE63A2DA8CE9726</RequestId><HostId>LXoeTEXV7h3wN5DICvZg/U6uWIeyOgkJ1JnyEOQCpKNkQtpst4qKhkxuTgPnJeV3uB1jOexYP+s=</HostId></Error>'
@thomasrockhu
Copy link
Contributor

@blueyed this is quite old, but if it's still useful, happy to merge in.

codecov/__init__.py Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Sep 14, 2020

Codecov Report

Merging #148 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #148   +/-   ##
=====================================
  Coverage      88%    88%           
=====================================
  Files           2      2           
  Lines           9      9           
=====================================
  Hits            8      8           
  Misses          1      1           

@thomasrockhu
Copy link
Contributor

🤷‍♂️ it can't hurt

@thomasrockhu thomasrockhu merged commit 9137404 into codecov:master Sep 15, 2020
@blueyed blueyed deleted the elapsed branch September 15, 2020 15:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants