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 a few bugs #1542

Merged
merged 1 commit into from
Nov 6, 2022
Merged

Fix a few bugs #1542

merged 1 commit into from
Nov 6, 2022

Conversation

natalieparellano
Copy link
Member

Summary

Fixes a few bugs uncovered during #1539

Documentation

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

Related

Resolves #___

@github-actions github-actions bot added this to the 0.28.0 milestone Nov 3, 2022
@github-actions github-actions bot added type/chore Issue that requests non-user facing changes. type/enhancement Issue that requests a new feature or improvement. labels Nov 3, 2022
@natalieparellano natalieparellano changed the base branch from main to refactor-lifecycle-execution November 3, 2022 19:18
@natalieparellano
Copy link
Member Author

Blocking on #1539

@codecov
Copy link

codecov bot commented Nov 3, 2022

Codecov Report

Merging #1542 (92a4fb4) into main (69fd327) will decrease coverage by 0.03%.
The diff coverage is 90.56%.

❗ Current head 92a4fb4 differs from pull request most recent head a675dc3. Consider uploading reports for the commit a675dc3 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1542      +/-   ##
==========================================
- Coverage   81.14%   81.12%   -0.02%     
==========================================
  Files         155      155              
  Lines       10090    10094       +4     
==========================================
+ Hits         8187     8188       +1     
- Misses       1420     1422       +2     
- Partials      483      484       +1     
Flag Coverage Δ
os_linux 79.85% <90.56%> (-0.05%) ⬇️
os_macos 77.37% <90.56%> (-0.02%) ⬇️
os_windows 81.00% <90.56%> (-0.02%) ⬇️
unit 81.12% <90.56%> (-0.02%) ⬇️

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

@dfreilich dfreilich force-pushed the refactor-lifecycle-execution branch from 96eaa06 to 03f4c69 Compare November 3, 2022 19:33
Base automatically changed from refactor-lifecycle-execution to main November 3, 2022 19:56
Fix (restore): when gid is provided, it shouldn't override other flags
Fix (analyze): when cache image is used, flags should not also contain -cache-dir
Fix (restore): when cache image is used, flags should not also contain -cache-dir
- Variable rename (ops for operations)
Fix (restore): provide registry credentials when using a cache image

Signed-off-by: Natalie Arellano <narellano@vmware.com>
@github-actions github-actions bot removed the type/chore Issue that requests non-user facing changes. label Nov 4, 2022
@natalieparellano natalieparellano marked this pull request as ready for review November 4, 2022 14:37
@natalieparellano natalieparellano requested review from a team as code owners November 4, 2022 14:37
@sambhav sambhav merged commit 8f8cdb3 into main Nov 6, 2022
@sambhav sambhav deleted the fix/few-bugs branch November 6, 2022 04:55
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.

2 participants