Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(app-shell-odd): make sure there is enough space on device before copying app tar file. #12461

Merged
merged 2 commits into from
Apr 12, 2023

Conversation

vegano1
Copy link
Contributor

@vegano1 vegano1 commented Apr 10, 2023

Overview

If the scp or clean-up function failed for whatever reason, we would end up in a state where we did not have enough space on the system to scp the new app shell tar file. This pr will remove the tar file before attempting to scp the new one, in case it was not previously removed.

Test Plan

  • test pushing file to odd
  • cancel push midway and push file again

Changelog

  • attempt to remove the old app shell tar file in case it was not previously removed.

Review requests

  • test on ODD

Risk assessment

Low, flex work

@vegano1 vegano1 requested a review from a team as a code owner April 10, 2023 18:53
@vegano1 vegano1 requested review from shlokamin and sfoster1 April 10, 2023 18:54
@codecov
Copy link

codecov bot commented Apr 10, 2023

Codecov Report

Merging #12461 (5e76da5) into edge (6af2495) will decrease coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             edge   #12461      +/-   ##
==========================================
- Coverage   73.81%   73.78%   -0.03%     
==========================================
  Files        2241     2246       +5     
  Lines       61707    61905     +198     
  Branches     6420     6507      +87     
==========================================
+ Hits        45549    45679     +130     
- Misses      14650    14705      +55     
- Partials     1508     1521      +13     
Flag Coverage Δ
app 72.50% <ø> (-0.12%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 13 files with indirect coverage changes

Copy link
Member

@sfoster1 sfoster1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@vegano1 vegano1 merged commit 44daef5 into edge Apr 12, 2023
@vegano1 vegano1 deleted the push-odd-out-of-space-fix branch April 12, 2023 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants