Issue 444435 in chromium: CrOSH fails to detect battery when using battery_test

[chro...@googlecode.com]

Status: Unconfirmed
Owner: ----
Labels: Pri-2 Via-Wizard Type-Bug-Regression OS-Chrome

New issue 444435 by frankt3...@gmail.com: CrOSH fails to detect battery
when using battery_test
https://code.google.com/p/chromium/issues/detail?id=444435

UserAgent: Mozilla/5.0 (X11; CrOS x86_64 6457.36.0) AppleWebKit/537.36
(KHTML, like Gecko) Chrome/40.0.2214.43 Safari/537.36
Platform: 6457.36.0 (Official Build) beta-channel parrot

Steps to reproduce the problem:
1. Open CrOSH
2. type battery_test
3. see error lines; no battery detected; fails to find path: No battery
found. Insert the battery and reboot the device.
crosh> ls: cannot access /sys/class/power_supply/*/online: No such file or
directory
Unknown command: 'ls:'
crosh> dirname: missing operand

What is the expected behavior?
battery test is run

What went wrong?
does not run battery test.

Did this work before? N/A

Chrome version: 40.0.2214.43 Channel: beta
OS Version: 6457.36.0
Flash Version: Shockwave Flash 16.0 r0

tried two restarts; two good battery packs; reset EC. Reported in
Chromebook TC RS Private Forum:
https://productforums.google.com/forum/#!private-topic/chromebook-central+rs-mentor/evXTiaTq1NI
Confirmed by mutliple TC's and RS's/ Feedback Reports filed.
THIS IS A REGRESSION

--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

[chro...@googlecode.com]

Comment #1 on issue 444435 by frankt3...@gmail.com: CrOSH fails to detect

battery when using battery_test
https://code.google.com/p/chromium/issues/detail?id=444435

Did this work before? YES

[chro...@googlecode.com]

Updates:
Labels: Cr-OS-Kernel-Power

Comment #2 on issue 444435 by kathrelk...@chromium.org: CrOSH fails to

detect battery when using battery_test
https://code.google.com/p/chromium/issues/detail?id=444435

(No comment was entered for this change.)

[chro...@googlecode.com]

Updates:
Cc: amesterh...@chromium.org
Labels: Hotlist-ConOps

Comment #4 on issue 444435 by tza...@chromium.org: CrOSH fails to detect

battery when using battery_test
https://code.google.com/p/chromium/issues/detail?id=444435

Related Feedback:
http://feedback/#/report/2407562047

[chro...@googlecode.com]

Comment #6 on issue 444435 by bugdro...@chromium.org: CrOSH fails to detect
battery when using battery_test
https://code.google.com/p/chromium/issues/detail?id=444435#c6

Project : chromiumos/platform2
Branch : master
Author : Gediminas Ramanauskas <ge...@google.com>
Committer: ChromeOS Commit Bot <chromeos-...@chromium.org>
Commit : ce9b8e7973fc74f22cf416ff613e4990b3be31ea

Code-Review 0 : ChromeOS Commit Bot, Gediminas Ramanauskas, Royans Tharakan
Code-Review +2: Kees Cook, Mike Frysinger
Commit-Queue 0 : ChromeOS Commit Bot, Kees Cook, Mike Frysinger, Royans
Tharakan
Commit-Queue +1: Gediminas Ramanauskas
Verified 0 : ChromeOS Commit Bot, Kees Cook, Mike Frysinger, Royans
Tharakan
Verified +1: Gediminas Ramanauskas
Commit Queue : Chumped
Change-Id : I4b4b0690c7fc02b1f214eb9f7fb501747b3d7843
Reviewed-at : https://chromium-review.googlesource.com/238524

crosh: fix battery_test path expansion

It's safe to hardcode battery path instead of using auto expansion
due to the implementation maturity.

BUG=chromium:444435
TEST=run crosh battery_test

crosh/crosh

[chro...@googlecode.com]

Updates:
Status: Fixed

Comment #7 on issue 444435 by ge...@chromium.org: CrOSH fails to detect
battery when using battery_test

https://code.google.com/p/chromium/issues/detail?id=444435

(No comment was entered for this change.)

[chro...@googlecode.com]

Comment #8 on issue 444435 by bugdro...@chromium.org: CrOSH fails to detect
battery when using battery_test
https://code.google.com/p/chromium/issues/detail?id=444435#c8

Project : chromiumos/platform2
Branch : master
Author : Gediminas Ramanauskas <ge...@google.com>
Committer: ChromeOS Commit Bot <chromeos-...@chromium.org>
Commit : ce9b8e7973fc74f22cf416ff613e4990b3be31ea

Code-Review 0 : ChromeOS Commit Bot, Daniel Erat, Gediminas Ramanauskas,

Royans Tharakan
Code-Review +2: Kees Cook, Mike Frysinger

Commit-Queue 0 : ChromeOS Commit Bot, Daniel Erat, Kees Cook, Mike

Frysinger, Royans Tharakan
Commit-Queue +1: Gediminas Ramanauskas

Verified 0 : ChromeOS Commit Bot, Daniel Erat, Kees Cook, Mike

Frysinger, Royans Tharakan
Verified +1: Gediminas Ramanauskas
Commit Queue : Chumped
Change-Id : I4b4b0690c7fc02b1f214eb9f7fb501747b3d7843
Reviewed-at : https://chromium-review.googlesource.com/238524

crosh: fix battery_test path expansion

It's safe to hardcode battery path instead of using auto expansion
due to the implementation maturity.

BUG=chromium:444435
TEST=run crosh battery_test

crosh/crosh

[chro...@googlecode.com]

Comment #9 on issue 444435 by WillG...@gmail.com: CrOSH fails to detect

battery when using battery_test
https://code.google.com/p/chromium/issues/detail?id=444435

Can this be merged into Beta? This test is really important for
troubleshooting battery issues. We request posters to run the test
frequently for battery issues in Chromebook Central.

[chro...@googlecode.com]

Comment #10 on issue 444435 by ge...@chromium.org: CrOSH fails to detect

battery when using battery_test
https://code.google.com/p/chromium/issues/detail?id=444435

Are you able to do some more testing on canary on a few more devices? There
are some doubts on a CL regarding the assumptions I made hardcoding paths.
I can look into it more tomorrow as well.

[chro...@googlecode.com]

Comment #15 on issue 444435 by bugdro...@chromium.org: CrOSH fails to

detect battery when using battery_test

https://code.google.com/p/chromium/issues/detail?id=444435#c15

Project : chromiumos/platform2
Branch : master
Author : Gediminas Ramanauskas <ge...@google.com>
Committer: ChromeOS Commit Bot <chromeos-...@chromium.org>
Commit : ce9b8e7973fc74f22cf416ff613e4990b3be31ea

Code-Review 0 : ChromeOS Commit Bot, Dan Erat, Gediminas Ramanauskas,
Royans Tharakan, Steven Jian

Code-Review +2: Kees Cook, Mike Frysinger

Commit-Queue 0 : ChromeOS Commit Bot, Dan Erat, Kees Cook, Mike Frysinger,
Royans Tharakan, Steven Jian
Commit-Queue +1: Gediminas Ramanauskas
Verified 0 : ChromeOS Commit Bot, Dan Erat, Kees Cook, Mike Frysinger,
Royans Tharakan, Steven Jian

Verified +1: Gediminas Ramanauskas
Commit Queue : Chumped
Change-Id : I4b4b0690c7fc02b1f214eb9f7fb501747b3d7843
Reviewed-at : https://chromium-review.googlesource.com/238524

crosh: fix battery_test path expansion

It's safe to hardcode battery path instead of using auto expansion
due to the implementation maturity.

BUG=chromium:444435
TEST=run crosh battery_test

crosh/crosh

[chro...@googlecode.com]

Comment #16 on issue 444435 by bugdro...@chromium.org: CrOSH fails to

detect battery when using battery_test

https://code.google.com/p/chromium/issues/detail?id=444435#c16

Project : chromiumos/platform2
Branch : master
Author : Daniel Erat <de...@chromium.org>

Committer: ChromeOS Commit Bot <chromeos-...@chromium.org>

Commit : 980592907ce1e67329695a444356076f5445f8d7

Code-Review 0 : ChromeOS Commit Bot, Dan Erat

Code-Review +2: Gediminas Ramanauskas, Mike Frysinger
Commit-Queue 0 : ChromeOS Commit Bot, Gediminas Ramanauskas, Mike Frysinger
Commit-Queue +1: Dan Erat
Trybot-Ready 0 : ChromeOS Commit Bot, Gediminas Ramanauskas, Mike Frysinger
Trybot-Ready +1: Dan Erat
Verified 0 : ChromeOS Commit Bot, Gediminas Ramanauskas, Mike Frysinger
Verified +1: Dan Erat
Commit Queue : Chumped
Change-Id : I3a110d6f50889016a70e938a6e800809725e0cce
Reviewed-at : https://chromium-review.googlesource.com/240161

crosh: Use dump_power_status instead of sysfs.

Update crosh's battery_test command to call
the dump_power_status binary instead of reading data from
sysfs. Also improve style (local, awk, etc.).

BUG=chromium:444435
TEST=manual: ran it
CQ-DEPEND=https://chromium-review.googlesource.com/#/q/Ib1ff5b899c39e44169eb1e45ea6adb2eed948807,n,z

[chro...@googlecode.com]

Comment #17 on issue 444435 by jleig...@sheboygan.k12.wi.us: CrOSH fails to

detect battery when using battery_test
https://code.google.com/p/chromium/issues/detail?id=444435

I'm on Stable channel and just ran into this same problem after the
update. Must not have gotten the fix incorporated in this version.

Chrome OS Version: 40.0.2214.93
Platform: 6457.83.0
Firmware Google_Peppy.4389.89.0

[chro...@googlecode.com]

Updates:
Status: Verified

Comment #21 on issue 444435 by helenzh...@chromium.org: CrOSH fails to

detect battery when using battery_test
https://code.google.com/p/chromium/issues/detail?id=444435

verified M42 (6778.0.0 / 42.0.2296.0

[chro...@googlecode.com]

Comment #22 on issue 444435 by seahorse...@gmail.com: CrOSH fails to detect

battery when using battery_test
https://code.google.com/p/chromium/issues/detail?id=444435

Version 40.0.2214.115 (64-bit)
Platform 6457.107.0 (Official Build) stable-channel falco
Firmware Google_Falco.4389.92.0

Still showing up on Stable Falco. Any fixes in the works?

Attachments:
Screenshot 2015-03-01 at 9.10.35 PM.png 35.6 KB

[chro...@googlecode.com]

Comment #23 on issue 444435 by de...@chromium.org: CrOSH fails to detect

battery when using battery_test
https://code.google.com/p/chromium/issues/detail?id=444435

seahorsebaker: The fix is in M41, not M40.