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

Bug fixes for some tier 1 tests for CBL-Mariner on QEMU. #1721

Merged
merged 2 commits into from
Jan 29, 2022

Conversation

cwize1
Copy link
Contributor

@cwize1 cwize1 commented Jan 28, 2022

  1. Add OS type for CBL-Mariner. While CBL-Mariner uses RPM
    and so should share the package handling logic, it is not
    a derivative of Fedora and has substantial differences.
    Hence, the RPM handling logic was split off into a new
    base class that both Fedora and CBLMariner classes
    inherit from.

  2. Fix a few tests that didn't declare their required
    supported_features.

  3. Restrict tests that rely on VMBus to the AZURE platform.
    (When the Hyper-V support is added, it can be added as a
    platform to these tests as well.)

  4. Add support for CBL-Mariner to Docker tool.

romoh
romoh previously approved these changes Jan 28, 2022
and so should share the package handling logic, it is not
a derivative of Fedora and has substantial differences.
Hence, the RPM handling logic was split off into a new
base class that both `Fedora` and `CBLMariner` classes
inherit from.

2. Fix a few tests that didn't declare their required
`supported_features`.

3. Restrict tests that rely on VMBus to the `AZURE` platform.
(When the Hyper-V support is added, it can be added as a
platform to these tests as well.)

4. Add support for CBL-Mariner to Docker tool.
@squirrelsc squirrelsc merged commit 14dfc8a into microsoft:main Jan 29, 2022
@cwize1 cwize1 deleted the user/chrisgun/TestFixes3 branch March 4, 2022 18:26
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