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
Add Image Feature Requirements #3255
Conversation
b57517c
to
65dfcd5
Compare
8627bf6
to
91c93f3
Compare
@LiliDeng Do you have a Community Gallery Image I can test my changes on? |
community_gallery_image: "southcentralus/FreeBSDCGTest-d8a43fa5-745a-4910-9f71-0c9da2ac22bf/FreeBSD-14.0-RC3-amd64-ufs-gen1-testing/latest" please specify the location southcentralus |
91c93f3
to
8600acc
Compare
8600acc
to
707e9af
Compare
b081dce
to
d929fc8
Compare
d929fc8
to
751e829
Compare
@LiliDeng LGTM |
@kamalca could you please run a test with gallery arm64 image? |
Shared Image Gallery or Community Gallery Image? |
You need cover all scenarios, I hit one issue when run a test with Shared Image Gallery. |
751e829
to
00aa945
Compare
I fixed a typo. Thank you for helping test the changes. |
@LiliDeng Any updates on this getting merged? I have to do more work to continually rebase, merge conflicts, and retest changes the longer this goes unmerged. |
Tags from Azure Images are now parsed to determine features such as Architecture, Security Profile, and Disk Controller Type.
00aa945
to
e806ace
Compare
Each feature can now implement a function
create_image_requirement
to determine a requirement based on the raw_capabilities of the image. The requirement is then merged with the platform requirement.