Skip to content

Add ebbr testing via fwt (New) - #2656

Merged
seankingyang merged 4 commits into
mainfrom
add-ebbr-in-ce-oem
Jul 20, 2026
Merged

Add ebbr testing via fwt (New)#2656
seankingyang merged 4 commits into
mainfrom
add-ebbr-in-ce-oem

Conversation

@baconYao

@baconYao baconYao commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Description

Add ebbr testing via fwts

Resolved issues

Jira: https://warthogs.atlassian.net/browse/OEMQA-6767

Documentation

Tests

Sideload Test on g1200: https://certification.canonical.com/hardware/202404-33953/submission/499957/

@baconYao
baconYao requested review from a team and Copilot July 7, 2026 09:36
@baconYao baconYao added the ce-oem The PR only contains to the ce-oem provider under the contrib area label Jul 7, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a new CE OEM EBBR test plan to the checkbox-provider-ce-oem contrib provider, wiring FWTS EBBR test enumeration into templated Plainbox jobs so EBBR cases can be executed and their FWTS logs validated.

Changes:

  • Introduces EBBR test plans (full/manual/automated) and a dedicated category.
  • Adds a resource-driven templated job set that runs fwts per EBBR case and checks logs.
  • Adds two helper scripts: one to generate EBBR cases from fwts --ebbr --show-tests, and one to parse FWTS logs for failures.

Reviewed changes

Copilot reviewed 3 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
contrib/checkbox-ce-oem/checkbox-provider-ce-oem/units/ebbr/test-plan.pxu Adds full/manual/automated EBBR test plans, bootstrapping a resource generator and including generated jobs.
contrib/checkbox-ce-oem/checkbox-provider-ce-oem/units/ebbr/jobs.pxu Defines the resource + jinja2 templated FWTS-per-case shell jobs and log checking.
contrib/checkbox-ce-oem/checkbox-provider-ce-oem/units/ebbr/category.pxu Adds a CE OEM EBBR category for grouping results.
contrib/checkbox-ce-oem/checkbox-provider-ce-oem/bin/sleep_test_log_check.py Adds a provider-local FWTS log parser with extra handling for tests lacking summary sections.
contrib/checkbox-ce-oem/checkbox-provider-ce-oem/bin/ebbr_resource.py Implements FWTS EBBR case enumeration for templated job generation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@seankingyang seankingyang left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I leave some in line comment.

baconYao added 4 commits July 14, 2026 09:44
Jira: https://warthogs.atlassian.net/browse/OEMQA-6767

Signed-off-by: baconYao <patrick.chang@canonical.com>
Deduplication. Leverage the sleep_test-log_check.py which
exists in base provider.

Signed-off-by: baconYao <patrick.chang@canonical.com>
Signed-off-by: baconYao <patrick.chang@canonical.com>
Signed-off-by: baconYao <patrick.chang@canonical.com>
@baconYao
baconYao force-pushed the add-ebbr-in-ce-oem branch from 451b77b to 5aa8277 Compare July 14, 2026 01:50
@baconYao
baconYao requested a review from a team July 14, 2026 01:51

@seankingyang seankingyang left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM~

@seankingyang
seankingyang merged commit d2023bf into main Jul 20, 2026
25 checks passed
@seankingyang
seankingyang deleted the add-ebbr-in-ce-oem branch July 20, 2026 02:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ce-oem The PR only contains to the ce-oem provider under the contrib area

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants