[image] Filter unsupported images at manifest parsing#4702
[image] Filter unsupported images at manifest parsing#4702deepakshirkem wants to merge 1 commit intocanonical:mainfrom
Conversation
ricab
left a comment
There was a problem hiding this comment.
Hi @deepakshirkem, thank you for your interest. I am afraid this approach is too superficial. We'd be giving up an essential benefit in dropping unsupported images: not having to maintain any code to dealt with them at all. That would mean filtering them out at the root (when parsing simplestreams) and removing any code that dealt with them specifically. There would be no more --show-unsupported in find or UnsupportedImageException. Saying multipass launch bionic would effectively be handled just like multipass launch unknown.
|
Hi @ricab, I should have though this through more carefully. |
|
Hi @deepakshirkem, yes, that sounds reasonable. Note that the code is already aware of what images are considered unsupported. It should now skip them altogether. Be sure to have a look at our contribution guidelines at https://github.com/canonical/multipass/blob/main/CONTRIBUTING.md and good luck! |
99b9691 to
d7ff252
Compare
[image] Filter unsupported images at manifest parsing
d7ff252 to
07dc880
Compare
|
Hi @ricab, I have implemented requested changes please review. |
Description
What does this PR do?
Filters out unsupported (EOL) images at the simplestreams manifest parsing level, so they never enter the system at all.
Attempting to launch an unsupported image by name is handled just like launching an unknown image. URL-based launches remain available.
Why is this change needed?
Previously, Multipass was aware of unsupported images and had special code to deal with them (UnsupportedImageException, --show-unsupported flag). This change removes that complexity entirely by filtering at the root.
Related Issue(s)
Closes #3934
Testing
multipass launch xenial— returns unknown image errormultipass launch 22.04— launches successfullymultipass find— should only show supported imagesScreenshots (if applicable)
Checklist
Additional Notes
NA