Updated DeviceStatusReportRequest to contain new monitoring data.

components/policy/proto/device_management_backend.proto

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 syntax = "proto2";
 
 option optimize_for = LITE_RUNTIME;
 
 package enterprise_management;
 
@@ skipping 462 matching lines @@
   // formatted as an ASCII string with 12 hex digits. Example: A0B1C2D3E4F5.
   optional string mac_address = 2;
 
   // MEID (if applicable) of the corresponding network device. Formatted as
   // ASCII string composed of 14 hex digits. Example: A10000009296F2.
   optional string meid = 3;
 
   // IMEI (if applicable) of the corresponding network device. 15-16 decimal
   // digits encoded as ASCII string. Example: 355402040158759.
   optional string imei = 4;
+
+  // The device path associated with this network interface.
+  optional string device_path = 5;
+}
+
+
+// Information about configured/visible networks - this is separate from
+// NetworkInterface because a configured network may not be associated with
+// any specific interface, or may be visible across multiple interfaces.
+message NetworkState {
+  // The current state of this network.
+  enum ConnectionState {
+    IDLE=0;

[ygorshenin1, 2014/12/05 15:55:41]

nit: could you please surround '=' characters by spaces in the enum?

[Andrew T Wilson (Slow), 2014/12/11 00:09:55]

Done.

+    CARRIER=1;
+    ASSOCIATION=2;
+    CONFIGURATION=3;
+    READY=4;
+    PORTAL=5;
+    OFFLINE=6;
+    ONLINE=7;
+    DISCONNECT = 8;
+    FAILURE = 9;
+    ACTIVATION_FAILURE=10;
+    UNKNOWN=11;
+  }
+
+  // For networks associated with a device, the path of the device.
+  optional string device_path = 1;
+
+  // Current state of this connection as reported by shill.
+  optional ConnectionState connection_state = 2;
+
+  // For wireless networks, the signal_strength in dBm.
+  optional int32 signal_strength = 3;
 }
 
 // Details about a device user.
 message DeviceUser {
   // Types of device users which can be reported.
   enum UserType {
     // A user managed by the same domain as the device.
     USER_TYPE_MANAGED = 0;
 
     // A user not managed by the same domain as the device.
     USER_TYPE_UNMANAGED = 1;
   }
 
   // The type of the user.
   required UserType type = 1;
 
   // Email address of the user. Present only if the user type is managed.
   optional string email = 2;
 }
 
+// Information about a single disk volume.

[pneubeck (no reviews), 2014/12/09 03:00:10]

are the changes down from here required for / used in this CL?

[Andrew T Wilson (Slow), 2014/12/11 00:09:55]

Not strictly required, but I'm trying to put all of the reporting proto changes
in a single CL, even though I'm not filling in every field yet.

+message VolumeInfo {
+  optional string volume_id = 1;
+  optional int64 storage_used = 2;
+  optional int64 storage_free = 3;
+}
+
 // Report device level status.
 message DeviceStatusReportRequest {
   // The OS version reported by the device is a platform version
   // e.g. 1435.0.2011_12_16_1635.
   optional string os_version = 1;
   optional string firmware_version = 2;
 
   // "Verified", "Dev". Same as verified mode.
   // If the mode is unknown, this field should not be set.
   optional string boot_mode = 3;
@@ skipping 10 matching lines @@
   repeated ActiveTimePeriod active_period = 6;
 
   // The device location.
   optional DeviceLocation device_location = 7;
 
   // List of network interfaces.
   repeated NetworkInterface network_interface = 8;
 
   // List of recent device users, in descending order by last login time.
   repeated DeviceUser user = 9;
+
+  // Disk space + other info about mounted/connected volumes.
+  repeated VolumeInfo volume_info = 10;
+
+  // List of visible/configured networks
+  repeated NetworkState network_state = 11;
+
+  // Samples of CPU utilization (0-100), sampled once every 60 seconds.
+  repeated int32 cpu_utilization_pct = 12;
+
+  // RAM free (as reported on a per-process level, unreliable due to GC).
+  optional int64 system_ram_free = 13;
+  optional int64 system_ram_used = 14;
+}
+
+// Provides status information for an installed app/extension.
+message AppStatus {
+  // ID of the installed app/extension
+  required string app_id = 1;
+
+  // Currently installed version of the app.
+  required string extension_version = 2;
+
+  // Self-reported status summary (via chrome.reporting APIs)
+  optional string status = 3;
+
+  // If true, the application is currently in a self-reported error state.
+  optional bool error = 4;
 }
 
 // Report session (a user on one device) level status.
 message SessionStatusReportRequest {
   // Installed apps for this user on this device.
-  repeated string installed_app_id = 1;
+  // No longer used -- use installed_apps instead.
+  repeated string installed_app_id = 1 [deprecated = true];
 
   // Installed extensions for this user on this device.
-  repeated string installed_extension_id = 2;
+  // No longer used -- use installed_extensions instead.
+  repeated string installed_extension_id = 2 [deprecated = true];
 
   // One stat per app for top 30 apps.
   repeated InstallableLaunch app_launch_stat = 3;
+
+  // If this is a kiosk session, this is the device local account ID.
+  optional string device_local_account_id = 4;
+
+  // Information about installed apps for this user on this device.
+  repeated AppStatus installed_apps = 5;
+
+  // Information about installed extensions for this user on this device.
+  repeated AppStatus installed_extensions = 6;
 }
 
 // Response from DMServer to update devices' status.
 // It is possible that status report fails but policy request succeed.  In such
 // case, the DeviceStatusReportResponse will contain an error code and the
 // device should re-send status report data in the next policy request.  The
 // device should re-send report data if policy request fails, even if
 // DeviceStatusReportResponse contains no error code.
 message DeviceStatusReportResponse {
   optional int32 error_code = 1;
@@ skipping 320 matching lines @@
 
   // Device-state retrieval.
   optional DeviceStateRetrievalResponse device_state_retrieval_response = 11;
 
   // Response to device pairing request.
   optional DevicePairingResponse device_pairing_response = 12;
 
   // Response to check device pairing request.
   optional CheckDevicePairingResponse check_device_pairing_response = 13;
 }