Skip to content

Conversation

@davidcassany
Copy link
Contributor

No description provided.

@davidcassany davidcassany requested a review from a team as a code owner March 6, 2024 14:23
Copy link
Contributor

@anmazzotti anmazzotti left a comment

Choose a reason for hiding this comment

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

Tested successfully, so looks good to me.

Signed-off-by: David Cassany <[email protected]>
@davidcassany davidcassany enabled auto-merge (rebase) March 6, 2024 15:07
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.53%. Comparing base (c4bea71) to head (d5a3fac).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1991   +/-   ##
=======================================
  Coverage   72.52%   72.53%           
=======================================
  Files          76       76           
  Lines        8906     8909    +3     
=======================================
+ Hits         6459     6462    +3     
  Misses       1913     1913           
  Partials      534      534           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@davidcassany davidcassany disabled auto-merge March 6, 2024 16:14
@davidcassany
Copy link
Contributor Author

With the current change it can't run the fallback test as the downloaded image does not include snapper. Which is good, it fails early in the process and it simply refuses to upgrade. Merging as the issue is in test-fallback itself, it should try to upgrade to the self image.

@davidcassany davidcassany merged commit c03dc35 into rancher:main Mar 6, 2024
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