Skip to content

Add OCI to test suite runner - #579

Open
sshmulev wants to merge 2 commits into
osbuild:mainfrom
sshmulev:add_oci_to_suit_runner
Open

Add OCI to test suite runner#579
sshmulev wants to merge 2 commits into
osbuild:mainfrom
sshmulev:add_oci_to_suit_runner

Conversation

@sshmulev

Copy link
Copy Markdown
Collaborator

No description provided.

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hey - I've left some high level feedback:

  • As more cloud providers are added, consider refactoring the get_default_test_suite_paths conditional chain into a mapping/dictionary of cloud_provider -> test suite path to make it easier to extend and maintain.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- As more cloud providers are added, consider refactoring the `get_default_test_suite_paths` conditional chain into a mapping/dictionary of `cloud_provider -> test suite path` to make it easier to extend and maintain.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

knivnia
knivnia previously approved these changes Jul 23, 2026

@knivnia knivnia left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

👍

@sshmulev
sshmulev force-pushed the add_oci_to_suit_runner branch from 636dce0 to 1abe961 Compare July 23, 2026 11:19
@sshmulev
sshmulev enabled auto-merge (rebase) July 23, 2026 11:20
@sshmulev
sshmulev force-pushed the add_oci_to_suit_runner branch from 1abe961 to 049b6d9 Compare July 23, 2026 11:44
OCI images always have a separate /boot partition, so
test_boot_mount_presence should not assert its absence.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

2 participants