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

Do not mount volatile paths for local Linux targets #94

Conversation

pyrco
Copy link
Contributor

@pyrco pyrco commented Aug 25, 2023

These are now mounted when the local Linux target is loaded.

(DIS-1965)

@pyrco pyrco force-pushed the feature/dis-1965_do_not_mount_volatile_paths_for_local_linux_targets branch from 0606ee8 to baf61da Compare August 25, 2023 09:35
@codecov
Copy link

codecov bot commented Aug 25, 2023

Codecov Report

Merging #94 (e861a98) into main (12b54de) will increase coverage by 0.19%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #94      +/-   ##
==========================================
+ Coverage   44.11%   44.30%   +0.19%     
==========================================
  Files          22       22              
  Lines        2779     2767      -12     
==========================================
  Hits         1226     1226              
+ Misses       1553     1541      -12     
Flag Coverage Δ
unittests 44.30% <100.00%> (+0.19%) ⬆️

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

Files Changed Coverage Δ
acquire/acquire.py 41.03% <100.00%> (+0.50%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

These are now mounted when the local Linux target is loaded.

(DIS-1965)
@pyrco pyrco force-pushed the feature/dis-1965_do_not_mount_volatile_paths_for_local_linux_targets branch from baf61da to e861a98 Compare August 29, 2023 11:37
@pyrco pyrco merged commit 49e177e into main Aug 30, 2023
18 checks passed
@pyrco pyrco deleted the feature/dis-1965_do_not_mount_volatile_paths_for_local_linux_targets branch August 30, 2023 09:26
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.

2 participants