default searchable snapshot to enabled on cloud when creating a new p…

…olicy
elastic · Jan 18, 2021 · 9c2956a · 9c2956a
1 parent f7fdda5
commit 9c2956a
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/...y/components/phases/shared_fields/searchable_snapshot_field/searchable_snapshot_field.tsx b/...y/components/phases/shared_fields/searchable_snapshot_field/searchable_snapshot_field.tsx
@@ -51,23 +51,28 @@ export const SearchableSnapshotField: FunctionComponent<Props> = ({ phase }) =>
   const {
     services: { cloud },
   } = useKibana();
-  const { getUrlForApp, policy, license } = useEditPolicyContext();
+  const { getUrlForApp, policy, license, isNewPolicy } = useEditPolicyContext();
   const { isUsingSearchableSnapshotInHotPhase, isUsingRollover } = useConfigurationIssues();
 
   const searchableSnapshotPath = `phases.${phase}.actions.searchable_snapshot.snapshot_repository`;
 
   const [formData] = useFormData({ watch: searchableSnapshotPath });
   const searchableSnapshotRepo = get(formData, searchableSnapshotPath);
 
+  const isColdPhase = phase === 'cold';
   const isDisabledDueToLicense = !license.canUseSearchableSnapshot();
-  const isDisabledInColdDueToHotPhase = phase === 'cold' && isUsingSearchableSnapshotInHotPhase;
-  const isDisabledInColdDueToRollover = phase === 'cold' && !isUsingRollover;
+  const isDisabledInColdDueToHotPhase = isColdPhase && isUsingSearchableSnapshotInHotPhase;
+  const isDisabledInColdDueToRollover = isColdPhase && !isUsingRollover;
 
   const isDisabled =
     isDisabledDueToLicense || isDisabledInColdDueToHotPhase || isDisabledInColdDueToRollover;
 
   const [isFieldToggleChecked, setIsFieldToggleChecked] = useState(() =>
-    Boolean(policy.phases[phase]?.actions?.searchable_snapshot?.snapshot_repository)
+    Boolean(
+      // New policy on cloud should have searchable snapshot on in cold phase
+      (isColdPhase && isNewPolicy && cloud?.isCloudEnabled) ||
+        policy.phases[phase]?.actions?.searchable_snapshot?.snapshot_repository
+    )
   );
 
   useEffect(() => {

diff --git a/...ndex_lifecycle_management/public/application/sections/edit_policy/edit_policy_context.tsx b/...ndex_lifecycle_management/public/application/sections/edit_policy/edit_policy_context.tsx
@@ -32,7 +32,7 @@ export const EditPolicyContextProvider = ({
   return <EditPolicyContext.Provider value={value}>{children}</EditPolicyContext.Provider>;
 };
 
-export const useEditPolicyContext = () => {
+export const useEditPolicyContext = (): EditPolicyContextValue => {
   const ctx = useContext(EditPolicyContext);
   if (!ctx) {
     throw new Error('useEditPolicyContext can only be called inside of EditPolicyContext!');