New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 605546 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
User never visited
Closed: Apr 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

crosh network_diag should test kiosk mode hosts

Project Member Reported by jayhlee@google.com, Apr 21 2016

Issue description

Chrome Version: current stable

Please specify Cr-* of the system to which this bug/feature applies (add
the label below).
Cr-OS-Systems-Network

Steps To Reproduce:
(1) Run crosh network_diag --hosts command.
(2) compare hosts to those listed in help article at: https://support.google.com/chrome/a/answer/6334001

Expected Result:
We should be testing all hosts so admins can confirm all hosts are properly whitelisted.

Patch coming shortly.
 
Cc: snanda@chromium.org
Owner: jleong@chromium.org
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 21 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/cde8b1c212a4461ac3bc986fb98533872f2d51f5

commit cde8b1c212a4461ac3bc986fb98533872f2d51f5
Author: Jay Lee <jayhlee@google.com>
Date: Thu Apr 21 14:31:40 2016

crosh: add network_diag hosts necessary for kiosk mode

Add kiosk hosts, re-order other hosts to match help article.
BUG= chromium:605546 
TEST=run network_diag --hosts from crosh

Change-Id: Ia6b1d19c3cc559d8c2a9a5c064936ea97379e369
Reviewed-on: https://chromium-review.googlesource.com/340220
Commit-Ready: Jay Lee <jayhlee@google.com>
Tested-by: Jay Lee <jayhlee@google.com>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/cde8b1c212a4461ac3bc986fb98533872f2d51f5/crosh/network_diag

Comment 3 by jayhlee@google.com, Apr 22 2016

Status: Fixed (was: Unconfirmed)
Steps to verify:

-Open crosh shell with CTRL+ALT+T
-Run: network_diag --hosts
-Confirm all hosts listed at:
  https://support.google.com/chrome/a/answer/6334001
are tested except for cache.pack.google.com (see crbug.com/405052#c17 for reason behind this exclusion)
Bulk verified
Status: Verified (was: Fixed)
bulk verified

Sign in to add a comment