New issue
Advanced search Search tips

Issue 747930 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2017
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Task

Blocking:
issue 747907



Sign in to add a comment

Change default value of EcryptfsMigrationStrategy user policy back to disallow migration

Project Member Reported by pmarko@chromium.org, Jul 24 2017

Issue description

User policy EcryptfsMigrationStrategy has been introduced for M61 ( bug 747907 ).

The default value for managed users is planned to be DISALLOW_ARC_NO_MIGRATION, but was changed to ASK_USER until the server-side changes are implemented.

Be sure to revert the default back to DISALLOW_ARC_NO_MIGRATION before M61 goes to stable.
 

Comment 1 by pmarko@chromium.org, Jul 24 2017

Blocking: 747907

Comment 2 by pmarko@chromium.org, Jul 24 2017

Summary: Re-enable user policy EcryptfsMigrationStrategy (was: Enable user policy EcryptfsMigrationStrategy)

Comment 3 by pmarko@chromium.org, Jul 24 2017

Description: Show this description

Comment 4 by pmarko@chromium.org, Jul 24 2017

Summary: Change default value of EcryptfsMigrationStrategy user policy back to disallow migration (was: Re-enable user policy EcryptfsMigrationStrategy)
Project Member

Comment 6 by bugdroid1@chromium.org, Aug 30 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/715b9c42cf11b38a86a613be4ed195e49e8abfe8

commit 715b9c42cf11b38a86a613be4ed195e49e8abfe8
Author: Pavol Marko <pmarko@chromium.org>
Date: Wed Aug 30 18:30:57 2017

Introduce additional EcryptfsMigrationStrategy values

Introduce additional values for the EcryptfsMigrationStrategy policy:
- MinimalMigrate (value 4) in preparation for minimal migration mode.
  For now, this behaves as wipe (so if we merge only this back to M-61,
  the M-61 client will use "Wipe" when this policy comes in).
- AskForEcryptfsArcUsers (value 5) in preparation for the new default
  for EDU.

Additionally, this CL:
- Changes the default value (if the cached policy is too old to contain
  the value) to kDisallowMigration as per  crbug.com/747930 
- Fixes handling of max value in DecodeMigrationActionFromPolicy.

BUG= 747907 , 747930 

Change-Id: Ib800b7d34aade8507a2c8f853d224fb33df40b95
Reviewed-on: https://chromium-review.googlesource.com/641871
Reviewed-by: Maksim Ivanov <emaxx@chromium.org>
Reviewed-by: Achuith Bhandarkar <achuith@chromium.org>
Reviewed-by: Igor <igorcov@chromium.org>
Commit-Queue: Pavol Marko <pmarko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#498542}
[modify] https://crrev.com/715b9c42cf11b38a86a613be4ed195e49e8abfe8/chrome/browser/chromeos/arc/policy/arc_policy_util.h
[modify] https://crrev.com/715b9c42cf11b38a86a613be4ed195e49e8abfe8/chrome/browser/chromeos/login/existing_user_controller.cc
[modify] https://crrev.com/715b9c42cf11b38a86a613be4ed195e49e8abfe8/components/policy/resources/policy_templates.json

Project Member

Comment 7 by bugdroid1@chromium.org, Sep 5 2017

Labels: merge-merged-3163
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a83b253ba61617231915dc02628dfead3eefcbb4

commit a83b253ba61617231915dc02628dfead3eefcbb4
Author: Pavol Marko <pmarko@chromium.org>
Date: Tue Sep 05 10:44:13 2017

[Merge to M61] Introduce additional EcryptfsMigrationStrategy values

Introduce additional values for the EcryptfsMigrationStrategy policy:
- MinimalMigrate (value 4) in preparation for minimal migration mode.
  For now, this behaves as wipe (so if we merge only this back to M-61,
  the M-61 client will use "Wipe" when this policy comes in).
- AskForEcryptfsArcUsers (value 5) in preparation for the new default
  for EDU.

Additionally, this CL:
- Changes the default value (if the cached policy is too old to contain
  the value) to kDisallowMigration as per  crbug.com/747930 
- Fixes handling of max value in DecodeMigrationActionFromPolicy.

BUG= 747907 , 747930 
TBR=pmarko@chromium.org

(cherry picked from commit 715b9c42cf11b38a86a613be4ed195e49e8abfe8)

Change-Id: Ib800b7d34aade8507a2c8f853d224fb33df40b95
Reviewed-on: https://chromium-review.googlesource.com/641871
Reviewed-by: Maksim Ivanov <emaxx@chromium.org>
Reviewed-by: Achuith Bhandarkar <achuith@chromium.org>
Reviewed-by: Igor <igorcov@chromium.org>
Commit-Queue: Pavol Marko <pmarko@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#498542}
Reviewed-on: https://chromium-review.googlesource.com/649695
Reviewed-by: Pavol Marko <pmarko@chromium.org>
Cr-Commit-Position: refs/branch-heads/3163@{#1098}
Cr-Branched-From: ff259bab28b35d242e10186cd63af7ed404fae0d-refs/heads/master@{#488528}
[modify] https://crrev.com/a83b253ba61617231915dc02628dfead3eefcbb4/chrome/browser/chromeos/arc/policy/arc_policy_util.h
[modify] https://crrev.com/a83b253ba61617231915dc02628dfead3eefcbb4/chrome/browser/chromeos/login/existing_user_controller.cc
[modify] https://crrev.com/a83b253ba61617231915dc02628dfead3eefcbb4/components/policy/resources/policy_templates.json

Status: Fixed (was: Assigned)
Note: the following summary only applies to managed users. Unmanaged users can always choose if they want to migrate.
Default value for not retrievable user policy or if the EcryptfsMigrationStrategy user policy value is missing are:
M61: 0 (DisallowMigration)
M62: GAIA user: 0 (DisallowMigration)
     AD user: 3 (AskUser)

Sign in to add a comment