Merge pull request #531 from konstructio/fix-ami-instancesize

fix: send api calls with ami type

containers/ClusterForms/shared/SetupForm.tsx

@@ -95,6 +95,9 @@ const SetupForm: FunctionComponent = () => {
   const handleRegionChange = useCallback(
     (region: string) => {
       setSelectedRegion(region);
+      if (installType == InstallationType.AWS) {
+        return;
+      }
       // if using google hold off on grabbing instances
       // since it requires the zone as well
 
@@ -112,6 +115,20 @@ const SetupForm: FunctionComponent = () => {
     [dispatch, installType, values],
   );
 
+  const handleAmiTypeSelect = useCallback(
+    (amiType: string) => {
+      dispatch(
+        getInstanceSizes({
+          installType,
+          region: selectedRegion as string,
+          values,
+          amiType,
+        }),
+      );
+    },
+    [dispatch, installType, selectedRegion, values],
+  );
+
   const handleZoneSelect = useCallback(
     (zone: string) => {
       if (cloudRegion) {
@@ -228,6 +245,7 @@ const SetupForm: FunctionComponent = () => {
             value: amiType,
           }))}
           defaultValue={AWS_AMI_TYPES[0]}
+          onChange={handleAmiTypeSelect}
         />
       )}
       <ControlledAutocomplete

package.json

@@ -125,5 +125,5 @@
   "volta": {
     "node": "18.18.0",
     "yarn": "1.22.19"
-  }
+  },
 }

redux/thunks/api.thunk.ts

@@ -173,19 +173,26 @@ export const getCloudDomains = createAsyncThunk<
 
 export const getInstanceSizes = createAsyncThunk<
   string[],
-  { region: string; installType?: InstallationType; values?: InstallValues; zone?: string },
+  {
+    region: string;
+    installType?: InstallationType;
+    values?: InstallValues;
+    zone?: string;
+    amiType?: string;
+  },
   {
     dispatch: AppDispatch;
     state: RootState;
   }
->('api/getInstanceSizes', async ({ region, installType, values, zone }) => {
+>('api/getInstanceSizes', async ({ region, installType, values, zone, amiType }) => {
   const { instance_sizes } = (
     await axios.post<{ instance_sizes: string[] }>('/api/proxy', {
       url: `/instance-sizes/${installType}`,
       body: {
         ...values,
         cloud_region: region,
         cloud_zone: zone,
+        ami_type: amiType,
       },
     })
   ).data;