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

plugins: fastboot: add commands for interacting with logical partitions #2748

Merged
merged 4 commits into from
Jul 26, 2022

Conversation

amartinz
Copy link
Member

@amartinz amartinz commented Jul 6, 2022

This implements the following new action:

  - actions:
      - fastboot:create_logical_partition:
          partition: "partition_name"
          size: 3221225472

      - fastboot:delete_logical_partition:
          partition: "partition_name"

      - fastboot:resize_logical_partition:
          partition: "partition_name"
          size: 3221225472

These actions are required for certain devices with
dynamic partitions, like the Fairphone 4.

Requirements

promise-android-tools needs support for these actions:

Then we need a new release of it and dependencies need to be updated.

This implements the following new action:

  - actions:
      - fastboot:create_logical_partition:
          partition: "partition_name"
          size: 3221225472

      - fastboot:delete_logical_partition:
          partition: "partition_name"

      - fastboot:resize_logical_partition:
          partition: "partition_name"
          size: 3221225472

These actions are required for certain devices with
dynamic partitions, like the Fairphone 4.

Change-Id: I02e0dc1dcfa3674bbdb2525e4abdff7dfbe868b4
Signed-off-by: Alexander Martinz <amartinz@shiftphones.com>
@amartinz
Copy link
Member Author

amartinz commented Jul 6, 2022

This is required for the Fairphone 4 installer bringup.

/cc @Flohack74 @fredldotme @z3ntu

Flohack74
Flohack74 previously approved these changes Jul 6, 2022
mariogrip
mariogrip previously approved these changes Jul 6, 2022
@Flohack74
Copy link
Member

@amartinz is this failing in CI on purpose?

@amartinz
Copy link
Member Author

@amartinz is this failing in CI on purpose?

Yes, see Requirements.

@sunweaver
Copy link

@NeoTheThird Any chance you can review this some time soon? We need this change to get into the installer for deploying Ubuntu Touch on e.g. Fairphone 4. Thanks.

@mariogrip mariogrip dismissed stale reviews from Flohack74 and themself via 1a8058d July 26, 2022 13:18
@codecov
Copy link

codecov bot commented Jul 26, 2022

Codecov Report

Merging #2748 (1a8058d) into master (5f5a494) will increase coverage by 0.34%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #2748      +/-   ##
==========================================
+ Coverage   77.24%   77.59%   +0.34%     
==========================================
  Files          29       29              
  Lines         967      982      +15     
==========================================
+ Hits          747      762      +15     
  Misses        220      220              
Impacted Files Coverage Δ
src/core/plugins/fastboot/plugin.js 55.26% <100.00%> (+6.77%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5f5a494...1a8058d. Read the comment docs.

@mariogrip mariogrip merged commit 984caef into ubports:master Jul 26, 2022
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.

4 participants