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

Add Image Feature Requirements #3255

Merged
merged 1 commit into from May 17, 2024
Merged

Add Image Feature Requirements #3255

merged 1 commit into from May 17, 2024

Conversation

kamalca
Copy link
Collaborator

@kamalca kamalca commented Apr 8, 2024

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.

@kamalca kamalca force-pushed the kameroncarr/image-features branch 8 times, most recently from b57517c to 65dfcd5 Compare April 9, 2024 19:47
lisa/schema.py Outdated Show resolved Hide resolved
@kamalca kamalca force-pushed the kameroncarr/image-features branch 3 times, most recently from 8627bf6 to 91c93f3 Compare April 23, 2024 21:36
@kamalca kamalca requested a review from squirrelsc April 23, 2024 21:41
@kamalca
Copy link
Collaborator Author

kamalca commented Apr 23, 2024

@LiliDeng Do you have a Community Gallery Image I can test my changes on?

@LiliDeng
Copy link
Collaborator

@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

@kamalca kamalca force-pushed the kameroncarr/image-features branch 9 times, most recently from b081dce to d929fc8 Compare April 29, 2024 16:17
@kamalca kamalca force-pushed the kameroncarr/image-features branch 4 times, most recently from d929fc8 to 751e829 Compare April 29, 2024 23:52
@squirrelsc
Copy link
Member

@LiliDeng LGTM

@LiliDeng
Copy link
Collaborator

LiliDeng commented May 6, 2024

@kamalca could you please run a test with gallery arm64 image?

@kamalca
Copy link
Collaborator Author

kamalca commented May 6, 2024

@kamalca could you please run a test with gallery arm64 image?

Shared Image Gallery or Community Gallery Image?

@LiliDeng
Copy link
Collaborator

LiliDeng commented May 6, 2024

@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.

@kamalca kamalca force-pushed the kameroncarr/image-features branch from 751e829 to 00aa945 Compare May 7, 2024 16:26
@kamalca
Copy link
Collaborator Author

kamalca commented May 7, 2024

@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.

I fixed a typo. Thank you for helping test the changes.

@kamalca
Copy link
Collaborator Author

kamalca commented May 16, 2024

@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.
@kamalca kamalca force-pushed the kameroncarr/image-features branch from 00aa945 to e806ace Compare May 16, 2024 22:08
@LiliDeng LiliDeng merged commit 2e7180d into main May 17, 2024
45 checks passed
@LiliDeng LiliDeng deleted the kameroncarr/image-features branch May 17, 2024 14:44
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.

None yet

3 participants