Save default boot option

Author: Nykseli

src/app.tsx

@@ -36,12 +36,12 @@ const _ = cockpit.gettext;
 
 type Pages = "advanced" | "boot-options" | "kernel-params";
 
-const SelectedPage = ({ page, grub }: { page: Pages, grub: GrubFile }) => {
+const SelectedPage = ({ page, grub, setBootEntry }: { page: Pages, grub: GrubFile, setBootEntry: (entry: string) => void }) => {
     switch (page) {
     case "advanced":
         return <AdvancedValues grub={grub} />;
     case "boot-options":
-        return <BootOptions />;
+        return <BootOptions setBootEntry={setBootEntry} />;
     case "kernel-params":
         return <KernelParameters grub={grub} />;
     default:
@@ -113,10 +113,11 @@ export const Application = () => {
     const [page, setPage] = React.useState<Pages>("kernel-params");
     const [hasGrub, setHasGrub] = useState<boolean | undefined>(undefined);
     const [updatingGrub, setUpdatingGrub] = useState(false);
+    const [bootEntry, setBootEntry] = useState<string | null>(null);
     const [authenticated, setAuthenticated] = React.useState(superuser.allowed);
 
     const updateGrub = React.useCallback(() => {
-        if (grub) {
+        if (grub && page !== "boot-options") {
             setUpdatingGrub(true);
             cockpit.file('/etc/default/grub', { superuser: "require" })
                             .replace(grub.toFile()).then(() => {
@@ -131,8 +132,16 @@ export const Application = () => {
                                 console.error(reason);
                                 setUpdatingGrub(false);
                             });
+        } else if (bootEntry) {
+            setUpdatingGrub(true);
+            cockpit.spawn(["grub2-set-default", bootEntry], { superuser: "require" })
+                            .then(() => console.log("grub2-set-default success"))
+                            .catch((reason) => {
+                                console.error(reason);
+                            })
+                            .finally(() => setUpdatingGrub(false));
         }
-    }, [grub]);
+    }, [grub, bootEntry]);
 
     const resetGrub = () => {
         // setting grub value to null first forces the state to reload
@@ -209,7 +218,7 @@ export const Application = () => {
                         </FlexItem>
                     </Flex>
                 </PageSection>
-                {grub ? <SelectedPage page={page} grub={grub} /> : null}
+                {grub ? <SelectedPage page={page} grub={grub} setBootEntry={setBootEntry} /> : null}
             </Page>
         </WithDialogs>
     );

src/pages/boot_options.tsx

@@ -2,19 +2,24 @@ import React, { useEffect, useState } from 'react';
 
 import { Form, FormGroup, FormSelect, FormSelectOption } from '@patternfly/react-core';
 import cockpit from 'cockpit';
-import { fsinfo } from 'cockpit/fsinfo';
+// import { fsinfo } from 'cockpit/fsinfo';
 
 const _ = cockpit.gettext;
 
-export const BootOptions = () => {
+export const BootOptions = ({ setBootEntry }: { setBootEntry: (entry: string) => void }) => {
     const [bootEntries, setBootEntries] = useState<string[]>([]);
-    const [hasGrubBoot, setHasGrubBoot] = useState<boolean | undefined>(undefined);
-    const [selectedBoot, setSelectedBoot] = useState<string>("defult?");
+    // const [hasGrubBoot, setHasGrubBoot] = useState<boolean | undefined>(undefined);
+    const [selectedBoot, setSelectedBoot] = useState<string>("");
+
+    const setEntry = (entry: string) => {
+        setBootEntry(entry);
+        setSelectedBoot(entry);
+    };
 
     useEffect(() => {
-        fsinfo('/boot/grub2/grub.cfg', [])
-                        .then(() => setHasGrubBoot(true))
-                        .catch(() => setHasGrubBoot(false));
+        // fsinfo('/boot/grub2/grub.cfg', [])
+        //                 .then(() => setHasGrubBoot(true))
+        //                 .catch(() => setHasGrubBoot(false));
 
         const bootConfig = cockpit.file('/boot/grub2/grub.cfg', { superuser: 'require' });
         bootConfig.watch(content => {
@@ -32,7 +37,7 @@ export const BootOptions = () => {
     return (
         <Form>
             <FormGroup label={_("Console resolution")} fieldId="graphical-console-resolution">
-                <FormSelect value={selectedBoot} onChange={(_event, value) => setSelectedBoot(value)}>
+                <FormSelect value={selectedBoot} onChange={(_event, value) => setEntry(value)}>
                     {bootEntries.map((value, idx) =>
                         <FormSelectOption key={idx} label={value} value={value} />
                     )}