Host overview could reuse the already existing getInstancesOnHost selector

[rtorrero]

The Host overview right now collects the sap systems each host belongs to using:

const getSapSystemsByHost = (sapSystems, hostId) => {
  const appInstances = sapSystems
    .flatMap((sapSystem) => sapSystem.application_instances)
    .filter((sapSystem) => sapSystem.host_id === hostId)
    .map((sapSystem) => ({ type: 'sap-systems', instance: sapSystem }));
  const dbInstances = sapSystems
    .flatMap((sapSystem) => sapSystem.database_instances)
    .filter((sapSystem) => sapSystem.host_id === hostId)
    .map((sapSystem) => ({ type: 'databases', instance: sapSystem }));

  return appInstances.concat(dbInstances);
};

we could instead use the getInstancesOnHost from @state/selectors. I did a quick try but it seems that calling useSelector on each host iteration on the list breaks the app after a refresh.

We should improve this.

[rtorrero]

This was addressed by @arbulu89