-
Notifications
You must be signed in to change notification settings - Fork 346
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 image pattern for FedoraSecondary provider on s390x #6082
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #6082 +/- ##
===========================================
+ Coverage 54.32% 68.66% +14.33%
===========================================
Files 202 202
Lines 21892 21899 +7
===========================================
+ Hits 11892 15036 +3144
+ Misses 10000 6863 -3137 ☔ View full report in Codecov by Sentry. |
792b6f2
to
d0fabf4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @harvey0100, this does seem to address the issue at hand, but I spotted a few blockers. Thanks for working on this!
3aff299
to
f5209e7
Compare
Updated the image pattern in FedoraSecondaryImageProvider to match the available Fedora image file names for the s390x architecture. This resolves the issue where the vmimage utility could not download the correct image due to a pattern mismatch. Reference: avocado-framework#6071 Signed-off-by: Harvey Lynden <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
Updated the image pattern in FedoraSecondaryImageProvider to match the available Fedora image file names for the s390x architecture. This resolves the issue where the vmimage utility could not download the correct image due to a pattern mismatch.
Reference: #6071