New issue
Advanced search Search tips

Issue 615914 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

[autotest frontend] Reinstall button does not work and should be removed

Project Member Reported by cywang@chromium.org, May 31 2016

Issue description

The python binary is missing on DUT, therefore, 'Repair' function does not work. However, 'Reinstall' button should be used for fixing this machine, but now we got the error 'No test found: autoupdate:repair' when 'Reinstall' is clicked.
 

Comment 1 by cywang@chromium.org, May 31 2016

BTW, the DUT 'chromeos4-row9-rack10-host18' could be used for testing.

Comment 2 by cywang@chromium.org, May 31 2016

Cc: dgarr...@chromium.org xixuan@chromium.org

Comment 3 by d...@chromium.org, May 31 2016

Components: -Infra>Labs
Labels: Hardware-Lab
Components: Infra>Client>ChromeOS
Labels: -Hardware-Lab
Repair can specifically fix a problem with a missing Python.

I believe there's no intent to fix 'Reinstall'.  It's far
more likely that we'll remove it.
Labels: Hotlist-Fixit
Summary: [autotest frontend] Reinstall button does not work and should be removed (was: [autotest frontend] Reinstall button does not work)
Moving to FixIt & renaming 
Owner: xixuan@chromium.org
Status: Started (was: Untriaged)
Project Member

Comment 7 by bugdroid1@chromium.org, Feb 4 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/9d29c26beb33a13422479e98b3604b2a4eacf80b

commit 9d29c26beb33a13422479e98b3604b2a4eacf80b
Author: xixuan <xixuan@chromium.org>
Date: Sat Feb 04 04:01:48 2017

autotest: Remove Reinstall-related codes.

This CL removes all codes related to host 'Reinstall'.

BUG= chromium:615914 
TEST=Run local afe, verify that there's no Reinstall on 'Host List' and
'View Host'. Kick off a verify job, and verify it runs well.

Change-Id: Id51a9ba5a7668a42cfdb2ccdd670c1f866f88526
Reviewed-on: https://chromium-review.googlesource.com/435487
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Richard Barnette <jrbarnette@google.com>

[modify] https://crrev.com/9d29c26beb33a13422479e98b3604b2a4eacf80b/frontend/client/src/autotest/afe/AfeUtils.java
[modify] https://crrev.com/9d29c26beb33a13422479e98b3604b2a4eacf80b/frontend/client/src/autotest/afe/HostListView.java
[modify] https://crrev.com/9d29c26beb33a13422479e98b3604b2a4eacf80b/frontend/client/src/autotest/public/AfeClient.html
[modify] https://crrev.com/9d29c26beb33a13422479e98b3604b2a4eacf80b/frontend/client/src/autotest/afe/HostDetailView.java

Labels: cros-infra-fixedit-q117
Status: Fixed (was: Started)

Comment 9 by dchan@google.com, Apr 17 2017

Labels: VerifyIn-59

Comment 10 by dchan@google.com, May 30 2017

Labels: VerifyIn-60
Labels: VerifyIn-61

Comment 12 by dchan@chromium.org, Oct 14 2017

Status: Archived (was: Fixed)

Sign in to add a comment