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

prepare-root: On a non-A/B androidboot system, boot system slot a #2975

Merged
merged 2 commits into from
Aug 22, 2023

Conversation

ericcurtin
Copy link
Collaborator

Sometimes android bootloaders boot in a nonab way:

https://source.android.com/docs/core/ota/nonab

In this case, "androidboot." kargs are present but not "androidboot.slot_suffix" specifically.

In this case, rather than getting stuck in a partially booted environment, boot system slot a.

@cgwalters
Copy link
Member

In the case of a Non-A/B androidboot system, boot system slot a

Our commit messages include a "topic prefix" usually unless they're system wide changes. In this case something like (also slightly condensed)

prepare-root: On a non-A/B androidboot system, boot slot a

src/switchroot/ostree-prepare-root.c Outdated Show resolved Hide resolved
src/switchroot/ostree-prepare-root.c Outdated Show resolved Hide resolved
src/switchroot/ostree-prepare-root.c Outdated Show resolved Hide resolved
src/switchroot/ostree-prepare-root.c Outdated Show resolved Hide resolved
src/switchroot/ostree-prepare-root.c Outdated Show resolved Hide resolved
src/switchroot/ostree-prepare-root.c Outdated Show resolved Hide resolved
@ericcurtin ericcurtin force-pushed the androidboot-single-slot-mode branch 3 times, most recently from bd0e4f4 to ba42ea0 Compare August 22, 2023 13:22
@ericcurtin ericcurtin changed the title In the case of a Non-A/B androidboot system, boot system slot a prepare-root: On a non-A/B androidboot system, boot system slot a Aug 22, 2023
Sometimes android bootloaders boot in a nonab way:

  https://source.android.com/docs/core/ota/nonab

In this case, "androidboot." kargs are present but not
"androidboot.slot_suffix" specifically.

In this case, rather than getting stuck in a partially booted
environment, boot system slot a.
Used strspn based on feedback from similar function.
@ericcurtin
Copy link
Collaborator Author

Hitting flakes in the CI I think

@ericcurtin
Copy link
Collaborator Author

/retest

@openshift-ci
Copy link

openshift-ci bot commented Aug 22, 2023

@ericcurtin: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/images 6e9e50d link true /test images

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@cgwalters
Copy link
Member

/test images

@cgwalters cgwalters merged commit 7c82340 into main Aug 22, 2023
20 checks passed
@ericcurtin ericcurtin deleted the androidboot-single-slot-mode branch August 23, 2023 09:00
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.

3 participants