Skip to content

Fetch lifecycle binaries from lifecycle image #2007

Merged
jjbustamante merged 2 commits intomainfrom
fix/extender-no-such-file
Dec 20, 2023
Merged

Fetch lifecycle binaries from lifecycle image #2007
jjbustamante merged 2 commits intomainfrom
fix/extender-no-such-file

Conversation

@natalieparellano
Copy link
Copy Markdown
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
@natalieparellano natalieparellano force-pushed the fix/extender-no-such-file branch from 4cc60a6 to e52a3e0 Compare December 13, 2023 19:54
…tainer

Hopefully fixes missing file issues on Linux

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

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
Copy Markdown
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
@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