DeviceManagementService should have more robust retry logic |
|||||||
Issue descriptionWe are seeing issues in the field for auto-starting kiosk sessions where the kiosk session starts before the internet connection is established, so device management activities (like policy fetch and status upload) that happen on startup fail. We should extend the retry logic in DeviceManagementService so that certain network errors (like INTERNET_DISCONNECTED) retry after a delay.
,
Apr 22 2016
,
Apr 26 2016
,
Apr 29 2016
Hi Márton, thank you for taking this bug. Any update?
,
Apr 29 2016
It's in progress, the CL is at https://codereview.chromium.org/1928013004, but I think there are still some cases where we don't retry (and also some browser tests are failing - they time out because of the retry delays).
,
May 13 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/efdd41f9d440254e720d7f2dc1a2958eea5c5df3 commit efdd41f9d440254e720d7f2dc1a2958eea5c5df3 Author: hunyadym <hunyadym@chromium.org> Date: Fri May 13 17:19:09 2016 Add delayed retry to DeviceManagementService requests. If a network related error occurs while executing a request, DeviceManagementService retries to execute it 3 more times, with exponential backoff. BUG= 602993 Review-Url: https://codereview.chromium.org/1928013004 Cr-Commit-Position: refs/heads/master@{#393562} [modify] https://crrev.com/efdd41f9d440254e720d7f2dc1a2958eea5c5df3/chrome/browser/chromeos/policy/user_affiliation_browsertest.cc [modify] https://crrev.com/efdd41f9d440254e720d7f2dc1a2958eea5c5df3/chrome/browser/extensions/api/enterprise_device_attributes/enterprise_device_attributes_apitest.cc [modify] https://crrev.com/efdd41f9d440254e720d7f2dc1a2958eea5c5df3/chrome/browser/policy/cloud/cloud_policy_manager_browsertest.cc [modify] https://crrev.com/efdd41f9d440254e720d7f2dc1a2958eea5c5df3/components/policy/core/common/cloud/device_management_service.cc [modify] https://crrev.com/efdd41f9d440254e720d7f2dc1a2958eea5c5df3/components/policy/core/common/cloud/device_management_service.h [modify] https://crrev.com/efdd41f9d440254e720d7f2dc1a2958eea5c5df3/components/policy/core/common/cloud/device_management_service_unittest.cc
,
May 23 2016
,
May 23 2016
,
May 24 2016
Working with Managed Chrome App in Peppy Device.Tried various connect disconnect of the Network and uploaded new policy files to check. M ChromeOS Chrome ARC Type Channel 52 8352.0.0 52.0.2739.0 2890844 release dev |
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by sdurais...@google.com
, Apr 14 2016