Skip to content

Commit f9156ef

Browse files
committed
fix machines in opsrequests
Signed-off-by: Samiul <samiul@appscode.com>
1 parent 4d25520 commit f9156ef

3 files changed

Lines changed: 38 additions & 29 deletions

File tree

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

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -664,8 +664,7 @@ export const useFunc = (model) => {
664664

665665
function getMachines() {
666666
const presets = storeGet('/kubedbuiPresets') || {}
667-
const dbDetails = getValue(discriminator, '/dbDetails')
668-
const limits = dbDetails?.spec?.podTemplate?.spec?.resources?.requests || {}
667+
const limits = getLimits()
669668

670669
const avlMachines = presets.admin?.machineProfiles?.available || []
671670
let arr = []
@@ -714,9 +713,8 @@ export const useFunc = (model) => {
714713
}
715714

716715
function setMachine() {
717-
const dbDetails = getValue(discriminator, '/dbDetails')
718-
const limits = dbDetails?.spec?.podTemplate?.spec?.resources?.requests || {}
719-
const annotations = dbDetails?.metadata?.annotations || {}
716+
const limits = getLimits()
717+
const annotations = getValue(discriminator, '/dbDetails/metadata/annotations') || {}
720718
const instance = annotations['kubernetes.io/instance-type']
721719

722720
let parsedInstance = {}
@@ -1721,9 +1719,17 @@ export const useFunc = (model) => {
17211719
})
17221720
}
17231721

1724-
function isMachineValid() {
1722+
function getLimits() {
17251723
const dbDetails = getValue(discriminator, '/dbDetails')
1726-
const limits = dbDetails?.spec?.podTemplate?.spec?.resources?.requests || {}
1724+
const containers = dbDetails?.spec?.podTemplate?.spec?.containers || []
1725+
const kind = dbDetails?.kind
1726+
const resource = containers.filter((ele) => ele.name === kind?.toLowerCase())
1727+
const limits = resource[0]?.resources?.requests || {}
1728+
return limits
1729+
}
1730+
1731+
function isMachineValid() {
1732+
const limits = getLimits()
17271733

17281734
const selectedMachine = getValue(discriminator, '/machine')
17291735
const selectedLimits = { cpu: selectedMachine.cpu, memory: selectedMachine.memory }

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

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -769,10 +769,7 @@ export const useFunc = (model) => {
769769

770770
function getMachines() {
771771
const presets = storeGet('/kubedbuiPresets') || {}
772-
const dbDetails = getValue(discriminator, '/dbDetails')
773-
const containers = dbDetails?.spec?.podTemplate?.spec?.containers || []
774-
const hazelcastContainer = containers.find((c) => c.name === 'hazelcast')
775-
const limits = hazelcastContainer?.resources?.requests || {}
772+
const limits = getLimits()
776773

777774
const avlMachines = presets.admin?.machineProfiles?.available || []
778775
let arr = []
@@ -822,9 +819,7 @@ export const useFunc = (model) => {
822819

823820
function setMachine() {
824821
const dbDetails = getValue(discriminator, '/dbDetails')
825-
const containers = dbDetails?.spec?.podTemplate?.spec?.containers || []
826-
const hazelcastContainer = containers.find((c) => c.name === 'hazelcast')
827-
const limits = hazelcastContainer?.resources?.requests || {}
822+
const limits = getLimits()
828823

829824
const annotations = dbDetails?.metadata?.annotations || {}
830825
const instance = annotations['kubernetes.io/instance-type']
@@ -1831,11 +1826,17 @@ export const useFunc = (model) => {
18311826
})
18321827
}
18331828

1834-
function isMachineValid() {
1829+
function getLimits() {
18351830
const dbDetails = getValue(discriminator, '/dbDetails')
18361831
const containers = dbDetails?.spec?.podTemplate?.spec?.containers || []
1837-
const hazelcastContainer = containers.find((c) => c.name === 'hazelcast')
1838-
const limits = hazelcastContainer?.resources?.requests || {}
1832+
const kind = dbDetails?.kind
1833+
const resource = containers.filter((ele) => ele.name === kind?.toLowerCase())
1834+
const limits = resource[0]?.resources?.requests || {}
1835+
return limits
1836+
}
1837+
1838+
function isMachineValid() {
1839+
const limits = getLimits()
18391840

18401841
const selectedMachine = getValue(discriminator, '/machine')
18411842
const selectedLimits = { cpu: selectedMachine.cpu, memory: selectedMachine.memory }

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

Lines changed: 14 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -754,15 +754,10 @@ export const useFunc = (model) => {
754754
// vertical scaling
755755

756756
// machine profile stuffs
757-
// let machinesFromPreset = []
758757

759758
function getMachines() {
760759
const presets = storeGet('/kubedbuiPresets') || {}
761-
const dbDetails = getValue(discriminator, '/dbDetails')
762-
const containers = dbDetails?.spec?.podTemplate?.spec?.containers || []
763-
const limits =
764-
dbDetails?.spec?.podTemplate?.spec?.resources?.requests ??
765-
(containers[0]?.resources?.limits || {})
760+
const limits = getLimits()
766761

767762
const avlMachines = presets.admin?.machineProfiles?.available || []
768763
let arr = []
@@ -812,9 +807,8 @@ export const useFunc = (model) => {
812807

813808
function setMachine() {
814809
const dbDetails = getValue(discriminator, '/dbDetails')
815-
const containers = dbDetails?.spec?.podTemplate?.spec?.containers || []
816-
const igniteContainer = containers.find((c) => c.name === 'ignite')
817-
const limits = igniteContainer?.resources?.requests || {}
810+
const limits = getLimits()
811+
console.log(limits)
818812
const annotations = dbDetails?.metadata?.annotations || {}
819813
const instance = annotations['kubernetes.io/instance-type']
820814

@@ -830,6 +824,7 @@ export const useFunc = (model) => {
830824

831825
const machinePresets = machinesFromPreset.find((item) => item.id === machine)
832826
if (machinePresets) {
827+
console.log('machinePresets', machinePresets, machine)
833828
return {
834829
machine: machine,
835830
cpu: machinePresets.limits.cpu,
@@ -1820,11 +1815,18 @@ export const useFunc = (model) => {
18201815
})
18211816
}
18221817

1823-
function isMachineValid() {
1818+
function getLimits() {
18241819
const dbDetails = getValue(discriminator, '/dbDetails')
18251820
const containers = dbDetails?.spec?.podTemplate?.spec?.containers || []
1826-
const igniteContainer = containers.find((c) => c.name === 'ignite')
1827-
const limits = igniteContainer?.resources?.requests || {}
1821+
const kind = dbDetails?.kind
1822+
const resource = containers.filter((ele) => ele.name === kind?.toLowerCase())
1823+
const limits = resource[0]?.resources?.requests || {}
1824+
return limits
1825+
}
1826+
1827+
function isMachineValid() {
1828+
const dbDetails = getValue(discriminator, '/dbDetails')
1829+
let limits = getLimits()
18281830
delete limits.machine
18291831

18301832
const selectedMachine = getValue(discriminator, '/machine')

0 commit comments

Comments
 (0)