fix machine resource in scaling ops for mongos

Signed-off-by: shofiq <shofiq@appscode.com>

Author: Shofiqur-052

charts/opskubedbcom-mongodbopsrequest-editor/ui/create-ui.yaml

@@ -238,7 +238,7 @@ step:
           name: isMachineValid|standalone
           type: custom
         watcher:
-          func: onMachineChange|standalone|/spec/podTemplate/spec/resources
+          func: onMachineChange|standalone
           paths:
           - temp/properties/machine-standalone
       - elements:
@@ -307,7 +307,7 @@ step:
           name: isMachineValid|replicaSet
           type: custom
         watcher:
-          func: onMachineChange|replicaSet|/spec/podTemplate/spec/resources
+          func: onMachineChange|replicaSet
           paths:
           - temp/properties/machine-replicaSet
       - elements:
@@ -376,8 +376,11 @@ step:
           subtitle: Compare your current machine configuration with the proposed memory
             adjustments and make informed decisions for your database setup
           type: machine-compare
+          validation:
+            name: isMachineValid|configServer
+            type: custom
           watcher:
-            func: onMachineChange|configServer|/spec/shardTopology/configServer/podTemplate/spec/resources
+            func: onMachineChange|configServer
             paths:
             - temp/properties/machine-configServer
         - elements:
@@ -441,8 +444,11 @@ step:
           subtitle: Compare your current machine configuration with the proposed memory
             adjustments and make informed decisions for your database setup
           type: machine-compare
+          validation:
+            name: isMachineValid|mongos
+            type: custom
           watcher:
-            func: onMachineChange|mongos|/spec/shardTopology/mongos/podTemplate/spec/resources
+            func: onMachineChange|mongos
             paths:
             - temp/properties/machine-mongos
         - elements:
@@ -506,8 +512,11 @@ step:
           subtitle: Compare your current machine configuration with the proposed memory
             adjustments and make informed decisions for your database setup
           type: machine-compare
+          validation:
+            name: isMachineValid|shard
+            type: custom
           watcher:
-            func: onMachineChange|shard|/spec/shardTopology/shard/podTemplate/spec/resources
+            func: onMachineChange|shard
             paths:
             - temp/properties/machine-shard
         - elements:
@@ -581,10 +590,6 @@ step:
           label: Memory
           schema: schema/properties/spec/properties/verticalScaling/properties/exporter/properties/resources/properties/limits/properties/memory
           type: input
-          watcher:
-            func: onExporterResourceChange|memory
-            paths:
-            - schema/properties/spec/properties/verticalScaling/properties/exporter/properties/resources/properties/limits/properties/memory
         showLabels: true
         type: horizontal-layout
       hideBlock: true

charts/opskubedbcom-mongodbopsrequest-editor/ui/functions.js

@@ -432,9 +432,10 @@ export const useFunc = (model) => {
     const cluster = storeGet('/route/params/cluster')
     const namespace = storeGet('/route/query/namespace') || getValue(model, '/metadata/namespace')
     const name = storeGet('/route/params/name') || getValue(model, '/spec/databaseRef/name')
+    const version = storeGet('/route/params/version')
 
     if (namespace && name) {
-      const url = `/clusters/${owner}/${cluster}/proxy/kubedb.com/v1alpha2/namespaces/${namespace}/mongodbs/${name}`
+      const url = `/clusters/${owner}/${cluster}/proxy/kubedb.com/${version}/namespaces/${namespace}/mongodbs/${name}`
       const resp = await axios.get(url)
 
       setDiscriminatorValue('/dbDetails', resp.data || {})
@@ -810,13 +811,7 @@ export const useFunc = (model) => {
   function getMachines(type) {
     const presets = storeGet('/kubedbuiPresets') || {}
     const dbDetails = getValue(discriminator, '/dbDetails')
-    // const limits = dbDetails?.spec?.podTemplate?.spec?.resources?.limits || {}
-    const limits = (type === 'replicaSet' || type === 'standalone'
-      ? dbDetails?.spec?.podTemplate?.spec?.resources?.requests
-      : dbDetails?.spec?.shardTopology?.[type]?.podTemplate?.spec?.resources?.requests) || {
-      cpu: '',
-      memory: '',
-    }
+    const limits = getLimits(type)
 
     const avlMachines = presets.admin?.machineProfiles?.available || []
     let arr = []
@@ -866,13 +861,7 @@ export const useFunc = (model) => {
 
   function setMachine(type) {
     const dbDetails = getValue(discriminator, '/dbDetails')
-    // const limits = dbDetails?.spec?.podTemplate?.spec?.resources?.limits || {}
-    const limits = (type === 'replicaSet' || type === 'standalone'
-      ? dbDetails?.spec?.podTemplate?.spec?.resources?.requests
-      : dbDetails?.spec?.shardTopology?.[type]?.podTemplate?.spec?.resources?.requests) || {
-      cpu: '',
-      memory: '',
-    }
+    const limits = getLimits(type)
     const annotations = dbDetails?.metadata?.annotations || {}
     const instance = annotations['kubernetes.io/instance-type']
     let parsedInstance = {}
@@ -894,9 +883,8 @@ export const useFunc = (model) => {
     } else return { machine: 'custom', cpu: limits.cpu, memory: limits.memory }
   }
 
-  function onMachineChange(type, valPath) {
-    let selectedMachine = {}
-    selectedMachine = getValue(discriminator, `/machine-${type}`)
+  function onMachineChange(type) {
+    const selectedMachine = getValue(discriminator, `/machine-${type}`) || {}
     const machine = machinesFromPreset.find((item) => item.id === selectedMachine.machine)
 
     let obj = {}
@@ -1850,27 +1838,9 @@ export const useFunc = (model) => {
     return limitVal
   }
 
-  function onExporterResourceChange(type) {
-    const commitPath = `/spec/verticalScaling/exporter/resources/requests/${type}`
-    const valPath = `/spec/verticalScaling/exporter/resources/limits/${type}`
-    const val = getValue(model, valPath)
-    if (val)
-      commit('wizard/model$update', {
-        path: commitPath,
-        value: val,
-        force: true,
-      })
-  }
-
   function isMachineValid(type) {
     const dbDetails = getValue(discriminator, '/dbDetails')
-
-    const limits = (type === 'replicaSet' || type === 'standalone'
-      ? dbDetails?.spec?.podTemplate?.spec?.resources?.requests
-      : dbDetails?.spec?.shardTopology?.[type]?.podTemplate?.spec?.resources?.requests) || {
-      cpu: '',
-      memory: '',
-    }
+    const limits = getLimits(type)
 
     const selectedMachine = getValue(discriminator, `/machine-${type}`)
     const selectedLimits = { cpu: selectedMachine.cpu, memory: selectedMachine.memory }
@@ -1881,10 +1851,26 @@ export const useFunc = (model) => {
     return false
   }
 
+  function getLimits(type) {
+    const dbDetails = getValue(discriminator, '/dbDetails')
+    const isTopology = !!dbDetails?.spec?.shardTopology
+
+    const spec = isTopology
+      ? dbDetails?.spec?.shardTopology?.[type]
+      : dbDetails?.spec?.podTemplate?.spec
+    let containers = spec?.containers ?? []
+
+    if (containers.length === 0) {
+      if (spec?.resources?.requests) return spec?.resources?.requests
+      containers = spec?.podTemplate?.spec?.containers ?? []
+    }
+    const kind = dbDetails?.kind?.toLowerCase()
+    return containers.find((container) => container.name === kind)?.resources?.requests ?? {}
+  }
+
   return {
     isMachineValid,
     setExporter,
-    onExporterResourceChange,
     fetchAliasOptions,
     validateNewCertificates,
     disableAlias,