Skip to content
Snippets Groups Projects

CI: Add bootimg analysis integration test

Merged Newbyte requested to merge newbyte/bootimg-analysis-test into master
All threads resolved!

I'm just checking in a random boot image I had on my PC. I'm not sure if we want to do this, but I'm also not sure how to test this code otherwise. We could use unit tests, but that is not as rigorous.

CI is expected to fail until !2466 (merged) has been merged.

Edited by Newbyte

Merge request reports

Merge request pipeline #209795 passed

Merge request pipeline passed for d19de946

Approved by

Merged by Oliver SmithOliver Smith 4 months ago (Nov 6, 2024 11:56pm UTC)

Merge details

  • Changes merged into master with d19de946.
  • Deleted the source branch.
  • Auto-merge enabled

Pipeline #209796 failed

Pipeline failed for d19de946 on master

Deployed to dep‎loy‎ 4 months ago

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
    • Resolved by Oliver Smith

      The test is not passing in CI, do you want to look into it?

        File "/builds/postmarketOS/pmbootstrap/pmb/parse/bootimg.py", line 195, in bootimg
          mtk_label_kernel=output["mtk_label_kernel"],
                           ~~~~~~^^^^^^^^^^^^^^^^^^^^
      KeyError: 'mtk_label_kernel'
  • Newbyte added 2 commits

    added 2 commits

    • b524d6da - 1 commit from branch master
    • 39f9b4aa - CI: Add bootimg analysis integration test

    Compare with previous version

  • Oliver Smith added 28 commits

    added 28 commits

    Compare with previous version

  • Oliver Smith resolved all threads

    resolved all threads

  • Oliver Smith approved this merge request

    approved this merge request

  • Oliver Smith enabled an automatic merge when all merge checks for d19de946 pass

    enabled an automatic merge when all merge checks for d19de946 pass

  • merged

  • Please register or sign in to reply
    Loading