Skip to content

Always run snapshot_hook_* as root - #3076

Merged
cjreynol merged 1 commit into
google:version-1.56-devfrom
cjreynol:backport_snapshot_hook
Aug 21, 2026
Merged

Always run snapshot_hook_* as root#3076
cjreynol merged 1 commit into
google:version-1.56-devfrom
cjreynol:backport_snapshot_hook

Conversation

@cjreynol

Copy link
Copy Markdown
Collaborator

Backport of #2860 (and finish matching the 1.56 mirror of #2979)

When not run as root, this produces errors like

+ /system/bin/cmd nfc status
+ grep -q disabled
/vendor/bin/snapshot_hook_pre_suspend[40]: grep: inaccessible or not found
+ sleep 0.1
/vendor/bin/snapshot_hook_pre_suspend[40]: sleep: inaccessible or not found

This may have worked intermittently before depending on whether the runtime environment was also using adb root.

Bug: b/534832487

When not run as root, this produces errors like
```
+ /system/bin/cmd nfc status
+ grep -q disabled
/vendor/bin/snapshot_hook_pre_suspend[40]: grep: inaccessible or not found
+ sleep 0.1
/vendor/bin/snapshot_hook_pre_suspend[40]: sleep: inaccessible or not found
```

This may have worked intermittently before depending on whether the
runtime environment was also using `adb root`.

Bug: b/534832487
@cjreynol
cjreynol requested a review from Databean August 21, 2026 23:47
@cjreynol cjreynol self-assigned this Aug 21, 2026
@cjreynol cjreynol added kokoro:force-run Trigger a presubmit build unconditionally. kokoro:run Run e2e tests. labels Aug 21, 2026
@GoogleCuttlefishTesterBot GoogleCuttlefishTesterBot removed kokoro:run Run e2e tests. kokoro:force-run Trigger a presubmit build unconditionally. labels Aug 21, 2026
@cjreynol
cjreynol merged commit 89fc3b1 into google:version-1.56-dev Aug 21, 2026
19 of 29 checks passed
@cjreynol
cjreynol deleted the backport_snapshot_hook branch August 21, 2026 23:50
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