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

Fetch lifecycle binaries from lifecycle image #2007

Merged
merged 2 commits into from
Dec 20, 2023

Conversation

natalieparellano
Copy link
Member

Summary

Fetch lifecycle binaries from lifecycle image instead of restorer container

Hopefully fixes missing file issues on Linux (see #1979)

Before

Before, we copied the lifecycle binaries out of the restorer container and mounted them back during the extend (run) phase.

After

Now, we download the lifecycle image from the daemon, fetch the lifecycle layer, and create an ephemeral run image that has the lifecycle layer added.

Documentation

  • Should this change be documented?
    • Yes, see #___
    • No

Related

Resolves #1979 (I hope)

@github-actions github-actions bot added this to the 0.33.0 milestone Dec 13, 2023
@github-actions github-actions bot added the type/enhancement Issue that requests a new feature or improvement. label Dec 13, 2023
…tainer

Hopefully fixes missing file issues on Linux

Signed-off-by: Natalie Arellano <narellano@vmware.com>
Copy link

codecov bot commented Dec 19, 2023

Codecov Report

Attention: 127 lines in your changes are missing coverage. Please review.

Comparison is base (1749198) 80.45% compared to head (5f6aae6) 79.69%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2007      +/-   ##
==========================================
- Coverage   80.45%   79.69%   -0.75%     
==========================================
  Files         172      172              
  Lines       12926    13046     +120     
==========================================
- Hits        10398    10396       -2     
- Misses       1874     1995     +121     
- Partials      654      655       +1     
Flag Coverage Δ
os_linux 78.53% <6.62%> (-0.76%) ⬇️
os_macos 76.36% <6.62%> (-0.70%) ⬇️
os_windows 79.09% <6.62%> (-0.13%) ⬇️
unit 79.69% <6.62%> (-0.75%) ⬇️

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

Copy link
Member

@jjbustamante jjbustamante left a comment

Choose a reason for hiding this comment

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

Great Job!

@jjbustamante jjbustamante merged commit d9fddda into main Dec 20, 2023
17 of 18 checks passed
@jjbustamante jjbustamante deleted the fix/extender-no-such-file branch December 20, 2023 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement Issue that requests a new feature or improvement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error Using Extensions in Ubuntu Jammy: "stat /cnb/lifecycle/extender: no such file or directory: unknown"
2 participants