fix: Add "Tested with" before Ubuntu version in kernel crash dump OS …

…requirement tooltips (#5558) - Tooltips showing the minimum Ubuntu version for enabling kernel crash dumps now say "Tested with" before the version number.
canonical · Dec 5, 2024 · 0a7f880 · 0a7f880
1 parent 6f4d066
commit 0a7f880
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 10 deletions.
diff --git a/...chineForms/MachineActionFormWrapper/DeployForm/DeployFormFields/DeployFormFields.test.tsx b/...chineForms/MachineActionFormWrapper/DeployForm/DeployFormFields/DeployFormFields.test.tsx
@@ -726,7 +726,7 @@ describe("DeployFormFields", () => {
 
     await waitFor(() => {
       expect(screen.getByRole("tooltip")).toHaveTextContent(
-        "Ubuntu 24.04 LTS or higher."
+        "Tested with Ubuntu 24.04 LTS or higher."
       );
     });
   });

diff --git a/...ts/MachineForms/MachineActionFormWrapper/DeployForm/DeployFormFields/DeployFormFields.tsx b/...ts/MachineForms/MachineActionFormWrapper/DeployForm/DeployFormFields/DeployFormFields.tsx
@@ -291,13 +291,15 @@ export const DeployFormFields = (): JSX.Element => {
                       </span>
                     }
                   />{" "}
-                  and OS{" "}
+                  must meet the minimum requirements and the OS{" "}
                   <TooltipButton
                     iconName="help-mid-dark"
-                    message="Ubuntu 24.04 LTS or higher."
+                    message="Tested with Ubuntu 24.04 LTS or higher."
                   />{" "}
-                  must meet the minimum requirements and secure boot must be
-                  disabled. Check crash dump status in machine details.
+                  must support it. Check crash dump status in machine details.{" "}
+                  <ExternalLink to="https://ubuntu.com/server/docs/kernel-crash-dump">
+                    More about kernel crash dump
+                  </ExternalLink>
                 </>
               }
               label={

diff --git a/src/app/settings/views/Configuration/KernelParametersForm/KernelParametersForm.test.tsx b/src/app/settings/views/Configuration/KernelParametersForm/KernelParametersForm.test.tsx
@@ -128,7 +128,7 @@ describe("KernelParametersForm", () => {
 
     await waitFor(() => {
       expect(screen.getByRole("tooltip")).toHaveTextContent(
-        "Ubuntu 24.04 LTS or higher."
+        "Tested with Ubuntu 24.04 LTS or higher."
       );
     });
   });

diff --git a/src/app/settings/views/Configuration/KernelParametersForm/KernelParametersForm.tsx b/src/app/settings/views/Configuration/KernelParametersForm/KernelParametersForm.tsx
@@ -86,13 +86,12 @@ const KernelParametersForm = (): JSX.Element => {
                 </span>
               }
             />{" "}
-            and OS{" "}
+            must meet the minimum requirements and the OS{" "}
             <TooltipButton
               iconName="help-mid-dark"
-              message="Ubuntu 24.04 LTS or higher."
+              message="Tested with Ubuntu 24.04 LTS or higher."
             />{" "}
-            must meet the minimum requirements and secure boot must be disabled.
-            Check crash dump status in machine details.{" "}
+            must support it. Check crash dump status in machine details.{" "}
             <ExternalLink to="https://ubuntu.com/server/docs/kernel-crash-dump">
               More about kernel crash dump
             </ExternalLink>